Project

General

Custom queries

Profile

Actions

Bug #5909

closed

Editing host fails for non-admin user with fact filter

Added by m w almost 11 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Users, Roles and Permissions
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

When a non-administrator tries edit anything about a host, this error is displayed. This includes, but is not limited to, adding or removing a class from a host, changing a parameter, or adding a comment. Role filters are attached as a screenshot.

ActiveRecord::ReadOnlyRecord
ActiveRecord::ReadOnlyRecord
app/models/concerns/foreman/sti.rb:29:in `save_with_type'
app/controllers/hosts_controller.rb:117:in `block in update'
app/models/taxonomy.rb:41:in `block in no_taxonomy_scope'
app/models/taxonomy.rb:48:in `block (2 levels) in as_taxonomy'
app/models/concerns/foreman/thread_session.rb:143:in `as_location'
app/models/taxonomy.rb:47:in `block in as_taxonomy'
app/models/concerns/foreman/thread_session.rb:108:in `as_org'
app/models/taxonomy.rb:46:in `as_taxonomy'
app/models/taxonomy.rb:40:in `no_taxonomy_scope'
app/controllers/hosts_controller.rb:109:in `update'
app/models/concerns/foreman/thread_session.rb:33:in `clear_thread'
lib/middleware/catch_json_parse_errors.rb:9:in `call'


Files


Related issues 1 (1 open0 closed)

Blocks Foreman - Tracker #4552: New permissions/authorization system issuesNew

Actions
#4

Updated by Joseph Magen over 10 years ago

  • Priority changed from High to Normal
#6

Updated by Dominic Cleal about 10 years ago

  • Blocks Tracker #4552: New permissions/authorization system issues added
#7

Updated by Dominic Cleal about 10 years ago

  • Category set to Users, Roles and Permissions
#9

Updated by Dominic Cleal about 10 years ago

  • Subject changed from removing class from a host fails for non-admin user to Editing host fails for non-admin user with fact filter
  • Status changed from New to Assigned
  • Assignee set to Dominic Cleal
#10

Updated by The Foreman Bot about 10 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2286 added
  • Pull request deleted ()
#11

Updated by Marek Hulán about 10 years ago

  • Translation missing: en.field_release set to 35
#12

Updated by Dominic Cleal about 10 years ago

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

Updated by Dominic Cleal about 10 years ago

  • Translation missing: en.field_release changed from 35 to 50
Actions

Also available in: Atom PDF