Skip to content
This repository has been archived by the owner on Nov 7, 2018. It is now read-only.

<other-identifier> question which data is being displayed #69

Closed
siwhitehouse opened this issue Feb 20, 2013 · 4 comments
Closed

<other-identifier> question which data is being displayed #69

siwhitehouse opened this issue Feb 20, 2013 · 4 comments
Labels

Comments

@siwhitehouse
Copy link

Relates to previous issue #53 where I suspect I mislead you.

As per:

http://iatistandard.org/activities-standard/other-activity-identifiers/

We ought to be pulling through the element's content rather than the owner-ref attribute.

eg. http://aidview-dev.herokuapp.com/activity/AU-5-INJ559-FJ

displays AU-5 which is a reference to the activity's owner.

whereas: http://aidview-dev.herokuapp.com/activity/GB-CHC-285776-ZIM361 is blank when it should show ZIM361 as per this response from the API

http://data.aidinfolabs.org/xquery/woapi.xq?ID=GB-CHC-285776-ZIM361&search=&start-date=&end-date=&start=1&pagesize=20&result=full&callback=&corpus=main&test=yes

I hope this is a superficial fix.

@benfoxall
Copy link
Contributor

I've put in a fix so that the <other-identifier> content is displayed, though in the example above - the id isn't shown, because other-identifier is empty in the api request.

Does this make sense? Have I misunderstood the issue?

@siwhitehouse
Copy link
Author

Hi Ben

I'm really sorry, I messed up the issue - I've edited it now so it ought to make sense.

The first example ought to be blank because, as you say, other-identifier is empty in the api request. However, when I look in AidView - http://aidview-dev.herokuapp.com/activity/AU-5-INJ559-FJ - I see "AusAID" under "Other activity identifiers".

http://aidview-dev.herokuapp.com/activity/GB-CHC-285776-ZIM361

Is correctly showing ZIM361 under "Other activity identifiers"

So, if we fix the code that is pulling AusAID in the first example we ought to be good.

@benfoxall
Copy link
Contributor

Okay, so now if you have no content in the other-identifier element:

<other-identifier owner-ref="AU-5" owner-name="AusAID"/>

(from AU-5-INJ559-FJ ) … the other identifier will be skipped in the UI.

@siwhitehouse
Copy link
Author

Thanks Ben, I'm happy that this is ok to be closed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants