CommentFileSizeAuthor
#4 encrypt-fix_coding_standard-2823673-4.patch1.42 KBgisle
#2 fix_coding_standard_2823673-1.patch1.38 KBprince_kr
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

prince_zyxware created an issue. See original summary.

prince_kr’s picture

prince_kr CreditAttribution: prince_kr at Zyxware Technologies commented
FileSize
fix_coding_standard_2823673-1.patch1.38 KB

Updated the patch for the spacing issues found on the module.

prince_kr’s picture

prince_kr CreditAttribution: prince_kr at Zyxware Technologies commented
Status: Active » Needs review
gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
CreditAttribution: gisle at Hannemyr Nye Medier AS commented
Version: 7.x-3.x-dev » 7.x-2.x-dev
Assigned: Unassigned » gisle
FileSize
encrypt-fix_coding_standard-2823673-4.patch1.42 KB

Rerolled patch against latest snapshot of 7.x-2.x-dev.

  • gisle committed db3dc3c on 7.x-2.x authored by prince_kr 
    Issue #2823673 by prince_kr,gisle: Fixed Drupal coding standard issues
gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
CreditAttribution: gisle at Hannemyr Nye Medier AS commented
Status: Needs review » Fixed

Has fixed the coding standard issues reported by prince_kr, plus some additional coding standard issues reported by the current version of PAreview.

Remaining coding standard issue is tracked by #3042550: Code standards issue: The decrypt function. Moving this one to "Fixed".

Status: Fixed » Closed (fixed)

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