Project

General

Profile

Bug #10185

ADE conversion and runtime

Added by Teodor Gorghe about 1 year ago. Updated about 1 year ago.

Status:
Test
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
Due date:
% Done:

30%

billable:
No
vendor_id:
GCD
case_num:
version_reported:
version_resolved:
production:
No
env_name:
topics:

History

#2 Updated by Teodor Gorghe about 1 year ago

When passing a value from extent to a procedure, with OUTPUT TEMP-HANDLE, on compilation we have this issue:

    [javac] /home/tg/gcd/testcases2/testcases/src/com/goldencode/testcases/tests/table_/parameter/output/bind/TestHandleTable.java:201: error: no suitable constructor found for TableParameter(handle[])
    [javac]          silent(() -> RUN_CALL_SITE_6.clone().setInHandle(hproc).run(subscript(new TableParameter(hext), 1)));
    [javac]                                                                                ^
    [javac]     constructor TableParameter.TableParameter(Temporary) is not applicable
    [javac]       (argument mismatch; handle[] cannot be converted to Temporary)
    [javac]     constructor TableParameter.TableParameter(handle) is not applicable
    [javac]       (argument mismatch; handle[] cannot be converted to handle)
    [javac]     constructor TableParameter.TableParameter(TableWrapper) is not applicable
    [javac]       (argument mismatch; handle[] cannot be converted to TableWrapper)

4GL code:
    @Test.
    method public void SameStructureExtent():
        define variable hext as handle no-undo extent 1.
        hext[1] = temp-table ttTest:handle.
        run noopOutTable in hproc ( output table-handle hext[1] ) no-error.
        support.test.AssertExt:NotErrorNotWarning().
    end method.

On ABL, this test compiles and passes.

#3 Updated by Teodor Gorghe about 1 year ago

  • Status changed from New to WIP

Created task branch 10185a.

#4 Updated by Teodor Gorghe about 1 year ago

  • % Done changed from 0 to 100

Committed revision 15992 on task branch 10185a:
- Fixed subscript wrapper on new TableParameter()

#5 Updated by Teodor Gorghe about 1 year ago

  • Status changed from WIP to Review

#6 Updated by Constantin Asofiei about 1 year ago

  • reviewer Constantin Asofiei added

#7 Updated by Constantin Asofiei about 1 year ago

Please check also:
  • DATASET-HANDLE argument type
  • the handle array is an instance or static var in a .cls (not a local var)
  • the handle array is an instance or static property in a .cls

#8 Updated by Teodor Gorghe about 1 year ago

  • reviewer deleted (Constantin Asofiei)
I am attaching the conversion results for:

I don't notice any problem. The application compiles and the tests seems to work.

#9 Updated by Teodor Gorghe about 1 year ago

  • reviewer Constantin Asofiei added

#10 Updated by Teodor Gorghe about 1 year ago

  • Status changed from Review to WIP
  • % Done changed from 100 to 30
  • Subject changed from OUTPUT TABLE-HANDLE conversion problem when passing value from extent variable to ADE conversion and runtime

On FWD, HAS-LOBS attribute is supported?
This is the cause of the second conversion problem on ADE.
Unsupported method or attribute KW_HAS_LOBS.

There is a test tests.buffer.attributes.TestHasLobs.cls, which has conversion issues.

#11 Updated by Teodor Gorghe about 1 year ago

Committed revision 15993 of task branch 10185a:
- added runtime and conversion support for BUFFER:HAS-LOBS.

Constantin, please check if the implementation needs to be modified. This might not be the best way. Another solution was to store this value during the conversion, into an annotation.
I am checking right now if the adm2/dataquery.p compilation works now and if there are any other issues.

#12 Updated by Teodor Gorghe about 1 year ago

With my initial conversion include files, it seems that there are no longer conversion issues. I will update this task as long as I encounter other issues. I am currently testing the runtime.

#13 Updated by Teodor Gorghe about 1 year ago

I have added adm2/query.p and it seems that we have:
Unsupported method or attribute KW_IS_PART.
I am investigating this and implement this feature if needed.

#14 Updated by Teodor Gorghe about 1 year ago

Committed revision 15994 on task branch 10185a:
- Added conversion support for BUFFER:IS-PARTITIONED (currently is stubbed, always returns false).

We have another conversion problem, on adm2/browser.p, which is also an unimplemented method/attribute: Unsupported method or attribute KW_UNIQ_MAT.

#15 Updated by Teodor Gorghe about 1 year ago

  • Status changed from WIP to Review

Committed revision 15995 on task branch 10185a:
- Added a partial implementation of WIDGET:UNIQUE-MATCH.
- Added some ADD-LIKE-COLUMN method overloads for fixing compilation issues on adm2/browser.p
- Added a setWidgetId method overload on using int64 id type.

Committed revision 15996 on task branch 10185a:
- Fixed compilation issues on adm2/b2b.p (the fix was suggested by Constantin).

From this stage, it seems that all .p files from adm2 gets converted and running.

#16 Updated by Teodor Gorghe about 1 year ago

Committed revision 15997 on task branch 10185a:
- Fixed compilation issues of BROWSE:NO-EMPTY-SPACE. The interface method definition was missing.

#17 Updated by Teodor Gorghe about 1 year ago

Committed revision 15998 on task branch 10185a:
- history entry fix on NameConverter.java, java_templates.tpl, Buffer.java and ComboBoxWidget.java

#18 Updated by Constantin Asofiei about 1 year ago

  • Status changed from Review to Merge Pending

Conversion testing passed. Please merge 10185a after 9550a.

#19 Updated by Teodor Gorghe about 1 year ago

  • Status changed from Merge Pending to Test

Branch 10185a was merged to trunk rev 16016 and archived.

Also available in: Atom PDF