Cleanup

diff --git a/muamba.business.inc b/muamba.business.inc
index 8ca7648b740bc04f8418dba0976dd57149e3d521..2ec564449e3c5e55cc7de59cf42c3c943e97d39f 100644 (file)
--- a/muamba.business.inc
+++ b/muamba.business.inc
@@ -191,12 +191,7 @@ function muamba_accept($mid) {
   }
 
   // Update database
-  $update = db_update('muamba')
-    ->fields(array(
-      'status' => MUAMBA_ACCEPTED,
-    ))
-    ->condition('mid', $mid, '=')
-    ->execute();
+  muamba_update_status($mid, MUAMBA_ACCEPTED);
 
   // Notify item owner
   $message = theme('muamba_accept_message', array('transaction' => $transaction));
@@ -228,12 +223,7 @@ function muamba_reject($mid) {
   }
 
   // Update database
-  $update = db_update('muamba')
-    ->fields(array(
-      'status' => MUAMBA_REJECTED,
-    ))
-    ->condition('mid', $mid, '=')
-    ->execute();
+  muamba_update_status($mid, MUAMBA_REJECTED);
 
   // Notify item owner
   $message = theme('muamba_reject_message', array('transaction' => $transaction));
@@ -265,12 +255,7 @@ function muamba_cancel($mid) {
   }
 
   // Update database
-  $update = db_update('muamba')
-    ->fields(array(
-      'status' => MUAMBA_CANCELLED,
-    ))
-    ->condition('mid', $mid, '=')
-    ->execute();
+  muamba_update_status($mid, MUAMBA_CANCELLED);
 
   // Notify item owner
   $message = theme('muamba_cancel_message', array('transaction' => $transaction));
@@ -303,12 +288,7 @@ function muamba_return($mid) {
   }
 
   // Update database
-  $update = db_update('muamba')
-    ->fields(array(
-      'status' => MUAMBA_RETURNED,
-    ))
-    ->condition('mid', $mid, '=')
-    ->execute();
+  muamba_update_status($mid, MUAMBA_RETURNED);
 
   // Notify item owner
   $message = theme('muamba_return_message', array('transaction' => $transaction));
@@ -342,12 +322,7 @@ function muamba_recover($mid) {
   }
 
   // Update database
-  $update = db_update('muamba')
-    ->fields(array(
-      'status' => MUAMBA_RECOVERED,
-    ))
-    ->condition('mid', $mid, '=')
-    ->execute();
+  muamba_update_status($mid, MUAMBA_RECOVERED);
 
   // Notify item owner
   $message = theme('muamba_recover_message', array('transaction' => $transaction));
@@ -380,12 +355,7 @@ function muamba_lost($mid) {
   }
 
   // Update database
-  $update = db_update('muamba')
-    ->fields(array(
-      'status' => MUAMBA_LOST,
-    ))
-    ->condition('mid', $mid, '=')
-    ->execute();
+  muamba_update_status($mid, MUAMBA_LOST);
 
   // Notify item owner
   $message = theme('muamba_lost_message', array('transaction' => $transaction));

diff --git a/muamba.db.inc b/muamba.db.inc
index 6c2f74d7f98f737ab03da03344471cf315b5f0a0..3650e08c0c5fc7e9f2074c5e64afadb4e7e596f7 100644 (file)
--- a/muamba.db.inc
+++ b/muamba.db.inc
@@ -167,3 +167,21 @@ function muamba_check_availability($data) {
 
   return FALSE;
 }
+
+/**
+ * Update a transaction status.
+ *
+ * @param $mid
+ *   Transaction id.
+ *
+ * @param $status
+ *   New status.
+ */
+function muamba_update_status($mid, $status) {
+  $update = db_update('muamba')
+    ->fields(array(
+      'status' => $status,
+    ))
+    ->condition('mid', $mid, '=')
+    ->execute();
+}