Comment regarding available_themes parameter needs clarifying

[Nick Jones]

The existing comment regarding the available_themes parameter isn't strictly true in that the datatype is actually an array of hashes, not a hash. This can lead to some confusion especially when specifying data in YAML via Hiera.

The code could be refactored to take a hash keyed off a theme name, but the existing code in the template works as long as the source data is in the right format. The quickest win therefore is to update the comment.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/358013

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/358013
Committed: https://git.openstack.org/cgit/openstack/puppet-horizon/commit/?id=1f209355da86e6b696445ba9e0bdedcf58de9d13
Submitter: Jenkins
Branch: master

commit 1f209355da86e6b696445ba9e0bdedcf58de9d13
Author: Nick Jones <email address hidden>
Date: Fri Aug 19 17:50:31 2016 +0100

    Update comment regarding the available_themes parameter

    This commit clarifies the available_themes parameter and its usage so as to save
    any confusion, especially when using Hiera.

    Change-Id: I3bba9ab71eb94bacf262adb71822f7261c7e716c
    Closes-bug: 1615046

[Doug Hellmann (doug-hellmann)]

This issue was fixed in the openstack/puppet-horizon 9.2.0 release.