TalkPHP - Are Operators Valid?

TalkPHP (http://www.talkphp.com/forums.php)

- Javascript, AJAX, E4X (http://www.talkphp.com/javascript-ajax-e4x/)

- - Are Operators Valid? (http://www.talkphp.com/javascript-ajax-e4x/1219-operators-valid.html)

CMellor

09-24-2007 10:52 PM

Are Operators Valid?

Hey,

Quick question. My site currently has 4 invalid HTML errors. It's all down to this function:

Code:

// Error checking

$('send_pm').onsubmit = function() {

valid = $(this.to).present() && $(this.subject).present() && $(this.message).present();

                        

if(valid) {

        return true;

}

else {

        alert('Please fill in all fields');

}

        return false;

}

The validator is saying that the && operator is the cause of the error. Their's 4 errors, and 4 &'s.

I'm guessing it's my code, perhaps it's not wrote out correctly, though it does work. Does anybody have any suggestions? BTW: I'm using PrototypeJS

Cheers,

- Chris.

Wildhoney

09-24-2007 11:26 PM

&& in that context seems totally wrong. You're assigning the values and && is for checking, such as like: if kiwi is kiwi AND banana is banana. If you're attempting to concatenate the values then the plus symbol is the way to go with javascript:

Code:

valid = $(this.to).present() + $(this.subject).present() + $(this.message).present();

Although one & does work as well, I believe.

Salathe

09-24-2007 11:47 PM

No, in this case the && is correct. If you look at the code, it's checking if various fields are 'present' and assigning the valid variable if they all are.

The HTML validation errors are caused by the ampersands (&) in your JavaScript but you should not change them to anything else. Instead, it is advisable (not just because of validation) to put your JavaScript in its own file and reference it using a <script> HTML element.

Karl	09-24-2007 11:53 PM

Im not sure present is a function in Protoype, try changing present() to empty(), you'll have to negate each condition too.

Code:

valid = !$(this.to).empty() && !$(this.subject).empty() && !$(this.message).empty();

CMellor

09-25-2007 12:21 AM

I think having the code in an external file, or maybe even just between the <head> tags might change the outcome.

Karl, hi... present() is a Form Element in Prototype. It has an example code, which I pretty much copied, but used an alert() instead of displaying some text. See the document (http://www.prototypejs.org/api/form/element/present)

Salathe

09-25-2007 07:20 AM

If you're wanting to keep the code within the HTML document, you'll need to tell the validator not to check the script by making it a CDATA block (that's XML terminology). The two "commented" (//) lines are what's important here.

HTML Code:

<script type="text/javascript">

//<![CDATA[



alert("This is compatible with XHTML");



//]]>

</script>

CMellor

09-25-2007 09:17 PM

Cool, I'll have a stab at it.

Cheers.

Wildhoney

09-25-2007 09:49 PM

I've never been a big fan of CDATA as I think it looks somewhat messy, but it serves a purpose so why the hell not :) ! Everything in external files, for me.

All times are GMT. The time now is 04:21 AM.