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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0031425
TypeCategorySeverityReproducibilityDate SubmittedLast Update
design defect[Openbravo ERP] A. Platformtrivialhave not tried2015-11-11 11:022022-02-01 08:09
ReporteralostaleView Statuspublic 
Assigned ToTriage Platform Base 
PrioritynormalResolutionopenFixed in Version
StatusacknowledgedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0031425: many boolean columns incorrectly defined in AD and DB

DescriptionBoolean columns (reference Y/N) should be defined as:
1. Mandatory: other case when working with DAL a Boolean null object is obtained failing when doing "if (object)"
2. They should have standard DB check for Y/N values
Steps To ReproduceFor case 1, check with:
select t.tablename, c.columnname
 from ad_column c, ad_table t
where c.ad_reference_id = '20'
 and t.ad_table_id = c.ad_table_id
 and t.isview ='N'
 and c.ismandatory ='N'
 order by 1,2

For case 2, check with:
select t.tablename, c.columnname
 from ad_column c, ad_table t
where c.ad_reference_id = '20'
 and t.ad_table_id = c.ad_table_id
 and t.isview ='N'
 and not exists (
        select 1
        from pg_constraint ct, pg_class pt
        where pt.relname ilike t.tablename
        and ct.conrelid = pt.oid
        and contype ='c'
        and consrc ilike '%'||c.columnname||'%'
        and consrc ilike '%''Y''%'
        and consrc ilike '%''N''%')
order by 1,2
Proposed SolutionDecide whether we should enforce these two rules, if so, fix existent columns (note will be an API change) and automate test case to prevent it in future.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
duplicate of defect 0017609 acknowledgedTriage Platform Base Code-review for issue: 16639 

-  Notes
There are no notes attached to this issue.

- Issue History
Date Modified Username Field Change
2015-11-11 11:02 alostale New Issue
2015-11-11 11:02 alostale Assigned To => platform
2015-11-11 11:02 alostale Modules => Core
2015-11-11 11:02 alostale Triggers an Emergency Pack => No
2015-11-11 11:02 alostale Status new => acknowledged
2015-11-11 11:02 alostale Relationship added duplicate of 0017609
2022-02-01 08:09 alostale Assigned To platform => Triage Platform Base


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker