From 21efef42f7bcd2c3d865b807c30c5c3c9073a02d Mon Sep 17 00:00:00 2001
From: Virgil Dupras <hsoft@hardcoded.net>
Date: Wed, 26 Jan 2011 11:49:30 +0100
Subject: [PATCH] [#134 state:fixed] Removing all dupes from the results sets
 it in 'not modified' state.

---
 core/results.py            | 2 +-
 core/tests/results_test.py | 6 ++++++
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/core/results.py b/core/results.py
index 587c941b..b4c1ea01 100644
--- a/core/results.py
+++ b/core/results.py
@@ -272,7 +272,7 @@ class Results(Markable):
         for group in affected_groups:
             group.discard_matches()
         self.__dupes = None
-        self.is_modified = True
+        self.is_modified = bool(self.__groups)
     
     def save_to_xml(self, outfile):
         self.apply_filter(None)
diff --git a/core/tests/results_test.py b/core/tests/results_test.py
index 4fae5553..6da07880 100644
--- a/core/tests/results_test.py
+++ b/core/tests/results_test.py
@@ -225,6 +225,12 @@ class TestCaseResultsWithSomeGroups:
         self.results.load_from_xml(f, get_file)
         assert not self.results.is_modified
     
+    def test_is_modified_after_removing_all_results(self):
+        # Removing all results sets the is_modified flag to false.
+        self.results.mark_all()
+        self.results.perform_on_marked(lambda x:None, True)
+        assert not self.results.is_modified
+    
 
 class TestCaseResultsWithSavedResults:
     def setup_method(self, method):