The order of operation inside url() changed in D7, meaning that hook_domainpath() needs a refactor.

CommentFileSizeAuthor
#8 940762-domain-path-new.patch4.7 KBagentrickard
#7 940762-domain-path-new.patch4.67 KBagentrickard
#7 940762-path-HEAD.patch4.19 KBagentrickard
#6 940762-path-HEAD.patch4.18 KBagentrickard
#6 940762-domain-path-new.patch4.66 KBagentrickard
#5 940762-path-followup.patch1.08 KBagentrickard
#4 940762-path-followup.patch865 bytesagentrickard
#2 940762-path.patch3.39 KBagentrickard
#1 940762-path.patch2.88 KBagentrickard

Comments

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Active » Needs review
StatusFileSize
new 940762-path.patch2.88 KB

And a patch.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
StatusFileSize
new 940762-path.patch3.39 KB

Revised.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented

This affects domain_get_uri() as well.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
StatusFileSize
new 940762-path-followup.patch865 bytes

Follow-up for #3.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
StatusFileSize
new 940762-path-followup.patch1.08 KB

And another.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
StatusFileSize
new 940762-domain-path-new.patch4.66 KB
new 940762-path-HEAD.patch4.18 KB

Newer patches, against HEAD and 7--2.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
StatusFileSize
new 940762-path-HEAD.patch4.19 KB
new 940762-domain-path-new.patch4.67 KB

New patches, which incorporate #921984: Fatal error: undefined function filter_xss_bad_protocol().

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Needs review » Fixed
StatusFileSize
new 940762-domain-path-new.patch4.7 KB

Fixed with a slight optimization.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

indika16’s picture

indika16 commented
Issue summary: View changes

I receive the following warning, will this change the format of the page if I continue as is?

Domain path article/integrating-lgbt-expats-in-a-global-economy matches an existing domain path alias for http://www.relo.crown.stag.acquia.infotexstaging.net/.