Bug 727484 - The rpm packages' History time record is not correct
Summary: The rpm packages' History time record is not correct
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Hardware Certification Program
Classification: Retired
Component: Hardware Catalog
Version: 6.1
Hardware: All
OS: Unspecified
unspecified
low
Target Milestone: ---
: ---
Assignee: Pengfei Xue
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 722937
TreeView+ depends on / blocked
 
Reported: 2011-08-02 09:16 UTC by Yunlong Xue
Modified: 2011-09-19 06:20 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-09-19 06:20:27 UTC


Attachments (Terms of Use)
alter column upload_date in table cert_rpms (1.01 KB, patch)
2011-08-18 02:10 UTC, Pengfei Xue
no flags Details | Diff
revised script (990 bytes, text/plain)
2011-08-23 02:14 UTC, Pengfei Xue
pxue: review? (rlandry)
pxue: review? (tfu)
Details
ignore case when delete "ON UPDATE" attribute (735 bytes, patch)
2011-08-26 10:15 UTC, Pengfei Xue
pxue: review? (rlandry)
pxue: review? (tfu)
pxue: review? (wshen)
Details | Diff

Description Yunlong Xue 2011-08-02 09:16:48 UTC
The rpm packages' History time record is not correct.
The first time record will always same with the last one.

Such as:
1. I uploaded a rpm package to catalog at 2011-08-02 05:03 EST , then the History of this package will be:
History: 	2011-08-02 05:03 EST - yxue attached this package.

2. I mark it as reviewed at 2011-08-02 05:04 EST , then the Histroy become :
History: 	2011-08-02 05:04 EST - yxue attached this package.
		2011-08-02 05:04 EST - yxue marked package as reviewed.
You see , the first update's time record is become same with second update's record. 
		
3.If i uncheck the Leave as Reviewed at 2011-08-02 05:08 EST: 
History: 	2011-08-02 05:08 EST - yxue attached this package.
		2011-08-02 05:04 EST - yxue marked package as reviewed.
		2011-08-02 05:08 EST - yxue reset package. 
The first time record change again.
		
4. Then mark it again at 2011-08-02 05:09 EST:
History: 	2011-08-02 05:09 EST - yxue attached this package.
		2011-08-02 05:04 EST - yxue marked package as reviewed.
		2011-08-02 05:08 EST - yxue reset package.
		2011-08-02 05:09 EST - yxue marked package as reviewed.
And again.

Comment 6 Pengfei Xue 2011-08-18 02:10:31 UTC
Created attachment 518788 [details]
alter column upload_date in table cert_rpms

Comment 9 Rob Landry 2011-08-22 22:18:58 UTC
Will there be an updated script accounting for weishen's feedback in comment #8?

What testing has been applied to the script?

Comment 10 Pengfei Xue 2011-08-23 02:12:50 UTC
(In reply to comment #9)
> Will there be an updated script accounting for weishen's feedback in comment
> #8?
> 
> What testing has been applied to the script?

already deleted the unnecessary libs and comments, and tested it in my own box, included: upload new result package, check/uncheck the "Mark as Reviewed" box, provided revised script

Comment 11 Pengfei Xue 2011-08-23 02:14:37 UTC
Created attachment 519380 [details]
revised script

Comment 12 Pengfei Xue 2011-08-26 10:15:21 UTC
Created attachment 520053 [details]
ignore case when delete "ON UPDATE" attribute


Note You need to log in before you can comment on or make changes to this bug.