That's a bug in core, so it should definitely be fixed there.

(We probably also want to put a workaround in Metadata Sharing, so people
don't have to upgrade their core install.)

-Darius

On Thu, Aug 4, 2011 at 8:58 AM, Dave Thomas <[email protected]> wrote:

> Hi.  Is there a reason that PersonService doesn't implement OpenmrsService?
>
> I just spent all afternoon trying to figure out why MetadataSharing can't
> import a RelationshipType, and its because in MetadataSharing's
> OpenmrsClassScanner,  initServiceSaveMethodsCache calls getClasses which
> thinks PersonService isn't an OpenmrsService because it doesn't implement
> OpenmrsService, which then results in none of the 'save' methods in
> PersonService being added to the MetadataSharing import method cache, so
> importing RelationshipTypes and PersonAttributeType don't work.
>
> Is the solution to explicitly look for PersonService in the matcher in the
> OpenmrsClassScanner, or is it to just have PersonService extend
> OpenmrsService?
>
> If this should be explicitly handled in MetadataSharing, i'll make the
> change.
>
> d
> ------------------------------
> Click here to 
> unsubscribe<[email protected]?body=SIGNOFF%20openmrs-devel-l>from 
> OpenMRS Developers' mailing list

_________________________________________

To unsubscribe from OpenMRS Developers' mailing list, send an e-mail to 
[email protected] with "SIGNOFF openmrs-devel-l" in the  body (not 
the subject) of your e-mail.

[mailto:[email protected]?body=SIGNOFF%20openmrs-devel-l]

Reply via email to