[ https://issues.apache.org/jira/browse/OFBIZ-7754?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15363576#comment-15363576 ]
Hans Bakker commented on OFBIZ-7754: ------------------------------------ After more thought I would like to question the whole implementation of OFBIZ-7112. As far as i understood it, it was initiated because it was not possible to set a properties value to 'blank' or Empty, sure, accepted that is true. (although set in the properties file set to blank would be a solution?) But is it required? Please give me a number of examples where this is needed. As far as I can see a properties value always has content either Y/N or False/True or a value. Further, anybody just starting with the system will try to change the most important properties file 'general.properties' and will see no result. then as Kongrath (we call him Joe) mentioned an upgrade will be much more difficult if you set your parameters in general.properties or even worse set them in the DB and they get overwritten and sure should not be seed. So my proposal: please revert this change in OFBIZ-7112 it causes far more problems than it solves. > The big problem when load seed. > ------------------------------- > > Key: OFBIZ-7754 > URL: https://issues.apache.org/jira/browse/OFBIZ-7754 > Project: OFBiz > Issue Type: Improvement > Components: framework > Affects Versions: Trunk > Reporter: Kongrath Suankaewmanee > Assignee: Jacques Le Roux > Attachments: OFBIZ-7754.patch > > > Hi All, > Regarding, [OFBIZ-7112|https://issues.apache.org/jira/browse/OFBIZ-7112] > That's good for who start on use the ofbiz with initial setup, but not for > the site that already online and has to update the OFBiz core. Because when > has update OFBiz core they will use command load-seed for update. > The problem is if we use load-seed mean the configuration data that's already > exists will be replaced by the data from this file, > *CommonSystemPropertyData.xml* > So, for my suggestion should change the reader from *seed* to *seed-initial* > or remove systemPropertyValue from the data file. > Thank you, > Kongrath -- This message was sent by Atlassian JIRA (v6.3.4#6332)