Issue status update for http://drupal.org/node/20902 Project: Drupal Version: cvs Component: base system Category: feature requests Priority: normal Assigned to: dikini Reported by: dikini Updated by: Steven Status: patch As far as I know, this would be equivalent to just using $_GET['q'], which we use elsewhere. It is certainly nicer in code, but you should also update the places where this is used then. Steven Previous comments: ------------------------------------------------------------------------ April 20, 2005 - 10:33 : dikini Attachment: http://drupal.org/files/issues/bootstrap_2.patch (951 bytes) Although not essential, it would be good if arg() can return all request aruments. It is a minor enhancement, but IMO encourages consistency. ------------------------------------------------------------------------ April 20, 2005 - 10:34 : dikini Sorry, moved the version to cvs, it is applicable, but not really relevan for 4.6

Issue status update for http://drupal.org/node/20902 Project: Drupal Version: cvs Component: base system Category: feature requests Priority: normal Assigned to: dikini Reported by: dikini Updated by: Dries Status: patch There is no problem with using $_GET['q']. I'm not tempted to commit this patch. Dries Previous comments: ------------------------------------------------------------------------ April 20, 2005 - 10:33 : dikini Attachment: http://drupal.org/files/issues/bootstrap_2.patch (951 bytes) Although not essential, it would be good if arg() can return all request aruments. It is a minor enhancement, but IMO encourages consistency. ------------------------------------------------------------------------ April 20, 2005 - 10:34 : dikini Sorry, moved the version to cvs, it is applicable, but not really relevan for 4.6 ------------------------------------------------------------------------ April 20, 2005 - 13:26 : Steven As far as I know, this would be equivalent to just using $_GET['q'], which we use elsewhere. It is certainly nicer in code, but you should also update the places where this is used then. ------------------------------------------------------------------------ April 20, 2005 - 15:07 : dikini That's true. This is mainly a utility/readability concern. I can do that, but it is a bit of an effort, and since it won't break any existing functionality, I haven't even though of doing it. I thoght small patches are better :) There are 64 uses of $_GET currently in core, and each of them should be considered with care. There is a long term possibility, which might be worth exploring - capturing info not only from GET, but POST, PUT, ... requests.