Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0030135Openbravo ERPA. Platformpublic2015-06-09 13:112015-06-25 18:46
Reportermalsasua 
Assigned Tocaristu 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS10OS VersionCommunity Appliance
Product Version 
Target Version3.0PR15Q3Fixed in Version3.0PR15Q3 
Merge Request Status
Review Assigned ToAugustoMauch
OBNetwork customer
Web browserGoogle Chrome
ModulesCore
Support ticket
Regression levelProduction - QA Approved
Regression date2014-11-18
Regression introduced in release3.0PR15Q1
Regression introduced by commithttp://code.openbravo.com/erp/devel/pi/rev/f7b4f6a3f08247b63fa6448a07187614d49de952 [^]
Triggers an Emergency PackNo
Summary0030135: On windows the 'ant setup' tool writes incorrect source.path to Openbravo.properties (not using \\ for a \)
DescriptionUsing 'ant setup' the tool writes source.path automatically but does not follow proper encoding of java properties files which required \\ to be used when a \ character is wanted as is

Effect of this is that any code using source.path (i.e. MMC, obx module installation and/or updates does not work)

Workaround is to manually edit source.path to have \\ for each \ which is wanted.

Note: Same applies to any path in properties (i.e. also attach.path)
Steps To ReproduceIn Windows OS:
.as system administrator, go to module management window
. select a obx file to install module
. error is returned:

Error: Error
org.openbravo.service.web.ResourceNotFoundException: Source file [C:openbravoerpopenbravo\src-db\database\model\tables\AD_MODULE.xml] can not be found. Verify 'source.path' property is properly set in Openbravo.properties file.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
depends on backport 00302443.0PR15Q2.3 closed caristu On windows the 'ant setup' tool writes incorrect source.path to Openbravo.properties (not using \\ for a \) 
caused by defect 00275423.0PR15Q1 closed inigosanchez Error in executing setup-properties-windows.exe in Windows 7 Ultimate 
has duplicate defect 00302833.0PR15Q3 closed platform Openbravo does not work in windows 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2015-06-09 13:11malsasuaNew Issue
2015-06-09 13:11malsasuaAssigned To => platform
2015-06-09 13:11malsasuaWeb browser => Google Chrome
2015-06-09 13:11malsasuaModules => Core
2015-06-09 13:11malsasuaTriggers an Emergency Pack => No
2015-06-09 13:18malsasuaWeb browserGoogle Chrome => Google Chrome
2015-06-09 13:18malsasuaSeveritymajor => minor
2015-06-10 17:59shuehnerWeb browserGoogle Chrome => Google Chrome
2015-06-10 17:59shuehnerNote Added: 0078193
2015-06-10 17:59shuehnerSummaryin windows, obx module can not be installed => On windows the 'ant setup' tool writes incorrect source.path to Openbravo.properties (not using \\ for a \)
2015-06-10 17:59shuehnerDescription Updatedbug_revision_view_page.php?rev_id=8809#r8809
2015-06-15 10:02alostaleStatusnew => acknowledged
2015-06-19 11:24alostaleAssigned Toplatform => inigosanchez
2015-06-24 10:50inigosanchezWeb browserGoogle Chrome => Google Chrome
2015-06-24 10:50inigosanchezOS20 => 10
2015-06-24 10:50inigosanchezSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=8882#r8882
2015-06-24 17:20inigosanchezStatusacknowledged => scheduled
2015-06-24 19:18caristuNote Added: 0078496
2015-06-24 19:18caristuNote Edited: 0078496bug_revision_view_page.php?bugnote_id=0078496#r8892
2015-06-25 08:45caristuNote Deleted: 0078496
2015-06-25 11:20caristuAssigned Toinigosanchez => caristu
2015-06-25 11:20caristuStatusscheduled => acknowledged
2015-06-25 11:21caristuStatusacknowledged => scheduled
2015-06-25 11:21caristuNote Added: 0078498
2015-06-25 11:24caristuWeb browserGoogle Chrome => Google Chrome
2015-06-25 11:24caristuRegression level => Production - QA Approved
2015-06-25 11:24caristuRegression date => 2014-11-18
2015-06-25 11:24caristuRegression introduced in release => 3.0PR15Q1
2015-06-25 11:24caristuRegression introduced by commit => http://code.openbravo.com/erp/devel/pi/rev/f7b4f6a3f08247b63fa6448a07187614d49de952 [^]
2015-06-25 12:29caristuIssue Monitored: AugustoMauch
2015-06-25 12:29caristuReview Assigned To => AugustoMauch
2015-06-25 12:29caristuWeb browserGoogle Chrome => Google Chrome
2015-06-25 12:40hgbotCheckin
2015-06-25 12:40hgbotNote Added: 0078504
2015-06-25 12:40hgbotStatusscheduled => resolved
2015-06-25 12:40hgbotResolutionopen => fixed
2015-06-25 12:40hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/988d6933a2bd1809b5f8276533fd3d4191f3ec88 [^]
2015-06-25 12:40caristuWeb browserGoogle Chrome => Google Chrome
2015-06-25 12:40caristuTarget Version => 3.0PR15Q3
2015-06-25 15:52dmitry_mezentsevRelationship addedcaused by 0027542
2015-06-25 16:35caristuNote Deleted: 0078498
2015-06-25 18:46AugustoMauchNote Added: 0078520
2015-06-25 18:46AugustoMauchStatusresolved => closed
2015-06-25 18:46AugustoMauchFixed in Version => 3.0PR15Q3
2015-06-30 07:50alostaleRelationship addedhas duplicate 0030283

Notes
(0078193)
shuehner   
2015-06-10 17:59   
Custom installation guide now as a special note to remember the \ quoting rule and also points to this (not yet fixed) issue.
(0078504)
hgbot   
2015-06-25 12:40   
Repository: erp/devel/pi
Changeset: 988d6933a2bd1809b5f8276533fd3d4191f3ec88
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Thu Jun 25 12:38:04 2015 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/988d6933a2bd1809b5f8276533fd3d4191f3ec88 [^]

Fixes 30135: On windows the ant setup tool writes incorrect source.path

The ant setup tool was writing an incorrect source.path because the user.dir property obtained under Windows contains backslashes as file separator in the path.
Now this case is controlled, and in case we are under a Windows operative system, the backslashes are replaced with a slash. The slash can be safely used in all platforms because Java internally can handle it and do the translation into platform-specific paths

---
M src/org/openbravo/configuration/ConfigurationApp.java
---
(0078520)
AugustoMauch   
2015-06-25 18:46   
Code reviewed and verified in pi@431da3d3ec95

Tested both in Linux and in Windows. In windows tested both with and without blank spaces in the source.path.