Submit/Firing Order, Take 2

[John Kiernan]

Hey JavaScript gurus...

I'm going to try this again. I haven't gotten as much help as I have advice
on style<grin>. I appreciate (having programmed in other languages for
quite a while) that everyone has their own opinion on HOW things should be
done. However, if it does not comes AFTER helping me figure out my problem,
please... (fill in your own polite way to say 'save the bandwidth').

I have code in a MouseUp of my save button (yes, technically it should go
into the 'OnSubmit()' of the form, but I don't want to do that, because I'm
already hacking one pre-packaged class (in VFP) and I don't want to be
hacking a second). There is NO "OnSubmit" on the form itself (to pre-answer
a question).

On everything else in the JavaScript function I'm talking about, if I trap
for an input error, I return a 'false' and the form doesn't submit. If I
return a 'true', the Form submits just fine.

However, on the last part of the routine utilizes a 'Confirm()' -- within an
IF statement. As per a hint from someone here, I changed it from utilizing
a variable to simply returning a true or false. However, it doesn't work --
in the same way the variable didn't work. If I return a false, the form
doesn't submit, but even if I return a true, the form doesn't submit.

Using deductive reasoning in that I have the Confirm() within an IF
statement (and it's the only thing within it) and the whole thing works if I
don't enter the IF and it DOESN"T work if I do, I think I can narrow the
problem down to the Confirm().

I would simply put in a hard coded document.form.s ubmit(), except that
doesn't work with the back end VFP stuff I have interpreting the return form
variables.

Can anyone else duoplicate this or help!?!?

I can't imagine someone else sometime hasn't put in a confirm() in this way?
Could it be? Code pasted in below...

Lost in Santa Monica.... (HELLLPPPP!!!)

-- John Kiernan, KierPro Associates
Custom VFP/Accounting Programming
and Web interfaces
VFP and/or SQL back ends

function cliservsave() {
lvIsnew = document.getEle mentById('isnew ');
lvIsnewV = lvIsnew.value
lvRepf = document.getEle mentById('repfr eq');
lvRepfV = lvRepf.value;
lvNotes = document.getEle mentById('NOTES ');
lvNVal = lvNotes.value;
lvDue = document.getEle mentById('dueda te');
lvDueV = lvDue.value;
lvNotComp = document.getEle mentById('NOTCO MP');
lvNCVal = lvNotComp.check ed;

if (lvNVal == "" && lvNCVal == true) {
alert("If Not Completed, Notes MUST be entered.");
return false;
}

lvDescrip = document.getEle mentById('DESCR IP');
lvDVal = lvDescrip.value ;
if (lvDVal == "") {
alert("Descript ion Cannot be Blank.");
return false;
}

if (lvNCVal == true && (lvDueV !== "" && lvDueV !== "01/01/00") ) {
alert("If Not Completed, Due Date must be Blank.");
return false;
}

if (lvIsnewV == "true" && lvRepfV !== "None") {
return confirm("Are you sure you want to generate multiple records?")
}

return true;
}

[John Kiernan]

AHA!!!!!!!!!!!! !!!!

No more calls!!! We *have* a winner!!!

Thank you Richard! Thank You! Thank You! Thank You! Thank You! Thank
You!

The answer was I need to do what Richard said:

OnMouseUp="retu rn mousetest();"
Instead of what I was doing:
OnMouseUp="mous etest();"

It was running the function, but not RETURNing the value I was sending
back!!!!

(Even though I'm a novice to JS, I really should have known better!)

Thanks again to all who answered....

-- John Kiernan, KierPro Associates
Custom VFP/Accounting Programming
and Web interfaces
VFP and/or SQL back ends
"Richard Cornford" <Ri*****@litote s.demon.co.uk> wrote in message
news:cf******** ***********@new s.demon.co.uk.. . John Kiernan wrote:
<snip>

<p><input type="submit" value="Submit" name="B1"
OnMouseUp=mouse test()></p>

<snip>

The string value provided for an event handling attribute in HTML is
used by the browser to create a function that is assigned to a property
of the corresponding element within the DOM. Disregarding any provision
of custom scope chains that may be made by the browser, the above
onMouseUp handler is equivalent to:-

document.forms[0].elements['B1'].onmouseup = function(){
mousetest();
}

- executed with javascript. Thus the function object that represents the
event handling method of the submit button is a function that calls
another function, but returns no value. As a result it doesn't matter
much what the - mousetest - function returns as the event handler is not
passing that value on. It cannot cancel the event (unless the
proprietary returnValue property of the event object were set (on IE))
and so whatever default action follows from the event will happen.

A more viable formulation would be:-

OnMouseUp="retu rn mousetest();"

- so the value returned from - mousetest - is also returned form the
actual event handler. (note that the attribute value should be quoted as
it contains characters that are not allowed in an unquoted value
(according to the HTML specification). Validating the HTML would have
pointed that out, and valid HTML is a considerable aid in easy and
reliable scripting as it results in a consistently structured DOM that
is not subject to the vagaries of HTML error-correction.)

Richard.

[Alberto]

This depends on the fact that like, if I am not mistaken Michael told you,
a few very good guys loiter here :-)
No I don't surreptitiously put myself in the pack LOL, but it is always
BEAUTIFUL to see constructive answers that lead to sove a problem fo a
person. I like when this happens, no matter whose was the answer.

if you consider how good the group can be when persons mind only to answer
questions and not to make silly rebuttals (in the past happened), you really
realize that the quality, also human here, can be interesting. Persons that
fought with javascript for years pop in here :-) N ever dismiss these guys,
all of a sudden can make the difference lol.

And now say thank you again to Richard :-) - *joking*

ciao, good luck!
Alberto
http://www.unitedscripters.com/
"John Kiernan" <ki********@ver izon.net> ha scritto nel messaggio
news:7N******** ********@nwrddc 04.gnilink.net. ..

AHA!!!!!!!!!!!! !!!!

No more calls!!! We *have* a winner!!!

Thank you Richard! Thank You! Thank You! Thank You! Thank You! Thank
You!

The answer was I need to do what Richard said:

OnMouseUp="retu rn mousetest();"

Instead of what I was doing:
OnMouseUp="mous etest();"

It was running the function, but not RETURNing the value I was sending
back!!!!

(Even though I'm a novice to JS, I really should have known better!)

Thanks again to all who answered....

-- John Kiernan, KierPro Associates
Custom VFP/Accounting Programming
and Web interfaces
VFP and/or SQL back ends
"Richard Cornford" <Ri*****@litote s.demon.co.uk> wrote in message
news:cf******** ***********@new s.demon.co.uk.. .

(snip)

[Richard Cornford]

John Kiernan wrote:
<snip>

The answer was I need to do what Richard said:

OnMouseUp="retu rn mousetest();"

Instead of what I was doing:
OnMouseUp="mous etest();"

It was running the function, but not RETURNing the
value I was sending back!!!!

<snip>

It was more 'an answer' than "the answer", in the sense that it
addressed a critical flaw in the original implementation. The other
suggestions made in relation to this question are still very valid,
particularly where they question the choice of event handler. If you use
a onmouseup event you are creating something that is pointing device
dependent. That would be in direct opposition to the recommendations of
the Web Content Accessibility Guidelines, where it is recommended that
functionality should be device independent (and so may have legal
implications in some jurisdictions/contexts).

However, mouse centred functionality is probably not a good idea in any
case. While it seems common for individuals who almost exclusively use
the mouse themselves for this type of task to assume that everyone else
does likewise, in practice individuals interact with computers in very
varied ways, depending on their circumstances and experience. For
example, it was my observation that numerous colleagues who had been
issued with IBM laptops found the pointing device on that machine so
inconvenient that they learnt all of the keyboard shortcuts, and having
learnt them they used them on any computer, and individuals who can
touch-type also appear to be significantly more inclined to be using
keyboard navigation.

Whatever you are doing the chances are good that at some point it will
be used by someone who uses the keyboard to interact with the browser
(entirely or in part) and when they do they will bypass your onmouseup
event handler.

Suggestions that - onclick - might be a better handler are valid, as it
is usually triggered by keyboard activation in addition to with the
mouse, but in my opinion the - onsubmit - handler of the form element is
the ideal place to be validating form content, and cancelling the
submission if need be. It has the advantages of being very well
supported, and intercepting attempts to submit a form by any means or
device (with the exception of code directly calling the form's -
submit - method, but that is always a bad idea).

The - onsubmit - handler has a direct relationship with the action that
is of interest in form validation.

Richard.

[John Kiernan]

Dear Richard (et al) -
Yes, you're all right that if this was a public application, I should be
flogged in the public square for this. However, it's a private, company
wide system (accessed by nurses who work for the company, often from home
and at various 'homes' for the disabled).

As a result, we have the ability to dictate certain things, like they HAVE
to use IE (for the specific DHTML we use). We use a dynamic (rights driven
and built) JavaScript menu navigation system -- so if you don't have
scripting turned on, you ain't doing nuthin'.

As the script is an adjunct to the server side, which has a mirror of the
validation, I'm just trying to cut down on the traffic and 'round tripping'
by validating on the client.

As a geek in other areas -- and one who posts and monitors other boards
that help newbies in my area, I have noticed (and am guilty of it myself
some times) that we geeks tend to want show our knowledge and great brains
more than answer the question as asked. As a result, the poster (me, in
this case) has to come back again and again to 'explain himself'.

As I said... I am as guilty of this as anyone, but I -- for one -- am
going to try to be more cognizant of this in the future.

Thanks again Richard!
-- John Kiernan, KierPro Associates
Custom VFP/Accounting Programming
and Web interfaces
VFP and/or SQL back ends
"Richard Cornford" <Ri*****@litote s.demon.co.uk> wrote in message
news:cf******** ***********@new s.demon.co.uk.. .

<snip>
It was more 'an answer' than "the answer", in the sense that it
addressed a critical flaw in the original implementation. The other
<snip>