Merge pull request #15 from st-tech/validation

Add validation
st-tech · Apr 18, 2019 · 5bc2111 · 5bc2111
2 parents 431dc1f + 0b18fed
commit 5bc2111
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/lambda/crawler/crawler.rb b/lambda/crawler/crawler.rb
@@ -24,6 +24,7 @@ def search
   def save
     @fashion_check_tweets.each do |tweet|
       begin
+        next unless tweet.valid?
         @dynamodb.put_item(tweet.dynamo_params)
       rescue Aws::DynamoDB::Errors::ConditionalCheckFailedException
         puts "DynamoDB put_item failed. url: #{tweet.tweet_url}"

diff --git a/lambda/crawler/fashion_check_tweet.rb b/lambda/crawler/fashion_check_tweet.rb
@@ -33,7 +33,7 @@ def valid?
   private
 
   def parse
-    @account_id = @tweet.user.name
+    @account_id = @tweet.user.screen_name
     @tweet_url = @tweet.url.to_s
     @image_url = @tweet.user.profile_image_url.to_s
     score_text = @tweet.text.match(TWEET_TEMPLATE_JP) || @tweet.text.match(TWEET_TEMPLATE_EN)