[mythtv] patch - forget history triggering signal change

[Kirby Vandivort]

doneRecording was calling signalChange after updating oldrecorded but
forgetHistory was not.  Both can affect the list of scheduled recordings
so this inconsistency has been rectified, and both were modified to only
call signalChange if the SQL command was successful.

-- 

Kirby Vandivort                      Theoretical and 
Senior Research Programmer            Computational Biophysics 
Email: kvandivo at ks.uiuc.edu          3051 Beckman Institute
http://www.ks.uiuc.edu/~kvandivo/    University of Illinois
Phone: (217) 244-5711                405 N. Mathews Ave
Fax  : (217) 244-6078                Urbana, IL  61801, USA
-------------- next part --------------
Index: libs/libmythtv/scheduledrecording.cpp
===================================================================
RCS file: /var/lib/mythcvs/mythtv/libs/libmythtv/scheduledrecording.cpp,v
retrieving revision 1.54
diff -b -u -2 -r1.54 scheduledrecording.cpp
--- libs/libmythtv/scheduledrecording.cpp	23 Nov 2003 15:57:37 -0000	1.54
+++ libs/libmythtv/scheduledrecording.cpp	3 Dec 2003 02:04:48 -0000
@@ -701,9 +701,13 @@
     QSqlQuery result = db->exec(query);
     if (!result.isActive())
+    {
         MythContext::DBError("doneRecording", result);
-
+    }
+    else
+    {
     // The addition of an entry to oldrecorded may affect near-future
     // scheduling decisions, so recalculate
     signalChange(db);
+    }
 }
 
@@ -724,5 +728,13 @@
     QSqlQuery result = db->exec(query);
     if (!result.isActive())
+    {
         MythContext::DBError("forgetHistory", result);
+    }
+    else
+    {
+        // The removal of an entry from oldrecorded may affect near-future
+        // scheduling decisions, so recalculate
+        signalChange(db);
+    }
 }