[drupal-devel] [task] Use named constants for drupal_bootstrap()
Issue status update for http://drupal.org/node/27003 Post a follow up: http://drupal.org/project/comments/add/27003 Project: Drupal Version: cvs Component: base system Category: tasks Priority: normal Assigned to: drumm Reported by: drumm Updated by: drumm Status: patch Attachment: http://drupal.org/files/issues/bootstrap_constants.patch (5.94 KB) Named constants are much better to use than strings for this sort of thing. drumm
Issue status update for http://drupal.org/node/27003 Post a follow up: http://drupal.org/project/comments/add/27003 Project: Drupal Version: cvs Component: base system Category: tasks Priority: normal Assigned to: drumm Reported by: drumm Updated by: chx Status: patch It's fine with me. drumm says that if you do make a typo with the named constant a useful error is more likely. chx Previous comments: ------------------------------------------------------------------------ Mon, 18 Jul 2005 23:22:22 +0000 : drumm Attachment: http://drupal.org/files/issues/bootstrap_constants.patch (5.94 KB) Named constants are much better to use than strings for this sort of thing.
Issue status update for http://drupal.org/node/27003 Post a follow up: http://drupal.org/project/comments/add/27003 Project: Drupal Version: cvs Component: base system Category: tasks Priority: normal Assigned to: drumm Reported by: drumm Updated by: drumm Status: patch I am nearly finished preparing a patch to update.php, which has modified lines next to a line modified by this patch. I'd like this resolved first to avoid conflicts. drumm Previous comments: ------------------------------------------------------------------------ Mon, 18 Jul 2005 23:22:22 +0000 : drumm Attachment: http://drupal.org/files/issues/bootstrap_constants.patch (5.94 KB) Named constants are much better to use than strings for this sort of thing. ------------------------------------------------------------------------ Wed, 20 Jul 2005 07:01:20 +0000 : chx It's fine with me. drumm says that if you do make a typo with the named constant a useful error is more likely.
Issue status update for http://drupal.org/node/27003 Post a follow up: http://drupal.org/project/comments/add/27003 Project: Drupal Version: cvs Component: base system Category: tasks Priority: normal Assigned to: drumm Reported by: drumm Updated by: Dries Status: patch Committed to HEAD. Dries Previous comments: ------------------------------------------------------------------------ Mon, 18 Jul 2005 23:22:22 +0000 : drumm Attachment: http://drupal.org/files/issues/bootstrap_constants.patch (5.94 KB) Named constants are much better to use than strings for this sort of thing. ------------------------------------------------------------------------ Wed, 20 Jul 2005 07:01:20 +0000 : chx It's fine with me. drumm says that if you do make a typo with the named constant a useful error is more likely. ------------------------------------------------------------------------ Fri, 22 Jul 2005 23:30:49 +0000 : drumm I am nearly finished preparing a patch to update.php, which has modified lines next to a line modified by this patch. I'd like this resolved first to avoid conflicts.
participants (3)
-
chx -
Dries -
drumm