Mercurial > rsstweet
annotate app/models/tweet.rb @ 143:8a858aec1905
Stringify the cache key
| author | nanaya <me@nanaya.pro> |
|---|---|
| date | Wed, 13 Dec 2017 06:09:17 +0900 |
| parents | 2d873f57f424 |
| children | 1925b08153dc |
| rev | line source |
|---|---|
| 15 | 1 class Tweet |
| 137 | 2 TIMELINE_OPTIONS = { |
| 3 :count => 100, | |
| 4 :exclude_replies => false, | |
| 5 :include_rts => true, | |
| 6 :tweet_mode => :extended, | |
| 7 } | |
| 8 | |
| 135 | 9 def initialize(twitter_id) |
| 10 @clients = {} | |
| 11 @twitter_id = twitter_id | |
| 12 end | |
| 13 | |
| 97 | 14 def cache_expires_time |
|
107
de342c5df747
Extend cache time to up to 30 minutes
nanaya <me@myconan.net>
parents:
100
diff
changeset
|
15 (15 + rand(15)).minutes |
| 97 | 16 end |
| 17 | |
|
86
5bfc986200db
The caching becomes a bit confusing because of parameters
nanaya <me@myconan.net>
parents:
78
diff
changeset
|
18 def timeline |
| 135 | 19 initial_config_id = client_config_id |
|
76
0c023d35cd80
Allows usage of multiple twitter keys
nanaya <me@myconan.net>
parents:
73
diff
changeset
|
20 |
|
134
3646b3e319c7
Only try/catch the tweet downloading part
nanaya <me@nanaya.pro>
parents:
108
diff
changeset
|
21 @timeline ||= |
| 143 | 22 Rails.cache.fetch("timeline:v1:#{@twitter_id}", :expires_in => cache_expires_time) do |
|
134
3646b3e319c7
Only try/catch the tweet downloading part
nanaya <me@nanaya.pro>
parents:
108
diff
changeset
|
23 begin |
| 141 | 24 timeline = client.user_timeline(@twitter_id, TIMELINE_OPTIONS) |
|
134
3646b3e319c7
Only try/catch the tweet downloading part
nanaya <me@nanaya.pro>
parents:
108
diff
changeset
|
25 rescue Twitter::Error::TooManyRequests |
| 135 | 26 @client_config_id += 1 |
|
134
3646b3e319c7
Only try/catch the tweet downloading part
nanaya <me@nanaya.pro>
parents:
108
diff
changeset
|
27 |
| 135 | 28 if initial_config_id == client_config_id |
|
134
3646b3e319c7
Only try/catch the tweet downloading part
nanaya <me@nanaya.pro>
parents:
108
diff
changeset
|
29 raise |
|
3646b3e319c7
Only try/catch the tweet downloading part
nanaya <me@nanaya.pro>
parents:
108
diff
changeset
|
30 else |
|
3646b3e319c7
Only try/catch the tweet downloading part
nanaya <me@nanaya.pro>
parents:
108
diff
changeset
|
31 retry |
|
3646b3e319c7
Only try/catch the tweet downloading part
nanaya <me@nanaya.pro>
parents:
108
diff
changeset
|
32 end |
|
76
0c023d35cd80
Allows usage of multiple twitter keys
nanaya <me@myconan.net>
parents:
73
diff
changeset
|
33 end |
| 141 | 34 |
| 35 timeline.select do |tweet| | |
| 36 tweet.retweeted_status.nil? || tweet.user.id != tweet.retweeted_status.user.id | |
| 37 end.map do |tweet| | |
| 38 # Fails when there's Twitter::NullObject initiated somewhere in previous select | |
| 39 # Reference: https://github.com/sferik/twitter/issues/892 | |
| 40 tweet.to_h | |
| 41 end | |
| 137 | 42 end.map do |tweet_hash| |
| 43 Twitter::Tweet.new(tweet_hash) | |
| 78 | 44 end |
| 15 | 45 end |
| 46 | |
|
48
8983c426e256
Prevent exploding on empty timeline.
nanaya <me@myconan.net>
parents:
47
diff
changeset
|
47 def user |
|
8983c426e256
Prevent exploding on empty timeline.
nanaya <me@myconan.net>
parents:
47
diff
changeset
|
48 @user ||= |
|
8983c426e256
Prevent exploding on empty timeline.
nanaya <me@myconan.net>
parents:
47
diff
changeset
|
49 if timeline.any? |
|
8983c426e256
Prevent exploding on empty timeline.
nanaya <me@myconan.net>
parents:
47
diff
changeset
|
50 timeline.first.user |
|
8983c426e256
Prevent exploding on empty timeline.
nanaya <me@myconan.net>
parents:
47
diff
changeset
|
51 else |
| 143 | 52 Rails.cache.fetch("user:v1:#{@twitter_id}", :expires_in => cache_expires_time) do |
| 142 | 53 client.user(@twitter_id) |
|
48
8983c426e256
Prevent exploding on empty timeline.
nanaya <me@myconan.net>
parents:
47
diff
changeset
|
54 end |
|
8983c426e256
Prevent exploding on empty timeline.
nanaya <me@myconan.net>
parents:
47
diff
changeset
|
55 end |
|
8983c426e256
Prevent exploding on empty timeline.
nanaya <me@myconan.net>
parents:
47
diff
changeset
|
56 end |
|
8983c426e256
Prevent exploding on empty timeline.
nanaya <me@myconan.net>
parents:
47
diff
changeset
|
57 |
| 135 | 58 def client |
| 59 @clients[client_config_id] ||= | |
| 60 Twitter::REST::Client.new do |config| | |
| 61 $cfg[:twitter][client_config_id].each do |cfg_key, cfg_value| | |
| 138 | 62 config.public_send(:"#{cfg_key}=", cfg_value) |
| 135 | 63 end |
| 64 end | |
|
76
0c023d35cd80
Allows usage of multiple twitter keys
nanaya <me@myconan.net>
parents:
73
diff
changeset
|
65 end |
|
0c023d35cd80
Allows usage of multiple twitter keys
nanaya <me@myconan.net>
parents:
73
diff
changeset
|
66 |
| 135 | 67 def client_config_id |
| 68 @client_config_id ||= 0 | |
|
76
0c023d35cd80
Allows usage of multiple twitter keys
nanaya <me@myconan.net>
parents:
73
diff
changeset
|
69 |
|
136
59991d10f8a3
Fix the variable as well while at it
nanaya <me@nanaya.pro>
parents:
135
diff
changeset
|
70 @client_config_id %= $cfg[:twitter].size |
| 15 | 71 end |
| 72 end |
