Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0033440
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSmajoralways2016-07-06 12:162016-07-12 13:44
ReporteraaroncaleroView Statuspublic 
Assigned Toranjith_qualiantech_com 
PriorityhighResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision4edcaf0e555a
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tomarvintm
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0033440: OB.Dal.removeRemoteModels fails if there is a remote model defined without a tableName

DescriptionIis is possible to define OB.Data.ExtensibleModels (such as Products, BPs,...) without tableName, which can be used to store temporary results (e.g. query results, function results). These models can be defined as remote models.

The OB.Dal.removeRemoteModels is executed on the logout, removing remote model's tables from the local database.

If a model without a table defined is configured as a remote model, the OB.Dal.removeRemoteModels call fails (only in development/debug mode) and the logout is never executed.
Steps To ReproduceDefine the following dummy model:
  OB.Model.ATest = OB.Data.ExtensibleModel.extend({
    remote: 'OBPOS_remote.customer'
  });
Having the instance in development mode, login in web pos, and then logout.
Verify the following exception appears in the browser console:
Uncaught OB.Dal.removeAll: tableName not found
And the application stays 'Logging out' forever.
Proposed SolutionAdd a validation to the OB.Dal.removeRemoteModels to verify that the model has a tableName defined before calling to removeAll (OB.Dal.getTableName can be used for this).
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0088379)
hgbot (developer)
2016-07-12 10:56

Repository: tools/automation/pi-mobile
Changeset: 8a382fb0ad91f177ff6e05657622ced3727340b8
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Tue Jul 12 14:26:16 2016 +0530
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/8a382fb0ad91f177ff6e05657622ced3727340b8 [^]

Verifies issue 33440 : Added automated test 'I33440_VerifyRemoteModelOnRemove'

---
A src-test/org/openbravo/test/mobile/quarantine/pack/RAN/loginout/I33440_VerifyRemoteModelOnRemove.java
---
(0088380)
hgbot (developer)
2016-07-12 10:56

Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 4edcaf0e555acabaa6e0b1ea889402d40bb1a028
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Tue Jul 12 14:25:28 2016 +0530
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/4edcaf0e555acabaa6e0b1ea889402d40bb1a028 [^]

Fixes issue 33440 : Added validation to check the model has tablename or not

---
M web/org.openbravo.mobile.core/source/data/ob-dal.js
---

- Issue History
Date Modified Username Field Change
2016-07-06 12:16 aaroncalero New Issue
2016-07-06 12:16 aaroncalero Assigned To => Retail
2016-07-06 12:16 aaroncalero Resolution time => 1468965600
2016-07-06 12:16 aaroncalero Triggers an Emergency Pack => No
2016-07-06 12:30 marvintm Status new => acknowledged
2016-07-07 11:14 ranjith_qualiantech_com Assigned To Retail => ranjith_qualiantech_com
2016-07-07 11:14 ranjith_qualiantech_com Status acknowledged => scheduled
2016-07-12 10:56 hgbot Checkin
2016-07-12 10:56 hgbot Note Added: 0088379
2016-07-12 10:56 hgbot Checkin
2016-07-12 10:56 hgbot Note Added: 0088380
2016-07-12 10:56 hgbot Status scheduled => resolved
2016-07-12 10:56 hgbot Resolution open => fixed
2016-07-12 10:56 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/4edcaf0e555acabaa6e0b1ea889402d40bb1a028 [^]
2016-07-12 13:44 marvintm Review Assigned To => marvintm
2016-07-12 13:44 marvintm Status resolved => closed


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker