Issue status update for http://drupal.org/node/21379 Post a follow up: http://drupal.org/project/comments/add/21379 Project: Drupal Version: cvs Component: other Category: feature requests Priority: normal Assigned to: Anonymous Reported by: mathias Updated by: killes@www.drop.org -Status: patch (code needs review) +Status: patch (code needs work) I am not convinced this is neccessary. the devel output from devel.module is sometimes a bit annoying, I admit that, but it is just one mysql statement to switch it off. :) For me this is easier than to type a get parameter into the location bar. If at all a devel version of variable_get would be nice. The review brought to you by Patch Bingo! killes@www.drop.org Previous comments: ------------------------------------------------------------------------ Tue, 26 Apr 2005 20:05:57 +0000 : mathias Attachment: http://drupal.org/files/issues/devel_GET_params_core.patch (2.38 KB) This patch allows admins who are using devel.module to optionally toggle the page timer, query logger and page redirection for each request using GET parameters rather than navigating to admin/settings/devel and flipping the global settings back and forth. I added this since I usually don't need to print out this information for every page. Now I just use: http://example.com/index.php?q=node&dev_query=1 for the request I'm interested in. Two other params can also be toggled: dev_timer and dev_redirect. ------------------------------------------------------------------------ Tue, 26 Apr 2005 22:15:22 +0000 : mathias Attachment: http://drupal.org/files/issues/devel_GET_params_module.patch (2.82 KB) And the patch for devel.module ------------------------------------------------------------------------ Sat, 07 May 2005 04:09:57 +0000 : Steven Perhaps we should move the trinary operator and variable_get into a small function: it would be cleaner in the code. ------------------------------------------------------------------------ Sat, 07 May 2005 04:10:54 +0000 : Steven In fact, why not create a copy of variable_get which gets its value from the database, but overrides it with a get parameter if it exists? That would be the cleanest, it can be used in devel.module as well as elsewhere... ------------------------------------------------------------------------ Sat, 07 May 2005 13:41:38 +0000 : moshe weitzman thats a good idea, although some implementations might want only certain users (i.e. permission check) to be able to override via _GET param. I suppose that could be handled in the calling code.