Removed attr_accessor in order for the methods to be called

The following methods-
:v_derived_storage_used
:min_v_derived_storage_used
:max_v_derived_storage_used

that populate the trending data for datastore were not being called
earlier due to attr_accessor being specified.

https://bugzilla.redhat.com/show_bug.cgi?id=1089625
Removed hand pointer for no-hover class

https://bugzilla.redhat.com/show_bug.cgi?id=1142401
Fixed issues introduced during HAML conversion.

- Fixed syntax error in HAML views.
- Checkboxes in the options boxes were not sending up transaction fixed those as well.
Merge pull request #835 from h-kataria/haml_typo_fixed

Fixed issues introduced during HAML conversion.
Remove no longer needed String#to_datetime patch.

Fixes #826

From: https://bugzilla.redhat.com/show_bug.cgi?id=MANAGEIQ-5898

```
The root cause was that under Rails 2.2 SOAP4r calls a "to_datetime" method on
a string which is a method added by ActiveSupport.  The method was ignoring the
timezone and caused the VC time to be inaccurate.
```

With the patch:

```
irb(main):001:0> require 'patches/active_support_string_patch'
=> false
irb(main):002:0> Time.now.to_s.to_datetime.to_s
=> "2014-10-14T18:04:13-04:00
```

Without the patch:
**Note**: the timezone is no longer ignored, so the patch can be removed:

```
irb(main):001:0> Time.now.to_s.to_datetime.to_s
=> "2014-10-14T18:03:38-04:00"
```
Merge pull request #788 from AparnaKarve/bz1089625_fix_utilization_trend_data_for_datastore

Fixed utilization trend data (Max Used/Avg Used) for datastore
Storage view rewritten to HAML

The obsolete _details.html.erb was removed.
Deleted obsolete cim_base_storage_extent view
Merge pull request #838 from jrafanie/remove_string_to_datetime_patch

Remove no longer needed String#to_datetime patch.
Harden security for requests

- also harden MiqRequest, MiqRequestTask
- support yaml where symbols are stored (vs only strings)
- support yaml where only some values are invalid
  before: replace all values in a hash if any one value was invalid
- support password prefixes (vs whole value)
- ConfigurationEncoder#dump supports same options as load

https://bugzilla.redhat.com/show_bug.cgi?id=1151241
Dont redraw the whole tree when checkbox was checked/unchecked

This was causing delays and was resulting in saving incorrect data when checking/unchecking lots of nodes in the tree very fast.  We dont need to redraw the whole tree to change the text color of checked/unchecked nodes and it's children in the tree to show user the changes in the tree. Fixed to only render the node that was checked/unchecked and it's children to show differences in blue color.

https://bugzilla.redhat.com/show_bug.cgi?id=1153047
https://bugzilla.redhat.com/show_bug.cgi?id=1145799
Merge pull request #799 from kbrock/fix_auth_service_key

Harden security for requests
Merge pull request #795 from skateman/view-cim_base_storage_extent-to-haml

Deleted obsolete cim_base_storage_extent view
Merge pull request #781 from skateman/view-storage-to-haml

Storage view rewritten to HAML
Merge pull request #822 from epwinchell/vm_disk_styling_fix

Removed hand pointer for no-hover class
Merge pull request #852 from h-kataria/dont_redraw_dynatree_when_changing_node_text_color

Dont redraw the whole tree when checkbox was checked/unchecked
Remove VDI references from miqhost
Remove VDI logic from Lib directory
Remove VDI models, specs and other references.
Migration to remove VDI tables