Don't hack Core

Submitted by AjK on 12 December 2008 - 12:10am

Here's a good one that tried to sneak past in an install file as part of the module's installation procedure...

  1.   db_add_field($ret, 'node_type', 'menu_settings',
  2.     array<('type' => 'text', 'not null' => FALSE));

Need I say more? The title of this page says it all.

Update, and here's more (this time the application's motivation message rather than even bothering to look at any code)...


Motivation message:
I have modified 2 files, page.inc and book.module to improve the accessibility and I like upload this change. This change are to Drupal 5.

Sorry, I don't think so. Although being both well intentioned and well motivated Contrib is not Core. This is a case of n00b lacking in rtfm< skills.


»
»