From: olivier H. <oli...@ne...> - 2011-09-22 07:59:37
|
Erwan, Jacques, Ce Jira traine depuis 2 mois alors qu'il concerne une régression importante (que j'avais introduite par une modification précédente) J'ai rajouté ce matin un commentaire pour expliciter en quoi ma correction de Juillet est la bonne. Est ce que l'un de vous deux peu s'occuper de ce Jira (et donc commiter mon patch). Par avance Merci. Olivier Le 29/07/2011 10:36, Olivier Heintz a écrit : > Le 28/07/2011 11:14, Erwan de FERRIERES a écrit : >> Salut Olivier, >> >> il y a eu une réponse de la part de leon sur le jira >> https://issues.apache.org/jira/browse/OFBIZ-4349 >> >> Si tu peux t'en occuper, merci ! >> > > Leon, > > Maybe, my explanations was not clear, I try to be more clear. The > problem I want to solve was not about field name the method search to > read the international content, but on the pk (name) to retrieve the > correct record. > > In the method you give, search is doing to check if there is a alias > for field name to retrieve. > In my patch, I check to have correct pkName. > > As exemple, in projectmgr component, there are view name Project.... > with pk named projectId not workEffortId and workeffortName should be > renamed projectName. > With initiale code (before my first correction), it will be a call to > get method with name=workEffortName, but search in properties will > fail because this.get("workEffortId") return a error "workEffortId is > not a field for...." > |