OK, i'll make the tickets. thanks d
On Thu, Aug 4, 2011 at 6:12 PM, Darius Jazayeri <[email protected]>wrote: > 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 > > > ------------------------------ > 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]
