Project

General

Profile

Bug #7615

Unable to create a non-taxonomy role filter when current context set

Added by Thomas McKay almost 5 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Organizations and Locations
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

As admin, visit try creating a new role filter in the UI for resource type Role. After submit no error message is displayed but the create page remains (versus normal behavior of returning to full list of filters). Looking at log an error is present of "Failed to save: Organization ids You can't assign organizations to this resource".


Related issues

Related to Foreman - Bug #5541: Filter of resource type Organization can result in error condition when trying to access organization resourcesClosed2014-05-01

Associated revisions

Revision 76fca93b (diff)
Added by Marek Hulán over 4 years ago

Fixes #7615 - filter ignore current context

Revision d155f473 (diff)
Added by Marek Hulán over 4 years ago

Fixes #7615 - filter ignore current context

(cherry picked from commit 76fca93b0093e9b4174d1a888ffca9f3fcb886c9)

History

#1 Updated by Thomas McKay almost 5 years ago

  • Bugzilla link set to 1145814

#2 Updated by Dominic Cleal almost 5 years ago

  • Related to Bug #5541: Filter of resource type Organization can result in error condition when trying to access organization resources added

#3 Updated by Dominic Cleal almost 5 years ago

  • Subject changed from unable to create a role filter to Unable to create a non-taxonomy role filter when current context set

Only happens when multi-org's enabled and a current context is set, then the current context's added to the filter.

Looks like the add_current_* methods aren't being overridden correctly, they're overridden as class methods when they're actually instance methods.

#4 Updated by The Foreman Bot over 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2291 added
  • Pull request deleted ()

#5 Updated by Marek Hulán over 4 years ago

  • Category changed from Authorization to Organizations and Locations
  • Assignee set to Marek Hulán

Thanks Dom, fixed the scope of methods.

#6 Updated by Dominic Cleal over 4 years ago

  • Legacy Backlogs Release (now unused) set to 28

#7 Updated by Marek Hulán over 4 years ago

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

Also available in: Atom PDF