Project

General

Profile

Bug #10662

Prodataset ADD-PARENT-ID-RELATION issue

Added by Teodor Gorghe 10 months ago. Updated 10 months ago.

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

100%

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

History

#2 Updated by Teodor Gorghe 10 months ago

The following test passes on 4GL, but fails on FWD:

BLOCK-LEVEL ON ERROR UNDO, THROW.

CLASS tests.TestDynamicDataset:

    DEFINE TEMP-TABLE tt1
        FIELD name AS CHARACTER.
    DEFINE TEMP-TABLE tt2 LIKE tt1
        FIELD parentId AS RECID.

    @Test.
    METHOD PUBLIC VOID TestAddParentIdRelation():
        DEFINE VARIABLE dsHandle AS HANDLE NO-UNDO.
        CREATE DATASET dsHandle.
        dsHandle:SET-BUFFERS (TEMP-TABLE tt1:HANDLE, TEMP-TABLE tt2:HANDLE).
        dsHandle:ADD-PARENT-ID-RELATION (BUFFER tt1:HANDLE, BUFFER tt2:HANDLE, "parentId", "", "").
        FINALLY:
            IF VALID-HANDLE (dsHandle) THEN
                DELETE OBJECT dsHandle.
        END FINALLY.
    END METHOD.
END CLASS.

The issue is that, for parent-fields-before and parent-fields-after on DATASET:ADD-PARENT-ID-RELATION, the arguments can be empty character.
FWD is not allowing this and throws this error:

ADD-PARENT-ID-RELATION parent-fields-before entry  is not valid. (16029)

#3 Updated by Teodor Gorghe 10 months ago

  • Assignee set to Teodor Gorghe
  • Status changed from New to WIP

Created task branch 10662a.

#4 Updated by Teodor Gorghe 10 months ago

  • Status changed from WIP to Review
  • % Done changed from 0 to 100
  • reviewer Constantin Asofiei added

Committed revision 16191 on task branch 10662a:
- DATASET:ADD-PARENT-ID-RELATION allow parent field before and after to be empty character.

#5 Updated by Constantin Asofiei 10 months ago

Teodor, please merge after 10380a.

#6 Updated by Teodor Gorghe 10 months ago

  • Status changed from Review to Test

Branch 10662a was merged to trunk rev. 16204 and archived.

Also available in: Atom PDF