|Summary:||Missing the "ASSOCIATE..." and "DISSASSOCIATE SELECTED" buttons within a repository table "Content Sources Associated with This Repository"|
|Product:||[Other] RHQ Project||Reporter:||John Sefler <jsefler>|
|Component:||Content||Assignee:||Simeon Pinder <spinder>|
|Status:||CLOSED CURRENTRELEASE||QA Contact:||Corey Welton <cwelton>|
|Version:||1.4||CC:||cwelton, ian.springer, smohan, whayutin|
|Fixed In Version:||2.4||Doc Type:||Bug Fix|
|Doc Text:||Story Points:||---|
|Last Closed:||2010-08-12 16:56:10 UTC||Type:||---|
|oVirt Team:||---||RHEL 7.3 requirements from Atomic Host:|
|Bug Depends On:|
Description John Sefler 2010-04-26 18:52:34 UTC
Description of problem: In the current build, there is no way to associate a content source with a repository. The "ASSOCIATE..." and "DISSASSOCIATE SELECTED" buttons are missing from the "Content Sources Associated with This Repository" table within an opened repository. Version-Release number of selected component (if applicable): RHQ version: 3.0.0-SNAPSHOT build number: 4fd30d0 How reproducible: Steps to Reproduce: 1. Administration > Content > Repositories 2. open an existing repo or create a new one and open it Actual results: The "Content Sources Associated with This Repository" table is missing the "ASSOCIATE..." and "DISSASSOCIATE SELECTED" buttons
Comment 1 Ian Springer 2010-04-29 14:18:03 UTC
The associate/disassociate functionality was intentionally disabled (commented out), since it is needed for JON. For JON, we only need to support the use case of one content provider (i.e. the CSP or a local mirror of the CSP) per repo. So you can disable the associated tests. You might want to keep them around in case we have new use cases in a future release of JON that require restoring the functionality.
Comment 2 John Sefler 2010-05-03 13:39:49 UTC
Now that you have removed the associate/disassociate functionality, there is no need for the checkboxes. See attached screenshot. This applies to both the "Content Sources Associated with This Repository" table in the selected Repository (i.e. JBoss Patches) page AND the "Repositories" table in the selected Content Source page. Moving back to ON_DEV
Comment 3 John Sefler 2010-05-03 13:40:52 UTC
Created attachment 410997 [details] Now that you have removed the associate/disassociate functionality, there is no need for the checkboxes
Comment 4 Corey Welton 2010-05-03 14:01:05 UTC
The problem with the logic in comment #1 is is that even if we are only supporting a 1:1 content source:repo, you should still be able to create a new content source, a new repo, and associate the two for example, we have the CSP content source/repo preconfigured, but let's say i want to create another one -- perhaps a different CSP login, or one that uses http vs https or... any number of things. Even if I am not trying to associate multiple, new repos with a pre-existing Content source, if we have removed the associate/disassociate buttons, I can no longer connect the two.
Comment 6 Charles Crouch 2010-05-13 19:14:00 UTC
Referring to cswiii's comment. How much effort would it be to add these associate/disassociate buttons back
Comment 7 Ian Springer 2010-05-13 20:47:55 UTC
I just commented them out in the xhtml files, so it would not be much effort.
Comment 9 Simeon Pinder 2010-05-18 01:10:58 UTC
Re-enabled associate/dissassociate buttons. Available in successful master build >= 307, and commit hash: ce0d8ab908a52df1bc7826de06d848c8300a343f
Comment 10 John Sefler 2010-05-18 13:15:55 UTC
One more request... Can you update the InfoBlock messages to spell out the word "repo" -> "repository". Associated  content sources with repo -> Associated  content sources with repository. Disassociated  content sources from repo -> Disassociated  content sources from repository. moving back to ON_DEV
Comment 11 John Sefler 2010-05-18 13:17:13 UTC
Created attachment 414844 [details] Update the InfoBlock messages to spell out the word "repo" -> "repository"
Comment 12 Simeon Pinder 2010-05-21 12:38:54 UTC
Yep. Fixed it there and in a few other places in the code tree. Available in successful master builds >= 332. with commit hash 49e72b33dbb715fb390cdcfe6c64ed727addaa8b
Comment 13 John Sefler 2010-05-25 14:30:37 UTC
Recent build appears to have undone the Re-enablement of the associate/dissassociate buttons committed in comment #9 The recent build which is missing the buttons is a19ca6b which contains the comment #9 commit: $ git rev-list a19ca6b | grep ce0d8ab908a52df1bc7826de06d848c8300a343f ce0d8ab908a52df1bc7826de06d848c8300a343f Moving back to ON_DEV
Comment 14 Simeon Pinder 2010-05-26 09:09:35 UTC
This was caused by a curious reversion of the applied fix after checkin. I re-applied the fix and the error message from the screenshot should be fixed now in successful build of master >= 355, and git hash: bd8ee1ee291e656dfec223110ed51d0cb24d2219.
Comment 15 Corey Welton 2010-05-27 20:41:12 UTC
Comment 16 Corey Welton 2010-08-12 16:56:10 UTC
Mass-closure of verified bugs against JON.