[jira] [Comment Edited] (OFBIZ-5458) getSystemPropertyValue is never called/used

2014-01-07 Thread Adrian Crum (JIRA)

[ 
https://issues.apache.org/jira/browse/OFBIZ-5458?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13864118#comment-13864118
 ] 

Adrian Crum edited comment on OFBIZ-5458 at 1/7/14 11:23 AM:
-

I agree that we shouldn't have those hard-coded default values. If the 
configuration files have missing values, then we should throw an exception to 
indicate something is wrong.



was (Author: adri...@hlmksw.com):
I agree that we shouldn't have those default values. If the configuration files 
have missing values, then we should throw an exception to indicate something is 
wrong.


 getSystemPropertyValue is never called/used
 ---

 Key: OFBIZ-5458
 URL: https://issues.apache.org/jira/browse/OFBIZ-5458
 Project: OFBiz
  Issue Type: Bug
  Components: ALL APPLICATIONS, ALL COMPONENTS, framework
 Environment: multi-tenant multi-server
Reporter: Pierre Smits

 getSystemPropertyValue which accesses and retrieves tenant specific 
 properties is never called/used regarding default settings for a tenant.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Comment Edited] (OFBIZ-5458) getSystemPropertyValue is never called/used

2014-01-07 Thread Pierre Smits (JIRA)

[ 
https://issues.apache.org/jira/browse/OFBIZ-5458?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13864154#comment-13864154
 ] 

Pierre Smits edited comment on OFBIZ-5458 at 1/7/14 12:41 PM:
--

I did a quick assessment on how often UtilProperties.getPropertyValue and 
EntityUtilProperties.getProperties.GetPropertyValues occured in the code.

A search on UtilProperties.getPropertyValue matched 595 times across all 
components in apps, framework and specpo.
520 in .java files (more than 100 - I stopped counting -  including hard coded 
default value
72 in .groovy files (15 including hard coded default value)
3 in .xml files
This included also 61 EntityUtilProperties.getPropertyValue finds.

A search on EntityUtilProperties.GetPropertyValue revealed:
- in apps accounting, content, marketing, order, party, product and securityext
- in framework common, minilang, webapp and widget
- in specpo ebay, ecommerce and googlebase
- 34 finds had a hard coded default value
- 5 finds had no delegator reference

An earlier assessment on entries in .properties files revealed that there were 
over 1200 property definitions (not counting those in locale variations in 
specpo pos). But there might be more in other configuration files with 
different file extensions. 
So there might also be a mismatch between definitions and usage, indicating 
possible deprecated property definitions.


was (Author: pfm.smits):
I did a quick assessment on how often UtilProperties.getPropertyValue and 
EntityUtilProperties.getProperties.GetPropertyValues occured in the code.

A search on UtilProperties.getPropertyValue matched 595 times across all 
components in apps, framework and specpo.
520 in .java files (more than 100 - I stopped counting -  including hard coded 
default value
72 in .groovy files (15 including hard coded default value)
3 in .xml files
This included also 61 EntityUtilProperties.getPropertyValue finds.

A search on EntityUtilProperties.GetPropertyValue revealed:
- in apps accounting, content, marketing, order, party, product and securityext
- in framework common, minilang, webapp and widget
- in specpo ebay, ecommerce and googlebase
- 34 finds had a hard coded default value
- 5 finds had no delegator reference

An earlier assessment on entries in .properties files revealed that there were 
over 1200 property definitions (not counting those in locale variations in 
specpo pos). But there might be more in other configuration files with 
different file extensions. 
So there might also be a mismatch between definitions and usage, indicating 
deprecated properties.

 getSystemPropertyValue is never called/used
 ---

 Key: OFBIZ-5458
 URL: https://issues.apache.org/jira/browse/OFBIZ-5458
 Project: OFBiz
  Issue Type: Bug
  Components: ALL APPLICATIONS, ALL COMPONENTS, framework
 Environment: multi-tenant multi-server
Reporter: Pierre Smits

 getSystemPropertyValue which accesses and retrieves tenant specific 
 properties is never called/used regarding default settings for a tenant.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)


[jira] [Comment Edited] (OFBIZ-5458) getSystemPropertyValue is never called/used

2014-01-06 Thread Pierre Smits (JIRA)

[ 
https://issues.apache.org/jira/browse/OFBIZ-5458?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13863289#comment-13863289
 ] 

Pierre Smits edited comment on OFBIZ-5458 at 1/6/14 7:29 PM:
-

After further analysis I found that in EntityUtilProperties.xml there is a 
function called getPropertyValue that first tries to retrieve the value of the 
property from getSystemPropertyValue in the same file, and if it fails it will 
call on UtilProperties.getPropertyValue to retrieve the value from the 
.properties file.

This would mean that every reference in other files to 
UtilProperties.getPropertyValue should change to 
EntityUtilProperties.getPropertyValue.


was (Author: pfm.smits):
After further analysis I found that in EntityUtilProperties.xml there is a 
function called getPropertyValue that first tries to retrieve the value of the 
property from getSystemPropertyValue in the same file, and if it fails it will 
call on UtilProperties.getPropertyValue to retrieve the value from the 
.properties file.

This would mean that changing every reference in other files to 
UtilProperties.getPropertyValue should change to 
EntityUtilProperties.getPropertyValue.

 getSystemPropertyValue is never called/used
 ---

 Key: OFBIZ-5458
 URL: https://issues.apache.org/jira/browse/OFBIZ-5458
 Project: OFBiz
  Issue Type: Bug
  Components: ALL APPLICATIONS, ALL COMPONENTS, framework
 Environment: multi-tenant multi-server
Reporter: Pierre Smits

 getSystemPropertyValue which accesses and retrieves tenant specific 
 properties is never called/used regarding default settings for a tenant.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)