Fixes #38264 - prevent duplicate entries

[sbernhard]

What are the changes introduced in this pull request?

Prevent to bind to the on ContentLoad multiple times which would lead to duplicate entries in the Lifecycle Environment Selection on the CreateHost page.

Considerations taken when implementing this change?

Unbind to the event and then bind new to prevent duplicate entries.

What are the testing steps for this pull request?

• Use the create host page
• Select the Hostgroup which uses ContentSource and all Katello related stuff
• Unselect (x) the Content Source
• Select the same Content Source
• Try to pick another Hostgroup and Unselect/Select the Content Source for another test
  At all stages, the Lifcecycle Environment Selection should not have duplicate entries.

[sbernhard]

Looks like there is a issue then with the content view. Need to analyze.

[jeremylenz]

Is this fixing the same issue as #11329 ?

[sbernhard]

yes, but mine is using namespaces and clears all events of the given namespace - which is a little bit more readable :)

... and, I was not aware of #11329 - congratulation to the bug fix at @ofedoren ;-)

[ofedoren]

yes, but mine is using namespaces and clears all events of the given namespace - which is a little bit more readable :)

... and, I was not aware of #11329 - congratulation to the bug fix at @ofedoren ;-)

I don't mind any solution as long as it fixes the issue, doesn't introduce new ones and everyone is happy with the approach :)

[lfu]

Tested. LGTM 👍
Before

After

[jeremylenz]

Tests will fail (and thus merging is blocked) until #11120 is merged, probably early next week.