Problem/Motivation

When the project browser clear storage option was added, I don't think project browser had its own cache.

Now it does, so it might be useful to add a "clear project browser cache" option.

Steps to reproduce

N/A

Proposed resolution

Add a Project Browser->Clear cache option.

Remaining tasks

  1. In the menu deriver, add a "clear cache" option above "clear storage", if project browser module present.
  2. In routing.yml, add a route for clearing project browser storage.
  3. In NavigationProjectBrowserController, add a function to call the deleteAll() method of the "cache.project_browser" service.
  4. Add a test case for clearing project browser cache.

User interface changes

New clear cache option.

API changes

None.

Data model changes

None.

Issue fork navigation_extra_tools-3569483

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:

About issue forks

Comments

lostcarpark created an issue. See original summary.

nickolaj made their first commit to this issue’s fork.

nickolaj opened merge request !35

nickolaj’s picture

nickolaj
Primary language Ukrainian
commented
Status: Active » Needs review

Added "Clear cache" option to Project Browser menu that uses `cache.project_browser` service's `deleteAll()` method. Includes routing, menu deriver update, and functional test.

nickolaj’s picture

nickolaj
Primary language Ukrainian
commented

Added "Clear cache" option to Project Browser menu that uses `cache.project_browser` service's `deleteAll()` method. Includes routing, menu deriver update, and functional test.

lostcarpark’s picture

lostcarpark commented
Status: Needs review » Needs work

I've made a suggestion to change the permission on the new menu option, which will also need a change to the parent.

The "Upgrade" test is failing for "previous major". I think that's unrelated to this change, but I'd like to understand why.