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@drupal.org [mailto:development-bounces@drupal.org] On Behalf Of Steven Wittens Sent: Monday, April 09, 2007 10:31 PM To: development@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