Closed (fixed)
Project:
REST UI
Version:
8.x-1.x-dev
Component:
Code
Priority:
Minor
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
14 Apr 2017 at 08:28 UTC
Updated:
13 May 2017 at 22:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
Munavijayalakshmi 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 commentedComment #8
cjgratacos 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 commentedI'll try to work on the suggested changes by @Wim Leers.
Comment #12
leolandotan commentedHere I have applied the following:
Hope everything is in order.
Comment #13
leolandotan 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 commented@ clemens.tolboom Thanks, excited to be part of this awesome community