--- test/simplenews_roles/simplenews_roles.old	2006-03-20 12:53:55.000000000 -0500
+++ simplenews_roles/simplenews_roles.module	2006-07-27 11:55:10.000000000 -0400
@@ -217,7 +217,7 @@
         // Add user to all newsletters that have authenticated users as subscribers
         $snid = db_result(db_query('SELECT snid FROM {sn_subscriptions} WHERE mail = "%s"', $edit['email']));
         while ($newsletter = db_fetch_object($result)) {
-          db_query("INSERT INTO {sn_snid_tid} (snid, tid, role_subscription) VALUE (%d, %d, %d)", $snid, $newsletter->tid, 2);
+          db_query("INSERT INTO {sn_snid_tid} (snid, tid, role_subscription) VALUES (%d, %d, %d)", $snid, $newsletter->tid, 2);
         }
       }
     case 'update':
@@ -225,7 +225,7 @@
       $roles = $edit['roles'];
       if ($roles) { // No action needed if user doesn't belong to a role
         // Get user's subscription id
-        $snid = db_result(db_query('SELECT FROM {sn_subscriptions} WHERE s.uid = %d', $account->uid));
+        $snid = db_result(db_query('SELECT * FROM {sn_subscriptions} s WHERE s.uid = %d', $account->uid));
         // Delete all user's old role subscriptions
         db_query('DELETE FROM {sn_snid_tid} WHERE snid = %d AND role_subscription != 0', $snid);
 
@@ -301,4 +301,4 @@
   $query['query'] .= ' AND user_unsubscribed_self = NULL';
 }
 
-?>
\ No newline at end of file
+?>