0011124: It is not possible to define SPACE as a groupping separator for a number format

Description

When defining SPACE character as a groupping separator the resulted value is empty as the src-core/src/org/openbravo/base/VariablesBase.java trims session value before checking if it is empty.

Proposed Solution

Don't trim value when checking the emptiness. See the attached patch.

Tags

No tags attached.

Attached Files

Allow-Spaces-for-session-values.diff [^] (542 bytes) 2009-10-26 11:32 [Show Content] [Hide Content]

Index: src-core/src/org/openbravo/base/VariablesBase.java
===================================================================
--- src-core/src/org/openbravo/base/VariablesBase.java	(revision 14928)
+++ src-core/src/org/openbravo/base/VariablesBase.java	(working copy)
@@ -888,7 +888,7 @@
     String auxStr = null;
     try {
       auxStr = (String) session.getAttribute(sessionAttribute.toUpperCase());
-      if (auxStr==null)
+      if (auxStr==null || auxStr.equals(""))
         auxStr = defaultValue;
     }
     catch (Exception e) {

Relationships [ Relation Graph ] [ Dependency Graph ]

Notes
(0052557) AugustoMauch (administrator) 2012-09-25 12:33	Effort: 1 Impact: low Plan: short

Issue History
Date Modified	Username	Field	Change
2009-10-26 11:32	villind	New Issue
2009-10-26 11:32	villind	Assigned To	=> rafaroda
2009-10-26 11:32	villind	File Added: Allow-Spaces-for-session-values.diff
2010-02-11 18:26	rafaroda	Assigned To	rafaroda => alostale
2010-02-15 12:21	alostale	Status	new => scheduled
2010-02-15 12:21	alostale	fix_in_branch	=> pi
2012-03-05 09:59	alostale	Type	defect => design defect
2012-03-05 09:59	alostale	fix_in_branch	pi =>
2012-09-25 12:33	AugustoMauch	Note Added: 0052557
2012-09-25 12:33	AugustoMauch	Priority	normal => high
2017-03-31 14:36	alostale	Status	scheduled => acknowledged
2017-04-10 14:35	alostale	Assigned To	alostale => platform
2022-02-01 08:08	alostale	Assigned To	platform => Triage Platform Base