Shell is broken and field order is random

JSON and perl have no way to store the order of a hash so we should store it in a separate array.

(from redmine: issue id 4993, created on 2016-07-07, closed on 2017-02-14)

• Relations:
  • copied_to #4994 (closed)
  • copied_to #4997 (closed)
• Changesets:
  • Revision c47bdb29 by Côme Chilliet on 2016-07-07T10:02:57.000Z:

Fixes #4993 Added sections_order and attrs_order, and fixed shell

• Revision b2f58d58 by Côme Chilliet on 2016-07-07T10:07:32.000Z:

Fixes #4993 Added sections_order and attrs_order, and fixed shell

• Custom Fields:
  • Bug in version: 1.0.13

Hi Côme, I took the time to test how the data is returned with getfields() and formfields() I'm unsure the data content returned by getfields() is the one we expect, see :

formfields()

            [perso] => Array
                (
                    [name] => Personal information
                    [icon] => geticon.php?context=types&icon=user&size=16
                    [attrs] => Array
                        (
                            [sn] => Array
                                (
                                    [htmlid] => sn
                                    [label] => Last name
                                    [required] => 1
                                    [disabled] => 
                                    [description] => Last name of this user
(...)
                    [attrs_order] => Array
                        (
                            [0] => sn
                            [1] => givenName
                            [2] => description
                            [3] => jpegPhoto
                        )

                )

Seems to works as expected.

And with getfields() we have :

            [perso] => Array
                (
                    [name] => Personal information
                    [icon] => geticon.php?context=types&icon=user&size=16
                    [attrs] => Array
                        (
                            [cn] => Array
                                (
                                    [htmlid] => cn
                                    [label] => cn
                                    [required] => 
                                    [disabled] => 
                                    [description] => hidden
(...)
                    [attrs_order] => Array
                        (
                            [cn] => Array
                                (
                                    [htmlid] => cn
                                    [label] => cn
                                    [required] => 
                                    [disabled] => 
                                    [description] => hidden
(...)the exact same content as [attrs](...)

I don't know if we want to order the output of getfields() but, if not we could simply not include [attrs_order] for this function, wdyt ?

(from redmine: written on 2016-07-08)

Thanks, I opened #4997 (closed) for this specific problem.

(from redmine: written on 2016-07-11)

Close issue

(from redmine: written on 2017-02-14)

closed

added Fixed label

removed Bugs label

added PJ1802-0188 label