letteropend 2.0.0 → 2.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d5e35304268f8c5572b2229b6e9f1b652fc85558
-  data.tar.gz: 5ca98b6f08d5095cabe6c4f13a42baabdc01d024
+  metadata.gz: a5640da5f8ec0d3a72189d53470ba0b9fb7e2bd7
+  data.tar.gz: 11d582a99ebc2e009982bd24561c526eb895558d
 SHA512:
-  metadata.gz: f80309d6f2665d12fa045e36fb9fb212685468d8574f45b63dd9d47d2bd8e52f602f4b3079645de13c9d53b595f29ec2f235c1eea2e24514f174a3039043f09d
-  data.tar.gz: b9737ce23e168e491c00d6f579939378a4b51a7debb096b7d9abd224990ecfe7653c456533e7dc02b434fcfea828db2215083c295adb3b3b685069657eb5e9e0
+  metadata.gz: 34e4b9356825b72f9114fd6ed63db2e422fdbe41d19d55c7728cf628bf2dc62094036f69d74c566152bc6fde763182cb2e3cbac9c423a21bde4c65b91cf88318
+  data.tar.gz: 0f4b3cc40743af4a48dd04a798f610898b3528a90ec51ea0ee088af988cada98158cc648ea4733304b43199cd27257ebf654a82688f16722392701fc63b93761

data/examples/compareWatchlist.rb

@@ -1,3 +1,5 @@
+# Usage: ruby compareWatchlist.rb <username 1> <username 2>
+
 require_relative "../lib/letteropend"
 
 Letteropend::List.config do
@@ -6,11 +8,11 @@ Letteropend::List.config do
   end
 end
 
-print "First username: "
-user1 = gets.chomp
+# First username
+user1 = ARGV[0]
 
-print "Second username: "
-user2 = gets.chomp
+# Second username
+user2 = ARGV[1]
 
 user1_wl = Letteropend::List.new(user1, "watchlist")
 user2_wl = Letteropend::List.new(user2, "watchlist")

data/examples/getAllTime.rb

@@ -1,9 +1,12 @@
+# Usage: ruby getAllTime.rb <username> <list>
+
 require_relative "../lib/letteropend"
 
-print "Enter username: "
-un = gets().chomp
-print "Enter list: "
-ul = gets().chomp
+# Username
+un = ARGV[0]
+
+# List
+ul = ARGV[1]
 
 # global variable and function ; yeah I know it's jank
 $ct = 0

data/examples/getDirectedFilms.rb

@@ -1,3 +1,6 @@
+# Usage: ruby getDirectedFilms.rb
+# TODO: Allow the user to pass command line arguments
+
 require_relative "../lib/letteropend"
 
 puts "Looking at films directed by Alfred Hitchcock"

data/examples/getFilmDetails.rb

@@ -1,3 +1,6 @@
+# Usage: ruby getFilmDetails.rb
+# TODO: Allow the user to pass command line arguments
+
 require_relative "../lib/letteropend"
 
 Letteropend::Film.config do

data/examples/getNumberOfFilms.rb

@@ -1,9 +1,12 @@
+# Usage: ruby getNumberOfFilms.rb <username> <list>
+
 require_relative "../lib/letteropend"
 
-print "Enter username: "
-un = gets().chomp
-print "Enter list: "
-ul = gets().chomp
+# Username
+un = ARGV[0]
+
+# List
+ul = ARGV[1]
 
 Letteropend::List.config do
   on :new_page do

data/lib/letteropend/film.rb

@@ -4,33 +4,32 @@ require "open-uri"
 module Letteropend
   # The Film class
   class Film
-    attr_reader :url
+    attr_reader :slug, :url
     @@valid_attributes = [:title, :tagline, :overview, :runtime]
     @@valid_events = [:model_updated, :model_updating]
 
     # Creates a new film instance
     #
-    # @param url - the url letterboxd assigns the film
+    # @param id - the title letterboxd assigns the film for the url
     # @param details - the attributes the user assigns the film (instead of pulling from letterboxd)
     # @param events - block of user defined events
-    def initialize(url, *details, &events)
-      @url = url
+    def initialize(slug, details={}, &events)
+      @slug = slug.split("/").last
+      @url = "http://letterboxd.com/film/#{@slug}/"
       @pulled = false
       @pulling = false
       @events = {}
 
       # assign each detail to a method
-      details.each do |detail|
-        detail.each do |key, value|
-
-          # only assign valid attributes to a film
-          if @@valid_attributes.include? key
-            define_singleton_method(key, lambda{value})
-          else
-            puts "Error: trying to assign invalid film data | film: #{@url}, attribute: #{key}"
-          end
+      details.each do |key, value|
 
+        # only assign valid attributes to a film
+        if @@valid_attributes.include? key
+          define_singleton_method(key, lambda{value})
+        else
+          puts "Error: trying to assign invalid film data | film: #{@slug}, attribute: #{key}"
         end
+
       end
 
       # assign events to film object
@@ -83,7 +82,7 @@ module Letteropend
       model_updating
 
       # pull the page from the url
-      page = Nokogiri::HTML(open("http://www.letterboxd.com/film/#{@url}/"))
+      page = Nokogiri::HTML(open(@url))
   
       # get the title for the film
       title = page.css("h1.film-title").text
@@ -114,7 +113,7 @@ module Letteropend
   
     # Equivalence operator
     def ==(film)
-      @url == film.url
+      @slug == film.slug
     end
 
     # Method Missing implementation
@@ -127,7 +126,7 @@ module Letteropend
           pull_data
           self.send(sym)
         elsif !@pulled and @pulling
-          puts "Error: trying to get film data prematurely | film: #{@url}, method: #{sym}"
+          puts "Error: trying to get film data prematurely | film: #{@slug}, method: #{sym}"
         end
       elsif (@@valid_events.include? sym)
         # no method was defined for this event

data/lib/letteropend/version.rb

@@ -1,3 +1,3 @@
 module Letteropend
-  VERSION = "2.0.0"
+  VERSION = "2.0.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: letteropend
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.0.1
 platform: ruby
 authors:
 - Jesse Jurman
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-19 00:00:00.000000000 Z
+date: 2014-08-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri