convert include-period to strip-period

citation-style-language / utilities

Citation Style Language utilities

http://citationstyles.org/

MIT License

18 stars 7 forks source link

convert include-period to strip-period #5

Closed bdarcus closed 13 years ago

bdarcus commented 13 years ago

Bitbucket: https://bitbucket.org/bdarcus/csl-utils/issue/3
Originally Reported By: Bruce D'Arcus
Originally Created At: 2009-11-06 04:50:53

bdarcus commented 13 years ago

What's probably needed:

All abbreviated terms (form="short" or "verb-short") should receive a period in the locales-xx-XX.xml files when applicable. E.g.
``` p pp
```
<term name="editor" form="verb-short">ed</term>
```

should become

<term name="page" form="short">

      <single>p.</single>

      <multiple>pp.</multiple>

    </term>

<term name="editor" form="verb-short">ed.</term>

In the styles, strip-periods="true" should be applied to every element where abbreviated terms are used (cs:date-part, cs:text and cs:label), unless include-periods is set to "true".
The include-periods attribute should be removed from the styles (it can occur on cs:date-part, cs:text and cs:label).

Original Comment By: Rintze Zelle

bdarcus commented 13 years ago

Should be partially solved with [Author: rmzelle

Age: 17 months ago

Should remove include-period attribute from cs:text, and apply strip- periods="true" to any instance of cs:text with form="short" or "verb-short" as long as include-period is not "false". Still have to extend the patch to cover cs:lable and cs:date-part (the latter only if name="month").">r6:7fea7c0c2c0b]1

Rest should be easy, but it's bedtime now.

Original Comment By: Rintze Zelle

bdarcus commented 13 years ago

Maybe we shouldn't attach strip-period="true" for cs:text elements that don't call a term? The most important exception would be journal titles, e.g. automatic conversion of

<text variable="container-title" form="short"/>

<text variable="container-title" form="short" strip-periods="true"/>

would clearly be undesired.

Original Comment By: Rintze Zelle

bdarcus commented 13 years ago

Surprisingly, I seem unable to read my own comments (the point raised in comment 3 was already addressed in comment 1). The whole issue should now be solved with [Author: rmzelle

Age: 17 months ago

Removed conversion to localized dates * Reordered style sheet, some cleanup
Removed some bugs (namespaces were applied to some elements; cs:style didn't enclose other elements) * Extended include-period to strip-periods conversion logic to cs:label, cs:date-part. For cs:text it has been limited to elements calling a term.">r10:f6e2933010ba]1 , except for the need to modify the locales-xx-XX.xml files. Leaving the ticket open so we won't forget that.

Original Comment By: Rintze Zelle

bdarcus commented 13 years ago

Adding periods to abbreviated terms in locale files is now addressed in an issue in csl-locales tracker: http://bitbucket.org/bdarcus/csl- locales/issue/7/add-periods-for-abbreviations-form-short-when

Closing this one (as the conversion works as advertised)

abbreviations-form-short-when

Original Comment By: Rintze Zelle