moewew
commented
7 years ago Unfortunately I'm not really well-versed in the new date-time features, so I can only make tentative suggestions.
If I understand things correctly, we need to replace (all?/most? occurrences of) the two simple lines
\printfield{labelyear}%
\printfield{extrayear}by the much longer and harder to understand
\ifdefstring\blx@dateformat@labeldate{edtf}
{}
{\datecircaprint}%
\dateeraprintpre{labelyear}%
\printfield{labelyear}%
\printfield{extrayear}%
\dateuncertainprint%
\iffieldsequal{labeldateera}{labelenddateera}
{}
{\dateeraprint{labelyear}}%
\ifdefstring\blx@dateformat@labeldate{edtf}
{\datecircaprintedtf}
{}%
\iffieldundef{labelendyear}
{}
{\iffieldsequal{labelyear}{labelendyear}
{}
{\ifdefstring\blx@dateformat@labeldate{edtf}
{\slash}% strict EDTF
{\bibdaterangesep
\enddatecircaprint}%
\dateeraprintpre{labelendyear}%
\printfield{labelendyear}%
\enddateuncertainprint
\ifdefstring\blx@dateformat@labeldate{edtf}
{\enddatecircaprintedtf}
{}%
\dateeraprint{labelendyear}}}@PLK Do you think it could be possible to conflate the monstrous 28 lines above that have to be repeated over and over again in all sorts of places into one more comfortable standard command? Earlier it was fairly neat to have only the two lines in all sorts of places, but having to have the 20-odd lines everywhere makes the code just look terribly complicated.
Additionally, in all \ifboolexpr we probably want to replace
test {\iffieldundef{month}}by
test {\iffieldundef{season}}
and
test {\iffieldundef{month}}Plus one
\iffieldundef{month}needs to be
\ifboolexpr{
test {\iffieldundef{season}}
and
test {\iffieldundef{month}}
}This should address your issues (1) and (2). (3) is expected behaviour, dates for @article types always go in brackets, while those for other entry types normally go bare.
I have uploaded a amended authoryear.bbx at gist: https://gist.github.com/moewew/fe3baef052d16872ae5a5226de463064 You can use it to test my suggestions.
JohnLukeBentley
plk
Some of the datetime frills (approximate, uncertain, eras) break in the authoryear style when mergedate is other than the default ("compact" or "true").
With ....
... then output is as expected (bibliographic output in document, not alphabetic, order):
With
... then output breaks:
This is broken in three ways:
Where a bibliographic entry has a
datefield with datetime precision finer than a year the fulldate time is sometimes not printed out as a second date. Here the entry for Alberici is correctly printout out, but for SEP there is no second date supplying the season. I'd expect:Perhaps this is only an issue with seasons.
datefield with datetime precision of a month the second date sometimes is surrounded by parentheses (as in Kennett, an article type), sometimes not (as in OED online, a book type).Request fixing this up for
mergedate=basic, if not all the other mergedate options (At the moment I'm personally only intrested inmergedate=basicandmergedate=compact).In the following test files only the entries for Aquinas and Plato are directly relevant. However I supply other entries of varying datetime precision, as a guard against regression.
Gist "Biblatex-Bug-Mergedate" (Edit 04: note to self these are locally renamed Biblatex-Tester-EdtfBasics.*):
Edit01:
labeldateusetime=falseEdit02:
@book{oed_online_2016_oed, ...entry to Biblatex-Bug-Mergedate.bib to support this.Edit03:
"Edit 04: note to self these are locally renamed Biblatex-Tester-EdtfBasics.*"