From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Sat, 9 Mar 2013 15:08:00 +0000 (+0000)
Subject: Don't set cell value in wxDataViewEvent in one place only.
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/17beda75ef9780ebef5044e41889a9b598dcf186?ds=inline

Don't set cell value in wxDataViewEvent in one place only.

We should either set the cell value in the event object everywhere or not do
it anywhere and as currently the native GTK and OS X versions don't do it at
all and the generic version only does it for ITEM_CONTEXT_MENU events, it's
easier to not do it at all.

See #14163.

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@73623 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---

diff --git a/src/generic/datavgen.cpp b/src/generic/datavgen.cpp
index 8d09cf5b52..3841f24767 100644
--- a/src/generic/datavgen.cpp
+++ b/src/generic/datavgen.cpp
@@ -4000,10 +4000,6 @@ void wxDataViewMainWindow::OnMouse( wxMouseEvent &event )
             le.SetItem( item );
             le.SetColumn( col->GetModelColumn() );
             le.SetDataViewColumn( col );
-
-            wxVariant value;
-            model->GetValue( value, item, col->GetModelColumn() );
-            le.SetValue(value);
         }
 
         parent->ProcessWindowEvent(le);