[jira] [Updated] (OFBIZ-6276) Fix deprecation issues in framework/entity
[ https://issues.apache.org/jira/browse/OFBIZ-6276?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Gil Portenseigne updated OFBIZ-6276: Attachment: OFBIZ-6276-EntityStoreOptions-SVN.patch Hi Martin, i just try to use your patch, it's working fine, I regenerate a pure SVN patch, from it (using linux svn shell command), there it is ! > Fix deprecation issues in framework/entity > -- > > Key: OFBIZ-6276 > URL: https://issues.apache.org/jira/browse/OFBIZ-6276 > Project: OFBiz > Issue Type: Sub-task > Components: framework >Reporter: Adam Heath >Assignee: Jacques Le Roux >Priority: Minor > Attachments: OFBIZ-6276-EntityStoreOptions-SVN.patch, > OFBIZ-6276-EntityStoreOptions.patch, OFBIZ-6276-variant-1.patch, > OFBIZ-6276-variant-2.patch > > > It appears that the storeAll() variants are being refactor; basically, the > useCache parameter is being removed. However, this has caused the > entity engine to have deprecation with itself. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Updated] (OFBIZ-6276) Fix deprecation issues in framework/entity
[ https://issues.apache.org/jira/browse/OFBIZ-6276?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Martin Becker updated OFBIZ-6276: - Attachment: OFBIZ-6276-EntityStoreOptions.patch Added new patch which removes the use of deprecated API and the deprecated API itself from Delegator and GenericDelegator. Introduced the EntityStoreOptions class as mentioned in a comment before to substitute the createDummyFks parameter of storeAll method to avoid ambiguous API after removing the deprecated doCacheClear parameter. The decryptFieldValue method, just defined deprecated by Jacopo is left there as deprecated. Only the old deprecated methods where removed. Fix deprecation issues in framework/entity -- Key: OFBIZ-6276 URL: https://issues.apache.org/jira/browse/OFBIZ-6276 Project: OFBiz Issue Type: Sub-task Components: framework Reporter: Adam Heath Assignee: Jacques Le Roux Priority: Minor Attachments: OFBIZ-6276-EntityStoreOptions.patch, OFBIZ-6276-variant-1.patch, OFBIZ-6276-variant-2.patch It appears that the storeAll() variants are being refactor; basically, the useCache parameter is being removed. However, this has caused the entity engine to have deprecation with itself. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Updated] (OFBIZ-6276) Fix deprecation issues in framework/entity
[ https://issues.apache.org/jira/browse/OFBIZ-6276?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Martin Becker updated OFBIZ-6276: - Attachment: OFBIZ-6276-variant-2.patch OFBIZ-6276-variant-1.patch It has to be ensured, that with the future removal of the doCacheClear parameter from the method signature: from {{storeAll(ListGenericValue values, boolean doCacheClear, boolean createDummyFks)}} to {{storeAll(ListGenericValue values, boolean createDummyFks)}} the signature afterwards does not match the old signature with different meaning: {{storeAll(ListGenericValue values, boolean doCacheClear)}} where legacy code would get no compiler error and may miss the changed meaning. I implemented two variants of a solution for this: With the first I renamed the method with the concrete implementation from storeAll to storeAllInternal and changed the visibility to private, so there is no collision with the other signature with {{ListGenericValue, boolean}} params. The createDummyFks function is made public available with a new method {{storeAllWithDummyFks}}. The second variant is simpler and without the need of changed method names, it just switches the order of the parameters in the signature, which is not so nice in reading the signature but avoids special method name for the createDummyFks feature. The next step for me would be to remove the deprecated code from the Delegator implementation in the context of OFBIZ-6273. Fix deprecation issues in framework/entity -- Key: OFBIZ-6276 URL: https://issues.apache.org/jira/browse/OFBIZ-6276 Project: OFBiz Issue Type: Sub-task Components: framework Reporter: Adam Heath Priority: Minor Attachments: OFBIZ-6276-variant-1.patch, OFBIZ-6276-variant-2.patch It appears that the storeAll() variants are being refactor; basically, the useCache parameter is being removed. However, this has caused the entity engine to have deprecation with itself. -- This message was sent by Atlassian JIRA (v6.3.4#6332)