Cleaning up and adopting the API

Error message

  • Deprecated function: implode(): Passing glue string after array is deprecated. Swap the parameters in drupal_get_feeds() (line 394 of /var/www/pied-piper.ermarian.net/includes/common.inc).
  • Deprecated function: The each() function is deprecated. This message will be suppressed on further calls in menu_set_active_trail() (line 2405 of /var/www/pied-piper.ermarian.net/includes/menu.inc).
My long and arduous journey in becoming a Drupal Wizard can be measured in how little code I have to reinvent. Back in May 2006, when I first began using Drupal, I had so little understanding of the engine that I actually had to hack the path aliasing system to make my own path "menu" API. This klutz of a page (the Grand Endeavor) was finally shut down late in 2006 when I integrated my Drupal sites into a unique code base and had to throw out these dirty hacks - and now the site employs the API properly. The PPP was redesigned as a Drupal site only in November 2006. By then I knew more, but unfortunately I still had a bad grasp of the paging, sorting and tabling API of Drupal. I wrote messy SQL queries, re-invented the entire table renderer as well as the pager. I am currently working on moving PPP closer to the existing API as well as theming functions in general. The upside of thise is that the code gets a lot easier to extend and also upgrade for future Drupal versions. In fact, it is relatively easy to make a searching function now. Which I am doing.