[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.


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