Made .wtObj more defensive in some cases

jQuery.data would not error on trying to get the data for null, so let's
check if jsRef() is not null first.

Author: RockinRoel

src/Wt/WAbstractSpinBox.C

@@ -137,8 +137,8 @@ void WAbstractSpinBox::connectJavaScript(Wt::EventSignalBase& s,
 {
   std::string jsFunction = 
     "function(obj, event) {"
-    """var o = " + jsRef() + ".wtObj;"
-    """if (o) o." + methodName + "(obj, event);"
+    """var o = " + jsRef() + ";"
+    """if (o && o.wtObj) o.wtObj." + methodName + "(obj, event);"
     "}";
 
   s.connect(jsFunction);

src/Wt/WDateEdit.C

@@ -214,8 +214,8 @@ void WDateEdit::connectJavaScript(Wt::EventSignalBase& s,
 {
   std::string jsFunction = 
     "function(dobj, event) {"
-    """var o = " + jsRef() + ".wtDObj;"
-    """if (o) o." + methodName + "(dobj, event);"
+    """var o = " + jsRef() + ";"
+    """if (o && o.wtDObj) o.wtDObj." + methodName + "(dobj, event);"
     "}";
 
   s.connect(jsFunction);

src/Wt/WImage.C

@@ -293,10 +293,7 @@ std::string WImage::updateAreasJS()
     ss <<
       "(function(){"
       """var w = " << jsRef() << ";"
-      """if (w) {"
-      ""  "var o = " << jsRef() << ".wtObj;"
-      ""  "if (o) { o.updateAreas(); }"
-      """}"
+      """if (w && w.wtObj) { o.wtObj.updateAreas(); }"
       "})();";
   }
   return ss.str();

src/Wt/WLineEdit.C

@@ -538,8 +538,8 @@ void WLineEdit::connectJavaScript(Wt::EventSignalBase& s,
 {
   std::string jsFunction =
     "function(lobj, event) {"
-    """var o = " + jsRef() + ".wtLObj;"
-    """if (o) o." + methodName + "(lobj, event);"
+    """var o = " + jsRef() + ";"
+    """if (o && o.wtLObj) o.wtLObj." + methodName + "(lobj, event);"
     "}";
 
   s.connect(jsFunction);

src/Wt/WPopupWidget.C

@@ -82,11 +82,11 @@ void WPopupWidget::setHidden(bool hidden, const WAnimation& animation)
 
   if (!WWebWidget::canOptimizeUpdates() || isRendered()) {
     if (hidden)
-      doJavaScript("var o = " + jsRef() + ".wtPopup;"
-		   "if (o) o.hidden();");
+      doJavaScript("var o = " + jsRef() + ";"
+		   "if (o && o.wtPopup) o.wtPopup.hidden();");
     else
-      doJavaScript("var o = " + jsRef() + ".wtPopup;"
-		   "if (o) o.shown();");
+      doJavaScript("var o = " + jsRef() + ";"
+		   "if (o && o.wtPopup) o.wtPopup.shown();");
   }
 }
 

src/Wt/WSuggestionPopup.C

@@ -149,8 +149,8 @@ void WSuggestionPopup::connectObjJS(EventSignalBase& s,
 {
   std::string jsFunction = 
     "function(obj, event) {"
-    """var o = " + jsRef() + ".wtObj;"
-    """if (o) o." + methodName + "(obj, event);"
+    """var o = " + jsRef() + ";"
+    """if (o && o.wtObj) o.wtObj." + methodName + "(obj, event);"
     "}";
   s.connect(jsFunction);
 }

src/Wt/WTableView.C

@@ -827,8 +827,8 @@ void WTableView::defineJavaScript()
 
   if (canvas_) {
     app->addAutoJavaScript
-      ("{var obj = " + jsRef() + ".wtObj;"
-       "if (obj) obj.autoJavaScript();}");
+      ("{var obj = " + jsRef() + ";"
+       "if (obj && obj.wtObj) obj.wtObj.autoJavaScript();}");
 
     connectObjJS(canvas_->mouseWentDown(), "mouseDown");
     connectObjJS(canvas_->mouseWentUp(), "mouseUp");

src/Wt/WTimeEdit.C

@@ -198,8 +198,8 @@ void WTimeEdit::connectJavaScript(Wt::EventSignalBase& s,
 {
   std::string jsFunction =
     "function(dobj, event) {"
-    """var o = " + jsRef() + ".wtDObj;"
-    """if(o) o." + methodName + "(dobj, event);"
+    """var o = " + jsRef() + ";"
+    """if(o && o.wtDObj) o.wtDObj." + methodName + "(dobj, event);"
     "}";
   s.connect(jsFunction);
 }

src/Wt/WTreeTable.C

@@ -80,8 +80,8 @@ void WTreeTable::render(WFlags<RenderFlag> flags)
     resize(width(), height());
 
     WApplication::instance()->addAutoJavaScript
-      ("{var obj = " + jsRef() + ".wtObj;"
-       "if (obj) obj.autoJavaScript();}");
+      ("{var obj = " + jsRef() + ";"
+       "if (obj && obj.wtObj) obj.wtObj.autoJavaScript();}");
   }
 
   WCompositeWidget::render(flags);