Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0022274Openbravo ERPZ. Otherspublic2012-11-12 17:012012-11-20 12:18
Reporterngarcia 
Assigned Tongarcia 
PriorityimmediateSeveritycriticalReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned Tongarcia
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0022274: API Change: Delete NOT NULL constraint from some columns related with accounts
DescriptionThe resolution of this issue:

https://issues.openbravo.com/view.php?id=22264 [^]

needs the following API change:

https://code.openbravo.com/erp/devel/try/rev/76d92b3adbde [^]

It does not break the try-api test due to the following bug:

https://issues.openbravo.com/view.php?id=22176 [^]

Change:

Delete the NOT NULL constraint from the following columns:

C_ACCTSCHEMA_DEFAULT - CH_EXPENSE_ACCT
C_ACCTSCHEMA_DEFAULT - CB_EXPENSE_ACCT
C_ACCTSCHEMA_DEFAULT - CB_RECEIPT_ACCT
C_CASHBOOK_ACCT - CB_EXPENSE_ACCT
C_CASHBOOK_ACCT - CB_RECEIPT_ACCT

Reason:

As the associated fields has been hidden a new record cannot be saved in the General Ledger Configuration - Defaults tab.

Risk:

Low. It could be that somebody has created a development thinking that this column would always have a value
Steps To Reproducenone
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 0022264 closed ngarcia Some default accounts of General Ledger are mandatory but were set to inactive 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2012-11-12 17:01ngarciaNew Issue
2012-11-12 17:01ngarciaAssigned To => iciordia
2012-11-12 17:01ngarciaModules => Core
2012-11-12 17:03ngarciaDescription Updatedbug_revision_view_page.php?rev_id=4055#r4055
2012-11-12 17:04ngarciaStatusnew => feedback
2012-11-12 17:18ngarciaRelationship addedrelated to 0022264
2012-11-12 18:56ngarciaStatusfeedback => scheduled
2012-11-12 18:56ngarciaNote Added: 0054169
2012-11-12 18:56ngarciaAssigned Toiciordia => ngarcia
2012-11-12 18:56ngarciafix_in_branch => pi
2012-11-20 12:17ngarciaStatusscheduled => resolved
2012-11-20 12:17ngarciaFixed in SCM revision => none
2012-11-20 12:17ngarciaResolutionopen => fixed
2012-11-20 12:18ngarciaReview Assigned To => ngarcia
2012-11-20 12:18ngarciaNote Added: 0054339
2012-11-20 12:18ngarciaStatusresolved => closed

Notes
(0054169)
ngarcia   
2012-11-12 18:56   
approved on his behalf
(0054339)
ngarcia   
2012-11-20 12:18   
commit to api-checks skipped because the test did not fail