get_freaky 0.3.0 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: cd9a568627ed1bfc123b1379ef94ac04b27f35c4
-  data.tar.gz: 126773bf0a32042076d10ebc7994ae5e710fcd32
+  metadata.gz: 879240e6e90154c17f2dde2fd7a202e11669fb21
+  data.tar.gz: 58e08b94cb3a887cf007df433aa115da36eb7fb3
 SHA512:
-  metadata.gz: fd8f6dad18d40da39937eab8b8dbfb8198afe4997e3498ec4763ee1a1e78777a71da77e874e1f3f57452f996fce60138d3b36fe8d8e657e8612976a80fc5e3f8
-  data.tar.gz: b4aa75f699b37ddc9209aa867377a19c58401c64750190ae3b2e64626b13673b722c8a6f401d48ba50e115e85ff142591a5b95ed52eeb77a8bed0b2c820c1e99
+  metadata.gz: 3b57896d5b65dea702dd816c3be14858bc8ff0f083857d4a5fd596cbd83fb4dcdcb43ac4506e5396ef1869e0b5044a1e803e9e91e102bfb1308bae179de98499
+  data.tar.gz: 1f06ee9aa36eaa2178636732086644b99f4a17721f56b02c677e4640cd1e7b1a5f8e92c9a4d81c6c5b54d06cdb0d3515c56f6d2f377403e67c3d0f242467ff38

data/bin/get_freaky

@@ -54,11 +54,17 @@ command :event do |c|
   c.summary = 'Shows the videos available from confreaks for an event (an instance of a conference in a particular year)'
   c.description = c.summary
   c.example 'Display information about the railsberry2013', 'get_freaky event railsberry2013'
+  c.example 'Download all the videos from railsberry2013', 'get_freaky event railsberry2013 --all'
   c.action do |args, options|
+
     unless args[0]
       puts `get_freaky conf --help`
     else
       event = Event.find(args[0])
+      if options.include "all"
+        event.download_all_videos!
+        break
+      end
       videos = event.video_list
       video_title = choose("Choose a talk to see more information about it:\n\n", *videos)
       video = Video.find(event.name, video_title)
@@ -68,6 +74,18 @@ command :event do |c|
   end
 end
 
+command :download_event do |c|
+  c.syntax = 'get_freaky download_event EVENT_NAME [options]'
+  c.summary = 'Downloads all the confreaks videos for a particular event'
+  c.description = c.summary
+  c.example 'Download all the videos from RubyConf 2015', 'get_freaky download_event rubyconf2015'
+  c.action do |args, options|
+    event = Event.find(args[0])
+    event.download_all_videos!
+  end
+end
+
+
 command :download do |c|
   c.syntax = 'get_freaky download VIDEO_NAME EVENT_NAME [options]'
   c.summary = 'Downloads a confreaks video locally using the viddl-rb library. Note that the "event name" given needs to be formatted in the same way as the listing show by the "event" command. Be sure to put quotes around video titles of multiple words and escape special exclamation points'

data/lib/get_freaky.rb

@@ -3,8 +3,10 @@ require 'slugify'
 require 'viddl-rb'
 
 require_relative 'get_freaky/conference'
+require_relative 'get_freaky/error_object'
 require_relative 'get_freaky/event'
+require_relative 'get_freaky/info'
+require_relative 'get_freaky/null_conference'
+require_relative 'get_freaky/null_event'
+require_relative 'get_freaky/version'
 require_relative 'get_freaky/video'
-require_relative "get_freaky/version"
-require_relative "get_freaky/info"
-

data/lib/get_freaky/conference.rb

@@ -13,11 +13,7 @@ class Conference
     response = get("/conferences/#{name}.json")
     # TODO: I need to figure out a better way of dealing with 404 errors
     if response["status"] == 404
-      self.new(
-        "Not Found",
-        "Not Found",
-        "Not Found"
-      )
+      NullConference.new("No conference was found with that name")
     elsif response.success?
       self.new(
         response["name"],
@@ -36,10 +32,14 @@ class Conference
   end
 
   def event_list
-    if events == "Not Found"
-      "There was no conference found by that name."
-    else
+    if valid? 
       events.map { |event| event["short_code"] }
+    else
+      "There was no conference found by that name."
     end
   end
+
+  def valid?
+    true
+  end
 end

data/lib/get_freaky/error_object.rb

@@ -0,0 +1,13 @@
+class ErrorObject
+  def initialize(message)
+    @message = message
+  end
+
+  def valid?
+    false
+  end
+
+  def to_s
+    "Error: #{@message}"
+  end
+end

data/lib/get_freaky/event.rb

@@ -5,12 +5,12 @@ class Event
   attr_accessor :short_code, :conference_name, :conference_id, :video_count, :date
 
   # TODO: I'm pretty sure that at this number of arguments it should just take a hash
-  def initialize(short_code, conference_name, conference_id, video_count, date)
-    self.short_code = short_code
-    self.conference_name = conference_name
-    self.conference_id = conference_id
-    self.video_count = video_count
-    self.date = date
+  def initialize(params = {})
+    self.short_code = params[:short_code]
+    self.conference_name = params[:conference_name]
+    self.conference_id = params[:conference_id]
+    self.video_count = params[:video_count]
+    self.date = params[:date]
   end
 
   # Alias short_code as name; "short_code" is consistent with the api, but "name" is easier to remember
@@ -25,23 +25,16 @@ class Event
   def self.find(short_code)
     response = get("/events/#{short_code}.json")
     if response["status"] == 404
-      # TODO: The below is totally embarrassing...need to figure out a better way to handle this (though it's still better than nil)
-      # Look into using NULL Objects?
-      # Better exceptions?
-      self.new(
-        "Not Found",
-        "Not Found",
-        "Not Found",
-        "Not Found",
-        "Not Found"
-      )
+      NullEvent.new("No event was found with that name")
     elsif response.success?
       self.new(
-        response["short_code"],
-        response["conference"]["name"],
-        response["conference"]["id"],
-        response["video_count"],
-        fix_date(response["start_at"])
+        {
+          :short_code      => response["short_code"],
+          :conference_name => response["conference"]["name"],
+          :conference_id   => response["conference"]["id"],
+          :video_count     => response["video_count"],
+          :date            => fix_date(response["start_at"])
+        }
       )
     else
       # This raises the net/http response that was raised
@@ -55,8 +48,12 @@ class Event
 
   def videos
     response = HTTParty.get("#{BASE_URI}/events/#{short_code}/videos.json")
+    videos = []
     if response.success?
-      return response
+      response.each do |video|
+        videos << Video.find(short_code, video["title"])
+      end
+      return videos
     else
       # This raises the net/http response that was raised
       raise response.response
@@ -65,22 +62,25 @@ class Event
 
   # TODO: This implementation isn't really ideal. By calling the videos method you're hitting the api again.
   # Ideally you want to just hit the api once and store that information away since you know that hitting the API takes more resources (and time frankly) then just storing the names in memory which is all you really care about until you need to actually get the information for that particular video
-  #
   # Probably should do something like store the list of video names as strings as soon as you create the object and then keep that array for future use
   # This seems slightly better but I'm still not sure it's ideal
   def video_list
-    if self.videos == "Not Found"
-      "There was no conference found by that name."
-    else
-      videos.map { |video| video["title"] }
-    end
+    videos.map { |video| video.title }
+  end
+
+  def download_all_videos!
+    videos.each { |video| video.download }
   end
 
+  def valid?
+    true
+  end
+
+  private
+
   def self.fix_date(date)
     year, month, day = date[0..3].to_i, date[5..6].to_i, date[8..9].to_i
     Date.new(year, month, day)
   end
 
-  private
-
 end

data/lib/get_freaky/null_conference.rb

@@ -0,0 +1,5 @@
+class NullConference < ErrorObject
+  def name
+    to_s
+  end
+end

data/lib/get_freaky/null_event.rb

@@ -0,0 +1,20 @@
+class NullEvent < ErrorObject
+  def short_code; nil ; end
+  def conference_name; nil; end
+  def conference_id; nil; end
+  def video_count; nil; end
+  def date; nil; end
+  # Could use Avdi Grimm's Naught gem to simplify the above
+
+  def video_list
+    to_s
+  end
+
+  def name
+    to_s
+  end
+
+  def valid?
+    false
+  end
+end

data/lib/get_freaky/version.rb

@@ -1,4 +1,4 @@
 module GetFreaky
-  VERSION = "0.3.0"
-  DATE = %q{2015-05-14}
+  VERSION = "0.3.1"
+  DATE = %q{2015-11-14}
 end

data/lib/get_freaky/video.rb

@@ -1,4 +1,5 @@
 require 'paint'
+require 'youtube-dl.rb'
 
 class Video
   include HTTParty
@@ -36,7 +37,8 @@ class Video
   end
 
   def download
-    cmd = %x[ viddl-rb #{url} ]
+    puts "downloading #{title}"
+    YoutubeDL.download url, { output: "#{title}.mp4" }
   end
 
   def event

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: get_freaky
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.1
 platform: ruby
 authors:
 - Stephen Mariano Cabrera
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-14 00:00:00.000000000 Z
+date: 2015-11-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: commander
@@ -80,6 +80,20 @@ dependencies:
     - - '='
       - !ruby/object:Gem::Version
         version: 1.0.0
+- !ruby/object:Gem::Dependency
+  name: youtube-dl.rb
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: fabricas
   requirement: !ruby/object:Gem::Requirement
@@ -164,8 +178,11 @@ files:
 - bin/setup
 - lib/get_freaky.rb
 - lib/get_freaky/conference.rb
+- lib/get_freaky/error_object.rb
 - lib/get_freaky/event.rb
 - lib/get_freaky/info.rb
+- lib/get_freaky/null_conference.rb
+- lib/get_freaky/null_event.rb
 - lib/get_freaky/version.rb
 - lib/get_freaky/video.rb
 homepage: https://github.com/smcabrera/get_freaky
@@ -189,7 +206,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.7
+rubygems_version: 2.4.5.1
 signing_key: 
 specification_version: 4
 summary: Get Freaky!