[drupal-devel] [feature] enable arg() to return all request
arguments
Dries
drupal-devel at drupal.org
Sun Apr 24 13:03:39 UTC 2005
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.
More information about the drupal-devel
mailing list