Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0022084Openbravo ERP09. Financial managementpublic2012-10-24 10:282012-10-24 11:25
psanjuan 
eduardo_Argal 
urgentminoralways
scheduledopen 
20Ubuntu 8.04.1
 
 
Core
No
0022084: Account Tree name is not properly created if installing a CoA module while running the Initial Organization setup process
The "Account Tree" name is not properly created if installing a CoA module of the Generic CoA module while running the Initial Organization setup process.

Create a Client named "Openbravo" by running the initial Client Setup.

Do not include accounting.

Exit

Log in by selecting Openbravo Admin - Openbravo Role

Create an Organization named "Openbravo" by running the initial organization setup process.

Include accounting. Select for instance the Generic CoA Module.

Navigate to the Account Tree window and realize that the name of the Account Tree is:

Openbravo Account Account

That is not correct.

The name should be "Openbravo Chart of Accounts" (in English) and "Openbravo Plan de Cuentas" (In Spanish) regardless if accounting is applied at Client level (Initial Client Setup) or at Organization level (Initial Organization Setup).
The name should be "Openbravo Name of the window (Account Tree)" (in English) and "Openbravo Árbol de Cuentas" (In Spanish) regardless if accounting is applied at Client level (Initial Client Setup) or at Organization level (Initial Organization Setup).
No tags attached.
Issue History
2012-10-24 10:28psanjuanNew Issue
2012-10-24 10:28psanjuanAssigned To => dmiguelez
2012-10-24 10:28psanjuanModules => Core
2012-10-24 10:31psanjuanSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=3995#r3995
2012-10-24 10:31psanjuanProposed Solution updated
2012-10-24 10:33psanjuanNote Added: 0053592
2012-10-24 10:46eduardo_ArgalStatusnew => scheduled
2012-10-24 10:46eduardo_ArgalAssigned Todmiguelez => eduardo_Argal
2012-10-24 10:46eduardo_Argalfix_in_branch => pi
2012-10-24 11:25psanjuanfix_in_branchpi =>
2012-10-24 11:25psanjuanProposed Solution updated

Notes
(0053592)
psanjuan   
2012-10-24 10:33   
the piece of code to review is:

COAUtility.insertElement(String strClientName, String strAccountString)

the COAUtility.createAccounting() is appending the strAccountString variable to the strElementName in case the accounts are created for the organization. After that the COAUtility.insertElement() method is executed, which appends again the strAccountString

Proposed solution.

Change:
if (organization != null) {
      strElementName = organization.getName() + " " + strAccountString;
    }

by

if (organization != null) {
      strElementName = organization.getName();
    }