Merge pull request #1 from fguillot/master

1 files changed, 7 insertions, 2 deletions

diff --git a/app/Model/UserNotificationFilter.php b/app/Model/UserNotificationFilter.php
index d4afd278..780ddfc7 100644
--- a/app/Model/UserNotificationFilter.php
+++ b/app/Model/UserNotificationFilter.php
@@ -61,10 +61,11 @@ class UserNotificationFilter extends Base
      * @access public
      * @param  integer  $user_id
      * @param  string   $filter
+     * @return boolean
      */
     public function saveFilter($user_id, $filter)
     {
-        $this->db->table(User::TABLE)->eq('id', $user_id)->update(array(
+        return $this->db->table(User::TABLE)->eq('id', $user_id)->update(array(
             'notifications_filter' => $filter,
         ));
     }
@@ -87,17 +88,21 @@ class UserNotificationFilter extends Base
      * @access public
      * @param  integer    $user_id
      * @param  integer[]  $project_ids
+     * @return boolean
      */
     public function saveSelectedProjects($user_id, array $project_ids)
     {
+        $results = array();
         $this->db->table(self::PROJECT_TABLE)->eq('user_id', $user_id)->remove();
 
         foreach ($project_ids as $project_id) {
-            $this->db->table(self::PROJECT_TABLE)->insert(array(
+            $results[] = $this->db->table(self::PROJECT_TABLE)->insert(array(
                 'user_id' => $user_id,
                 'project_id' => $project_id,
             ));
         }
+
+        return !in_array(false, $results, true);
     }
 
     /**