Version 2.x is completely broken. with syntax errors and coding issues. This task will be used to fix all major errors.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

caminadaf created an issue. See original summary.

erickbj’s picture

The attached patch fixes all files permissions (from 755 to 644 where appropriate) and Drupal coding standards for files under the includes/ directory. Coding standards were checked with phpcs, using option standards as "Drupal" and "DrupalPractice".

atoledo@ciandt.com’s picture

Status: Active » Needs review
FileSize
50.42 KB

Attached a patch fixing the coding standards for the root files. Coding standards were checked with phpcs, using option standards as "Drupal" and "DrupalPractice".

atoledo@ciandt.com’s picture

FileSize
50.43 KB

Disregard patch in #3, please use this one after applying #2

caminadaf’s picture

Status: Needs review » Fixed

Hey guys, code seems really good now! Thank you SO MUCH for your effort into this!

I've commited this on 7.x-2.x, and now we can finally start doing real improvements on this version!

Status: Fixed » Closed (fixed)

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