Sockets - Kelly & Jansen

app/controllers/movies_controller.rb

@@ -16,17 +16,31 @@ def show
       status: :ok,
       json: @movie.as_json(
         only: [:title, :overview, :release_date, :inventory],
-        methods: [:available_inventory]
-        )
-      )
+        methods: [:available_inventory],
+      ),
+    )
+  end
+
+  def add_movie
+    movie = Movie.new(title: params[:title], overview: params[:overview], release_date: params[:release_date], image_url: params[:image_url], external_id: params[:external_id])
+
+    already_added = Movie.find_by(external_id: params[:external_id])
+
+    if already_added
+      render status: :conflict, json: {errors: "This movie is already in the library"}
+    elsif movie.save
+      render status: :ok, json: {}
+    else
+      render status: :bad_request, json: {errors: movie.errors.messages}
+    end
   end
 
   private
 
   def require_movie
     @movie = Movie.find_by(title: params[:title])
     unless @movie
-      render status: :not_found, json: { errors: { title: ["No movie with title #{params["title"]}"] } }
+      render status: :not_found, json: {errors: {title: ["No movie with title #{params["title"]}"]}}
     end
   end
 end

app/models/movie.rb

@@ -10,7 +10,7 @@ def image_url
     orig_value = read_attribute :image_url
     if !orig_value
       MovieWrapper::DEFAULT_IMG_URL
-    elsif external_id
+    elsif external_id && !orig_value.include?("https://image.tmdb.org/t/p/")
       MovieWrapper.construct_image_url(orig_value)
     else
       orig_value

config/routes.rb

@@ -9,6 +9,7 @@
   post "/rentals/:title/return", to: "rentals#check_in", as: "check_in"
   get "/rentals/overdue", to: "rentals#overdue", as: "overdue"
 
-  root 'movies#index'
+  post "/movies/add-movie", to: "movies#add_movie", as: "add_movie"
 
+  root "movies#index"
 end

lib/movie_wrapper.rb

@@ -8,8 +8,9 @@ class MovieWrapper
   DEFAULT_IMG_URL = "http://lorempixel.com/185/278/"
 
   def self.search(query)
+    sleep(1)
     url = BASE_URL + "search/movie?api_key=" + KEY + "&query=" + query
-    response =  HTTParty.get(url)
+    response = HTTParty.get(url)
     if response["total_results"] == 0
       return []
     else
@@ -28,11 +29,11 @@ def self.construct_movie(api_result)
       overview: api_result["overview"],
       release_date: api_result["release_date"],
       image_url: api_result["poster_path"], #(api_result["poster_path"] ? self.construct_image_url(api_result["poster_path"]) : nil),
-      external_id: api_result["id"])
+      external_id: api_result["id"],
+    )
   end
 
   def self.construct_image_url(img_name)
     return BASE_IMG_URL + DEFAULT_IMG_SIZE + img_name
   end
-
 end