dan-weiss / apa7-latex-cls-source

Source code for apa7 class
LaTeX Project Public License v1.3c
49 stars 13 forks source link

Avoid unnecessary biblatex options #23

Closed moewew closed 3 years ago

moewew commented 3 years ago

Currently the apa7 manual recommends loading and loads biblatex as follows

https://github.com/dan-weiss/apa7-latex-cls-source/blob/f269eaf595f00e18488f9e78673f9ddc4f6aa125/apa7/apa7.dtx#L791-L794

https://github.com/dan-weiss/apa7-latex-cls-source/blob/f269eaf595f00e18488f9e78673f9ddc4f6aa125/apa7/apa7.dtx#L1238

With a current version of biblatex-apa (which is assumed because we want 7th edition APA style) sortcites=true, is redundant, because it is implied by style=apa (cf. apa.bbx).

Furthermore, sorting=nyt, might actually be counter-productive since it forces the standard author-year sorting and not the APA sorting defined by biblatex-apa as apa (loaded in apa.bbx).

So I recommend loading biblatex simply as

\RequirePackage[style=apa, backend=biber]{biblatex}

\DeclareLanguageMapping has not been necessary in biblatex-apa since v7.5 came out in November 2017. For new documents I would not recommend adding those lines. Since you are working under the assumption that biblatex-apa produces 7th-edition APA style (so is at least at version v9.0 from November 2019), I think it makes sense to stop mentioning \DeclareLanguageMapping altogether in the documentation.

dan-weiss commented 3 years ago

@moewew Thanks for the suggestion. I like deleting code :)

moewew commented 3 years ago

Thank you very much! The biblatex examples look much less daunting now.