[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'),
"$arg/$query->name");
If it doesn't work, you might want to look at the regex in coder_6x.inc,
which I "think" works better.
Doug Green
904-583-3342
www.douggreenconsulting.com
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?
Steven
More information about the development
mailing list