Back to a sticky state with params changed not working

[molog]

Example steps:

1. User is at the state /books/1
2. User go to state /photos/1
3. User go to state /books/2
4. The page displays content about /books/1 instead of /books/2
   It seems params are not regarded as part of a state.
   I am not sure if this is intended. Is there a way to make above work so that it displays /books/2 ?

[christopherthielen]

I think that should work. Can you put it in a plunkr for me?

[molog]

http://plnkr.co/edit/Gy3ltQM7NLKKQAQslRkF?p=preview
if you click a book under the 'Books' tab and go to 'favorites' tab and click another book, you will see it goes back to the first book.

[BasitAli]

Hi guys.

I just noticed this behaviour in my app after updating both ui-router and ui-router-extras. I jumped in the ct-ui-router-extras.js and tracked the issue down to this,

      // Duplicates logic in $state.transitionTo, primarily to find the pivot state (i.e., the "keep" value)
      function equalForKeys(a, b, keys) {
        if (!keys) {
          keys = [];
          for (var n in a) keys.push(n); // Used instead of Object.keys() for IE8 compatibility
        }

        for (var i = 0; i < keys.length; i++) {
          var k = keys[i];
          if (a[k] != b[k]) return false; // Not '===', values aren't necessarily normalized
        }
        return true;
      }

'keys' is an Object and it fails getting the ".length", thus returning true. Adding

keys = Object.keys(keys)

works for now.

[christopherthielen]

@molog @BasitAli Thanks for the plunk and the analysis.

I updated your plunk with 0.0.12-prerelease http://plnkr.co/edit/YtIbH7emVB4h6YyiMTlT?p=preview

[astyfx]

+1