Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||||||
| ID | ||||||||||||
| 0030807 | ||||||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||||||
| design defect | [Openbravo ERP] A. Platform | minor | have not tried | 2015-09-09 17:10 | 2022-02-01 08:09 | |||||||
| Reporter | alostale | View Status | public | |||||||||
| Assigned To | Triage Platform Base | |||||||||||
| Priority | normal | Resolution | open | Fixed in Version | ||||||||
| Status | new | Fix in branch | Fixed in SCM revision | |||||||||
| Projection | none | ETA | none | Target Version | ||||||||
| OS | Any | Database | Any | Java version | ||||||||
| OS Version | Database version | Ant version | ||||||||||
| Product Version | SCM revision | |||||||||||
| Review Assigned To | ||||||||||||
| Web browser | ||||||||||||
| Modules | Core | |||||||||||
| Regression level | ||||||||||||
| Regression date | ||||||||||||
| Regression introduced in release | ||||||||||||
| Regression introduced by commit | ||||||||||||
| Triggers an Emergency Pack | No | |||||||||||
| Summary | 0030807: review login in AdvancedQueryBuilder.substituteContextParameters | |||||||||||
| Description | Logic in org.openbravo.service.json.AdvancedQueryBuilder.substituteContextParameters method should be reviewed. There are 2 cases where might be incorrect: 1. Value is tried to be obtained first from session and after from request, request should be more specific than session so it should be done in the other way around. Though this change needs to be carefully verified from the functional point of view. 2. In case it is used for a wherclause in a selector invoked from a P&E opened from a window, tab and window ids are incorrectly obtained (they should be retrieved from inpTabId and inpWindowId request parameters instead of from tabId and windowId ones), this makes that when the value is tried to be obtained from session to be got always from session context not being able to check stored in session fields in the opening tab. Doing this change is not trivial because some values (such as isSoTrx used in Action Regading Document field in Add Payment P&E) to be changed from true to Y making the selector not to show the proper values | |||||||||||
| Steps To Reproduce | Case 1 cannot be reproduced within current distribution Case 2 was reproducible before fix for 0030623 | |||||||||||
| Tags | No tags attached. | |||||||||||
| Attached Files | ||||||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
| There are no notes attached to this issue. |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2015-09-09 17:10 | alostale | New Issue | |
| 2015-09-09 17:10 | alostale | Assigned To | => platform |
| 2015-09-09 17:10 | alostale | Modules | => Core |
| 2015-09-09 17:10 | alostale | Triggers an Emergency Pack | => No |
| 2015-09-10 08:05 | alostale | Relationship added | related to 0030623 |
| 2022-02-01 08:09 | alostale | Assigned To | platform => Triage Platform Base |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |