Before this commit, attempting to delete a template that contains a repository generated the following error notice: "undefined method `eq' for nil:NilClass" The issue here is that the migrate script for the join table contains "id => false"; however, for models defined with ":has_many through" need to have an id specified on the join table.
This commit will ensure that a given tdl export does not list the same repo multiple times. Previously, this could happen if the user added a product to the template as well as an individual repo from that product.
Prior to this commit, when the user created a system template, if they added a product to a template (and that product contained at least one repo that had a distribution associated with it), the user could associate a distribution with the template. However, if the user instead added the repository to the template (instead of the product), the user could not associate a distribution with the template. I.e. there had to be a product. With this commit, if the template has a product and/or repo associated with it and they have a distro, the user can select it.
…t it in the Organizations / Subscriptions since it shows the bonus pools created as Virtual
Puppet step cpsetup can fail on slow machines sometimes due to default Puppet timeout value.