JavaScript Warnings

[Andrea W]

From your demo page (and similarly on my own web pages where the calendar is loaded), the following JavaScript warnings occur. It would be nice to fix these (I guess this is more of a feature request than bug, since they don't really hurt anything; I just don't like seeing that javascript warning triangle in my browser status bar...).

Again, I captured these from your demo page:

Warning: function olMain does not always return a value
Source File: http://www.easilysimplecalendar.com/...LIB/overlib.js
Line: 451, Column: 8
Source Code:
return;

Warning: redeclaration of var snapping
Source File: http://www.easilysimplecalendar.com/...LIB/overlib.js
Line: 658, Column: 7
Source Code:
var snapping = placeY % o3_snapy;

Warning: variable e hides argument
Source File: http://www.easilysimplecalendar.com/...LIB/overlib.js
Line: 676, Column: 5
Source Code:
var e = (e) ? e : event;

Warning: redeclaration of var hookPt
Source File: http://www.easilysimplecalendar.com/...LIB/overlib.js
Line: 1087, Column: 7
Source Code:
var hookPt=(hookType == 1 ? hookPt.before : hookPt.after);

Warning: function runHook does not always return a value
Source File: http://www.easilysimplecalendar.com/...LIB/overlib.js
Line: 1156, Column: 37
Source Code:
if (typeof l == 'undefined') return;

Warning: function runHook does not always return a value
Source File: http://www.easilysimplecalendar.com/...LIB/overlib.js
Line: 1159, Column: 23
Source Code:
if (!l.length) return;

Warning: function runHook does not always return a value
Source File: http://www.easilysimplecalendar.com/...LIB/overlib.js
Line: 1169, Column: 36
Source Code:
return eval(fnHookTo+'('+arS+')');

Warning: function runHook does not always return a value
Source File: http://www.easilysimplecalendar.com/...LIB/overlib.js
Line: 1171, Column: 28
Source Code:
return eval('l('+arS+')');

Warning: function runHook does not always return a value
Source File: http://www.easilysimplecalendar.com/...LIB/overlib.js
Line: 1174
Source Code:
}

Warning: redeclaration of var j
Source File: http://www.easilysimplecalendar.com/...LIB/overlib.js
Line: 1245, Column: 11
Source Code:
for (var j = 0; j < hookPt.length; j++) {

Warning: redeclaration of var i
Source File: http://www.easilysimplecalendar.com/...LIB/overlib.js
Line: 1250, Column: 13
Source Code:
for (var i = 0; i < fnRef.length; i++) if (hookPt[j] == fnRef[i]) {

Warning: redeclaration of var j
Source File: http://www.easilysimplecalendar.com/...LIB/overlib.js
Line: 1261, Column: 11
Source Code:
for (var j = 0; j < hookPt.length; j++) {

Warning: redeclaration of var i
Source File: http://www.easilysimplecalendar.com/...LIB/overlib.js
Line: 1263, Column: 12
Source Code:
for (var i = 0; i < order.length; i++) {

Warning: function reOrder does not always return a value
Source File: http://www.easilysimplecalendar.com/...LIB/overlib.js
Line: 1276, Column: 14
Source Code:
return hookPt;

Warning: assignment to undeclared variable DONOTHING
Source File: http://www.easilysimplecalendar.com/...LIB/overlib.js
Line: 1067

[There are a bunch of warnings about line 1067 since it loops. It complains about DONOTHING, INARRAY, CAPARRAY, STICKY, BACKGROUND, NOCLOSE, CAPTION, LEFT, RIGHT, CENTER, OFFSETX, OFFSETY, FGCOLOR, BGCOLOR, TEXTCOLOR, CAPCOLOR, CLOSECOLOR, WIDTH, BORDER, CELLPAD, STATUS, AUTOSTATUS, AUTOSTATUSCAP, HEIGHT, CLOSETEXT, SNAPX, SNAPY, FIXX, FIXY, RELX, RELY, FGBACKGROUND, BGBACKGROUND, PADX, PADY, FULLHTML, ABOVE, BELOW, CAPICON, TEXTFONT, CAPTIONFONT, CLOSEFONT, TEXTSIZE, CAPTIONSIZE, CLOSESIZE, TIMEOUT, FUNCTION, DELAY, HAUTO, VAUTO, CLOSECLICK, WRAP, FOLLOWMOUSE, MOUSEOFF, CLOSETITLE, CSSOFF, CSSCLASS, FGCLASS, BGCLASS, TEXTFONTCLASS, CAPTIONFONTCLASS, CLOSEFONTCLASS.]

Warning: assignment to undeclared variable hookPts
Source File: http://www.easilysimplecalendar.com/...LIB/overlib.js
Line: 1076

[Brian]

What browser and operating system are you using?

Do you get the same error if you visit overLIB's web site (http://www.bosrup.com/web/overlib/) ?

[Andrea W]

I do get javascript warnings when I visit that page.

I use Mozilla Firefox version 0.8 for my browser. It has a nice Javascript console. I'm running it on Win XP Pro.

Most of these warnings are very nitpicky. For instance on overlib page, this is one of the warnings I get:

Warning: function olMain does not always return a value
Source File: http://www.bosrup.com/web/overlib/overlib.js
Line: 451, Column: 8
Source Code:
return;

It's complaining that it just does a return instead of returning a value (like return true. But if this is just a prepackaged set of functions you use, then I understand that you shouldn't fix them and that the overlib developer should.

[Brian]

I've tested the calendar script with overLIB integration on several computers with both Firefox .8 and .9 (along with many other browsers) and don't see the errors that you are referring to. If I open the JavaScript console, it is blank? This is on XP Pro, XP, and 2000 (tested with Firefox).

What steps are you going through when you get these errors?

[Andrea W]

Off topic for a bit... I'm no longer getting instant email notifications to the threads I'm subscribed to. It was working a few days ago. Weird. Anyway...

I'm not doing anything special to see the warnings in the JavaScript Console. I just go to my page that the calendar is displayed on ( http://darius.mobius-soft.com/ ) and in Firefox 0.8, I select JavaScript Console from the tools menu. Click Clear in the console window. Refresh my page in Firefox, and then all the JS warnings are displayed in my console window. I use Firefox 0.8 on Win XP Pro (Version 2002 Service Pack 1 according to my Control Panel -> System -> General Properties).

[Brian]

I've tried, but I really can't make this occur on any test installation I have. My only suggestion is to make sure you are using the latest java download that's available from the Sun web site.

[ve9gra]

These are all warnings... I don't see a single compilation or execution error in there. I guess FireFox is a bit more picky about what it runs.... I can get to your site with no errors at all.

[Andrea W]

Yes, the title of my thread is "Warnings" and not "Errors." I agree Firefox is picky. This was just a suggestion. It would be nice to quell these warning messages.

Perhaps this thread should be moved from bugs and errors to feature requests. Sorry I posted it in the wrong place.