]> git.mxchange.org Git - quix0rs-gnu-social.git/commit
Fix for failure edge case in TwitterBridge outgoing repeat/retweets.
authorBrion Vibber <brion@status.net>
Fri, 12 Nov 2010 21:06:41 +0000 (13:06 -0800)
committerBrion Vibber <brion@status.net>
Fri, 12 Nov 2010 21:06:41 +0000 (13:06 -0800)
commit6291e8201f90ed50687f7670edc505645ea55bfb
tree0a0744cc4c4faa655eada3995b72e1401e921310
parentb6af5a25ba61ef3c86a3772abea7ac95778689f7
Fix for failure edge case in TwitterBridge outgoing repeat/retweets.

When the retweet failed with a 403 error (say due to it being a private tweet, which can't be retweeted) we would end up mishandling the return value from our internal error handling.
Instead of correctly discarding the message and closing out the queue item, we ended up trying to save a bogus twitter<->local ID mapping, which threw another exception and lead the queue system to re-run it.

- Fixed the logic check and return values for the retweet case in broadcast_twitter().
- Added doc comments explaining the return values on some functions in twitter.php
- Added check on Notice_to_status::saveNew() for empty input -- throw an exception before we try to actually insert into db. :)
plugins/TwitterBridge/Notice_to_status.php
plugins/TwitterBridge/twitter.php