{"push_id":2807722573,"size":1,"distinct_size":1,"ref":"refs/heads/trunk","head":"06944e3298de90818ac03e3c939c8d6ae2ecd71e","before":"ea3741db40f17999e4193ba7637af19787e40601","commits":[{"sha":"06944e3298de90818ac03e3c939c8d6ae2ecd71e","author":{"name":"Nicolas Malin","email":"468feb6fa71f8dd2720b54338a277cfe5de997c8@apache.org"},"message":"Improved: Refactoring ContactMechWorker.get[Entity]ContactMechValueMaps function to improve processing speed\n(OFBIZ-10514)\nContactMechWorker.get[Entity]ContactMechValueMaps are old historic functions that resolve all contact mech context related to an Entity (Party, Facility, Order, WorkEffort).\nThe problem that they create too many db call during their execution that decrease OFBiz performance when the contactMech history grow.\n\nA solution use to improve the process speed was replace all db call through *GenericValue.getRelated* function by *Delegator.makeValidValue* to instanciate each GenericValue needed after an unique search call on a viewEntity. For the four entities : Party, Facility, Order and WorkEffort, I use the same view structure with a sub view entity on ContactMechDetail. This offer more genericity and permit to slim code.\n\nThis commit introduce a regression on genericValue resolved for ContactMechType and ContactMechPurposeType that currently lost their description, it's actually not awkward because theses genericValue use the function .get(\"description\", locale) to resolve the description so call directly the label system.\n\nFor Order and WorkEffort, I merged the resolving process in one function, that can be extend later easily for some other entity like Invoice or Quote.\nThe new FtpAddress wasn't managed by this worker so I introduce it to keep the coherence.\n\nLast improvement, I use \n   getPartyContactMechValueMaps(Delegator delegator, String partyId, Timestamp date, String contactMechTypeId) \ninstead of\n   getPartyContactMechValueMaps(Delegator delegator, String partyId, boolean showOld, String contactMechTypeId) \nto offert also more possibility to resolve at date instead just use now or all.\n\nicing on the cake, I added groovy test to control this worker\n\ngit-svn-id: https://svn.apache.org/repos/asf/ofbiz/ofbiz-framework/trunk@1838349 13f79535-47bb-0310-9956-ffa450edef68","distinct":true,"url":"https://api.github.com/repos/apache/ofbiz-framework/commits/06944e3298de90818ac03e3c939c8d6ae2ecd71e"}]}
{"actor":{"display_login":"asfgit"}}