Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0022339 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] Z. Others | major | have not tried | 2012-11-19 09:57 | 2013-01-18 09:34 | |||
Reporter | ioritzCia | View Status | public | |||||
Assigned To | ioritzCia | |||||||
Priority | immediate | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | pi | Fixed in SCM revision | 751258826d60 | |||
Projection | none | ETA | none | Target Version | ||||
OS | Linux 32 bit | Database | PostgreSQL | Java version | 1.6.0_22 | |||
OS Version | Ubuntu 10.04 | Database version | 8.4 | Ant version | 1.7.1 | |||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | ||||||||
OBNetwork customer | OBPS | |||||||
Web browser | Google Chrome | |||||||
Modules | Core | |||||||
Support ticket | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0022339: Fix to issue 21886 causes an api change. | |||||||
Description | Due to this issue: https://issues.openbravo.com/view.php?id=21886 [^] I have to make an api-change. This is the change: http://pastebin.com/CpshXksC [^] Due to this issue https://issues.openbravo.com/view.php?id=22176 [^] api change does not detect the change, so I cannot paste the failure. - The change is the following: M_PRODUCT_PO.VENDORPRODUCTNO column goes from not nullable to nullable. - The change is required due to the following reason: This column is in a unique constraint that needs to be removed due to 21886 issue. The unique constraint is that each product needs to have one unique vendorproductno for each business partner and this is not right, because two partners may have the same vendortproductno for the same product. - The risk is low: Someone may have made a development assuming this field would never be null. If that development did not have the null check and now that column is allowed to be null, it would cause an exception. Anyway these should not be very likely. | |||||||
Steps To Reproduce | None. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
||||||||
|
![]() |
|
(0054302) iciordia (viewer) 2012-11-19 11:31 |
Approved. Ismael |
(0054313) ioritzCia (viewer) 2012-11-19 13:36 |
No change fixes this API-Change. As said in the description, due to issue 21176 this will not fail in the tests. The SCM revision was set to the original issue's fix as the field is mandatory. |
![]() |
|||
Date Modified | Username | Field | Change |
2012-11-19 09:57 | ioritzCia | New Issue | |
2012-11-19 09:57 | ioritzCia | Assigned To | => dmiguelez |
2012-11-19 09:57 | ioritzCia | Web browser | => Google Chrome |
2012-11-19 09:57 | ioritzCia | Modules | => Core |
2012-11-19 09:57 | ioritzCia | OBNetwork customer | => Yes |
2012-11-19 09:57 | ioritzCia | Web browser | Google Chrome => Google Chrome |
2012-11-19 09:57 | ioritzCia | Assigned To | dmiguelez => iciordia |
2012-11-19 09:57 | ioritzCia | Status | new => feedback |
2012-11-19 11:32 | iciordia | Note Added: 0054302 | |
2012-11-19 11:32 | iciordia | Assigned To | iciordia => ioritzCia |
2012-11-19 11:32 | iciordia | Status | feedback => scheduled |
2012-11-19 11:32 | iciordia | fix_in_branch | => pi |
2012-11-19 13:23 | ioritzCia | Relationship added | depends on 0021886 |
2012-11-19 13:36 | ioritzCia | Note Added: 0054313 | |
2012-11-19 13:36 | ioritzCia | Status | scheduled => resolved |
2012-11-19 13:36 | ioritzCia | Fixed in SCM revision | => 751258826d60 |
2012-11-19 13:36 | ioritzCia | Resolution | open => fixed |
2013-01-18 09:34 | ioritzCia | Status | resolved => closed |
Copyright © 2000 - 2009 MantisBT Group |