Picsure 16 patient mapping data #63
Conversation
| } | ||
|
|
||
| @Override | ||
| public Result runQuery(User user,Query query, Result result) throws ResourceInterfaceException { |
There was a problem hiding this comment.
if just super(), you don't actually need to override it
| for (PidType ps : patientDataResponse.getPatientData().getPidSet().getPid()){ | ||
| if (this.sourceWhiteList.contains(ps.getPatientId().getSource())){ | ||
| mrs.appendRow(); | ||
| mrs.updateString("Patient Id", ps.getPatientId().getValue()); |
There was a problem hiding this comment.
see screenshots: 
which one should we look into?
There was a problem hiding this comment.
direct from i2b2 org about PID:
There was a problem hiding this comment.
we really need both to come back, so you would pseu-do something like this:
select: Hospital-1 and Hospital-5
where: patients have lab results of some kind
and get something like this in return:
patient_id, Hospital-1, Hospital-5
100000001, 2000001961, 3000001821
...
There was a problem hiding this comment.
So instead of one column of just the patient id, we need multiple columns, where the heading of the columns is the patient_map_id source?? Meaning multiple columns with the same heading? This needs a lot more explanation, and to be in the ticket.
There was a problem hiding this comment.
Yep! All updated.
Unfortunately it's a little hard to tell what changes happened in I2B2XMLResourceImplementation, but it's mostly just moving declarations of variables a little bit and wrapping some of the logic in an if statement.