diff options
author | Brion Vibber <brion@pobox.com> | 2010-12-13 12:12:22 -0800 |
---|---|---|
committer | Brion Vibber <brion@pobox.com> | 2010-12-13 12:13:24 -0800 |
commit | 39cad55711897323fac5f651c003c4d815a51ae0 (patch) | |
tree | a2e2d2c6c1baa0bcc3825b900c2b6003fb9e2132 /plugins/TwitterBridge/daemons | |
parent | 3f9b8b293d5071357771ce7a88ffaf530ce229e8 (diff) |
TwitterBridge: partial merge of id_str usage from 0.9.x for improved 32-bit and pre-5.2.10 compatibility. (on 64-bit in 5.2.6 we can pull the integer IDs, but silently lose some precision on the end.)
Fixes for Twitter bridge breakage on 32-bit servers. New "Snowflake" 64-bit IDs have become too big to fit in the integer portion of double-precision floats, so to reliably use these IDs we need to pull the new string form now.
Machines with 64-bit PHP installation should have had no problems (except on Windows, where integers are still 32 bits)
Conflicts:
plugins/TwitterBridge/twitterimport.php <- as this hasn't been broken out, the import code is NOT FULLY UPDATED HERE.
Diffstat (limited to 'plugins/TwitterBridge/daemons')
-rwxr-xr-x | plugins/TwitterBridge/daemons/twitterstatusfetcher.php | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/plugins/TwitterBridge/daemons/twitterstatusfetcher.php b/plugins/TwitterBridge/daemons/twitterstatusfetcher.php index cef67b180..1b9cca8ec 100755 --- a/plugins/TwitterBridge/daemons/twitterstatusfetcher.php +++ b/plugins/TwitterBridge/daemons/twitterstatusfetcher.php @@ -218,8 +218,9 @@ class TwitterStatusFetcher extends ParallelizingDaemon } if (!empty($timeline)) { - Twitter_synch_status::setLastId($flink->foreign_id, 'home_timeline', $timeline[0]->id); - common_debug("Set lastId value '{$timeline[0]->id}' for foreign id '{$flink->foreign_id}' and timeline 'home_timeline'"); + $lastId = twitter_id($timeline[0]); + Twitter_synch_status::setLastId($flink->foreign_id, 'home_timeline', $lastId); + common_debug("Set lastId value '$lastId' for foreign id '{$flink->foreign_id}' and timeline 'home_timeline'"); } // Okay, record the time we synced with Twitter for posterity |