Closed (fixed)
Project:
Views (for Drupal 7)
Version:
6.x-2.8
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
20 Apr 2009 at 14:49 UTC
Updated:
10 Dec 2011 at 19:50 UTC
Jump to comment: Most recent file
Comments
Comment #1
dawehneri can reproduce the error:
what whould be the logic to solve this:
if there is set a destination
- check which display the user wanted to edit
- if the page of this changed display changes from the page before redirect to the new page
- else do nothing.
is this true?
Comment #2
esmerel commentedThat logic seems reasonable to me; this still happens in the most recent version.
Comment #3
MatthijsG commentedHere to. Changed a path from date to calender and saved it. VIews send me to the old /date.
Comment #4
iamjon commentedI'm assigning this to dereine and chaging the title to reflect that it's problem with admin_links
Comment #5
dawehnerComment #6
dawehnerHere is a patch.
Comment #7
dawehnerIt would be cool if you could test this patch iamjon.
Basically: Edit a view in the "frontend" by clicking "edit view" with a page display.
Then change the path of the page display.
Additional test a view with multiple page displays and change the path of another page display than the first one.
Comment #8
iamjon commentedTested against the latest 2.x head and it worked great. I had troubles with 2.x dev and 3.x dev..but i think that might have been user error.
Comment #9
iamjon commentedComment #10
merlinofchaos commentedDestinate isn't a word, though I like it. Try "Direct"
Also, this patch overwrites $destination if the path changed without caring what the $destination was. If the destination was the list of views, and you change the path, you don't go back to the list.
We must test to see if $destination == old path.
Comment #11
iamjon commentedAssigning to dereine to review
Comment #12
dawehnerFixed.
Also fixed the setting of the destination mentioned by earl.
Comment #13
dawehnerCommitted to both 6.x-3.x and 7.x-3.x