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 |