[development] Announce Coder 6.x

Doug Green douggreen at douggreenconsulting.com
Tue Apr 10 14:05:56 UTC 2007

Thanks Steven!  I updated coder, but used a slightly different regex.  The
warning references your script.

One question however, I'm not sure that you script works on cases with
functions as arguments.  When I tried to borrow the regex, it didn't work in
the matching phase.  I didn't run any tests, but the regex was off when I
used it in coder.  Does your script properly ignore?

return l(format_date($query->created, 'custom', 'F, Y'),

If it doesn't work, you might want to look at the regex in coder_6x.inc,
which I "think" works better.

Doug Green
Bringing Ideas to Life with Software Artistry and Invention...
Providing open source software political solutions

-----Original Message-----
From: development-bounces at drupal.org [mailto:development-bounces at drupal.org]
On Behalf Of Steven Wittens
Sent: Monday, April 09, 2007 10:31 PM
To: development at drupal.org
Subject: Re: [development] Announce Coder 6.x

> For anyone doing any 6.x development, I spent this afternoon making  
> coder (http://drupal.org/project/coder) compatible with Drupal HEAD  
> (6.x).  I also added a new upgrade review for 5.x to 6.x (which is  
> available in both the HEAD and DRUPAL-5 branches).  The 6.x upgrade  
> review is better than my past upgrade reviews, and (I think)  
> catches everything except the url() and l() argument changes.  (I  
> hope to add these checks before the code freeze.)

Note that the issue for the l() url() change contains a PHP script  
that can automatically convert all l()/url() calls in Drupal code.  
Perhaps you could adapt it?


More information about the development mailing list