search

vojtech-dobes / history.nette.ajax.js

Adds History API support to nette.ajax.js addon!
30 stars 27 forks source link

OnResponseHandler: append presenter parameters to payload #11

Closed vysinsky closed 10 years ago

vysinsky commented 10 years ago

Now when I use $this->redirect('edit', ['id' => 5]) parameter id is lost and url changes to /edit/ - without ID - so refreshing page breaks application. It would be much better to append current presenter's parameters to $payload->redirect

vojtech-dobes commented 10 years ago

I now understand. $this->link('this') doesn't keep parameters that are not arguments of actions, and there is no way to bypass that. I think this PR is correct then. Can you please rebase it and replace - in commit message to :?

vysinsky commented 10 years ago

@vojtech-dobes done

vysinsky commented 10 years ago

@vojtech-dobes sorry, wrong rebase. It is fixed now.