Convert Time Profile Editor to Angular

[AparnaKarve]

Purpose or Intent

This PR converts the Time Profile Editor to Angular.
It also includes directives that ensure that at least one day and one hour are checked.
Note that checkboxes have been replaced by bootstrap switches as shown in the screenshots below.

Screenshots:

Before:

After:

With directives in play:

[Fryguy]

@AparnaKarve Something is wrong with the bot...please don't react to the rubocop things just yet.

[AparnaKarve]

@Fryguy Thanks for the heads-up... I was about to address them.

[dclarizio]

@AparnaKarve I think we need to always show the detailed options below the All Days and All Hours option. It makes it easier to, for example, turn on all days, then just turn off Sat and Sun if I want the 5 weekdays. This is how it worked before.

In addition, if All is chosen and one of the detail options below is turned off, then All should turn off. Likewise, if I check all of the detailed options, All would turn on.

[miq-bot]

Checked commits AparnaKarve/manageiq@e7fa872~...7417262 with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1
4 files checked, 48 offenses detected

app/controllers/configuration_controller.rb

• ❗ - Line 397, Col 5 - Style/ConditionalAssignment - Use the return of the conditional for variable assignment and comparison.
• ❗ - Line 408, Col 103 - Style/SpaceInsideBlockBraces - Space missing inside }.
• ❗ - Line 408, Col 55 - Style/NestedTernaryOperator - Ternary operators must not be nested. Prefer if/else constructs instead.
• ❗ - Line 408, Col 7 - Style/ConditionalAssignment - Use the return of the conditional for variable assignment and comparison.
• ❗ - Line 408, Col 92 - Style/SpaceBeforeBlockBraces - Space missing to the left of {.
• ❗ - Line 408, Col 92 - Style/SpaceInsideBlockBraces - Space between { and | missing.
• ❗ - Line 408, Col 92 - Style/SymbolProc - Pass &:to_i as an argument to collect instead of a block.
• ❗ - Line 409, Col 109 - Style/SpaceInsideBlockBraces - Space missing inside }.
• ❗ - Line 409, Col 59 - Style/NestedTernaryOperator - Ternary operators must not be nested. Prefer if/else constructs instead.
• ❗ - Line 409, Col 7 - Style/ConditionalAssignment - Use the return of the conditional for variable assignment and comparison.
• ❗ - Line 409, Col 98 - Style/SpaceBeforeBlockBraces - Space missing to the left of {.
• ❗ - Line 409, Col 98 - Style/SpaceInsideBlockBraces - Space between { and | missing.
• ❗ - Line 409, Col 98 - Style/SymbolProc - Pass &:to_i as an argument to collect instead of a block.
• ❗ - Line 414, Col 11 - Style/AlignHash - Align the elements of a hash literal if they span more than one line.
• ❗ - Line 414, Col 11 - Style/IndentHash - Use 2 spaces for indentation in a hash, relative to the start of the line where the left curly brace is.
• ❗ - Line 416, Col 11 - Style/AlignHash - Align the elements of a hash literal if they span more than one line.
• ❗ - Line 442, Col 9 - Style/IndentHash - Use 2 spaces for indentation in a hash, relative to the start of the line where the left curly brace is.
• ❗ - Line 448, Col 49 - Style/SpaceBeforeBlockBraces - Space missing to the left of {.
• ❗ - Line 448, Col 49 - Style/SpaceInsideBlockBraces - Space between { and | missing.
• ❗ - Line 448, Col 49 - Style/SymbolProc - Pass &:to_i as an argument to collect instead of a block.
• ❗ - Line 448, Col 60 - Style/SpaceInsideBlockBraces - Space missing inside }.
• ❗ - Line 449, Col 52 - Style/SpaceBeforeBlockBraces - Space missing to the left of {.
• ❗ - Line 449, Col 52 - Style/SpaceInsideBlockBraces - Space between { and | missing.
• ❗ - Line 449, Col 52 - Style/SymbolProc - Pass &:to_i as an argument to collect instead of a block.
• ❗ - Line 449, Col 63 - Style/SpaceInsideBlockBraces - Space missing inside }.
• ❗ - Line 451, Col 9 - Style/IndentHash - Use 2 spaces for indentation in a hash, relative to the start of the line where the left curly brace is.
• ❕ - Line 439, Col 3 - Metrics/AbcSize - Assignment Branch Condition size for construct_edit_for_audit is too high. [33.78/20]
• ❕ - Line 461, Col 3 - Metrics/AbcSize - Assignment Branch Condition size for time_profile_form_fields is too high. [43.34/20]

app/views/configuration/_timeprofile_form.html.haml

• ⚠️ - Line 110 - Hash attribute should end with no space before the closing brace
• ⚠️ - Line 110 - Space inside } detected.
• ⚠️ - Line 127 - Hash attribute should end with no space before the closing brace
• ⚠️ - Line 127 - Space inside } detected.
• ⚠️ - Line 145 - Hash attribute should end with no space before the closing brace
• ⚠️ - Line 145 - Space inside } detected.
• ⚠️ - Line 162 - Hash attribute should end with no space before the closing brace
• ⚠️ - Line 162 - Space inside } detected.
• ⚠️ - Line 202 - Hash attribute should end with no space before the closing brace
• ⚠️ - Line 202 - Space inside } detected.
• ⚠️ - Line 32 - Prefer to_s over string interpolation.
• ⚠️ - Line 32 - Prefer to_s over string interpolation.
• ⚠️ - Line 32 - Use hash rockets syntax.
• ⚠️ - Line 33 - Line is too long. [174/160]
• ⚠️ - Line 49 - Hash attribute should end with no space before the closing brace
• ⚠️ - Line 49 - Space inside } detected.
• ⚠️ - Line 64 - Hash attribute should end with no space before the closing brace
• ⚠️ - Line 64 - Space inside } detected.
• ⚠️ - Line 94 - Hash attribute should end with no space before the closing brace
• ⚠️ - Line 94 - Space inside } detected.

[AparnaKarve]

@dclarizio Please retest/review. Thanks.
CI error looks unrelated to the PR changes.