Several changes:

- Issue #5834: do not update form data for widget if widget is not enabled - Widgets that are not enabled should not be exposed
emweb · Jul 28, 2017 · b263ccc · b263ccc
1 parent 387efbb
commit b263ccc
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/src/Wt/WApplication.C b/src/Wt/WApplication.C
@@ -524,6 +524,8 @@ bool WApplication::isExposed(WWidget *w) const
   if (!w->isVisible())
     return false;
   */
+  if (!w->isEnabled())
+    return false;
 
   if (w == domRoot_)
     return true;

diff --git a/src/web/WebSession.C b/src/web/WebSession.C
@@ -2848,9 +2848,12 @@ void WebSession::propagateFormValues(const WEvent& e, const std::string& se)
     std::string formName = i->first;
     WObject *obj = i->second;
 
-    if (!request.postDataExceeded())
+    if (!request.postDataExceeded()) {
+      WWidget *w = dynamic_cast<WWidget*>(obj);
+      if (w && !w->isEnabled())
+	continue; // Do not update form data of a disabled widget
       obj->setFormData(getFormData(request, se + formName));
-    else
+    } else
       obj->setRequestTooLarge(request.postDataExceeded());
   }
 }