diff --git a/src/core/Group.cpp b/src/core/Group.cpp
index fc9726f1d..87741ee0c 100644
--- a/src/core/Group.cpp
+++ b/src/core/Group.cpp
@@ -865,7 +865,9 @@ Group* Group::clone(Entry::CloneFlags entryFlags, Group::CloneFlags groupFlags)
 
 void Group::copyDataFrom(const Group* other)
 {
-    m_data = other->m_data;
+    if (set(m_data, other->m_data)) {
+        emit groupDataChanged(this);
+    }
     m_customData->copyDataFrom(other->m_customData);
     m_lastTopVisibleEntry = other->m_lastTopVisibleEntry;
 }
@@ -1079,7 +1081,7 @@ bool Group::GroupData::equals(const Group::GroupData& other, CompareItemOptions
     if (::compare(customIcon, other.customIcon) != 0) {
         return false;
     }
-    if (timeInfo.equals(other.timeInfo, options) != 0) {
+    if (!timeInfo.equals(other.timeInfo, options)) {
         return false;
     }
     // TODO HNH: Some properties are configurable - should they be ignored?
diff --git a/tests/TestGroup.cpp b/tests/TestGroup.cpp
index 6ea43b105..9abbd31d1 100644
--- a/tests/TestGroup.cpp
+++ b/tests/TestGroup.cpp
@@ -798,3 +798,46 @@ void TestGroup::testIsRecycled()
     db->recycleGroup(group4);
     QVERIFY(group4->isRecycled());
 }
+
+void TestGroup::testCopyDataFrom()
+{
+    QScopedPointer<Group> group(new Group());
+    group->setName("TestGroup");
+
+    QScopedPointer<Group> group2(new Group());
+    group2->setName("TestGroup2");
+
+    QScopedPointer<Group> group3(new Group());
+    group3->setName("TestGroup3");
+    group3->customData()->set("testKey", "value");
+
+
+    QSignalSpy spyGroupModified(group.data(), SIGNAL(groupModified()));
+    QSignalSpy spyGroupDataChanged(group.data(), SIGNAL(groupDataChanged(Group*)));
+
+    group->copyDataFrom(group2.data());
+    QCOMPARE(spyGroupModified.count(), 1);
+    QCOMPARE(spyGroupDataChanged.count(), 1);
+
+    // if no change, no signals
+    spyGroupModified.clear();
+    spyGroupDataChanged.clear();
+    group->copyDataFrom(group2.data());
+    QCOMPARE(spyGroupModified.count(), 0);
+    QCOMPARE(spyGroupDataChanged.count(), 0);
+
+    // custom data change triggers a separate modified signal
+    spyGroupModified.clear();
+    spyGroupDataChanged.clear();
+    group->copyDataFrom(group3.data());
+    QCOMPARE(spyGroupDataChanged.count(), 1);
+    QCOMPARE(spyGroupModified.count(), 2);
+}
+
+void TestGroup::testEquals()
+{
+    QScopedPointer<Group> group(new Group());
+    group->setName("TestGroup");
+
+    QVERIFY(group->equals(group.data(), CompareItemDefault));
+}
diff --git a/tests/TestGroup.h b/tests/TestGroup.h
index fe0fefda4..9fd5c2efd 100644
--- a/tests/TestGroup.h
+++ b/tests/TestGroup.h
@@ -43,6 +43,8 @@ private slots:
     void testLocate();
     void testAddEntryWithPath();
     void testIsRecycled();
+    void testCopyDataFrom();
+    void testEquals();
 };
 
 #endif // KEEPASSX_TESTGROUP_H