Re: RE : [sdx-developers] Conformité OAI-PMH

[Pierrick Brihaye]

Re,

Pierrick Brihaye a écrit:

> Je commence :

Je continue :

Dans

AbstractDocumentBaseOAIRepository, on a :

/**The number of records for before issuing a resumption tokent*/
protected int numberPerResponse = 1000;//defaulted

Wow ! Ne peut-on reprendre :

/** The default number of results per page */
public static final int HITS_PER_PAGE = 20;

dans  fr.gouv.culture.sdx.search.lucene.query.Results

et, en passant, remonter cette variable, et les variables connexes, dans 
la classe fr.gouv.culture.sdx.search.Results ?


Attention également à :
this.numberPerResponse = 
configuration.getAttributeAsInteger(ATTRIBUTE_NAME_NO_PER_RESPONSE, 1000);

Dans LuceneDocumentBaseOAIRepository :

je remarque :
else if (Utilities.checkString(request.getResumptionToken())) {
  new OAIError(OAIError.ERROR_BAD_RESUMPTION_TOKEN, "This repository 
does not support resumptionTokens for ListSets requests").toSAX(this);

Soit... :-)

avaliableFormats : typo (plusieurs occurences).

String mdPrefix = request.getMetadataPrefix();
String resumptionToken = request.getResumptionToken();
if (Utilities.checkString(resumptionToken))
  mdPrefix = getResumptionTokenProperty(resumptionToken, 
OAIRequest.URL_PARAM_NAME_METADATA_PREFIX);

cosmétique. Préférer :

String mdPrefix = null;
String resumptionToken = request.getResumptionToken();
if (Utilities.checkString(resumptionToken))
  mdPrefix = getResumptionTokenProperty(resumptionToken, 
OAIRequest.URL_PARAM_NAME_METADATA_PREFIX);
else mdPrefix = request.getMetadataPrefix();

???

buildExternalOaiId : utliser un ID generator ? Créer un package pour ces 
IDgenerators ?

if (exQuery != null)
  bq.add(exQuery, true, false);

Euh... n'est ce pas bq.add(exQuery, false, true); ? A propos, soyons 
logiques et traitons l'inclusion avant l'exclusion, non ?

createResumptionToken : créer un IDGenerator ?

java.util.Date untilDate = 
fr.gouv.culture.sdx.utils.Date.parseUtcISO8601Date(until);

if (untilDate == null && Utilities.checkString(until)) {
  new OAIError(OAIError.ERROR_BAD_DATE, "The value of the \"until\" 
argument, " + until + ", is invalid.").toSAX(this);
  return null;//Bad date
}
-- X-POINT --
if (untilDate == null)//no date param
  untilDate = fr.gouv.culture.sdx.utils.Date.parseDate("9999");

Euh... si untilDate == null, le point X ne sera jamais atteint, non ?

Voilà pour un premier examen plus profond.

A+

--
Pierrick Brihaye, informaticien
Service régional de l'Inventaire
DRAC Bretagne
mailto:address@hidden