path_load and drupal_lookup_path are inconsistent when choosing the 'current' path of an entity [#1934086]

While not mentioned in the API documentation, drupal_lookup_path will return the most recently created alias for a given path (based on having the highest pid).

Sister function path_load, will also return a single alias, but there is no explicit preference for one alias over another, so it is likely that it will return the oldest alias.

We should add an orderBy to the latter function, to make the consistent.

Comment	File	Size	Author
#6	drupal-1934086-path_load_order-6.patch	323 bytes	SebCorbin
#6
#1	drupal-1934086-path_load_order-1.patch	362 bytes	ianthomas_uk
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

ianthomas_uk

he/him

English

Brighton, UK

CreditAttribution: ianthomas_uk commented 5 March 2013 at 14:52

Status:

Active

» Needs review

File	Size
drupal-1934086-path_load_order-1.patch	362 bytes

Comment #2

Jody Lynn

she/her

English

CreditAttribution: Jody Lynn commented 10 April 2013 at 21:40

This is a duplicate of #1160764: URL alias load is inconsistent if there are more then one alias but I think you've found the problem and the solution best.

I'm running a live site in which the inability to change aliases and have them take effect consistently is driving editors crazy. I will try this patch and report back.

Comment #3

Jody Lynn

she/her

English

CreditAttribution: Jody Lynn commented 10 April 2013 at 21:56

Steps to reproduce the bug before the patch:

Create two aliases for a node, first alias foo and then alias bar.
View the node. It should show the newer alias, alias bar, as the URL.
Edit the node. It shows the older alias, alias foo in the edit form.
Change the alias when you edit the node. The alias for foo's pid is now changed in the database, but bar still shows when you view the node.

So, basically you can't edit your aliases without understanding the whole system and using the url alias administration page to delete the old one.

Comment #4

SebCorbin CreditAttribution: SebCorbin commented 19 June 2014 at 15:31

Issue summary:	View changes
Status:	Needs review	» Reviewed & tested by the community

This is a big problem when using globalredirect module for example, because the user is never presented the latest url alias and won't understand why.