details: https://code.openbravo.com/erp/devel/main/rev/ec93bcb5091f changeset: 22844:ec93bcb5091f user: Asier Lostalé <asier.lostale <at> openbravo.com> date: Wed Apr 30 15:48:32 2014 +0200 summary: fixed bug 26412: property fields based on computed columns with a FK fail
property fields (sent to backend as additionalProperties) failed because an extra unneeded join to _computedColumns was performed details: https://code.openbravo.com/erp/devel/main/rev/0d16a3f22763 changeset: 22845:0d16a3f22763 user: Augusto Mauch <augusto.mauch <at> openbravo.com> date: Fri May 02 12:03:07 2014 +0200 summary: Fixes issue 26412: Removes code that was meant just for debugging diffstat: modules/org.openbravo.service.json/src/org/openbravo/service/json/JsonUtils.java | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diffs (16 lines): diff -r f05c1d1e1c17 -r 0d16a3f22763 modules/org.openbravo.service.json/src/org/openbravo/service/json/JsonUtils.java --- a/modules/org.openbravo.service.json/src/org/openbravo/service/json/JsonUtils.java Wed Apr 30 15:49:22 2014 +0200 +++ b/modules/org.openbravo.service.json/src/org/openbravo/service/json/JsonUtils.java Fri May 02 12:03:07 2014 +0200 @@ -314,7 +314,11 @@ return properties; } result = currentEntity.getProperty(part); - properties.add(result); + if (!result.isProxy()) { + // do not add explicit join to _computedColumns + properties.add(result); + } + if (result.getTargetEntity() != null) { currentEntity = result.getTargetEntity(); } ------------------------------------------------------------------------------ "Accelerate Dev Cycles with Automated Cross-Browser Testing - For FREE Instantly run your Selenium tests across 300+ browser/OS combos. Get unparalleled scalability from the best Selenium testing platform available. Simple to use. Nothing to install. Get started now for free." http://p.sf.net/sfu/SauceLabs _______________________________________________ Openbravo-commits mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openbravo-commits
