Part of meta-issue #1518116: [meta] Make Core pass Coder Review

CommentFileSizeAuthor
#4 1533232-field-module-coder-review.diff6.95 KBgaëlg
#2 drupal-code_cleanup-1533232-3.patch105.72 KB-enzo-

Comments

NROTC_Webmaster’s picture

NROTC_Webmaster commented
Status: Active » Postponed

Postponed because of #1326634: Clean up API docs for field module

-enzo-’s picture

-enzo- commented
StatusFileSize
new drupal-code_cleanup-1533232-3.patch105.72 KB

I don't realize this issue was posponed :S

I did a code cleanup.

I guess after the issue #1326634 will be completed, this patch needs to be re-rolled.

gaëlg’s picture

gaëlg
he/him
Primary language French
Location Lille, France
commented
Issue summary: View changes
Status: Postponed » Needs work
Issue tags: +#amsterdam2014

I'm having a try with this.

gaëlg’s picture

gaëlg
he/him
Primary language French
Location Lille, France
commented
Issue tags: -#amsterdam2014
StatusFileSize
new 1533232-field-module-coder-review.diff6.95 KB

Well, this might not be the best thing to do during a DC sprint. The code moves too much! Anyway, here's a little diff of what I did. More info on encountered problems in the parent meta issue.

yannisc’s picture

yannisc commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 4: 1533232-field-module-coder-review.diff, failed testing.

The last submitted patch, 2: drupal-code_cleanup-1533232-3.patch, failed testing.

xjm’s picture

xjm
she/her
Primary language English
commented
Version: 8.0.x-dev » 8.1.x-dev
Priority: Normal » Minor
Status: Needs work » Postponed
Issue tags: -Novice

Thanks for all the work here so far. See #1518116-86: [meta] Make Core pass Coder Review. This issue is postponed until the meta issue is either closed or reopened.

pfrenssen’s picture

pfrenssen
Location Sofia
commented
Status: Postponed » Closed (duplicate)

Closing in favor of #2571965: [meta] Fix PHP coding standards in core, stage 1. In this issue the coding standards will be fixed on a sniff-per-sniff basis rather than a module-per-module basis.