{"action":"created","issue":{"url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3919","labels_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3919/labels{/name}","comments_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3919/comments","events_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3919/events","html_url":"https://github.com/ManageIQ/manageiq/pull/3919","id":101661007,"number":3919,"title":"[WIP] Tenants Basic CRUD UI","user":{"login":"h-kataria","id":3450808,"avatar_url":"https://avatars.githubusercontent.com/u/3450808?v=3","gravatar_id":"","url":"https://api.github.com/users/h-kataria","html_url":"https://github.com/h-kataria","followers_url":"https://api.github.com/users/h-kataria/followers","following_url":"https://api.github.com/users/h-kataria/following{/other_user}","gists_url":"https://api.github.com/users/h-kataria/gists{/gist_id}","starred_url":"https://api.github.com/users/h-kataria/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/h-kataria/subscriptions","organizations_url":"https://api.github.com/users/h-kataria/orgs","repos_url":"https://api.github.com/users/h-kataria/repos","events_url":"https://api.github.com/users/h-kataria/events{/privacy}","received_events_url":"https://api.github.com/users/h-kataria/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ManageIQ/manageiq/labels/enhancement","name":"enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/ManageIQ/manageiq/labels/ui","name":"ui","color":"5319e7"},{"url":"https://api.github.com/repos/ManageIQ/manageiq/labels/wip","name":"wip","color":"eeeeee"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-08-18T13:54:09Z","updated_at":"2015-08-22T08:12:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ManageIQ/manageiq/pulls/3919","html_url":"https://github.com/ManageIQ/manageiq/pull/3919","diff_url":"https://github.com/ManageIQ/manageiq/pull/3919.diff","patch_url":"https://github.com/ManageIQ/manageiq/pull/3919.patch"},"body":""},"comment":{"url":"https://api.github.com/repos/ManageIQ/manageiq/issues/comments/133653234","html_url":"https://github.com/ManageIQ/manageiq/pull/3919#issuecomment-133653234","issue_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/3919","id":133653234,"user":{"login":"miq-bot","id":7806257,"avatar_url":"https://avatars.githubusercontent.com/u/7806257?v=3","gravatar_id":"","url":"https://api.github.com/users/miq-bot","html_url":"https://github.com/miq-bot","followers_url":"https://api.github.com/users/miq-bot/followers","following_url":"https://api.github.com/users/miq-bot/following{/other_user}","gists_url":"https://api.github.com/users/miq-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/miq-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/miq-bot/subscriptions","organizations_url":"https://api.github.com/users/miq-bot/orgs","repos_url":"https://api.github.com/users/miq-bot/repos","events_url":"https://api.github.com/users/miq-bot/events{/privacy}","received_events_url":"https://api.github.com/users/miq-bot/received_events","type":"User","site_admin":false},"created_at":"2015-08-22T08:12:17Z","updated_at":"2015-08-22T08:12:17Z","body":"<rubocop />Checked commits https://github.com/h-kataria/manageiq/commit/841c943c272e0f62b5cd240eefde6e55344f4427 .. https://github.com/h-kataria/manageiq/commit/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586 with rubocop 0.32.1 and haml-lint 0.13.0\n17 files checked, 30 offenses detected\n\n**app/controllers/application_controller.rb**\n- [ ] :small_blue_diamond: - [Line 1312](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/application_controller.rb#L1312), Col 121 - [Metrics/LineLength](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Metrics/LineLength) - Line is too long. [123/120]\n\n**app/controllers/ops_controller.rb**\n- [ ] :large_orange_diamond: - [Line 646](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/ops_controller.rb#L646), Col 28 - [Style/MultilineTernaryOperator](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Style/MultilineTernaryOperator) - Avoid multi-line ?: (the ternary operator); use `if`/`unless` instead.\n\n**app/controllers/ops_controller/ops_rbac.rb**\n- [ ] :large_orange_diamond: - [Line 93](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/ops_controller/ops_rbac.rb#L93), Col 3 - [Style/Alias](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Style/Alias) - Use `alias_method` instead of `alias`.\n- [ ] :large_orange_diamond: - [Line 139](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/ops_controller/ops_rbac.rb#L139), Col 53 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Style/SpaceAroundOperators) - Operator `:` should be surrounded with a single space.\n- [ ] :large_orange_diamond: - [Line 276](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/ops_controller/ops_rbac.rb#L276), Col 23 - [Style/BlockDelimiters](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Style/BlockDelimiters) - Avoid using `{...}` for multi-line blocks.\n- [ ] :large_orange_diamond: - [Line 283](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/ops_controller/ops_rbac.rb#L283), Col 44 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Style/SpaceAroundOperators) - Operator `%` should be surrounded with a single space.\n- [ ] :large_orange_diamond: - [Line 684](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/ops_controller/ops_rbac.rb#L684), Col 50 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Style/SpaceAroundOperators) - Surrounding space missing for operator `=>`.\n- [ ] :large_orange_diamond: - [Line 774](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/ops_controller/ops_rbac.rb#L774), Col 11 - [Style/CaseIndentation](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Style/CaseIndentation) - Indent `when` as deep as `case`.\n- [ ] :large_orange_diamond: - [Line 788](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/ops_controller/ops_rbac.rb#L788), Col 7 - [Style/CaseIndentation](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Style/CaseIndentation) - Indent `when` as deep as `case`.\n- [ ] :small_blue_diamond: - [Line 95](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/ops_controller/ops_rbac.rb#L95), Col 3 - [Metrics/AbcSize](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Metrics/AbcSize) - Assignment Branch Condition size for rbac_tenant_edit is too high. [83.26/15]\n- [ ] :small_blue_diamond: - [Line 95](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/ops_controller/ops_rbac.rb#L95), Col 3 - [Metrics/CyclomaticComplexity](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Metrics/CyclomaticComplexity) - Cyclomatic complexity for rbac_tenant_edit is too high. [13/6]\n- [ ] :small_blue_diamond: - [Line 95](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/ops_controller/ops_rbac.rb#L95), Col 3 - [Metrics/MethodLength](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Metrics/MethodLength) - Method has too many lines. [47/25]\n- [ ] :small_blue_diamond: - [Line 95](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/ops_controller/ops_rbac.rb#L95), Col 3 - [Metrics/PerceivedComplexity](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Metrics/PerceivedComplexity) - Perceived complexity for rbac_tenant_edit is too high. [13/7]\n- [ ] :small_blue_diamond: - [Line 103](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/ops_controller/ops_rbac.rb#L103), Col 121 - [Metrics/LineLength](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Metrics/LineLength) - Line is too long. [145/120]\n- [ ] :small_blue_diamond: - [Line 166](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/ops_controller/ops_rbac.rb#L166), Col 3 - [Metrics/AbcSize](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Metrics/AbcSize) - Assignment Branch Condition size for tenant_set_record_vars is too high. [17.03/15]\n- [ ] :small_blue_diamond: - [Line 270](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/ops_controller/ops_rbac.rb#L270), Col 3 - [Metrics/AbcSize](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Metrics/AbcSize) - Assignment Branch Condition size for rbac_tenant_delete is too high. [46.44/15]\n- [ ] :small_blue_diamond: - [Line 270](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/ops_controller/ops_rbac.rb#L270), Col 3 - [Metrics/PerceivedComplexity](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Metrics/PerceivedComplexity) - Perceived complexity for rbac_tenant_delete is too high. [8/7]\n- [ ] :small_blue_diamond: - [Line 1068](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/ops_controller/ops_rbac.rb#L1068), Col 121 - [Metrics/LineLength](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Metrics/LineLength) - Line is too long. [150/120]\n- [ ] :small_blue_diamond: - [Line 1069](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/controllers/ops_controller/ops_rbac.rb#L1069), Col 121 - [Metrics/LineLength](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Metrics/LineLength) - Line is too long. [147/120]\n\n**app/views/ops/_rbac_group_details.html.haml**\n- [ ] :red_circle: **Warn** - [Line 56](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/views/ops/_rbac_group_details.html.haml#L56), Col  -  - Line is too long. [124/120]\n\n**app/views/ops/_tenant_form.html.haml**\n- [ ] :red_circle: **Warn** - [Line 1](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/app/views/ops/_tenant_form.html.haml#L1), Col  -  - Style/VariableName: Use snake_case for variable names.\n\n**spec/controllers/ops_controller/ops_rbac_spec.rb**\n- [ ] :large_orange_diamond: - [Line 15](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/spec/controllers/ops_controller/ops_rbac_spec.rb#L15), Col 32 - [Style/SpaceInsideHashLiteralBraces](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Style/SpaceInsideHashLiteralBraces) - Space inside { detected.\n- [ ] :large_orange_diamond: - [Line 15](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/spec/controllers/ops_controller/ops_rbac_spec.rb#L15), Col 43 - [Style/SpaceInsideHashLiteralBraces](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Style/SpaceInsideHashLiteralBraces) - Space inside { detected.\n- [ ] :large_orange_diamond: - [Line 15](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/spec/controllers/ops_controller/ops_rbac_spec.rb#L15), Col 70 - [Style/SpaceInsideHashLiteralBraces](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Style/SpaceInsideHashLiteralBraces) - Space inside } detected.\n- [ ] :large_orange_diamond: - [Line 15](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/spec/controllers/ops_controller/ops_rbac_spec.rb#L15), Col 72 - [Style/SpaceInsideHashLiteralBraces](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Style/SpaceInsideHashLiteralBraces) - Space inside } detected.\n- [ ] :large_orange_diamond: - [Line 25](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/spec/controllers/ops_controller/ops_rbac_spec.rb#L25), Col 32 - [Style/SpaceInsideHashLiteralBraces](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Style/SpaceInsideHashLiteralBraces) - Space inside { detected.\n- [ ] :large_orange_diamond: - [Line 25](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/spec/controllers/ops_controller/ops_rbac_spec.rb#L25), Col 43 - [Style/SpaceInsideHashLiteralBraces](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Style/SpaceInsideHashLiteralBraces) - Space inside { detected.\n- [ ] :large_orange_diamond: - [Line 25](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/spec/controllers/ops_controller/ops_rbac_spec.rb#L25), Col 70 - [Style/SpaceInsideHashLiteralBraces](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Style/SpaceInsideHashLiteralBraces) - Space inside } detected.\n- [ ] :large_orange_diamond: - [Line 25](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/spec/controllers/ops_controller/ops_rbac_spec.rb#L25), Col 72 - [Style/SpaceInsideHashLiteralBraces](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Style/SpaceInsideHashLiteralBraces) - Space inside } detected.\n- [ ] :large_orange_diamond: - [Line 75](https://github.com/h-kataria/manageiq/blob/7f495d4c5c3c03f8d65a7cd61ef98bc9aac2e586/spec/controllers/ops_controller/ops_rbac_spec.rb#L75), Col 39 - [Style/AlignParameters](http://rubydoc.info/gems/rubocop/0.32.1/RuboCop/Cop/Style/AlignParameters) - Align the parameters of a method call if they span more than one line.\n"}}