Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
REST UI module needs to be converted to use short array syntax as per new coding standard.
Comment | File | Size | Author |
---|---|---|---|
#12 | interdiff-9-12.txt | 3.54 KB | leolandotan |
#12 | convert_module_to_use_short_array_syntax-2869724-12.patch | 17.11 KB | leolandotan |
#9 | interdiff-2869724-8-9.txt | 4.22 KB | clemens.tolboom |
#9 | convert_module_to_use-2869724-9.patch | 18.8 KB | clemens.tolboom |
#8 | short_array_syntax-2869724-8.patch | 17.08 KB | cjgratacos |
Comments
Comment #2
Munavijayalakshmi CreditAttribution: Munavijayalakshmi at Valuebound commentedComment #3
Wim LeersPlease wait to commit this until more important issues are committed, which would otherwise need rerolls.
Comment #4
Wim LeersThis can now continue again. Retesting.
Comment #6
Wim LeersComment #7
cjgratacos CreditAttribution: cjgratacos as a volunteer commentedComment #8
cjgratacos CreditAttribution: cjgratacos as a volunteer commentedRerolling Patch
Comment #9
clemens.tolboomI had some code style issues (PHP Storm formatted it for me). It looks OK to me now.
Comment #10
Wim LeersI don't see why we'd be reflowing the code too?
Same here…
Here it seems sensible though!
And here it doesn't…
Comment #11
leolandotan CreditAttribution: leolandotan as a volunteer and at Promet Source commentedI'll try to work on the suggested changes by @Wim Leers.
Comment #12
leolandotan CreditAttribution: leolandotan as a volunteer and at Promet Source commentedHere I have applied the following:
Hope everything is in order.
Comment #13
leolandotan CreditAttribution: leolandotan as a volunteer and at Promet Source commentedComment #15
clemens.tolboomThanks 2 all!
Comment #16
clemens.tolboom@cjgratacos congrats with your first Drupal patch in your first month as a member.
Comment #17
cjgratacos CreditAttribution: cjgratacos as a volunteer commented@ clemens.tolboom Thanks, excited to be part of this awesome community