Code clean-up

Fixed a syntax error.

I just ported this to D5 and created a D5 development release. I've updated the code style to my current standards for contrib modules. Please re-submit any additional problems as a new issue. Thanks!

How comes that I do not see a DRUPAL-5 branch?

Attached is a new patch against HEAD. Already thoroughly tested on two productive sites.

This patch does not apply cleanly for me on patching filter_default.info. I get the following filter_default.info.rej file:

It does apply cleanly on filter_default.module. I applied filter_default_3.patch directly to the 5.x-1.x-dev version; should I have applied the other patches in succession first?

Attached patch cleans the coding-style of filter_default adhering to Drupal Coding Standards, and fixes some minor bugs in current HEAD.
Btw: There is still no DRUPAL-5 branch visible in CVS. I have no clue, how you managed to release a development snapshot for 5.x, but for an official release, we need to branch filter_default to DRUPAL-5 and create a DRUPAL-5--1-0 tag afterwards.

These are the included fixes:

• The .info file should declare a package.
• admin/settings/filters/default should display a proper page title.
• !list_link would be the proper placeholder for HTML markup like hyperlinks, however, the string list tab would not be translatable, and thus, only the URL is passed to t() as replacement now.
• This repeatedly used for-condition is executed for each iteration, and thus replaced with a faster and better understandable syntax:
  
  <?php
-  for ($i = 1; $i < count($roles)+1; $i++) {
+  for ($i = 1, $ii = count($roles); $i <= $ii; $i++) {
?>
• Like in my previous patch, I've altered filter_default_admin_defaults_form_validate() for better readability. This time, I additionally included some inline comments for better understanding of the rather complex logic (thoroughly tested).
• The last if-condition in _filter_default_form_alter_filters() needs to additionally check if there are any children to iterate over. I already added this check in my last patch, since I encountered an error message somewhere (unfortunately I do not remember anymore, where).
• Added the missing form_id for user-created blocks to _filter_default_is_new().

#5 applied , it's huge, thanks.

Barry?

Barry has passed the torch. I've been working on issues, and just rolled a 1.0 release of this module to get the ball rolling, but I'll look at these next.