We should use arg() instead.
Actually current_path() is the preferred function compared to arg().
I'm fine with this for the module, but I dont want to touch any of the code in xhprof.inc unless its a bug fix. That file contains hacked up versions of the original non-drupal xhprof ui that are pretty ugly.
xhprof.inc will be removed in the next version in favor of https://github.com/msonnabaum/xhproflib.
The 8.x-1.x was broken without this change, so I committed a fix for 8.x-1.x and 7.x-1.x.
http://drupalcode.org/project/XHProf.git/commit/6bfbbc6 http://drupalcode.org/project/XHProf.git/commit/a4b9863
Automatically closed -- issue fixed for 2 weeks with no activity.
The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.
Comments
Comment #1
dave reidActually current_path() is the preferred function compared to arg().
Comment #2
msonnabaum commentedI'm fine with this for the module, but I dont want to touch any of the code in xhprof.inc unless its a bug fix. That file contains hacked up versions of the original non-drupal xhprof ui that are pretty ugly.
xhprof.inc will be removed in the next version in favor of https://github.com/msonnabaum/xhproflib.
Comment #3
amateescu commentedThe 8.x-1.x was broken without this change, so I committed a fix for 8.x-1.x and 7.x-1.x.
http://drupalcode.org/project/XHProf.git/commit/6bfbbc6
http://drupalcode.org/project/XHProf.git/commit/a4b9863