Project

General

Profile

Actions

Bug #16828

closed

Cloned roles should have `builtin` value 0

Added by Swapnil Abnave over 7 years ago. Updated over 5 years ago.

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

Description

Ensure that `role.builtin` for cloned roles is always 0.

This is because `role.builtin` is being assigned a boolean value whereas the PG datatype is INTEGER.

https://github.com/theforeman/foreman/blob/develop/app/controllers/roles_controller.rb#L97

The attached bugzilla is not reproducable as upstream is fixed by commit: https://github.com/theforeman/foreman/blob/12612809c5deb885b13ceaf36b6a147c21688eb5/db/migrate/20160715131352_set_role_builtin_default.rb


Related issues 2 (1 open1 closed)

Related to Foreman - Feature #18321: Provide migration to fix cloned rolesClosedLukas Zapletal01/31/2017Actions
Related to Foreman - Refactor #15621: Change role builtin attribute to booleanNew07/08/2016Actions
Actions #1

Updated by The Foreman Bot over 7 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/3925 added
Actions #2

Updated by Dominic Cleal over 7 years ago

  • Category set to Users, Roles and Permissions
  • translation missing: en.field_release set to 189
Actions #3

Updated by Swapnil Abnave over 7 years ago

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

Updated by Swapnil Abnave over 7 years ago

  • Target version set to 1.15.6
Actions #5

Updated by Swapnil Abnave over 7 years ago

  • Target version changed from 1.15.6 to 1.15.1
Actions #6

Updated by Lukas Zapletal about 7 years ago

  • Related to Feature #18321: Provide migration to fix cloned roles added
Actions #7

Updated by Lukas Zapletal about 7 years ago

Workaround (for googlers):

echo "Role.where(:builtin => nil).update_all(:builtin => 0)" | foreman-rake console

To fix all cloned roles.

Actions #8

Updated by Dominic Cleal almost 7 years ago

  • Related to Refactor #15621: Change role builtin attribute to boolean added
Actions

Also available in: Atom PDF