A simple automated code review at ventral.org shows 41 coding standards issues including one potential security vulnerability.

You can see the result of the review here:
http://ventral.org/pareview/httpgitdrupalorgprojectcontextredirectgit

If the module ever gets an active maintainer it would be a good idea to clean it up to the d.o. accepted standards and make sure the codebase is commented/documented correctly.

Comments

thedavidmeister’s picture

thedavidmeister commented

I've cleaned up the 7.x-1.x branch and I'm now pretty sure everything in the Ventral.org report is a false positive.

The potential security issue is not a problem because $name comes from l() and the class names are unfortunately dictated by the non-camel-case style of Context itself.

6.x-1.x still needs some love.

thedavidmeister’s picture

thedavidmeister commented
Status: Active » Fixed

6.x-1.x now only has the same issues flagged by Ventral as 7.x-1.x

Status: Fixed » Closed (fixed)

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