Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0033024Openbravo ERPA. Platformpublic2016-05-03 12:352016-05-24 07:41
ReporterJONHM 
Assigned Tocaristu 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version3.0PR16Q1 
Target Version3.0PR16Q1.2Fixed in Version3.0PR16Q1.2 
Merge Request Status
Review Assigned Toalostale
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelPre packaging ( pi )
Regression date2016-04-08
Regression introduced in releasepi
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/8c53c9e9c26febf892a11cd30f5f5f81342cb2c2 [^]
Triggers an Emergency PackNo
Summary0033024: Cannot export CSV from Widgets
DescriptionAt trying to export CSV from any widget no file is exported nor error window showed
Steps To Reproduce1) As F&B International Group Admin create new widget with data on it
2) Click on the pencil and press "Export as CSV"

*Log shows: ERROR org.openbravo.service.datasource.DataSourceServlet - AccessTableNoView
org.openbravo.client.kernel.OBUserException: AccessTableNoView
Proposed SolutionSince the following changeset[1] 'tabId' is being sent in the request instead of 'tab'. So, before that changeset as the parameter name was not 'tabId' the following check was never true:

if (!hasAccess(request, parameters.get("tabId"))) {

Two possible solutions regarding the renamed parameter (tabId):

a) Take into account undefined value in if clause

b) Prevent sending undefined value from the client side and send null instead

[1] https://code.openbravo.com/erp/devel/pi/rev/8c53c9e9c26febf892a11cd30f5f5f81342cb2c2 [^]
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0032838 closed alostale Cannot export CSV from Widgets 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-05-23 12:17caristuTypedefect => backport
2016-05-23 12:17caristuTarget Version => 3.0PR16Q1.2
2016-05-23 12:38hgbotCheckin
2016-05-23 12:38hgbotNote Added: 0086646
2016-05-23 12:38hgbotStatusscheduled => resolved
2016-05-23 12:38hgbotResolutionopen => fixed
2016-05-23 12:38hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/50cfbabf4fe370d4f75bacfff17905437b9c324e [^] => http://code.openbravo.com/erp/backports/3.0PR16Q1.2/rev/0352a751baf7deec61225d7e48070a8ebcac3adf [^]
2016-05-23 12:40caristuReview Assigned Tocaristu => alostale
2016-05-23 12:40caristuAssigned Toalostale => caristu
2016-05-24 07:41alostaleStatusresolved => closed
2016-05-24 07:41alostaleFixed in Version => 3.0PR16Q1.2

Notes
(0086646)
hgbot   
2016-05-23 12:38   
Repository: erp/backports/3.0PR16Q1.2
Changeset: 0352a751baf7deec61225d7e48070a8ebcac3adf
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Mon May 23 12:36:08 2016 +0200
URL: http://code.openbravo.com/erp/backports/3.0PR16Q1.2/rev/0352a751baf7deec61225d7e48070a8ebcac3adf [^]

fixes issue 33024: Cannot export CSV from Widgets

  The name of the parameter used to sent the tab ID when exporting to CSV changed, this
  affected a check on DataSourceServlet that was always true when exporting widgets until now.

  Created an utility method to check whether a json value can be considered as
  empty which takes into account common cases of null and undefined literals.

---
M modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/DataSourceServlet.java
M modules/org.openbravo.service.json/src/org/openbravo/service/json/JsonConstants.java
M modules/org.openbravo.service.json/src/org/openbravo/service/json/JsonUtils.java
---