CivicActions is reviewing and upgrading multiple modules for use on client sites. Part of this work is a coding standards review. Attached you will find a patch based on a review with the coder module and a careful examination of the code. Thanks!

This patch includes mostly code style fixes in the emfield and emfield_video modules along with the removal of the unused $allowScriptAcess (sic) variable.

CommentFileSizeAuthor
emfield-codereview.patch25.98 KBwmostrey
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

stella’s picture

stella CreditAttribution: stella commented
Status: Active » Needs review
Alex UA’s picture

Alex UA CreditAttribution: Alex UA commented
Status: Needs review » Fixed

Sorry, I thought this was also within your other patch. I committed your changes- thanks for your help!

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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