diff options
| author | Evan Prodromou <evan@controlyourself.ca> | 2008-09-24 11:08:34 -0400 |
|---|---|---|
| committer | Evan Prodromou <evan@controlyourself.ca> | 2008-09-24 11:08:34 -0400 |
| commit | 392137b4033793620fcd36d641625b5591b046bf (patch) | |
| tree | d72c88c8653d76d3b9ae226b374896acee252329 /lib/util.php | |
| parent | 845db5ec168e26759c79de4f8216d1b54f7c0d8a (diff) | |
change foreign links flags code to use bitmasks
Zach was doing a lot of integer comparisons on the Foreign_link sync
fields. I switched them to use named bitmasks instead. I also switched
the semantics of bit 3 to be the opposite of what Zach had -- I find
lots of double-negatives in a checkbox to be hard to read.
darcs-hash:20080924150834-5ed1f-54cd945f61e43bc06768037c60c1e6180a8feead.gz
Diffstat (limited to 'lib/util.php')
| -rw-r--r-- | lib/util.php | 17 |
1 files changed, 6 insertions, 11 deletions
diff --git a/lib/util.php b/lib/util.php index a45f4a214..901e3ac46 100644 --- a/lib/util.php +++ b/lib/util.php @@ -1110,17 +1110,12 @@ function common_broadcast_notice($notice, $remote=false) { // Check to see if notice should go to Twitter $flink = Foreign_link::getForeignLink($notice->profile_id, 1); // 1 == Twitter - if ($flink->noticesync >= 1) { - $ok_to_send = true; - - // Check to see whether user wants to filter @-replies - if ($flink->noticesync == 3) { - if (preg_match('/(?:^|\s)@([A-Za-z0-9_\-\.]{1,64})/', $notice->content)) { - $ok_to_send = false; - } - } - - if ($ok_to_send) { + if ($flink->noticesync & FOREIGN_NOTICE_SEND) { + + // If it's not a reply, or if the user WANTS to send replies... + + if (!$notice->reply_to || ($flink->noticesync & FOREIGN_NOTICE_SEND_REPLY)) { + $result = common_twitter_broadcast($notice, $flink); if (!$result) { |