Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0004893Openbravo ERPA. Platformpublic2008-09-09 17:262009-01-30 14:43
Reporterjordimas 
Assigned Toshuehner 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionout of date 
PlatformOS5OS Version
Product Versionpi 
Target VersionFixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0004893: Request to review FormatUtilities.replaceTildes
DescriptionHello,

The FormatUtilities class contains the following code:

  public static String replaceTildes(String strIni) {
    //Delete tilde characters
    return strIni.replace('á', 'a').replace('é', 'e').replace('í', 'i').replace('ó', 'o').replace('ú', 'u').replace('Á', 'A').replace('É', 'E').replace('Í', 'I').replace('Ó', 'O').replace('Ú', 'U');
  }

Which from an internationalization point of view is most likely wrong.

Steps To Reproduce
Proposed SolutionSuggested actions:

· Someone reviews the code and understands why is used
· Deletes the members or replaces it by a I18N friendly code
Additional Information
TagsNo tags attached.
Relationships
blocks feature request 0004446 acknowledged Triage Platform Base Review list of allowed characters in Tab/Window-names and normalization for internal use (i.e. FormatUtilities.replaceTildes) 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2008-09-09 17:26anonymousNew Issue
2008-09-09 17:26anonymousAssigned To => shuehner
2008-09-09 17:26anonymousStatusnew => scheduled
2008-09-09 17:26anonymousfix_in_branch => 2.40
2009-01-30 14:43shuehnerRegression testing => No
2009-01-30 14:43shuehnerStatusscheduled => closed
2009-01-30 14:43shuehnerNote Added: 0012906
2009-01-30 14:43shuehnerResolutionopen => out of date

Notes
(0012906)
shuehner   
2009-01-30 14:43   
no change required for 2.40 as original defect was changed to feature request to check the allowed characters for Tab-/Window-names and normalization of those for internal use.