Erwan, could you please explain why this patch was committed to the portletWidget branch? There were some objections in Jira and in general there was no general approval for the inclusion. Also, it was a patch for the trunk, not the branch.
This is not the way to go, the branch is not the playground of one committer and we cannot use it as an easy way (a lot of traffic, less reviews from committers) to see the code we like committed to trunk. If this is the general trend, I am tempted to say that the experiment of branches (mostly) used by one committer is failing: branches make sense only if a relevant part of the committer group is working on new stuff, not just one. Kind regards, Jacopo PS: a message to all: since I am not going to review each and every commit done on this branch, I am going to vote -1 to the merging of the portletWidget branch with the trunk until I will get enough guarantees from the people that worked on it that the changes will be only related to the original purpose of the branch. On Oct 30, 2012, at 10:10 PM, er...@apache.org wrote: > Author: erwan > Date: Tue Oct 30 21:10:10 2012 > New Revision: 1403870 > > URL: http://svn.apache.org/viewvc?rev=1403870&view=rev > Log: > Applying a patch from Olivier Heintz on branch OFBIZ-4949 add a new attribute > for for entity-engine-xml tag, put-other-field-to-null= true, if it exist at > the beginning data file, all update will put to null all field not detail in > this file > > Modified: > > ofbiz/branches/20120329_portletWidget/framework/entity/src/org/ofbiz/entity/util/EntitySaxReader.java