For the body classes, get actual URI instead of relying on drupal_get_path_alias($_GET['q']) [#1835078]

Hello,

I was just working on a site that uses two url aliases for a couple of nodes. This happens because if the node is viewed in one context it would require a different styling. These different contexts are being controlled with the url alias. /about for the normal site and /iframe/about (for example. we are not using iframes).

Problem:
Currently the path (as used in the body classes) are being generated with a call to drupal_get_path_alias. This doesn't get the actual path that is currently being viewed.

Proposed Solution:
Use request_uri to get the actual path that is being requested.

Patch to follow.

Comment	File	Size	Author
#1	put-actual-path-in-body-class-1835078-1.patch	784 bytes	frob

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

frob

English

CreditAttribution: frob commented 8 November 2012 at 02:57

Status:

Active

» Needs review

File	Size
put-actual-path-in-body-class-1835078-1.patch	784 bytes

Here is the patch.

Comment #2

JohnAlbin

he/they/他

English

Taipei, Taiwan

CreditAttribution: JohnAlbin commented 28 July 2013 at 18:35

Status:

Needs review

» Needs work

Using the PHP function request_uri() isn't going to play well with other modules that manipulate the path, like the internationalization module, path alias, etc.

I'm tempted to won't fix this since it is such an edge case. But I'll leave it open to see what comes up.