hook to force page refresh on context url changing is not working

[Michael Hudson-Doyle]

There is a hook in base-layout-macros.pt to force a page refresh if an ajax action causes the URL of a page to change:

            Y.on('lp:context:web_link:changed', function(e) {
                  window.location = e.new_value;
            });

This isn't working. You can reproduce this by editing the task of a bug or by editing the name of a recipe.

[Michael Hudson-Doyle (mwhudson)]

FWIW, poking with firebug suggests that the cache isn't being updated by the edits (LP.cache.context.web_link is unchanged by edits that should change it)

[Michael Hudson-Doyle (mwhudson)]

Some more poking suggests that rev 14634 broke this.

The code checks update events need to be fired for an object in the cache by comparing lp_original_uri of the entry object that is constructed out of the response from the PATCH with the self_link of the object in the cache. Rev 14634 changed things so that the entry is constructed with the _new_ URI as lp_original_uri and so it doesn't match anything in the cache.

[Launchpad QA Bot (lpqabot)]

Fixed in stable r15685 <http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/15685>.