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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0014045
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] Z. Otherscriticalhave not tried2010-07-26 13:112010-07-29 00:00
ReportershuehnerView Statuspublic 
Assigned Tomirurita 
PriorityurgentResolutionfixedFixed in Versionpi
StatusclosedFix in branchpiFixed in SCM revision6834c4694499
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

0014045: try api-check 79 fails

DescriptionThis (try) api-check build fails:

http://builds.openbravo.com/view/try/job/try-api/79/changes [^]

Two issues:
a.) Unique constraint added: table: FIN_FINACC_PAYMENTMETHOD - Unique constraint: FIN_FINACC_PAYMENTMETHOD_UN

b.) Changed column value AD_MESSAGE.VALUE -ID:968C37CD41AD45C3AD92D394CC357A90 from [FIN_FINACC_PAYMETHOD_UNIQUEKEY] to [FIN_FINACC_PAYMENTMETHOD_UN]
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0029556)
shuehner (administrator)
2010-07-26 13:14

Changeset relevant for this build:
https://code.openbravo.com/erp/devel/try/rev/832c0e77063b [^]

a.) is a false positive, as can be seen in the changeset. In addition to create this unique constraint a unique-index with exactly the same definition was removed. The the restriction placed on the is identical -> no data problem -> false positive

b.) Changing of the 'name' of a ad_message entry. Is done to correspond to the unique-index -> unique-constraint move. Potential issue: someone using the old name manully. However this name/message is only (in core) for this specific constraint (with the same name) and i think its unlikely that its reused in any custom code.

Suggestion for b.) accept as change with low risk
(0029570)
pjuvara (reporter)
2010-07-27 09:07

API change accepted.

Please send notice with risk level none.
(0029577)
mirurita (developer)
2010-07-27 10:49

Fixed on pi revision 7922:6834c4694499

https://code.openbravo.com/erp/devel/pi/rev/6834c4694499 [^]

and

api-checks repository
https://code.openbravo.com/erp/devel/api-checks/rev/0628a7ed4126 [^]

- Issue History
Date Modified Username Field Change
2010-07-26 13:11 shuehner New Issue
2010-07-26 13:11 shuehner Assigned To => mirurita
2010-07-26 13:14 shuehner Note Added: 0029556
2010-07-27 09:07 pjuvara Status new => scheduled
2010-07-27 09:07 pjuvara Note Added: 0029570
2010-07-27 09:07 pjuvara fix_in_branch => pi
2010-07-27 10:49 mirurita Note Added: 0029577
2010-07-27 10:49 mirurita Status scheduled => resolved
2010-07-27 10:49 mirurita Fixed in Version => pi
2010-07-27 10:49 mirurita Fixed in SCM revision => 6834c4694499
2010-07-27 10:49 mirurita Resolution open => fixed
2010-07-28 10:43 mirurita Status resolved => closed
2010-07-29 00:00 anonymous sf_bug_id 0 => 3036289


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker