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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0019059
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] Z. Otherscriticalalways2011-11-14 10:152011-12-02 14:39
ReporterdmiguelezView Statuspublic 
Assigned Todmiguelez 
PriorityimmediateResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision06f744fa2469
ProjectionnoneETAnoneTarget Version
OSLinux 32 bitDatabasePostgreSQLJava version1.6.0_18
OS VersionCommunity ApplianceDatabase version8.3.9Ant version1.7.1
Product VersionSCM revision 
Review Assigned To
Web browserGoogle Chrome
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0019059: Issue 0018796: try-api fails. Requesting API change.

DescriptionDue to this issue,

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

I have to make an this API change:

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

I have pushed the change to try and this is the failure:

http://builds.openbravo.com/view/try/job/try-api/529/console [^]

- The change is the following:
The ReportGeneralLedgerJournalDetail was executing inside ReportGeneralLedgerJournal process.

- The change is needed due to the following reason:
In 3.0 this forces you to re-open ReportGeneralLedgerJournal after opening the Detail process if you want to execute ReportGeneralLedgerJournal again because there is not backwards button.

- The risk of this api change is very low because of the following reason:
The Detail's mapping is deleted in ReportGeneralLedgerJournal's process and a new mapping is created in ReportGeneralLedgerJournalDetail's process. The only risk is if someone is developing a module that opens tabs and uses the old mapping for the ReportGeneralLedgerJournalDetail. This is very unlikely to happen.
Steps To ReproduceNone
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0042810)
iciordia (manager)
2011-11-14 12:41

Approved. Ismael
(0042838)
hgbot (developer)
2011-11-15 16:00

Repository: erp/devel/api-checks
Changeset: 06f744fa2469998a3876a0c1e2c6bcded53dae79
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Mon Nov 14 13:16:13 2011 +0100
URL: http://code.openbravo.com/erp/devel/api-checks/rev/06f744fa2469998a3876a0c1e2c6bcded53dae79 [^]

Fixed issue 0019059: Issue 0018796: try-api fails. Requesting API change.

---
M model/src-db/database/sourcedata/AD_MODEL_OBJECT.xml
M model/src-db/database/sourcedata/AD_MODEL_OBJECT_MAPPING.xml
---
(0043120)
dmiguelez (developer)
2011-11-23 16:39

Testing plan

No testing plan required.

Low regression risk.
(0043695)
mirurita (developer)
2011-12-02 14:39

Very low risk api change approved by Isma.

- Issue History
Date Modified Username Field Change
2011-11-14 10:15 dmiguelez New Issue
2011-11-14 10:15 dmiguelez Assigned To => iciordia
2011-11-14 10:15 dmiguelez Web browser => Google Chrome
2011-11-14 10:15 dmiguelez Modules => Core
2011-11-14 12:41 iciordia Note Added: 0042810
2011-11-14 12:41 iciordia Assigned To iciordia => dmiguelez
2011-11-15 16:00 hgbot Checkin
2011-11-15 16:00 hgbot Note Added: 0042838
2011-11-15 16:00 hgbot Status new => resolved
2011-11-15 16:00 hgbot Resolution open => fixed
2011-11-15 16:00 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/api-checks/rev/06f744fa2469998a3876a0c1e2c6bcded53dae79 [^]
2011-11-23 16:39 dmiguelez Note Added: 0043120
2011-12-02 14:39 mirurita Note Added: 0043695
2011-12-02 14:39 mirurita Status resolved => closed


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker