PKP Bugzilla – Bug 6566
Publicly display keywords in article abstract pages, if they exist
Last modified: 2011-06-07 13:46:10 PDT
We are moving to Git Issues for bug tracking in future releases. During transition, content will be in both tools. If you'd like to file a new bug, please create an issue.
Keywords aren't currently displayed publicly on article abstract pages (except in the RT, if enabled, which is a little buried). I think it's pretty standard that they be publicly displayed if they exist. The following code does just this (I've placed it between the abstract and citation code chunks in in templates/article/article.tpl), but should probably be refactored slightly (eg. this should probably have its own locale key):
Created attachment 3530 [details]
adds "keywords" section to article view
This patch adds a "keywords" section to the article view, between abstract and citations; I made it use the "article.subject" key which we already had mapping to "keywords" so we won't need a new key for this.
there are a lot of <br/> in this template, incidentally. Should these be engineered out of existence (in the spirit of html/css separation)?
No, leave the <br/> tags for now; best not to disrupt the CSS. This will probably need porting to OCS as well, if you get the chance.
Created attachment 3535 [details]
this patches the same issue in OCS, in the templates/paper/paper.tlp file
On closer look -- the keywords should be fully escaped, rather than permitting limited HTML, as per the use of getLocalizedSubject in other template files. (This is for both OJS and OCS.)
Created attachment 3545 [details]
patch against ojs-stable-2_3
adds keywords between abstract and citations (fully escaped) in OJS
Created attachment 3546 [details]
patch against ocs-stable-2_3
adds keywords between abstract and citations (fully escaped) in OCS
Committed to ojs-stable-2_3, ojs master, ocs-stable-2_3, ocs master.