Problem/Motivation

Lots of code in the hal, serialization and rest modules has significant coding standards violations.

This makes maintenance and patches more painful.

Proposed resolution

Fix coding standard errors.

Remaining tasks

Review & commit.

User interface changes

None.

API changes

None.

Data model changes

None.

Original => https://www.drupal.org/node/2362495

CommentFileSizeAuthor
rest-collection-2100637-18.patch54.33 KBSaphyel
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Saphyel created an issue. See original summary.

clemens.tolboom’s picture

clemens.tolboom
Primary language Dutch
Location Groningen, 🇳🇱/🇪🇺
CreditAttribution: clemens.tolboom commented
Status: Needs work » Closed (outdated)

As stated in #2571965: [meta] Fix PHP coding standards in core

Instead we will now split up the work per type of coding standard. For example in separate issues we'll deal with trailing whitespace, inline comments, line lengths, etc. This will be much easier to review, since every patch will only contain very similar fixes.

I close #2362495: Fix coding standard errors in REST, HAL and Serialization modules and #2709151: Fix coding standard errors in REST, HAL and Serialization modules as both do not comply to the workflow defined in #2571965: [meta] Fix PHP coding standards in core