Ticket #1310 (closed enhancement: fixed)

Opened 16 months ago

Last modified 15 months ago

Plural Article Headers

Reported by: josowski Owned by: npeterson
Priority: medium Milestone: 0.9.3_rc2
Component: ambra Version: 0.9.1_rc2-SNAPSHOT
Keywords: Cc:

Description

In browse Issues, article type headers need to sometimes be plural

Attachments

plurals.txt (1.8 KB) - added by josowski 16 months ago.
List of plural terms

Change History

Changed 16 months ago by josowski

List of plural terms

Changed 16 months ago by josowski

  • priority changed from critical to medium
  • version set to 0.9.1_rc2-SNAPSHOT
  • milestone set to 0.9.3_rc2

Changed 16 months ago by josowski

  • status changed from new to assigned
  • owner changed from dragisak to josowski

Changed 16 months ago by npeterson

  • type changed from defect to enhancement

Changed 15 months ago by josowski

(In [7704]) Added capability to display plural article types when browsing issues.

References #1310

Changed 15 months ago by josowski

  • owner changed from josowski to npeterson
  • status changed from assigned to verify
  • resolution set to fixed

Changed 15 months ago by rcave

  • owner changed from npeterson to josowski
  • status changed from verify to reopened
  • resolution fixed deleted

in browseIssue.ftl, rather than throw a nasty error if a <typeHeadingPlural> doesn't exist, it would be better to use the <typeHeading>.

And if that doesn't exist, the page should trap the error gracefully.

2009-06-05 14:53:32,756 ERROR runtime(PLoSONE)>  [TP-Processor3 freemarker.runtime]
Expression articleGrp.pluralHeading is undefined on line 91, column 19 in journals/plosone/article/browseIssue.ftl.
The problematic instruction:
----------
==> ${articleGrp.pluralHeading} [on line 91, column 17 in journals/plosone/article/browseIssue.ftl]
 in include "${templateFile}" [on line 31, column 1 in templates/main_template.ftl]
----------

Java backtrace for programmers:
----------
freemarker.core.InvalidReferenceException: Expression articleGrp.pluralHeading is undefined on line 91, column 19 in journals/plosone/article/browseIssue.ftl.
        at freemarker.core.TemplateObject.assertNonNull(TemplateObject.java:124)
...
...
2009-06-05 14:53:32,760 ERROR UnhandledException(PLoSONE)> Expression articleGrp.pluralHeading is undefined on line 91, column 19 in journals/plosone/article/browseIssue.ftl. [TP-Processor3 org.topazproject.ambra.UnhandledException]
Expression articleGrp.pluralHeading is undefined on line 91, column 19 in journals/plosone/article/browseIssue.ftl.
The problematic instruction:
----------
==> ${articleGrp.pluralHeading} [on line 91, column 17 in journals/plosone/article/browseIssue.ftl]
 in include "${templateFile}" [on line 31, column 1 in templates/main_template.ftl]
----------

Java backtrace for programmers:
----------
freemarker.core.InvalidReferenceException: Expression articleGrp.pluralHeading is undefined on line 91, column 19 in journals/plosone/article/browseIssue.ftl.
        at freemarker.core.TemplateObject.assertNonNull(TemplateObject.java:124)
...
...

Changed 15 months ago by josowski

(In [7731]) Improving how we handle issue headers. If a plural heading is not available we default to use the singular heading. If no heading is available default to "No Header Defined"

References #1310

Changed 15 months ago by josowski

  • owner changed from josowski to npeterson
  • status changed from reopened to verify
  • resolution set to fixed

Changed 15 months ago by npeterson

  • status changed from verify to closed
Note: See TracTickets for help on using tickets.