From 143eadb58e2d4790a223385e9d0f6b1111c40b3e Mon Sep 17 00:00:00 2001
From: Doc West <dr.west@zoho.com>
Date: Wed, 4 Jul 2018 23:21:55 +0200
Subject: [PATCH] Use setData() instead of emtitting dataChanged() which does
 not work on CI. Also Fixes the remaining issue with subviews not updating due
 to only the modified flag emitting a change, which prevented the widget
 mapper from working for updates.

---
 apps/opencs/model/world/commands.cpp | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/apps/opencs/model/world/commands.cpp b/apps/opencs/model/world/commands.cpp
index a572ca632..c67de6350 100644
--- a/apps/opencs/model/world/commands.cpp
+++ b/apps/opencs/model/world/commands.cpp
@@ -323,11 +323,16 @@ void CSMWorld::RevertCommand::redo()
         // updated in the view unless the user had selected another item or forced a
         // repaint with other means
         int count = mModel.columnCount (index.parent ());
-        if (count > 0)
+        for (int i=0; i<count; ++i)
         {
-            emit mModel.dataChanged(mModel.index (index.row(), 0, index.parent ()),
-                                    mModel.index (index.row(), count - 1, index.parent ()));
+            if (i != index.column())
+            {
+                auto colIndex = mModel.index (index.row (), i, index.parent ());
+                auto data = mModel.data (colIndex);
+                mModel.setData (colIndex, data);
+            }
         }
+
         mModel.setData (index, static_cast<int> (RecordBase::State_BaseOnly));
     }
 }