Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0013794Openbravo ERPA. Platformpublic2010-06-28 10:302010-07-16 00:00
alostale 
alostale 
urgentminoralways
closedfixed 
5
 
 
Core
No
0013794: Send dependencies to check consistency
When scanning for updates and checking consistency for remote module installation, information about locally installed modules is sent.

The problem is that the only sent info is module module_id + version, without any additional info about dependencies.

This makes that in case the locally installed module versions are not present in central repository they are not taken into account when calculating consistency.
1- ERP instance with Initial Data Load 1.0.0 installed
2- Create a new module depending depending on Initial Data Load 1.0.0
3- Scan for updates -> Latest Initial Data Load version is proposed to be installed

Note that 3 is not correct because the module defined in 2 wouldn't satisfy dependencies if latest version of Initial Data Load is installed since it is a different major version than the one defined as dependency.
Additionally to module_id + version send the complete dependency list.

Note that this changes are already implemented in Central Repository, it is only pending to implement them in Openbravo ERP.
No tags attached.
Issue History
2010-06-28 10:30alostaleNew Issue
2010-06-28 10:30alostaleAssigned To => alostale
2010-06-28 10:31alostaleProposed Solution updated
2010-06-28 12:14alostaleSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=552#r552
2010-06-28 12:44hgbotCheckin
2010-06-28 12:44hgbotNote Added: 0028812
2010-06-28 12:44hgbotStatusnew => resolved
2010-06-28 12:44hgbotResolutionopen => fixed
2010-06-28 12:44hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/de99c80d26508b85b6e2a71b903e65465367784c [^]
2010-06-28 13:34hgbotCheckin
2010-06-28 13:34hgbotNote Added: 0028813
2010-06-30 12:17hudsonbotCheckin
2010-06-30 12:17hudsonbotNote Added: 0028914
2010-06-30 12:17hudsonbotCheckin
2010-06-30 12:17hudsonbotNote Added: 0028915
2010-07-15 10:40iperdomoNote Added: 0029332
2010-07-15 10:40iperdomoStatusresolved => closed
2010-07-16 00:00anonymoussf_bug_id0 => 3030267

Notes
(0028812)
hgbot   
2010-06-28 12:44   
Repository: erp/devel/pi
Changeset: de99c80d26508b85b6e2a71b903e65465367784c
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Mon Jun 28 12:43:45 2010 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/de99c80d26508b85b6e2a71b903e65465367784c [^]

fixed issue 13794: Send dependencies to check consistency

  Dependencies of locally installed modules are sent to central repository
  to check consistency. In this way it is possible to guarantee consistency
  even for module versions that are not registered in central repository.

---
M src/org/openbravo/erpCommon/modules/ImportModule.java
M src/org/openbravo/erpCommon/modules/VersionUtility.java
---
(0028813)
hgbot   
2010-06-28 13:34   
Repository: erp/devel/pi
Changeset: 1b3eed45fb417051b57208e730f307120666412f
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Mon Jun 28 13:33:56 2010 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/1b3eed45fb417051b57208e730f307120666412f [^]

issue 13794: removed unneded argument

---
M src/org/openbravo/erpCommon/modules/ImportModule.java
M src/org/openbravo/erpCommon/modules/VersionUtility.java
---
(0028914)
hudsonbot   
2010-06-30 12:17   
A changeset related to this issue has been promoted to main after passing a series of tests and an OBX has been generated:

Changeset: http://code.openbravo.com/erp/devel/main/rev/de99c80d2650 [^]
Merge Changeset: http://code.openbravo.com/erp/devel/main/rev/0670375d782d [^]
Tests: http://builds.openbravo.com/view/int/ [^]
OBX: http://builds.openbravo.com/erp/core/obx/OpenbravoERP-2.50CI.17726.obx [^]
(0028915)
hudsonbot   
2010-06-30 12:17   
A changeset related to this issue has been promoted to main after passing a series of tests and an OBX has been generated:

Changeset: http://code.openbravo.com/erp/devel/main/rev/1b3eed45fb41 [^]
Merge Changeset: http://code.openbravo.com/erp/devel/main/rev/0670375d782d [^]
Tests: http://builds.openbravo.com/view/int/ [^]
OBX: http://builds.openbravo.com/erp/core/obx/OpenbravoERP-2.50CI.17726.obx [^]
(0029332)
iperdomo   
2010-07-15 10:40   
Tested on pi @ rev 161d540df8db