Project

General

Profile

Actions

Bug #6065

closed

Inverting the admin flag has no effect on menu cache

Added by Greg Sutcliffe over 10 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Category:
Web Interface
Target version:
Difficulty:
easy
Triaged:
Fixed in Releases:
Found in Releases:

Description

The TopbarSweeper cache clearing system only ever acts on User.current. This means if an admin toggles the admin flag on another user/group, that user's topbar does not refresh and show updated entries until the cache naturally times out (or is cleared with =foreman-rake tmp:cache:clear=).

Steps to reproduce:
1. Create a user without admin flag or roles
2. In a separate browser sesson, log in with that user.
=> observe they can only see Hosts and Bookmarks
3. Create a user group, give it the admin flag, and assign the above user to it
4. Refresh the user's page

Expected results:
The user can now see all menu entries on the topbar

Actual result:
The user still only sees Hosts and Bookmarks


Related issues 2 (0 open2 closed)

Related to Foreman - Bug #6831: When creating a new organization as an admin, the switcher doesn't list itClosedTomer Brisker07/30/2014Actions
Related to Foreman - Bug #6873: Error during db:seed from 1.4 to 1.6: undefined method `expire_topbar_cache' for nil:NilClassClosedDominic Cleal08/01/2014Actions
Actions #1

Updated by Greg Sutcliffe over 10 years ago

  • Status changed from New to Assigned
  • Assignee set to Greg Sutcliffe
  • Target version set to 1.8.2
  • Difficulty set to easy
Actions #2

Updated by Greg Sutcliffe over 10 years ago

  • Status changed from Assigned to Ready For Testing
Actions #3

Updated by marcelo veglienzone over 10 years ago

I can confirm the patch works as expected.

Actions #4

Updated by Daniel Lobato Garcia over 10 years ago

  • Bugzilla link set to https://bugzilla.redhat.com/show_bug.cgi?id=1105251
Actions #5

Updated by Anonymous over 10 years ago

  • Target version changed from 1.8.2 to 1.8.1
Actions #6

Updated by Dominic Cleal over 10 years ago

  • Translation missing: en.field_release set to 18
Actions #7

Updated by Anonymous over 10 years ago

  • Target version changed from 1.8.1 to 1.8.0
Actions #8

Updated by The Foreman Bot over 10 years ago

  • Pull request https://github.com/theforeman/foreman/pull/1495 added
Actions #9

Updated by Greg Sutcliffe over 10 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #10

Updated by Dominic Cleal over 10 years ago

  • Related to Bug #6831: When creating a new organization as an admin, the switcher doesn't list it added
Actions #11

Updated by Dominic Cleal over 10 years ago

  • Related to Bug #6873: Error during db:seed from 1.4 to 1.6: undefined method `expire_topbar_cache' for nil:NilClass added
Actions

Also available in: Atom PDF