Skip to content

Fix #422: Add "last modified" line on cover page#423

Draft
tomschr wants to merge 1 commit intomainfrom
feature/422-modified-date
Draft

Fix #422: Add "last modified" line on cover page#423
tomschr wants to merge 1 commit intomainfrom
feature/422-modified-date

Conversation

@tomschr
Copy link
Collaborator

@tomschr tomschr commented Oct 29, 2020

This is a proof-of-concept and a draft ATM. It contains the following changes:

  • Introduce parameter last.modified for FO and XHTML, by default empty
  • Introduce a last.modified key in language file (currently only en ATM)
  • When the cover page is created, only add "Last modified" line when $last.modified != ''.

Currently, the parameter $last.modified is used "as-is"; it is included in the respective line no matter if it's a date or not.

However, when a date is passed, it is currently inconsistent with the publication date. This looks like this (English):

Publication Date: October 29, 2020
Last modified: 2020-10-01

That first date is created from this line:

<date><?dbtimestamp format="B d, Y"?></date>

In other words, to make both dates consistent, the last modified date needs to be transformed into the format of the PI inside <date>.

We need to decide if we want a minimal solution first and improve it later, or, if we want to make it more consistent.

This is a proof-of-concept:

* Introduce parameter "last.modified" for FO and XHTML
  by default it's empty
* Introduce a "last.modified" key in language file
  (currently en ATM)
* When the cover page is created, only add "Last modified"
  line when $last.modified != ''.

Currently, the parameter $last.modified is used "as-is"; it
will be included in the respective line no matter if it's a
date or not.
However, if it's a date, it is currently inconsistent with the
publication date. That date is created from this line:

<date><?dbtimestamp format="B d, Y"?></date>

In other words, to make both dates to be consistent, the
last modified date needs to be transformed into the format
of the PI inside <date>.
@tomschr tomschr added format-html html, xhtml, html5, webhelp, jsp support format-pdf pdf support styles-2013 "suse2013", "suse2013-ns", "daps2013", "daps2013-ns", "opensuse2013", "opensuse2013-ns" styles labels Oct 29, 2020
@tomschr tomschr requested a review from a user October 29, 2020 15:44
Base automatically changed from master to main February 10, 2021 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

format-html html, xhtml, html5, webhelp, jsp support format-pdf pdf support styles-2013 "suse2013", "suse2013-ns", "daps2013", "daps2013-ns", "opensuse2013", "opensuse2013-ns" styles

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant