... and i've checked in a small hack to metadatasharing so that its working OK for now for importing RelationshipTypes and PersonAttributeTypes.
d On Thu, Aug 4, 2011 at 6:25 PM, Dave Thomas <[email protected]> wrote: > 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]
