[Interpreter] Should a segment array appear as an array of arrays with one repetition? about edifact HOT 9 CLOSED

@swiffer what I was thinking is to have a flag is a similar fashion to patchFiles which is a boolean enabling the segment patching feature
https://github.com/php-edifact/edifact/blob/master/src/EDI/Interpreter.php#L45

from edifact.

For usage in implementation, it's difficult to know if it's an array of arrays (multiple returns) or just an array for one result.

I was wondering why you haven't choose to make return in stdclass ?

from edifact.

I would check with is_array... The choice was done initially because it was easy enough to work with the array and to json_encode it :-)

from edifact.

@sabas - 6 years later I got hit by this as I was reading an EDIFACT with could have multiple DTM segments but is not having them all the time. As I'm interating over the array this fails when there is only one DTM segment. Would be great to always return an array of results for consistancy whenever maxrepeat is > 1

from edifact.

@swiffer wow 6 years time flies hahaha

I actually need some time to remember the issue, but I suspect it would need to tweak the doAddArray function
https://github.com/php-edifact/edifact/blob/master/src/EDI/Interpreter.php#L578

So to always make it an array, something like

if (isset($array[$jsonMessage['key']])) {
$array[$jsonMessage['key']] = [];
}
$array[$jsonMessage['key']][] = $jsonMessage['value'];

Would you try something like this? In case we could do some sort of flag to control this mode.

from edifact.

@sabas for now i'm using something like this:

    if (!array_is_list($detail['SG6'][0]['date/time/period'])) {
        $detail['SG6'][0]['date/time/period'] = [$detail['SG6'][0]['date/time/period']];
    }

you mean having an option passed to:

  new Interpreter(,...) ?

from edifact.

sounds good to me

from edifact.

@swiffer Hi Matthias, would you please send me (mail is in profile) an EDI sample so I can think about the solution?
Perhaps also highlighting what would be the desired structure... :)

Also offtopic, I updated the mappings and I wanted to alert you that now the names don't contain slashes (I found out xml doesn't want them in the name attribute) so date/time/period => dateOrtimeOrperiod

from edifact.

Ok, I made a new flag, if set true then if the maxrepeat is greater than 1 it makes an array also for a single segment
$interpreter->forceArrayWhenRepeatable(false);

from edifact.