Bug 987766 - Revision History date problems
Summary: Revision History date problems
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: PressGang CCMS
Classification: Community
Component: DocBook-builder
Version: 1.1
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 1.2
Assignee: Lee Newson
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-07-24 06:50 UTC by Misty Stanley-Jones
Modified: 2014-06-18 07:05 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2013-10-17 23:49:33 UTC
Embargoed:


Attachments (Terms of Use)

Description Misty Stanley-Jones 2013-07-24 06:50:40 UTC
1. The day abbreviations don't accept all the choices (Thu, Thur, Thurs, Thursday). Not sure if this is easily fixable. It is a very low priority.

2. Multiple revision entries on the same day show up as out of order, even though they are perfectly fine. This is a little bit bigger of a deal.

For more info, view http://docbuilder.usersys.redhat.com/21333/#appe-Induction_Guide-Revision_History.

Comment 1 Lee Newson 2013-07-25 01:26:43 UTC
Had a look into the issue quickly this morning.

1. This doesn't seem to be something Java by default supports at all so to do that we'd possibly have to write our own parser or try and find one that does accept various day abbreviations.

2. It's not the duplicate dates that is the problem here, the problem is the "Mon July 27 2012" date. The Joda DateTime library which is used to parse the date is parsing the date as 23rd of July 2012, which is the main problem since the next date is the 25th of July 2012. I fixed the date up in that topic so that it is "Fri July 27 2012" and it is now working.

So for 1. unless I can find a library to handle that it is something to look at some other time. As for 2. I'll take a look into why it is parsing the date wrong, as the Day of the Week shouldn't be changing the date.

Comment 2 Misty Stanley-Jones 2013-07-25 01:30:51 UTC
Please consider this a low priority as we should be getting the dates correct in the revision history anyway.

Comment 3 Lee Newson 2013-07-28 23:58:05 UTC
Fixed point 2 in csprocessor 0.33.6, as for point 1 though since it's low priority I'll come back to it at a latter stage.

Comment 4 Misty Stanley-Jones 2013-07-29 01:40:48 UTC
How can I test point 2?

Comment 5 Lee Newson 2013-07-29 01:51:01 UTC
Misty if you use any revision history and change the day of the week to something that isn't correct (ie Mon July 26 2013, which is actually a Friday) and then include a revision before that (ie Wed July 24 2013), then the book should build fine. To be sure that it wasn't because of the duplicate dates feel free to include that was well. You can use Content Spec 16913 and Revision History 16906 to test if you want.

example:

<revision>
	<revnumber>0.32.2-0</revnumber>
  <date>Mon July 26 2013</date>
	...
</revision>
<revision>
	<revnumber>0.32.1-0</revnumber>
  <date>Wed July 24 2013</date>
	...
</revision>
<revision>
	<revnumber>0.32.0-0</revnumber>
  <date>Wed July 24 2013</date>
	...
</revision>

I should note that basically the day of the week is ignored now in a similar way to how publican handles it.

Comment 6 Lee Newson 2013-10-01 05:09:16 UTC
I've updated the builder to try and sanitise the Day name so that it can be successfully parsed by the Java libraries when a medium length day abbreviation is used. This will likely need to be improved as more combinations are found. Currently I've allowed for:

Tues
Thur
Thurs

Comment 8 Matthew Casperson 2013-10-15 04:25:46 UTC
"Thurs" does not work:

This topic has invalid Docbook XML. Invalid revision, the date "Thurs Oct 10 2013" is not in a valid format.

"Thur" and "Tues" work as advertised.

Comment 9 Lee Newson 2013-10-15 05:01:08 UTC
Fixed the problem in csprocessor 1.2-SNAPSHOT build 201310151455.

The problem was caused by replacing "Thur" before "Thurs" which meant "Thurs" became "Thus" and therefore wasn't replaced.

Comment 10 Matthew Casperson 2013-10-16 00:44:07 UTC
Confirmed that Japanese spec 13879 builds with an instance of "Thurs" in the revision history.


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