Bug 444997

Summary: for every update the catalog shows postdate group removed
Product: [Retired] Red Hat Ready Certification Tests Reporter: Rob Landry <rlandry>
Component: web siteAssignee: XINSUN <xisun>
Status: CLOSED CURRENTRELEASE QA Contact: Yu Shao <yshao>
Severity: low Docs Contact:
Priority: high    
Version: 2CC: bxu, dwa, xiqin, xisun, 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:29:14 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 and do some code-optimization
none
Delete the error mechanism when it can't figure out the group, becuase the group exists by default in bug db and can always be figured out . none

Description Rob Landry 2008-05-02 17:09:01 UTC 
Description of problem:

for every update the catalog seems to be calling the bz event logger and is
creating records of the postdate group being removed even if it wasn't set to
begin with.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:
Comment 1 XINSUN 2008-05-13 01:44:22 UTC 
Created attachment 305191 [details]
The patch to fix this bug and do some code-optimization
Comment 2 Rob Landry 2008-05-13 16:32:34 UTC 
+    #If failed to find the hwcert_postdate group ID, indicate that group
dosen't exist.Then return

Doesn't actually indicate anything to the user, as such silent failure is a bad
thing; we should investigate where this happens in the code to see if it should
error out or just bail on this one thing.

+    #Set/Reset the post-date 

Do we really want it to not set the group but set the date?  This may be a
reasonable feature I don't know about the data integretity that BZ might attempt
to enforce.

sub UnsetPostDate {

Should likely have the same error mechanism as setPostDate does for when it
can't figure out the group.
Comment 3 XINSUN 2008-05-14 03:42:28 UTC 
Created attachment 305325 [details]
Delete the error mechanism when it
can't figure out the group, becuase the group exists by default in bug db and can always be figured out .
Comment 4 Rob Landry 2008-05-16 15:44:49 UTC 
Looks good on the surface, if this works properly when tested go ahead and check
it in.
Comment 5 XINSUN 2008-05-16 16:36:34 UTC 
Have tested well, The patch have checked into cvs.
Nicho