Problem/Motivation

The module currently assumes that the views_ui module is installed. Some functionality and routes rely on UI-specific elements provided by views_ui, such as:

UI-specific admin routes and tabs under the Views UI interface

However, views_ui is an optional module in Drupal core. Sites can use Views without enabling the Views UI module. When views_ui is not enabled, the module’s administrative route and UI integration fail to work correctly.

This creates an unnecessary dependency on views_ui, even though the functionality of this module does not require the Views UI module itself.

The module should work independently of views_ui.

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

n-m-daz created an issue. See original summary.

n-m-daz’s picture

Issue summary: View changes

n-m-daz’s picture

Status: Active » Needs review

Created a fix for this.

n-m-daz’s picture

StatusFileSize
new135.54 KB

Removed:
Dependency on Views UI admin tabs (views_ui.settings_tab).
Usage of Views UI local task integration for the rebuild page.
Assumption that the views_ui module is installed in order to access the module’s administrative interface.

Added:
A standalone admin configuration route at
/admin/config/search/views-url-alias to rebuild the url alias table.

Rebuild page

rachel_norfolk’s picture

Now that is a very fine catch! Code looks reasonable but I’ll see if someone wants to test before I accept.

jan.fetalvero’s picture

Verified that the Views URL Alias rebuild functionality
no longer depends on the Views UI module.

Steps tested:
1. Enabled views_url_alias
2. Disabled views_ui
views_ui

3. Accessed /admin/config/search/views-url-alias

rebuild

Result:
The rebuild page is accessible and works correctly
without Views UI installed.

Also tested with views_ui enabled and functionality
remains unchanged.

works as expected RTBC.

jan.fetalvero’s picture

Status: Needs review » Reviewed & tested by the community

rachel_norfolk’s picture

Status: Reviewed & tested by the community » Fixed

Merged - thank you!!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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