Views UI preview does not respect changed 'path' option until saving

Created on 20 November 2015, over 9 years ago
Updated 26 January 2025, 3 months ago

Problem/Motivation

When editing a View with a Page display, if the Path is changed in the settings, the change is not reflected in the Preview pane despite Auto preview being checked.
(The display's name and path are displayed below the result, at the bottom of the page)
You must save the view before, only then the setting is used.

Steps to reproduce

  1. Create or edit a view with page display
  2. Make sure auto preview is enabled
  3. Update the page path more than once
  4. Notice the path changes are not shown in the preview

Proposed resolution

Show the latest Path in the preview.

Remaining tasks

  1. Commit patch

User interface changes

API changes

Data model changes

Release notes snippet

🐛 Bug report
Status

Needs work

Version

11.0 🔥

Component

views_ui.module

Created by

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • 🇮🇳India jaydev bhatt Pune, Maharashtra

    i'm working on this.

  • Pipeline finished with Failed
    2 months ago
    Total: 113s
    #435702
  • Pipeline finished with Failed
    2 months ago
    Total: 103s
    #435724
  • Pipeline finished with Failed
    2 months ago
    Total: 117s
    #435745
  • Pipeline finished with Failed
    about 2 months ago
    Total: 115s
    #435839
  • Pipeline finished with Failed
    about 2 months ago
    Total: 102s
    #435891
  • Pipeline finished with Failed
    about 2 months ago
    Total: 95s
    #436364
  • Pipeline finished with Failed
    about 2 months ago
    Total: 104s
    #436371
  • Pipeline finished with Failed
    about 2 months ago
    Total: 102s
    #436603
  • Pipeline finished with Failed
    about 2 months ago
    Total: 128s
    #436640
  • 🇮🇳India jaydev bhatt Pune, Maharashtra

    I have tested the patch, and it works with Drupal 11.1.1. I’ve also updated the patch to align with the latest codebase, as the original was created 4 years ago.

    Additionally, I have resolved all PHPCS and JavaScript linting issues. However, I’m encountering PHPStan errors related to missing return types.

    Locally, all level 1 issues appear to be fixed, but the CI pipeline still fails due to return type errors. Despite updating all methods in the ViewUI file, the number of reported issues has increased.

    I need guidance on resolving this discrepancy between local and pipeline results. Marking this as NR.

  • 🇺🇸United States smustgrave

    The view changes almost seem out of scope but the baseline would need to be regenerated

  • Pipeline finished with Failed
    about 2 months ago
    Total: 493s
    #441816
  • Pipeline finished with Failed
    about 2 months ago
    Total: 563s
    #441856
  • 🇮🇳India mohit_aghera Rajkot

    Fixed all the phpstan issues.
    Resolved merge conflicts and removed changes related to views as this wasn't necessary.
    There is one more failure in the CI, I'm going through that.

    I've hidden the patches in favor of MR approach.

  • Pipeline finished with Failed
    about 2 months ago
    Total: 555s
    #441996
Production build 0.71.5 2024