When I browsed to an image gallery I use this module with, I noticed that the terms had been reverted back to the normal paths. I'm not sure what caused this, since it worked last time I've added arguments to other view types, used url alias to add aliases and added some blocks. The problem must lie somewhere around there. I can't track down this bug because I can't get the overidden paths back. The links from node pages are also reverted, so it's not my block that's causing the problem. It still works on another view type with another vocabulary though. I know it worked on both before.
I've tried clearing the cache, it won't work either.

CommentFileSizeAuthor
#4 views_tpo-798886-4.patch3.4 KBplach

Comments

plach’s picture

Category: support » bug

You should try to reproduce the bug. You could set up a fresh Drupal + Views installation and start configuring an environment similar to the one you are currently working with.

plach’s picture

Title: Help tracking down bug - overidden terms are reverted » Overidden terms are reverted when saving another View
Priority: Normal » Critical

I reproduced the bug: working on it. Hope to come up with a patch soon.

Toxid’s picture

I've got it working on multiple views now, it seems that it works as long as the last thing you do is to open the argument, press update and then save the view. I discovered this recently so I don't know how stable this solution is, but we'll see. It didn't break when I cleared cache data, so it looks promising.

plach’s picture

Version: 6.x-1.0-alpha1 » 6.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new3.4 KB

@Toxid:

The attached patch should fix the issue. Please test it and report.

Toxid’s picture

I've tested it and it works, I can now save views and TPO doesn't break. There's one problem though. The checkbox to override taxonomy paths in views is gone. It's still there for the views that got it checked already, but not for new views.

plach’s picture

There's one problem though. The checkbox to override taxonomy paths in views is gone. It's still there for the views that got it checked already, but not for new views.

The checkbox will appear only after defining a path for the display and making the arguments override the default ones.

Toxid’s picture

Status: Needs review » Reviewed & tested by the community

Ah, I didn't know I had to override the default view first :) It seems to work flawlessly now. Good job!

plach’s picture

Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thanks :)

Status: Fixed » Closed (fixed)

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