98107 - This page returns "java.lang.NumberFormatException 660099" error

Status: RESOLVED WONTFIX

(Core Graveyard :: Java: OJI, defect, P3)

Description

[Benjamin Mucci]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Macintosh; U; PPC; en-US; rv:0.9.3+) Gecko/20010903
BuildID:    2001090305

The error "java.lang.NumberFormatException 660099" is related to the stock
graphic above "Option sur action" which isn't displayed in Moz. MacOS 8.6 / MRJ
2.2.5. WFM in NS4.7.

Reproducible: Always

Comment 1

[Oliver Klee]

Confirming on 2001083103 Win2k. OS/Platform=All. The error message appears in
the Java console:

java.lang.NumberFormatException:  660099
 at java.lang.Integer.parseInt(Unknown Source)
 at Outils.convertColor(Outils.java:745)
 at IntradGraph.init(IntradGraph.java:57)
 at sun.applet.AppletPanel.run(Unknown Source)
 at java.lang.Thread.run(Unknown Source)

Comment 2

[Alexei V. Mokeev]

Reproduced on Linux and jdk1.3.1 too.
Moving to OJI.

Comment 3

[Alexei V. Mokeev]

The reason of this bug is invalid parameter handling in applet code

Applet tag on this page contain PARAM :
<param NAME=triBasColor  VALUE=" 660099">
and value of this param begins with space. In applet code this
value parsed via Integer.parseInt method. But it is illegal, because exception
thrown if(specification):
"..Any character of the string is not a digit of the specified radix, except
that the first character may be a minus sign '-' ('\u002d') provided that the
string is longer than length 1. "

So i suggest avoid spaces from parameter values in HTML or correct applet code
to trim spaces before parsing.

Internal Java of NS4.x and MSIE always performs trim of parameters so this
applet work in these browsers. 

Think that we may close this bug as INVALID

Comment 4

[Benjamin Mucci]

Same error here:

http://www.pch.gc.ca/english.htm

Comment 5

[Alexei V. Mokeev]

Do not see any errors at http://www.pch.gc.ca/english.htm
Could you please provide console output ?

Comment 6

[Benjamin Mucci]

An exception occurred:
java.lang.NumberFormatException: _self
	at java.lang.Integer.parseInt(Compiled Code)
	at java.lang.Integer.parseInt(Integer.java)
	at rp.get(Compiled Code)
	at aTicker.readFile(Compiled Code)
	at aTicker.init(aTicker.java)
	at 
com.apple.mrj.JManager.JMAppletViewer_OLD.doAppletInit(JMAppletViewerOld.java)
	at 
com.apple.mrj.JManager.JMAppletViewer_OLD.setState(JMAppletViewerOld.java)
	at com.apple.mrj.JManager.JMViewerEvent.post(JMAppletViewerOld.java)
	at com.apple.mrj.JManager.AVDispatcherThread.run(JMAppletViewerOld.java)
An exception occurred:
java.lang.NumberFormatException: _self
	at java.lang.Integer.parseInt(Compiled Code)
	at java.lang.Integer.parseInt(Compiled Code)
	at rp.get(Compiled Code)
	at aTicker.readFile(Compiled Code)
	at aTicker.init(aTicker.java)
	at 
com.apple.mrj.JManager.JMAppletViewer_OLD.doAppletInit(JMAppletViewerOld.java)
	at 
com.apple.mrj.JManager.JMAppletViewer_OLD.setState(JMAppletViewerOld.java)
	at com.apple.mrj.JManager.JMViewerEvent.post(JMAppletViewerOld.java)
	at com.apple.mrj.JManager.AVDispatcherThread.run(JMAppletViewerOld.java)

with
User-Agent: Mozilla/5.0 (Macintosh; U; PPC; en-US; rv:0.9.3+) Gecko/20010903
BuildID:    2001090305

Comment 7

[edburns]

I see this problem.

Comment 8

[edburns]

Attachment: cvs diff -u of fix for this bug, first iteration.

Comment 9

[Peter Lubczynski]

Comment on attachment 49252 [details] [diff] [review]
cvs diff -u of fix for this bug, first iteration.

r=peterl

Comment 10

[lchiang]

Ed, is this a change you think we might need on the 0.9.4 branch for the next 
netscape release?  per PDT

Comment 11

[edburns]

lchiang.  This is a low risk, mild benefit change.  I don't know how many apps 
out there have PARAM tags with values that contain trailing or leading spaces.  
My take is that we should try to get it in.

I'm looking for sr for the trunk now.

Comment 12

[rpotts (gone)]

Comment on attachment 49252 [details] [diff] [review]
cvs diff -u of fix for this bug, first iteration.

sr=[e-mail]

Comment 13

[Asa Dotzler [:asa]]

0.9.4 is out the door

Comment 14

[edburns]

Fix checked in to trunk.

Comment 15

[Patty Mac]

Verified on trunk build (2001-10-05-05-0.9.4). However, on the branch build,
(2001-10-05-05-0.9.4) the stock graph on the upper left still not load yet.

Comment 16

[Patty Mac]

I meant the trunk build (2001-10-03-03-trunk)

Comment 17

[Patty Mac]

The stock graph on the upper left is loading fine on windows 98 and Linux redhat
6.2 except Mac OS 9.2 (Commercialt trunk: 2001-10-25-08-trunk).
However, it still not able to load it on all platforms (commercial branch:
2001-10-22-18-0.9.4).

Comment 18

[Patty Mac]

The stock graph on the upper left is loading fine on windows 98 both commercial
trunk(2002-01-30-08-trunk)and branch build (2002-01-30-08-0.9.8).
It also loads fine on Mac OS 10.1 (commercial trunk: 2002-01-30-08-trunk), 
but not on OS 9.2(commercial trunk: 2002-01-30-08-trunk).
Therefore,I'm reopening this bug.

Comment 19

[Igor Nekrestyanov]

Would it be cleaner to mark this bug "Mac System 9.x" only?

Comment 20

[Patty Mac]

change the OS to Mac System 9.x only since this bug is currently happened on Mac 
os 9.2.
       

Comment 21

[Fuzzy Gorilla]

The Target Milestone (0.9.5) is long passed.  This bug should be retargeted.
There is no active patch for the remaining problem, so the Status Whiteboard
should be updated.  The keyword 'pp' should be added since this is now specific
to one platform.

Comment 22

[Igor Nekrestyanov]

Updated milestone and whiteboard status. Added pp keyword.

Ed does not work on OJI anymore
 -> Back to Joe

Comment 23

[Patty Mac]

Chris Petersen is a new QA contact for oji component. His email is:
[e-mail]

Comment 24

[Matthias Versen [:Matti]]

fixing small error for [e-mail] (filter with : SPAMMAILSUCKS)

Comment 25

[Simon Paquet [:sipaq]]

bug only happened on mac os 9, which is no longer supported. -> WONTFIX