Ticket #952 (closed defect: fixed)

Opened 6 months ago

Last modified 4 months ago

DOI server name hard coded in java code.

Reported by: amit Assigned to: jkirton
Priority: low Milestone: 0.9.0
Component: ambra Version: 0.9-SNAPSHOT
Keywords: doi configuration Cc:
Blocking: Blocked By:

Description

DOI Server hard coded into code. Please look at line 219 of CitationUtils?.java

sb.append("http://dx.plos.org");

Dependency Graph

Change History

06/06/08 22:28:56 changed by ronald

It's also defaulted in source:head/ambra/webapp/src/main/resources/org/plos/article/service/pmc2obj.xslt and not overriden by the Ingester.

06/09/08 16:21:49 changed by amit

  • priority changed from high to low.

Not necessary for push to production, but crucial for official 0.9 release.

06/10/08 17:19:57 changed by russ

it's also in many .java files.

07/09/08 10:05:47 changed by jkirton

  • status changed from new to assigned.

07/10/08 09:03:28 changed by jkirton

(In [6149]) addresses #952

Created new config key: "ambra.platform.doiAlias" designed to hold the doi server name relevant to the server impl. Currently, it defaults to "dx.doi.org".

Updated Ingester to honor this new config key. Updated CitationUtils? to honor this new config key. Genericized pmc2obj.xslt replacing dx.plos.org with ds.doi.org as the default for the 'doi-url-prefix' parameter.

This needs to be tested..

07/10/08 12:15:00 changed by jkirton

(In [6150]) addresses #952

Re-named config key: "ambra.platform.doiAlias" to "ambra.platform.doiUrlPrefix" which now has prefixing "http://" and suffixing "/".

Updated Ingester and CitationUtils? accordingly. Updated CitationInfo? to use this config key removing the hard-coded "http://dx.doi.org".

07/10/08 13:43:34 changed by amit

  • status changed from assigned to closed.
  • resolution set to fixed.

Tested. Fixed with r6149 and r6150.