Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0015900Openbravo ERPA. Platformpublic2011-02-09 14:152011-05-24 10:56
Reportershuehner 
Assigned Toshuehner 
PriorityurgentSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0015900: ModelProvider.getTables() should return tables usable via dal/ModelProvider (and not all tables)
DescriptionMost methods of the ModelProvider class do return the tables/columns/entities etc usable via dal.

However the public getTables() method does return all tables including those which are not usable via DAL (i.e. tables not having any column defined as primary key).

Steps To Reproduce
Proposed SolutionChange implementation to be consistent with the other function and not return the tables which are not usable by DAL.

Nice side-effect this info is already loaded in the in memory model, so the changed getTables could be used without any db-reads necessary.
Additional Information
TagsclosingMay2011
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2011-02-09 14:15shuehnerNew Issue
2011-02-09 14:15shuehnerAssigned To => shuehner
2011-02-09 14:15shuehnerModules => Core
2011-02-09 14:15shuehnerOBNetwork customer => No
2011-02-09 15:12hgbotCheckin
2011-02-09 15:12hgbotNote Added: 0034101
2011-02-09 15:12hgbotStatusnew => resolved
2011-02-09 15:12hgbotResolutionopen => fixed
2011-02-09 15:12hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/d98f5745a83031ee6f841afa2810c2b96182b5c6 [^]
2011-05-24 10:08dalsasuaTag Attached: closingMay2011
2011-05-24 10:56dalsasuaStatusresolved => closed

Notes
(0034101)
hgbot   
2011-02-09 15:12   
Repository: erp/devel/pi
Changeset: d98f5745a83031ee6f841afa2810c2b96182b5c6
Author: Stefan Hühner <stefan.huehner <at> openbravo.com>
Date: Wed Feb 09 15:12:04 2011 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/d98f5745a83031ee6f841afa2810c2b96182b5c6 [^]

Fixed 15900. Change getTables() to only return tables usable with DAL.
This changes the list of tables returned by ModelProvider.getTables() to only
return the list of tables usable with dal and not all the tables which are
defined in ad_table. Difference between those are tables not supported by dal
i.e. which do not have any column defined which is marked as primary key.

---
M src/org/openbravo/base/model/ModelProvider.java
---