JacquesLeRoux commented on PR #635: URL: https://github.com/apache/ofbiz-framework/pull/635#issuecomment-1539454339
Hi @thahn27, I checked the usage in Java and Groovy code of the previous methods I talked about above. Most of them are not used at all OOTB. getPropertyAsBoolean and getPropertyAsBigDecimal are both used twice in Java, getPropertyAsInteger is used 21 times in Java and 6 in Groovy. It could be that OFBiz users are using other them more. But it's really easy to replace them by UtilProperties ones. So maybe we could remove them all together OOTB. And replace those used by calls to the new ones you introduce. I'll start a discussion on dev ML about that. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@ofbiz.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
