Hi Jacques,

I will dig into this further in the next days.

Thanks,

Michael

Am 30.08.22 um 11:28 schrieb Jacques Le Roux:
Le 30/08/2022 à 10:20, Jacques Le Roux a écrit :
Hi,

It's a long time now I, from time to time, have a look at this issue. At some point I had at least 13 related links, remain open only:
https://issues.apache.org/jira/browse/OFBIZ-7112 "EntityUtilProperties"
https://issues.apache.org/jira/browse/OFBIZ-6712 "Increase the number of EntityUtilProperties methods which really use getSystemPropertyValue()" https://issues.apache.org/jira/browse/OFBIZ-7754 "The big problem when loading seed." https://issues.apache.org/jira/browse/OFBIZ-6164 "Improve configurability of OFBiz"

And somehow related (to review when the rest is done):
https://cwiki.apache.org/confluence/display/OFBIZ/Extension+in+Multitenancy+support
https://cwiki.apache.org/confluence/display/OFBIZ/Multitenancy+support

We discussed a lot about that (notably*) but did not decide about it.
* https://markmail.org/message/md6fuoouan377c6w (look rather at bottom of the thread where some opinionated conclusions show)

For OFBIZ-7754 I notably suggested (comment there https://s.apache.org/cyx1j):

   <<Since the master DB makes only sense when using tenants, I believe we should 1st separate properties, keep generic ones in properties files and
   put others in DB.>>

But Michael is basically against that, see bottom of the thread (markmail) above.

I don't think anybody is against changing from seed to seed-initial for CommonSystemPropertyData.xml rather than removing it as suggested by Hans (who created it). I mean maybe some users are relying on that now. Note that it goes in the direction I suggested above.

Then, if nobody is against, I'll close OFBIZ-7754 (and hopefully other but 7112) and create a new Jira task to separate all properties as Hans did for CommonSystemPropertyData

Afterward I'll get back to OFBIZ-7112

What do you think?

Note that I'll not wait forever for opinions. It will not be longer than next week.

TIA

Jacques
Regarding Michael's then reluctance, I already suggested a maybe solution: instead of using load-initial to load alike properties from files (those that can be tenant specific) we could create a load-tenant reader that would be only used if <<multitenant=Y>> or if the user prefer using DB data instead of properties in file for all properties types (ie business and system)

Michael, what do you think about that?

Another option, somehow related, is to use Taher's suggestion and have all properties in a sole file as HTTPD does, I'm not much inclined to that. (It's only in my memories, I can't find a reference back)

I have created
https://issues.apache.org/jira/browse/OFBIZ-12689 "Separate business properties from system properties" https://issues.apache.org/jira/browse/OFBIZ-12688 "Change SystemProperty entity name to BusinessProperty" (depends upon 12689)

Jacques

Reply via email to