From a8c27750ef1cf9d0ad1a2a6ebe33307da900a5c1 Mon Sep 17 00:00:00 2001
From: Michael Bianco <iloveitaly@gmail.com>
Date: Thu, 10 May 2012 10:18:56 -0400
Subject: [PATCH 1/9] Spree 1.1 compatibility. Minor fix to shipping option
 filtering in checkout process

---
 app/overrides/modify_shipping_options_display.rb | 3 +--
 spree_digital.gemspec                            | 2 +-
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/app/overrides/modify_shipping_options_display.rb b/app/overrides/modify_shipping_options_display.rb
index e05104a..69cf011 100644
--- a/app/overrides/modify_shipping_options_display.rb
+++ b/app/overrides/modify_shipping_options_display.rb
@@ -20,10 +20,9 @@ Deface::Override.new(:virtual_path => "spree/checkout/_delivery",
   <%== t 'digital_shipping', :email => @order.email %> (<%= number_to_currency @order.digital_shipping_method[:cost] %>)
 </label>
 <% else %>
-<% filtered_rate_hash = @order.rate_hash.select { |m| !(@order.digital_shipping_method && shipping_method[:id] == @order.digital_shipping_method[:id]) } %>
+<% filtered_rate_hash = @order.rate_hash.select { |m| !(@order.digital_shipping_method && m[:id] == @order.digital_shipping_method[:id]) } %>
 <% if filtered_rate_hash.count > 0 %>
 <% filtered_rate_hash.each do |shipping_method| %>
-<% next if @order.digital_shipping_method && shipping_method[:id] == @order.digital_shipping_method[:id] %>
   <label>
     <%= radio_button(:order, :shipping_method_id, shipping_method[:id]) %>
     <% if Spree::Config[:shipment_inc_vat] %>
diff --git a/spree_digital.gemspec b/spree_digital.gemspec
index c4118bf..aa17873 100644
--- a/spree_digital.gemspec
+++ b/spree_digital.gemspec
@@ -11,7 +11,7 @@ Gem::Specification.new do |s|
   s.require_path = 'lib'
   s.requirements << 'none'
   s.required_ruby_version = '>= 1.8.7'
-  s.add_dependency('spree_core', '~> 1.0.0')
+  s.add_dependency('spree_core', '>= 1.0.0')
 
   # test suite
   s.add_development_dependency 'shoulda-matchers'

From 9b193c217d3cd2127127570ea4f8c00bb923fb98 Mon Sep 17 00:00:00 2001
From: Jakob <jakob@platogo.com>
Date: Mon, 14 May 2012 10:58:05 +0200
Subject: [PATCH 2/9] let's see if it also works with spree 1.1.0

---
 spree_digital.gemspec | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/spree_digital.gemspec b/spree_digital.gemspec
index c4118bf..9b4a0d2 100644
--- a/spree_digital.gemspec
+++ b/spree_digital.gemspec
@@ -11,7 +11,7 @@ Gem::Specification.new do |s|
   s.require_path = 'lib'
   s.requirements << 'none'
   s.required_ruby_version = '>= 1.8.7'
-  s.add_dependency('spree_core', '~> 1.0.0')
+  s.add_dependency('spree_core', '~> 1.1.0')
 
   # test suite
   s.add_development_dependency 'shoulda-matchers'

From b7ef0a5b9d9d7c943212bcdfc6efc05ed9074966 Mon Sep 17 00:00:00 2001
From: Jakob <jakob@platogo.com>
Date: Mon, 14 May 2012 14:56:38 +0200
Subject: [PATCH 3/9] specify available attributes for update

---
 app/models/spree/digital.rb | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/app/models/spree/digital.rb b/app/models/spree/digital.rb
index d1d4bd1..30282a4 100644
--- a/app/models/spree/digital.rb
+++ b/app/models/spree/digital.rb
@@ -6,6 +6,8 @@ module Spree
     has_attached_file :attachment, :path => ":rails_root/private/digitals/:id/:basename.:extension"
   
     # TODO: Limit the attachment to one single file. Paperclip supports many by default :/
+
+    attr_accessible :variant_id, :attachment
       
   end
 end
\ No newline at end of file

From 3025683638f4d1e554211fd8aaca567068ebf288 Mon Sep 17 00:00:00 2001
From: Jakob <jakob@platogo.com>
Date: Mon, 14 May 2012 15:05:21 +0200
Subject: [PATCH 4/9] moved partial into other directory

---
 app/views/spree/digitals/_digital.html.erb | 5 +++++
 1 file changed, 5 insertions(+)
 create mode 100644 app/views/spree/digitals/_digital.html.erb

diff --git a/app/views/spree/digitals/_digital.html.erb b/app/views/spree/digitals/_digital.html.erb
new file mode 100644
index 0000000..c68a94b
--- /dev/null
+++ b/app/views/spree/digitals/_digital.html.erb
@@ -0,0 +1,5 @@
+<% if digital.attachment_file_name.present? %>
+  <%= digital.attachment_file_name %> (<%= number_to_human_size(digital.attachment_file_size) %>)
+<% else %>
+  <%=t 'broken_file' %>
+<% end %>

From 831b513e6fdb7dbd07a5d7621ce67ceeedae4d96 Mon Sep 17 00:00:00 2001
From: Jakob <jakob@platogo.com>
Date: Mon, 14 May 2012 15:48:00 +0200
Subject: [PATCH 5/9] specify available attributes for update

---
 app/models/spree/digital_link.rb | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/app/models/spree/digital_link.rb b/app/models/spree/digital_link.rb
index 1b794bc..f8c0032 100644
--- a/app/models/spree/digital_link.rb
+++ b/app/models/spree/digital_link.rb
@@ -27,6 +27,8 @@ module Spree
       self.secret = SecureRandom.hex(15)
       self.access_counter = 0
     end
-  
+
+    attr_accessible :digital_id, :line_item_id, :secret
+
   end
 end
\ No newline at end of file

From 1fd0cd2916bb5671385a9abeda1204b3542bb48e Mon Sep 17 00:00:00 2001
From: Jakob <jakob@platogo.com>
Date: Mon, 14 May 2012 15:54:57 +0200
Subject: [PATCH 6/9] specify available attributes for update

---
 app/models/spree/digital_link.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/models/spree/digital_link.rb b/app/models/spree/digital_link.rb
index f8c0032..1bf1ee7 100644
--- a/app/models/spree/digital_link.rb
+++ b/app/models/spree/digital_link.rb
@@ -28,7 +28,7 @@ module Spree
       self.access_counter = 0
     end
 
-    attr_accessible :digital_id, :line_item_id, :secret
+    attr_accessible :digital, :line_item, :secret
 
   end
 end
\ No newline at end of file

From 5cfa1175af738bdaf9bc91ac708e4c64ebd9c78b Mon Sep 17 00:00:00 2001
From: Michael Bianco <iloveitaly@gmail.com>
Date: Thu, 7 Jun 2012 10:50:40 -0400
Subject: [PATCH 7/9] Updated testing to use spree 1.1 and new factorygirl
 syntax

---
 .rspec                                 |  1 +
 Gemfile                                |  4 ++--
 spec/factories.rb                      |  6 ------
 spec/factories/digital_factory.rb      |  6 ++++--
 spec/factories/digital_link_factory.rb |  8 +++++---
 spec/models/digital_link_spec.rb       | 18 +++++++++---------
 spec/models/digital_spec.rb            |  4 ++--
 spec/models/line_item_spec.rb          |  8 ++++----
 spec/models/order_spec.rb              | 26 +++++++++++++-------------
 spec/spec_helper.rb                    |  4 ++--
 10 files changed, 42 insertions(+), 43 deletions(-)
 create mode 100644 .rspec
 delete mode 100644 spec/factories.rb

diff --git a/.rspec b/.rspec
new file mode 100644
index 0000000..cf6add7
--- /dev/null
+++ b/.rspec
@@ -0,0 +1 @@
+--colour
\ No newline at end of file
diff --git a/Gemfile b/Gemfile
index caa8242..d272462 100644
--- a/Gemfile
+++ b/Gemfile
@@ -1,8 +1,6 @@
 source 'http://rubygems.org'
 
 group :test do
-  # without ffaker in test it wont init
-  # https://github.com/spree/spree/pull/833
   gem 'ffaker'
   gem 'shoulda-matchers'
   gem 'guard-rspec'
@@ -13,4 +11,6 @@ group :test do
   end
 end
 
+gem 'spree', '~> 1.1.1'
+
 gemspec
diff --git a/spec/factories.rb b/spec/factories.rb
deleted file mode 100644
index c0ae53a..0000000
--- a/spec/factories.rb
+++ /dev/null
@@ -1,6 +0,0 @@
-require 'factory_girl'
-
-Dir["#{File.dirname(__FILE__)}/factories/**"].each do |f|
-  fp =  File.expand_path(f)
-  require fp
-end
diff --git a/spec/factories/digital_factory.rb b/spec/factories/digital_factory.rb
index 1b029ca..c8e6f3c 100644
--- a/spec/factories/digital_factory.rb
+++ b/spec/factories/digital_factory.rb
@@ -1,3 +1,5 @@
-Factory.define :digital, :class => Spree::Digital do |f|
-  f.variant { |p| p.association(:variant) }
+FactoryGirl.define do
+  factory :digital, :class => Spree::Digital do |f|
+    f.variant { |p| p.association(:variant) }
+  end
 end
\ No newline at end of file
diff --git a/spec/factories/digital_link_factory.rb b/spec/factories/digital_link_factory.rb
index 140c644..0565be7 100644
--- a/spec/factories/digital_link_factory.rb
+++ b/spec/factories/digital_link_factory.rb
@@ -1,4 +1,6 @@
-Factory.define :digital_link, :class => Spree::DigitalLink do |f|
-  f.digital { |p| p.association(:digital) }
-  f.line_item { |p| p.association(:line_item) }
+FactoryGirl.define do
+  factory :digital_link, :class => Spree::DigitalLink do |f|
+    f.digital { |p| p.association(:digital) }
+    f.line_item { |p| p.association(:line_item) }
+  end
 end
\ No newline at end of file
diff --git a/spec/models/digital_link_spec.rb b/spec/models/digital_link_spec.rb
index eb75cf3..d59714b 100644
--- a/spec/models/digital_link_spec.rb
+++ b/spec/models/digital_link_spec.rb
@@ -10,29 +10,29 @@ describe Spree::DigitalLink do
 
   context "#create" do
     it "should have an appropriately long secret" do
-      Factory(:digital_link).secret.length.should == 30
+      FactoryGirl.create(:digital_link).secret.length.should == 30
     end
     
     it "should have the access counter being an Integer on zero" do
-      Factory(:digital_link).access_counter.should == 0
+      FactoryGirl.create(:digital_link).access_counter.should == 0
     end
   end
   
   context "#update" do
     it "should not change the secret when updated" do
-      digital_link = Factory(:digital_link)
+      digital_link = FactoryGirl.create(:digital_link)
       secret = digital_link.secret
       digital_link.increment(:access_counter).save
       digital_link.secret.should == secret
     end
     
     it "should enforce to have an associated digital" do
-      link = Factory(:digital_link)
+      link = FactoryGirl.create(:digital_link)
       lambda { link.update_attributes!(:digital => nil) }.should raise_error(ActiveRecord::RecordInvalid)
     end
 
     it "should not allow an empty or too short secret" do
-      link = Factory(:digital_link)
+      link = FactoryGirl.create(:digital_link)
       lambda { link.update_attributes!(:secret => nil) }.should raise_error(ActiveRecord::RecordInvalid)
       lambda { link.update_attributes!(:secret => 'x' * 25) }.should raise_error(ActiveRecord::RecordInvalid)
     end
@@ -40,14 +40,14 @@ describe Spree::DigitalLink do
   
   context "authorization" do
    it "should increment the counter using #authorize!" do
-     link = Factory(:digital_link)
+     link = FactoryGirl.create(:digital_link)
      link.access_counter.should == 0
      link.authorize!
      link.access_counter.should == 1
    end
    
    it "should not be #authorized? when the access_counter is too high" do
-     link = Factory(:digital_link)
+     link = FactoryGirl.create(:digital_link)
      link.stub(:access_counter => Spree::DigitalConfiguration[:authorized_clicks] - 1)
      link.authorizable?.should be_true
      link.stub(:access_counter => Spree::DigitalConfiguration[:authorized_clicks])
@@ -55,7 +55,7 @@ describe Spree::DigitalLink do
    end
       
    it "should not be #authorize! when the created_at date is too far in the past" do
-     link = Factory(:digital_link)
+     link = FactoryGirl.create(:digital_link)
      link.authorize!.should be_true
      link.stub(:created_at => (Spree::DigitalConfiguration[:authorized_days] * 24 - 1).hours.ago)
      link.authorize!.should be_true
@@ -64,7 +64,7 @@ describe Spree::DigitalLink do
    end
    
    it "should not be #authorized? when both access_counter and created_at are invalid" do
-     link = Factory(:digital_link)
+     link = FactoryGirl.create(:digital_link)
      link.authorizable?.should be_true
      link.stub(:access_counter => Spree::DigitalConfiguration[:authorized_clicks], :created_at => (Spree::DigitalConfiguration[:authorized_days] * 24 + 1).hours.ago)
      link.authorizable?.should be_false
diff --git a/spec/models/digital_spec.rb b/spec/models/digital_spec.rb
index 758d275..5ea6acf 100644
--- a/spec/models/digital_spec.rb
+++ b/spec/models/digital_spec.rb
@@ -13,8 +13,8 @@ describe Spree::Digital do
   
   context "#destroy" do
     #it "should destroy associated digital_links" do
-    #  digital = Factory(:digital)
-    #  3.times { digital.digital_links.create! :order => Factory(:order) }
+    #  digital = FactoryGirl.create(:digital)
+    #  3.times { digital.digital_links.create! :order => FactoryGirl.create(:order) }
     #  DigitalLink.count.should == 3      
     #  digital.destroy
     #  DigitalLink.count.should == 0      
diff --git a/spec/models/line_item_spec.rb b/spec/models/line_item_spec.rb
index f6b27df..b8a64dd 100644
--- a/spec/models/line_item_spec.rb
+++ b/spec/models/line_item_spec.rb
@@ -4,16 +4,16 @@ describe Spree::LineItem do
 
   context "#save" do
     it "should create one link for a single digital Variant" do
-      digital_variant = Factory(:variant, :digital => Factory(:digital))
-      line_item = Factory(:line_item, :variant => digital_variant)
+      digital_variant = FactoryGirl.create(:variant, :digital => FactoryGirl.create(:digital))
+      line_item = FactoryGirl.create(:line_item, :variant => digital_variant)
       links = digital_variant.digital.digital_links
       links.all.size.should == 1
       links.first.line_item.should == line_item
     end
 
     it "should create a link for each quantity of a digital Variant, even when quantity changes later" do
-      digital_variant = Factory(:variant, :digital => Factory(:digital))
-      line_item = Factory(:line_item, :variant => digital_variant, :quantity => 5)
+      digital_variant = FactoryGirl.create(:variant, :digital => FactoryGirl.create(:digital))
+      line_item = FactoryGirl.create(:line_item, :variant => digital_variant, :quantity => 5)
       links = digital_variant.digital.digital_links
       links.all.size.should == 5
       links.each { |link| link.line_item.should == line_item }
diff --git a/spec/models/order_spec.rb b/spec/models/order_spec.rb
index 83eb072..404d055 100644
--- a/spec/models/order_spec.rb
+++ b/spec/models/order_spec.rb
@@ -8,18 +8,18 @@ describe Spree::Order do
   
   context "#add_variant" do
     it "should add digital Variants of quantity 1 to an order" do
-      order = Factory(:order)
-      order.add_variant variant1 = Factory(:variant, :digital => Factory(:digital))
-      order.add_variant variant2 = Factory(:variant, :digital => Factory(:digital))
-      order.add_variant variant3 = Factory(:variant, :digital => Factory(:digital))
+      order = FactoryGirl.create(:order)
+      order.add_variant variant1 = FactoryGirl.create(:variant, :digital => FactoryGirl.create(:digital))
+      order.add_variant variant2 = FactoryGirl.create(:variant, :digital => FactoryGirl.create(:digital))
+      order.add_variant variant3 = FactoryGirl.create(:variant, :digital => FactoryGirl.create(:digital))
       order.line_items.first.variant.should == variant1
       order.line_items.second.variant.should == variant2
       order.line_items.third.variant.should == variant3
     end
     
     it "should handle quantity higher than 1 when adding one specific digital Variant" do
-      order = Factory(:order)
-      digital_variant = Factory(:variant, :digital => Factory(:digital))
+      order = FactoryGirl.create(:order)
+      digital_variant = FactoryGirl.create(:variant, :digital => FactoryGirl.create(:digital))
       order.add_variant digital_variant, 3
       order.line_items.first.quantity.should == 3
       order.add_variant digital_variant, 2
@@ -29,23 +29,23 @@ describe Spree::Order do
   
   context "line_item analysis" do
     it "should understand that all products are digital" do
-      order = Factory(:order)
+      order = FactoryGirl.create(:order)
       3.times do
-        order.add_variant Factory(:variant, :digital => Factory(:digital))
+        order.add_variant FactoryGirl.create(:variant, :digital => FactoryGirl.create(:digital))
       end
       order.digital?.should be_true
-      order.add_variant Factory(:variant, :digital => Factory(:digital)), 4
+      order.add_variant FactoryGirl.create(:variant, :digital => FactoryGirl.create(:digital)), 4
       order.digital?.should be_true
     end
     
     it "should understand that not all products are digital" do
-      order = Factory(:order)
+      order = FactoryGirl.create(:order)
       3.times do
-        order.add_variant Factory(:variant, :digital => Factory(:digital))
+        order.add_variant FactoryGirl.create(:variant, :digital => FactoryGirl.create(:digital))
       end
-      order.add_variant Factory(:variant) # this is the analog product
+      order.add_variant FactoryGirl.create(:variant) # this is the analog product
       order.digital?.should be_false
-      order.add_variant Factory(:variant, :digital => Factory(:digital)), 4
+      order.add_variant FactoryGirl.create(:variant, :digital => FactoryGirl.create(:digital)), 4
       order.digital?.should be_false
     end
   end
diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
index 93c5af0..cfcaac1 100644
--- a/spec/spec_helper.rb
+++ b/spec/spec_helper.rb
@@ -31,11 +31,11 @@ RSpec.configure do |config|
   config.use_transactional_fixtures = true
 end
 
-require File.expand_path("../factories", __FILE__)
+Dir[File.join(File.dirname(__FILE__), "factories/*.rb")].each {|f| require f }
 
 # not sure if this really adds anything, but this existed in the intial version of the spree_digital rspec testing
 RSpec::Matchers.define :have_valid_factory do |factory_name|
   match do |model|
-    Factory(factory_name).new_record?.should be_false
+    FactoryGirl.create(:factory_name).new_record?.should be_false
   end
 end
\ No newline at end of file

From 346322b8d3b722694fa25b132895772dd0e72c83 Mon Sep 17 00:00:00 2001
From: Michael Bianco <iloveitaly@gmail.com>
Date: Thu, 7 Jun 2012 10:54:22 -0400
Subject: [PATCH 8/9] Spec helper typo. Tests are passing.

---
 spec/spec_helper.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
index cfcaac1..aa638f7 100644
--- a/spec/spec_helper.rb
+++ b/spec/spec_helper.rb
@@ -36,6 +36,6 @@ Dir[File.join(File.dirname(__FILE__), "factories/*.rb")].each {|f| require f }
 # not sure if this really adds anything, but this existed in the intial version of the spree_digital rspec testing
 RSpec::Matchers.define :have_valid_factory do |factory_name|
   match do |model|
-    FactoryGirl.create(:factory_name).new_record?.should be_false
+    FactoryGirl.create(factory_name).new_record?.should be_false
   end
 end
\ No newline at end of file

From 094b648e2f255581d2a9f1a9d7068cd4307ab494 Mon Sep 17 00:00:00 2001
From: Michael Bianco <iloveitaly@gmail.com>
Date: Thu, 7 Jun 2012 10:55:48 -0400
Subject: [PATCH 9/9] Versionfile updates

---
 README.textile | 2 ++
 Versionfile    | 3 ++-
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/README.textile b/README.textile
index 9fe62d0..a6e7dfa 100644
--- a/README.textile
+++ b/README.textile
@@ -4,6 +4,8 @@ This is a spree extension to enable downloadable products. The design goal is to
 
 NOTE: This is still under development, but since things were falling into place, I thought I'd publish it.
 
+The master branch may work with spree 1.0, but the latest confirmed working version if referenced in the Versionfile.
+
 The idea is simple. You attach a file to a Product (or a Variant of this Product) and when people buy it, they will receive a link via email where they can download it once. There are a few assumptions that spree_digital (currently) makes and it's important to be aware of them. These might change, but since I programmed digital_spree over night, this is what you get ;)
 
 * The table structure of spree_core is not touched. Spree_digital lives parallel to spree_core and does not do any changes to your existing database, except adding two new tables.
diff --git a/Versionfile b/Versionfile
index 42e13a1..595dd62 100644
--- a/Versionfile
+++ b/Versionfile
@@ -1,2 +1,3 @@
 "0.7.x" => { :ref => "eddaea63959586d123007cbca3be5bf9c5edb1a7" }
-"1.0.x" => { :branch => "master" }
+"1.0.x" => { :ref => "a8c27750ef1cf9d0ad1a2a6ebe33307da900a5c1" }
+"1.1.x" => { :branch => "master" }