Problem/Motivation

To make it easier for other developers to contribute code, let's update Term Merge to follow the Drupal coding standards.

Proposed resolution

Update the code to follow Drupal coding standards

Remaining tasks

  1. Write a patch
  2. Review
  3. Commit

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#2 term_merge-coding_standards-2980479-2.patch31.36 KBidebr
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

idebr created an issue. See original summary.

idebr’s picture

idebr CreditAttribution: idebr at ezCompany commented
Status: Active » Needs review
FileSize
term_merge-coding_standards-2980479-2.patch31.36 KB

Attached patch updates the code to follow Drupal coding standards.

legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
CreditAttribution: legolasbo at Ordina Digital Services for Sijthoff Media commented
Status: Needs review » Reviewed & tested by the community

Looks good, thanks!

  • legolasbo committed 54a3280 on 8.x-1.x authored by idebr 
    Issue #2980479 by idebr, legolasbo: Follow Drupal coding standards
legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
CreditAttribution: legolasbo at Ordina Digital Services for Sijthoff Media commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x-1.x, Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.