If you click the "edit" link in the hovering menu on a View page, change the path of that page and click save, you get taken to the OLD path which generates a 404. No biggy, but it might as well be in the queue. =)

Comments

dawehner’s picture

i 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?

esmerel’s picture

Version: 6.x-2.x-dev » 6.x-2.8

That logic seems reasonable to me; this still happens in the most recent version.

MatthijsG’s picture

Here to. Changed a path from date to calender and saved it. VIews send me to the old /date.

iamjon’s picture

Title: Changing the path of a page causes temporary 404 » Changing the path of a page causes temporary 404 in the admin_links

I'm assigning this to dereine and chaging the title to reflect that it's problem with admin_links

dawehner’s picture

Assigned: Unassigned » dawehner
dawehner’s picture

Status: Active » Needs review
StatusFileSize
new1.05 KB

Here is a patch.

dawehner’s picture

Assigned: dawehner » iamjon

It 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.

iamjon’s picture

Tested 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.

iamjon’s picture

Status: Needs review » Reviewed & tested by the community
merlinofchaos’s picture

Status: Reviewed & tested by the community » Needs work
+  // Destinate the user to the right url, if the path of the display has changed.

Destinate 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.

iamjon’s picture

Assigned: iamjon » dawehner

Assigning to dereine to review

dawehner’s picture

Status: Needs work » Needs review
StatusFileSize
new1.11 KB

Destinate isn't a word, though I like it. Try "Direct"

Fixed.

Also fixed the setting of the destination mentioned by earl.

dawehner’s picture

Status: Needs review » Fixed

Committed to both 6.x-3.x and 7.x-3.x

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.