Bug 439952

Summary: Only the "Make certification public" checkbox should mark all previous comments to private
Product: [Retired] Red Hat Ready Certification Tests Reporter: YangKun <ykun>
Component: web siteAssignee: XINSUN <xisun>
Status: CLOSED CURRENTRELEASE QA Contact: Yu Shao <yshao>
Severity: low Docs Contact:
Priority: medium    
Version: betaCC: averma, bxu, rlandry, xiqin, ykun, yshao
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: (2.18rh) 2008/05/23 push Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-06-23 07:44:26 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
The patch to fix this bug none

Description YangKun 2008-04-01 02:53:36 UTC 
Description of problem:

currently there're two checkboxes in our catalog controlling whether to public a
cert: the "Make certification public" and "TAM ACK". a cert can be public only
when both checkboxes were clicked.

and only the "Make certification public" checkbox should mark all previous
comments to private. However this is not true. Current catalog behaviour is: if
"Make certification public" is clicked first, the previous public comments will
NOT be changed to private, they'll be backed to private only when both "Make
certification public" and "TAM ACK" are clicked.
Comment 1 YangKun 2008-04-14 07:05:44 UTC 
turns out that "TAM ACK" will not mark all previous comments to private. the
real problem is: once "Make certification public" is provided, any further
changes(the "Save Changes" was clicked) in the cert will mark all previous
comments to private.

But the expected behaviour should be: only the first time when we "Make
certification public" mark all preivous comments to private, any further changes
in that cert should not change the status of previous comments.
Comment 2 XINSUN 2008-04-14 08:09:35 UTC 
Yk, 
  The patch to fix this bug is attached. and my ~/xisun2 tree on bugdev have
patch  ed it and you can test it as your need.

Best Regards!
Nicho
Comment 3 XINSUN 2008-04-14 08:10:17 UTC 
Created attachment 302309 [details]
The patch to fix this bug
Comment 4 YangKun 2008-04-14 08:44:24 UTC 
Nicho,

That's quick! Thanks :)

I verified this change on -xisun2
(http://bugdev.devel.redhat.com/hwcert-xisun2/show.cgi?id=429952)

It works as expected: after the "Make certification public" is checked, all
further changes(the "Save Changes" was clicked) in the cert won't mark all
previous comments to private(including the "TAM Ack").

Rob, can we get this patch into the live catalog ?

Thanks
-YK
Comment 5 XINSUN 2008-05-14 13:38:31 UTC 
Patch has been summited into cvs.