PatientEventListener.java

        @Override
        public String getExternalId()
        {
            try {
                for (Object identifierUncast : this.getData("identifiers")) {
                    Map.Entry<String, String> identifier = (Map.Entry<String, String>) identifierUncast;
                    if (identifier.getKey().equalsIgnoreCase("external_id")){
                        return identifier.getValue();
                    }
                }
            } catch (Exception ex) {
                return null;
            }
            return null;
        }

this.getData returns PatientData<T> which extends List. List has a get( int ), so I can't get("external_id"), like you suggested. Should I do it like this, or get the XWikiDocument and get the external_id from it?

Do it (almost) like this.

for (ImmutablePair<String, String> identifier : this.<ImmutablePair<String, String>>getData("identifiers")) {