search

chapitreneuf / nova

Maquette Lodel compatible avec OpenEdition Journals
GNU General Public License v3.0
4 stars 4 forks source link

Fix doi display #123

Closed oliviercrouzet closed 7 months ago

oliviercrouzet commented 8 months ago

1er patch : Le doi de numero s'affiche aussi pour les numéro à paraitre. 2ème patch : pour que l'insertion du paramètre doi dans l'url des COINS tienne compte du type de document

oliviercrouzet commented 8 months ago

euh.. à la réflexion je crois que la modif sur macros_head.html (Do not display doi for upcoming issues) est inutile de même que la condition sur ![#PARAITRE] dans inserting doi parameter in COINS url

brrd commented 8 months ago

Je ne suis pas sûr de comprendre le but de ce commit. Faut-il que le DOI s'affiche sur les numéros à paraître ou non ?

oliviercrouzet commented 8 months ago

oui, j'avoue, c'est pas des plus clairs. La réponse est non : l'idée c'est de ne pas afficher le DOI sur les numéros à paraitre. Pour ça, la modification sur la macro BASE_DOI dans macros_base.html suffit (et celle de macros_head.html est inutile car elle porte sur les metatags qui ne sont pas insérés dans les articles). mais je vais refaire 2 PR dissociés, un pour ça et l'autre qui porte sur les COINS.