[development] coder and 6.x core remaining upgrades

Jeff Eaton jeff at viapositiva.net
Sat Jun 2 15:55:09 UTC 2007

On Jun 2, 2007, at 10:36 AM, Doug Green wrote:

> f you’re working on core for 6.x, I’d like you to know that http:// 
> drupal.org/project/coder’s 6.x upgrade review is now current.  The  
> HEAD version of coder works on the current Drupal HEAD, and if you  
> run it on Drupal core itself it finds about 30 problems ranging  
> including many argument changes to l(), url(), watchdog t()’s no  
> longer needed.  Since these upgrades span several modules and since  
> I’m not aware of the fix really needed (for example, #multistep in  
> poll.module), it seemed more appropriate just to let the core  
> developers know that there are a few minor issues, and that coder  
> should help you find many of them.

I've used Coder in the past when cleaning up the VotingAPI code and  
the results were really nice. It's awesome that we're starting to  
apply our coding standards to the core modules that were around  
before those standards evolved. ;)

The #multistep issue in poll is fixed in the $form-state patch; it's  
easy to miss in the mass of stuff there. ;)


More information about the development mailing list