picasa-downloader 0.0.4 → 0.0.5

data/.gitignore

@@ -1,2 +1,3 @@
 *.gem
 Gemfile.lock
+tmp

data/README.md

@@ -23,6 +23,10 @@ Download all Picasa albums:
 
 `picasa-downloader all`
 
+Download a particular album:
+
+`picasa-downloader album "my album's name"`
+
 ## Development
 
 Run tests:

data/TODO.md

@@ -2,6 +2,5 @@
 
 * Possibility to determine the target root directory via CLI
 * Cucumber tests for happy paths
-* Use median year to determine the target directory
 * Video download
 * Allow synchronising with online albums

data/lib/picasa-downloader/album_persister.rb

@@ -8,26 +8,30 @@ module PicasaDownloader
 
     def download
       FileUtils.mkdir_p(get_temp_album_dir)
-      @client.list_photos(@album.id).each { |photo|
+      photos = @client.list_photos(@album.id)
+      download_and_persist_to_disk(photos)
+      move_to_final_dir(photos)
+    end
+
+    private
+
+    def download_and_persist_to_disk(photos)
+      def persist_to_disk(photo, photo_data)
+        File.open(get_temp_album_dir + photo.name, 'wb') { |file|
+          file.write(photo_data)
+          FileUtils.touch(file.path, :mtime => photo.created_date)
+        }
+      end
+      photos.each { |photo|
         photo_data = @client.download_photo(photo)
         if photo_data
-          File.open(get_temp_album_dir + photo.name, 'wb') { |file|
-            file.write(photo_data)
-            FileUtils.touch(file.path, :mtime => photo.created_date)
-          }
+          persist_to_disk(photo, photo_data)
         end
       }
-      move_to_final_dir
     end
 
-    private
-
-    def move_to_final_dir
-      file_modified_years = Dir.entries(get_temp_album_dir).map { |file|
-        File.new(get_temp_album_dir + file).mtime.year
-      }.reject { |year| year == nil }
-      album_min_year = file_modified_years.min
-      final_dir = get_final_album_dir_root(album_min_year)
+    def move_to_final_dir(photos)
+      final_dir = get_final_album_dir_root(PhotoHelper.median_year(photos))
       FileUtils.mkdir_p(final_dir)
       FileUtils.mv(get_temp_album_dir, final_dir)
     end

data/lib/picasa-downloader/photo_helper.rb

@@ -0,0 +1,8 @@
+module PicasaDownloader
+  class PhotoHelper
+    def self.median_year(photos)
+      years = photos.map { |photo| photo.created_date.year }
+      median = years[years.length / 2]
+    end
+  end
+end

data/picasa-downloader.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |s|
   s.name          = 'picasa-downloader'
-  s.version       = '0.0.4'
+  s.version       = '0.0.5'
   s.license       = 'MIT'
   s.summary       = 'Download your Google Picasa photo albums'
   s.description   =

data/spec/album_persister_spec.rb

@@ -1,6 +1,4 @@
-require 'vcr'
 require 'test_helper'
-require_relative '../lib/picasa-downloader'
 
 module PicasaDownloader
   describe AlbumPersister do

data/spec/client_spec.rb

@@ -1,6 +1,4 @@
-require 'vcr'
-require_relative 'test_helper'
-require_relative '../lib/picasa-downloader'
+require 'test_helper'
 
 module PicasaDownloader
   describe Client do

data/spec/env_credentials_spec.rb

@@ -1,5 +1,4 @@
 require 'test_helper'
-require_relative '../lib/picasa-downloader'
 
 module PicasaDownloader
   describe EnvCredentials do

data/spec/photo_helper_spec.rb

@@ -0,0 +1,26 @@
+require 'test_helper'
+
+module PicasaDownloader
+  describe PhotoHelper do
+    it "calculates median year for one photo" do
+      photo1 = double('photo')
+      photo1.stub('created_date' => Time.utc(2008, 7, 8))
+      PhotoHelper.median_year([photo1]).should be 2008
+    end
+
+    it "calculates median year for five photos" do
+      photo1 = double('photo')
+      photo1.stub('created_date' => Time.utc(2000, 7, 8))
+      photo2 = double('photo')
+      photo2.stub('created_date' => Time.utc(2003, 7, 8))
+      photo3 = double('photo')
+      photo3.stub('created_date' => Time.utc(2009, 7, 8))
+      photo4 = double('photo')
+      photo4.stub('created_date' => Time.utc(2010, 7, 8))
+      photo5 = double('photo')
+      photo5.stub('created_date' => Time.utc(2011, 7, 8))
+      PhotoHelper.median_year([photo1, photo2, photo3, photo4, photo5]).should
+        be 2009
+    end
+  end
+end

data/spec/test_helper.rb

@@ -1,4 +1,6 @@
+require 'vcr'
 require 'require_relative'
+require_relative '../lib/picasa-downloader'
 
 VCR.configure do |config|
   config.cassette_library_dir = "fixtures/cassettes"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: picasa-downloader
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-07-22 00:00:00.000000000 Z
+date: 2012-07-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: gdata_19
@@ -199,10 +199,12 @@ files:
 - lib/picasa-downloader/env_credentials.rb
 - lib/picasa-downloader/errors.rb
 - lib/picasa-downloader/photo.rb
+- lib/picasa-downloader/photo_helper.rb
 - picasa-downloader.gemspec
 - spec/album_persister_spec.rb
 - spec/client_spec.rb
 - spec/env_credentials_spec.rb
+- spec/photo_helper_spec.rb
 - spec/test_helper.rb
 homepage: https://github.com/laurilehmijoki/picasa-downloader
 licenses:
@@ -237,4 +239,5 @@ test_files:
 - spec/album_persister_spec.rb
 - spec/client_spec.rb
 - spec/env_credentials_spec.rb
+- spec/photo_helper_spec.rb
 - spec/test_helper.rb