Ran PAReview on ventral.org and discovered some additional problems with the code in the 2.x branch and with the fact there is still a master branch. See http://ventral.org/pareview/httpgitdrupalorgsandboxadamatmobomo1271006git for the results. I will fix when I get time.

Comments

timwood’s picture

timwood
Primary language English
Location Rockville, Maryland
commented
Status: Active » Fixed

I've corrected all the PAReview suggestions EXCEPT the "Line exceeds 80 characters..." warnings.

Status: Fixed » Closed (fixed)

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

  • Commit f84681a on 6.x-2.x, 7.x-3.x, 7.x-4.x by timwood: 
    More code cleanup per PAReview recommendations. Fixes #1423238.