Details
-
Type: Bug
-
Status: Closed
-
Priority: Major
-
Resolution: Fixed
-
Affects Version/s: 1.5.2
-
Component/s: ICE-Components
-
Labels:None
-
Environment:Operating System: Windows XP
Platform: PC
Description
the number formatting used in EffectsArguments puts a comma on numbers > 999 and
so using, say, 1000, generates the following javascript which breaks the effect:
new Effect.Highlight(id,{startcolor:'#ffff99',duration:1,000})
suggested fix:
---
C:\work\frameworks\ICEfaces-1.5.3-src\icefaces\core\src\com\icesoft\faces\context\effects\EffectsArguments.java
2006-12-04 10:55:20.000000000 -0700
+++
C:\work\frameworks\ICEfaces-1.5.3-src-Inovis\icefaces\core\src\com\icesoft\faces\context\effects\EffectsArguments.java
2007-02-12 12:05:42.000000000 -0700
@@ -44,13 +44,13 @@
private Map map = new HashMap();
private List parameter = new ArrayList();
private NumberFormat floatFormat = NumberFormat.getNumberInstance(Locale.US);
public EffectsArguments(){
-
+ floatFormat.setGroupingUsed(false);
}
so using, say, 1000, generates the following javascript which breaks the effect:
new Effect.Highlight(id,{startcolor:'#ffff99',duration:1,000})
suggested fix:
---
C:\work\frameworks\ICEfaces-1.5.3-src\icefaces\core\src\com\icesoft\faces\context\effects\EffectsArguments.java
2006-12-04 10:55:20.000000000 -0700
+++
C:\work\frameworks\ICEfaces-1.5.3-src-Inovis\icefaces\core\src\com\icesoft\faces\context\effects\EffectsArguments.java
2007-02-12 12:05:42.000000000 -0700
@@ -44,13 +44,13 @@
private Map map = new HashMap();
private List parameter = new ArrayList();
private NumberFormat floatFormat = NumberFormat.getNumberInstance(Locale.US);
public EffectsArguments(){
-
+ floatFormat.setGroupingUsed(false);
}
Activity
Philip Breau
created issue -
Icefaces Administrator
made changes -
Field | Original Value | New Value |
---|---|---|
issue.field.bugzillaimportkey | 1331 | 12527 |
Ken Fyten
made changes -
Fix Version/s | 1.6 [ 10031 ] |
Ken Fyten
made changes -
Fix Version/s | 1.6DR1 [ 10035 ] | |
Fix Version/s | 1.6 [ 10031 ] |
Ken Fyten
made changes -
Fix Version/s | 1.6 [ 10031 ] |
Ken Fyten
made changes -
Status | Resolved [ 5 ] | Closed [ 6 ] |
Assignee | Rob Mayhew [ rob.mayhew ] |
Fixed in head r13226