Enabled vocabularies are displayed with the custom view page correctly, yet disabled vocabularies which should be displayed by the default views taxonomy term page override are completely ignoring views altogether and are being displayed by Drupal's core taxonomy term display.

This may be a duplicate of #1107972: TVI Doesn't overtake core taxonomy/term/% path.

Members fund testing for the Drupal project. Drupal Association Learn more


cwithout’s picture

Status: Active » Needs review
1.39 KB

I've tracked this down to tvi.module line 248 where it checks the status property before setting the $display variable. The default view never has a status set. As a result, in tvi_render_view() on lines 199-202, the variable $display is empty and the default view doesn't render.

The attached patch removes the conditional check on the configuration form that excludes the status box. Since the checkbox was purposefully excluded from that form, I assumed there was probably a reason for that, but I wasn't able to find one. In addition to fixing the issue with the default view never being found, this allows admins to decide whether or not there will be a default view or default to core functionality.

To test, after installing the patch, you'll need to resave your configuration settings at admin/config/user-interface/tvi with the "Use view override" option checked.

elpino’s picture

Ok, great, I'll test as soon as I can.

Pierco’s picture

Maybe the checkbox is not needed on the settings page so you can simply change the submission handler like I did in this patch.

cwithout’s picture

Yes, you could just set the value without also adding an option for site admins to disable the default. But I can't see a reason why we wouldn't want admins to be able to disable the default. If someone only wanted to use the module on a certain vocabulary but use Drupal's taxonomy handling for all others, including the ability to disable this module's default would allow that. Forcing it to enabled forces an admin to use the module's taxonomy handling and override each case where they don't want to use it.

kevinquillen’s picture

Version: 7.x-1.0-beta3 » 7.x-1.x-dev
Status: Needs review » Fixed

Committed to dev

Status: Fixed » Closed (fixed)

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