Project

General

Profile

bug #9384

DefaultReferenceCacheStrategy.createShortCitation() working not correctly for Teams with two members

Added by Andreas Kohlbecker about 1 month ago. Updated about 1 month ago.

Status:
Closed
Priority:
Highest
Category:
cdmlib
Target version:
Start date:
01/12/2021
Due date:
% Done:

100%

Severity:
critical
Found in Version:
Tags:

Description

see https://api.phycobank.org/phycobank/registrationDTO.json?identifier=http://phycobank.org/102662

The Team "Molinari Novoa, E.A. & Guiry, M.D.R." is truncated to "Molinari Novoa & al. 2020",


Related issues

Related to Edit - feature request #9279: Improve type designation formatting and TypeDesignationSetManager In Progress 11/05/2020

Associated revisions

Revision 854244ff (diff)
Added by Andreas Kohlbecker about 1 month ago

fix #9384 fixing author team truncation

Revision 619db306 (diff)
Added by Andreas Müller about 1 month ago

fix #9384, ref #9279 fixed incorrect implementation of createShortCitation and added tests

Revision f4c60bdf (diff)
Added by Andreas Kohlbecker about 1 month ago

ref #9384 adapting test to modified createShortCitation method

History

#1 Updated by Andreas Kohlbecker about 1 month ago

  • Tags set to phycobank
  • Description updated (diff)
  • Category set to cdmlib
  • Assignee set to Andreas Kohlbecker
  • Priority changed from New to Highest
  • Target version changed from Unassigned CDM tickets to Release 5.19
  • Severity changed from normal to critical

#2 Updated by Andreas Kohlbecker about 1 month ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 50

#3 Updated by Andreas Kohlbecker about 1 month ago

  • Assignee changed from Andreas Kohlbecker to Andreas Müller

please review

Tests for this method are missing and should be implemented asap. Please could you open a new ticket for that.

#4 Updated by Andreas Müller about 1 month ago

  • Related to feature request #9279: Improve type designation formatting and TypeDesignationSetManager added

#5 Updated by Andreas Müller about 1 month ago

  • Assignee changed from Andreas Müller to Andreas Kohlbecker

The fix was not fully correct as it created teams like Name1, Name2 & al. but there should be always a maximum number of 2 names only. So only Name1 or Name1 & Name2 or Name1 et al. is allowed.

I fixed it and added tests (by the way there were some indirect tests in TypeDesignationSetManagerTest, but they were not enough, but at least the reason why I found that the new implementation was not correct either).

Please review.

#6 Updated by Andreas Kohlbecker about 1 month ago

  • Subject changed from DefaultReferenceCacheStrategy.createShortCitation() working not correctly for Teams with twi members to DefaultReferenceCacheStrategy.createShortCitation() working not correctly for Teams with two members

#7 Updated by Andreas Kohlbecker about 1 month ago

  • Assignee changed from Andreas Kohlbecker to Andreas Müller

The implementation is correct as far as the the implemented formatting rule for more than two authors is holding true.
Is this rule embodied in the ICN and also the same in other codes? The ticket can be closed if the answer to this question is yes.

#8 Updated by Andreas Müller about 1 month ago

  • Status changed from Resolved to Closed
  • % Done changed from 50 to 100

as we talk about bibliographic references here the ICN (or any other code) is not relevant here. It is the rule I have seen at many places and also shortly discussed with RH for cyprus.
In general the formatting should be project specific. But this requires project specific cache strategies which currently does not yet exist.

I close the ticket. Please reopen if you think more discussion is needed.

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)