No changes, just rearranging code a little

diff --git a/message/index.php b/message/index.php
index 0fed964cadaa1cf42de9dbec51ed9413dc20781c..7ccde85af5d2dbce5eed9a1fe62e756f02dd3525 100644 (file)
--- a/message/index.php
+++ b/message/index.php
@@ -7,16 +7,24 @@ require('lib.php');
 require_login(0, false);
 
 /// optional variables that may be passed in
-$tab            = optional_param('tab',       'contacts'); // current tab - default to contacts
-$addcontact     = optional_param('addcontact',     false); // adding a contact
-$removecontact  = optional_param('removecontact',  false); // removing a contact
-$blockcontact   = optional_param('blockcontact',   false); // blocking a contact
-$unblockcontact = optional_param('unblockcontact', false); // unblocking a contact
-
-if (($addcontact     !== false) and confirm_sesskey()) message_add_contact($addcontact);
-if (($removecontact  !== false) and confirm_sesskey()) message_remove_contact($removecontact);
-if (($blockcontact   !== false) and confirm_sesskey()) message_block_contact($blockcontact);
-if (($unblockcontact !== false) and confirm_sesskey()) message_unblock_contact($unblockcontact);
+$tab            = optional_param('tab', 'contacts'); // current tab - default to contacts
+$addcontact     = optional_param('addcontact',     0, PARAM_INT); // adding a contact
+$removecontact  = optional_param('removecontact',  0, PARAM_INT); // removing a contact
+$blockcontact   = optional_param('blockcontact',   0, PARAM_INT); // blocking a contact
+$unblockcontact = optional_param('unblockcontact', 0, PARAM_INT); // unblocking a contact
+
+if ($addcontact and confirm_sesskey()) {
+    message_add_contact($addcontact);
+}
+if ($removecontact and confirm_sesskey()) {
+    message_remove_contact($removecontact);
+}
+if ($blockcontact and confirm_sesskey()) {
+    message_block_contact($blockcontact);
+}
+if ($unblockcontact and confirm_sesskey()) {
+    message_unblock_contact($unblockcontact);
+}
 
 
 /// a print function is associated with each tab