[Exist-open] [ exist-Bugs-3309569 ] wrong if statement in editor.js in eXide

[SourceForge.net <no...@so...>]

Bugs item #3309569, was opened at 2011-05-31 09:40
Message generated for change (Tracker Item Submitted) made by danruebe
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=117691&aid=3309569&group_id=17691

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: danruebe (danruebe)
Assigned to: Nobody/Anonymous (nobody)
Summary: wrong if statement in editor.js in eXide

Initial Comment:
eXist trunk rev. 14590
In editor.js in eXide there is a wrong if statement at the beginning of the function "validateQuery" in eXide.edit.Editor. Line 721 says
if (!this.activeDoc.isXQuery) {
but it should be
if (!this.activeDoc.isXQuery()) {
The first version checks the existance of the function, the second the return value.

Patch below. Cheers,
Daniel

Patch:
Index: editor.js
===================================================================
--- editor.js	(revision 14590)
+++ editor.js	(working copy)
@@ -718,7 +718,7 @@
 		},
 
 		validateQuery: function() {
-			if (!this.activeDoc.isXQuery) {
+			if (!this.activeDoc.isXQuery()) {
 				return;
 			}
 			this.pendingCheck = true;


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=117691&aid=3309569&group_id=17691