Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0026412Openbravo ERPA. Platformpublic2014-04-29 11:462014-05-07 18:12
Reportercaristu 
Assigned Toalostale 
PriorityimmediateSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version3.0MP31.2 
Target VersionFixed in Version3.0PR14Q2 
Merge Request Status
Review Assigned ToAugustoMauch
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2013-12-20
Regression introduced in release
Regression introduced by commithttp://code.openbravo.com/erp/devel/pi/rev/b718b3ee2462bdd381bc76b332a779e3d8ecbe3d [^]
Triggers an Emergency PackNo
Summary0026412: Property fields based on computed columns with a FK fail
DescriptionHaving a computed column which is a FK to another entity. if a property column is defined to show any property of that FK entity, it fails.

In authorize.net module there are examples for this in Payment In > Lines tab. Ex.:

_computedColumns.obautnFinPaymentScheduledetail.active

Steps To Reproduce- Install authorize.net module
- Go to Payment In window, select a record
- Go to Lines -> Error appears
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 0024495 closed alostale Lazy computed columns API chage 
related to defect 00252483.0MP31 closed alostale Error after sorting by a computed column 
related to defect 0026432 closed alostale can't select a computed column for a property field. 
Attached Filespng error.png (29,514) 2014-04-29 11:46
https://issues.openbravo.com/file_download.php?file_id=6891&type=bug
png
Issue History
Date ModifiedUsernameFieldChange
2014-04-29 11:46caristuNew Issue
2014-04-29 11:46caristuAssigned To => AugustoMauch
2014-04-29 11:46caristuFile Added: error.png
2014-04-29 11:46caristuModules => Core
2014-04-29 11:46caristuTriggers an Emergency Pack => No
2014-04-29 11:46caristuIssue Monitored: networkb
2014-04-29 11:47caristuRegression level => Production - Confirmed Stable
2014-04-29 11:47caristuRelationship addedrelated to 0024495
2014-04-29 11:50caristuSummaryProperty field based on computed columns are not working => Property field based on computed columns are not working using the new API
2014-04-29 11:50caristuDescription Updatedbug_revision_view_page.php?rev_id=5768#r5768
2014-04-29 11:58caristuSummaryProperty field based on computed columns are not working using the new API => Property fields based on computed columns are not working using the new API
2014-04-29 12:10caristuSeveritymajor => critical
2014-04-30 09:06caristuSeveritycritical => minor
2014-04-30 15:38alostaleReview Assigned To => AugustoMauch
2014-04-30 15:38alostaleRegression date => 2013-12-20
2014-04-30 15:38alostaleRegression introduced by commit => http://code.openbravo.com/erp/devel/pi/rev/b718b3ee2462bdd381bc76b332a779e3d8ecbe3d [^]
2014-04-30 15:38alostaleSummaryProperty fields based on computed columns are not working using the new API => Property fields based on computed columns with a FK fail
2014-04-30 15:38alostaleDescription Updatedbug_revision_view_page.php?rev_id=5776#r5776
2014-04-30 15:38alostaleSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=5778#r5778
2014-04-30 15:38alostaleRelationship addedrelated to 0025248
2014-04-30 15:43alostaleRelationship addedrelated to 0026432
2014-04-30 15:46alostaleAssigned ToAugustoMauch => alostale
2014-04-30 15:50hgbotCheckin
2014-04-30 15:50hgbotNote Added: 0066712
2014-04-30 15:50hgbotStatusnew => resolved
2014-04-30 15:50hgbotResolutionopen => fixed
2014-04-30 15:50hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/cc9011c7b09ff1cefd032f2616db3ae0e4404e60 [^]
2014-04-30 15:51alostaleNote Added: 0066714
2014-05-02 09:32AugustoMauchNote Added: 0066779
2014-05-02 09:32AugustoMauchStatusresolved => new
2014-05-02 09:32AugustoMauchResolutionfixed => open
2014-05-02 12:13hgbotCheckin
2014-05-02 12:13hgbotNote Added: 0066788
2014-05-02 12:13hgbotStatusnew => resolved
2014-05-02 12:13hgbotResolutionopen => fixed
2014-05-02 12:13hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/cc9011c7b09ff1cefd032f2616db3ae0e4404e60 [^] => http://code.openbravo.com/erp/devel/pi/rev/bdc2ec814d75ecd4892096c7d9cda96a86b63bb8 [^]
2014-05-02 12:16AugustoMauchNote Added: 0066789
2014-05-02 12:16AugustoMauchStatusresolved => closed
2014-05-02 12:16AugustoMauchFixed in Version => PR14Q2
2014-05-07 18:12hudsonbotCheckin
2014-05-07 18:12hudsonbotNote Added: 0066988
2014-05-07 18:12hudsonbotCheckin
2014-05-07 18:12hudsonbotNote Added: 0066993

Notes
(0066712)
hgbot   
2014-04-30 15:50   
Repository: erp/devel/pi
Changeset: cc9011c7b09ff1cefd032f2616db3ae0e4404e60
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Wed Apr 30 15:48:32 2014 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/cc9011c7b09ff1cefd032f2616db3ae0e4404e60 [^]

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

---
M modules/org.openbravo.service.json/src/org/openbravo/service/json/JsonUtils.java
---
(0066714)
alostale   
2014-04-30 15:51   
Tested again all cases defined in 0025248
(0066779)
AugustoMauch   
2014-05-02 09:32   
Wrong if condition:

 if (true || !result.isProxy()) {
(0066788)
hgbot   
2014-05-02 12:13   
Repository: erp/devel/pi
Changeset: bdc2ec814d75ecd4892096c7d9cda96a86b63bb8
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Fri May 02 12:03:07 2014 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/bdc2ec814d75ecd4892096c7d9cda96a86b63bb8 [^]

Fixes issue 26412: Removes code that was meant just for debugging

---
M modules/org.openbravo.service.json/src/org/openbravo/service/json/JsonUtils.java
---
(0066789)
AugustoMauch   
2014-05-02 12:16   
Code reviewed and verified in pi@bdc2ec814d75
(0066988)
hudsonbot   
2014-05-07 18:12   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/02ca2648bcdc [^]
Maturity status: Test
(0066993)
hudsonbot   
2014-05-07 18:12   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/02ca2648bcdc [^]
Maturity status: Test