More consistent return values from send-based CAPI functions

send_chat() and bankickunban() now follow through and return the request ID of the sent message, or False if one was not sent. In the event of trying to whisper a user not in the channel, a malformed whisper request is no longer sent after the user isn't found.

capi.py

@@ -209,22 +209,24 @@ def send_chat(self, message, mtype="channel", target=None):
                 user = self.get_user(target)
                 if user is None:
                     self.parent.error(bncs.ERROR_NOTLOGGEDON)
+                    return False
                 else:
                     payload["user_id"] = user.id
 
-            self.send_command(send_message_types.get(mtype), payload)
+            return self.send_command(send_message_types.get(mtype), payload)
 
     def bankickunban(self, target, action="ban"):
         user = self.get_user(target)
         if action.lower() != "unban" and user is None:
             self.parent.error(bncs.ERROR_NOTLOGGEDON)
+            return False
         else:
             action = bku_actions.get(action.lower())
             if action is None:
                 raise ValueError("Invalid ban/kick/unban action - must be %s" % ', '.join(bku_actions.keys()))
 
             payload = {"toon_name": target} if user is None else {"user_id": user.id}
-            self.send_command(action, payload)
+            return self.send_command(action, payload)
 
     def run(self):
         while self.connected():