Tickets :: [#9717] Searches do not update url hash

6.0.0-beta1

11/4/25

Summary	Searches do not update url hash
Queue	IMP
Queue Version	Git master
Type	Bug
State	Resolved
Priority	2. Medium
Owners	slusarz (at) horde (dot) org
Requester	rui.carneiro (at) portugalmail (dot) net
Created	03/25/2011 (5338 days ago)
Due
Updated	06/09/2011 (5262 days ago)
Assigned
Resolved	06/09/2011 (5262 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	Yes

06/09/2011 06:32:16 AM	Michael Slusarz	Comment #5 Assigned to Michael Slusarz State ⇒ Resolved	Reply to this comment
No real reason not to allow it. Only downside: quicksearches are not allowed to be edited. But that is acceptable.

06/09/2011 06:31:24 AM	Git Commit	Comment #4	Reply to this comment
Changes have been made in Git for this ticket: ~~Request #9717~~: Allow searches to update URL hash in dynamic view 1 files changed, 1 insertions(+), 3 deletions(-) http://git.horde.org/horde-git/-/commit/d35a7938adf5b201f1c50abd84975318702e9306

03/25/2011 09:07:29 PM	rui (dot) carneiro (at) portugalmail (dot) net	Comment #3	Reply to this comment
This was intentional, although I can't remember why off the top of my head. Maybe because sooner or later that url would be invalid? But anyway, "example.com/imp/#null" its not a good url after a search :)

03/25/2011 06:53:59 PM	Michael Slusarz	Comment #2	Reply to this comment
This was intentional, although I can't remember why off the top of my head.

03/25/2011 06:41:14 PM	rui (dot) carneiro (at) portugalmail (dot) net	Comment #1 Priority ⇒ 2. Medium New Attachment: 0001-We-should-update-hash-on-search-too.patch Patch ⇒ Yes Milestone ⇒ Summary ⇒ Searches do not update url hash Type ⇒ Bug State ⇒ Unconfirmed Queue ⇒ IMP	Reply to this comment
Removed some code so if the current view its a search it will fall through to the general code.