dugway 0.12.2 → 1.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2f39a4ac63900661151c83b9fb34e373e9df16e5ee4ed6896a4489681592ccc3
-  data.tar.gz: 7fdd8f77279734cd5df737c2fce1fdefd09de681cf7dad3a0434e128fb9d49d9
+  metadata.gz: a7327098642f17da47ce2769de014026ed204725960b39133769b02943ceaae4
+  data.tar.gz: b158bab3913e4fedd00064531937abe052cda1325a0dace29332ab4f77cba70e
 SHA512:
-  metadata.gz: ef056b3003a09886d4727ce3d1392abd45d40b2a234fecb1284398e8525060e2366f72002c340bba0cf23aecc42298c335c826bab596e8d4d916d5cca6fada78
-  data.tar.gz: b63d62a8ab48d74226dd851e6cd8ff6f2c9a06bfb53f3bda7ff5dbd14c47cb4548f57198f6b6e501303dbb7173d7682990c44ed578fe77084aa978668edbb085
+  metadata.gz: 17dd683322eeb0b8abfd1e2f9c729a14ecaea0e72328e07160e355e6891da1a845777cc553f88d4e7ddc23187b54337c6d5dda7d4d04f23865ae47d2c5e560c6
+  data.tar.gz: 71b0394669bb047a8003bbd61e713fffe985e70ffbc76561e1ea5acdb744f26ad3c4e380ce8b2b9875c67985936f895774a64112e3eb43642ea44edec26f3dad

data/README.md

@@ -182,7 +182,7 @@ Run your theme in any browser by starting the Dugway server:
 dugway server
 ```
 
-By default this will serve your theme at http://0.0.0.0:9292. You can then stop
+By default this will serve your theme at http://127.0.0.1:9292/. You can then stop
 the server by hitting CTRL+C.
 
 ### Pow

data/dugway.gemspec

@@ -24,10 +24,10 @@ Gem::Specification.new do |s|
   s.add_dependency('activesupport', '~> 5.2')
   s.add_dependency('liquid', '~> 3.0.6')
   s.add_dependency('coffee-script', '~> 2.4.1')
-  s.add_dependency('sass', '~> 3.2.5')
+  s.add_dependency('sass', '~> 3.4.25')
   s.add_dependency('sprockets', '~> 2.0')
   s.add_dependency('sprockets-sass', '~> 1.3.1')
-  s.add_dependency('compass', '~> 0.12.2')
+  s.add_dependency('compass', '~> 1.0.3')
   s.add_dependency('httparty', '~> 0.10.0')
   s.add_dependency('better_errors', '~> 0.9.0')
   s.add_dependency('will_paginate', '~> 3.0.4')
@@ -36,7 +36,7 @@ Gem::Specification.new do |s|
   s.add_dependency('thor', '~> 0.20.3')
   s.add_dependency('rubyzip', '~> 0.9.9')
   s.add_dependency('uglifier', '~> 4.1')
-  s.add_dependency('thin', '~> 1.7.2')
+  s.add_dependency('thin', '~> 1.8.0')
   s.add_dependency('bigcartel-theme-fonts')
   s.add_dependency('bigcartel-currency-locales')
 

data/lib/dugway/cart.rb

@@ -31,7 +31,7 @@ module Dugway
     end
 
     def shipping
-      { 
+      {
         'enabled' => false,
         'amount' => 0.0,
         'strict' => false,
@@ -40,14 +40,14 @@ module Dugway
     end
 
     def tax
-      { 
+      {
         'enabled' => false,
         'amount' => 0.0
       }
     end
 
     def discount
-      { 
+      {
         'enabled' => false,
         'pending' => false,
         'amount' => 0.0

data/lib/dugway/cli/templates/source/contact.html

@@ -22,14 +22,9 @@
       <label for="message">Message</label>
       {{ contact | contact_input: 'message' }}
     </li>
-    <li>
-      <label for="captcha">Spam check</label>
-      <p>Please enter the characters from the image.</p>
-      <div>{{ contact.captcha }}</div>
-      {{ contact | contact_input: 'captcha' }}
-    </li>
     <li>
       <button type="submit" name="submit" title="Send us an email">Send</button>
+      <div>{{ contact.recaptcha }}</div>
     </li>
   </ul>
 </form>

data/lib/dugway/contact_form_validator.rb

@@ -12,25 +12,19 @@ module Dugway
         'All fields are required'
       elsif param_does_not_match(:email, email_format)
         'Invalid email address'
-      elsif param_does_not_match(:captcha, captcha_format)
-        'Spam check was incorrect'
       end
     end
 
     private
 
     def required_fields
-      [ :name, :email, :subject, :message, :captcha ]
+      [ :name, :email, :subject, :message ]
     end
 
     def email_format
       /^([^@\s]+)@((?:[-a-zA-Z0-9]+\.)+[a-zA-Z]{2,})$/
     end
 
-    def captcha_format
-      /^rQ9pC$/i
-    end
-
     def param_does_not_match(param_name, regex)
       !(params[param_name.to_sym] =~ regex)
     end

data/lib/dugway/liquid/drops/base_drop.rb

@@ -42,7 +42,7 @@ module Dugway
 
         nil
       end
-      
+
       def method_missing(method, *args, &block)
         before_method(method.to_s)
       end

data/lib/dugway/liquid/drops/contact_drop.rb

@@ -21,6 +21,15 @@ module Dugway
         %{<img id="captcha_image" src="https://s3.amazonaws.com/bigcartel/captcha/28e3d1288cbc70c0cd1a2d10845f8e11e1a90d14.png">}
       end
 
+      def recaptcha
+        @recaptcha ||= begin
+          html = "This site is protected by reCAPTCHA and the Google "
+          html += '<a href="https://policies.google.com/privacy">Privacy Policy</a> and '
+          html += '<a href="https://policies.google.com/terms">Terms of Service</a> apply.'
+          html
+        end
+      end
+
       def sent
         request.path == '/contact' && request.post? && errors.blank?
       end

data/lib/dugway/liquid/drops/pages_drop.rb

@@ -4,6 +4,10 @@ module Dugway
       def all
         @all ||= source.select { |page| page['category'] == 'custom' }
       end
+
+      def cart
+        @cart ||= source.find { |page| page['permalink'] == 'cart' }
+      end
     end
   end
 end

data/lib/dugway/liquid/drops/products_drop.rb

@@ -68,7 +68,7 @@ module Dugway
 
       def search_terms
         params[:search]
-      end  
+      end
 
       def page
         if @context['internal'].present? && @context['internal'].has_key?('page') # has_key? here because 'page' will be nil for get blocks

data/lib/dugway/liquid/filters/url_filters.rb

@@ -28,7 +28,7 @@ module Dugway
         if name.is_a?(Drops::ThemeDrop)
           '/theme.js'
         elsif name == 'api'
-          'http://cache0.bigcartel.com/api/1/api.usd.js'
+          'http://cache0.bigcartel.com/api/5/api.usd.js'
         else
           name
         end

data/lib/dugway/version.rb

@@ -1,3 +1,3 @@
 module Dugway
-  VERSION = "0.12.2"
+  VERSION = "1.0.3"
 end

data/spec/features/contact_form_spec.rb

@@ -16,18 +16,12 @@ feature 'Contact form' do
     expect(page).to have_content('Invalid email address')
   end
 
-  scenario 'with an invalid captcha' do
-    submit_contact_form(:captcha => 'blah')
-    expect(page).to have_content('Spam check was incorrect')
-  end
-
   def submit_contact_form(fields={})
     fields.reverse_merge!(
       :name => 'Joe',
       :email => 'joe@example.com',
       :subject => 'Testing',
       :message => 'Hi there',
-      :captcha => 'rQ9pC'
     )
 
     visit '/contact'
@@ -36,7 +30,6 @@ feature 'Contact form' do
     fill_in 'Email', :with => fields[:email]
     fill_in 'Subject', :with => fields[:subject]
     fill_in 'Message', :with => fields[:message]
-    fill_in 'Spam check', :with => fields[:captcha]
 
     click_button 'Send'
   end

data/spec/features/page_rendering_spec.rb

@@ -50,7 +50,7 @@ feature 'Page rendering' do
     visit '/contact'
     expect(page).to have_content('Dugway') # layout.html
     expect(page).to have_content('Contact')
-    expect(page).to have_content('Spam check')
+    expect(page).to have_content('protected by reCAPTCHA')
   end
 
   scenario 'maintenance.html' do

data/spec/fixtures/store/page/about-us.json

@@ -1 +1 @@
-{"category":"custom","content":"<p>We're really cool!</p>","permalink":"about-us","name":"About Us","id":95821979,"url":"/about-us"}
+{"category":"custom","content":"<p>We're really cool!</p>","permalink":"about-us","name":"About Us","id":95821979,"url":"/about-us"}

data/spec/fixtures/theme/contact.html

@@ -22,14 +22,9 @@
       <label for="message">Message</label>
       {{ contact | contact_input: 'message' }}
     </li>
-    <li>
-      <label for="captcha">Spam check</label>
-      <p>Please enter the characters from the image.</p> 
-      <div>{{ contact.captcha }}</div>       
-      {{ contact | contact_input: 'captcha' }}      
-    </li>
     <li>
       <button type="submit" name="submit" title="Send us an email">Send</button>
+      <div>{{ contact.recaptcha }}</div>
     </li>
   </ul>
 </form>

data/spec/units/dugway/contact_form_validator_spec.rb

@@ -11,7 +11,6 @@ describe Dugway::ContactFormValidator do
       :email => "name@example.com",
       :subject => "subject",
       :message => "message",
-      :captcha => "rQ9pc",
     }
   end
 
@@ -36,21 +35,11 @@ describe Dugway::ContactFormValidator do
       assert_required_fields_error
     end
 
-    it "returns an error for a missing captcha" do
-      validator.params[:captcha] = "   "
-      assert_required_fields_error
-    end
-
     it "returns an error for invalid email format" do
       validator.params[:email] = "foo-at-foo-dot-net"
       expect(validator.error_message).to eq "Invalid email address"
     end
 
-    it "returns an error for incorrect captcha" do
-      validator.params[:captcha] = "oops"
-      expect(validator.error_message).to eq "Spam check was incorrect"
-    end
-
     def assert_required_fields_error
       expect(validator.error_message).to eql "All fields are required"
     end

data/spec/units/dugway/liquid/drops/contact_drop_spec.rb

@@ -13,7 +13,7 @@ describe Dugway::Drops::ContactDrop do
     Rack::MockRequest::DEFAULT_ENV.update({
     'PATH_INFO' => '/contact'
   })}
-  
+
   let(:request) { Dugway::Request.new(env) }
 
   let(:errors) {
@@ -96,6 +96,12 @@ describe Dugway::Drops::ContactDrop do
     end
   end
 
+  describe "#recaptcha" do
+    it "returns the recaptcha branding text" do
+      contact.recaptcha.should == %{This site is protected by reCAPTCHA and the Google <a href="https://policies.google.com/privacy">Privacy Policy</a> and <a href="https://policies.google.com/terms">Terms of Service</a> apply.}
+    end
+  end
+
   describe "#sent" do
     it "should return false before the form has been sent" do
       contact.sent.should be(false)

data/spec/units/dugway/liquid/drops/pages_drop_spec.rb

@@ -1,7 +1,18 @@
 require 'spec_helper'
 
 describe Dugway::Drops::PagesDrop do
-  let(:pages) { Dugway::Drops::PagesDrop.new(Dugway.store.pages.map { |p| Dugway::Drops::PageDrop.new(p) }) }
+  let(:pages) do
+    Dugway::Drops::PagesDrop.new(
+      Dugway.store.pages.map do |p|
+        case p["permalink"]
+        when "cart"
+          Dugway::Drops::CartDrop.new(p)
+        else
+          Dugway::Drops::PageDrop.new(p)
+        end
+      end
+    )
+  end
 
   describe "#all" do
     it "should return an array of all pages" do
@@ -15,6 +26,14 @@ describe Dugway::Drops::PagesDrop do
     end
   end
 
+  describe "#cart" do
+    it "returns the cart drop instead of the cart class instance" do
+      cart = pages.cart
+      cart.should be_an_instance_of(Dugway::Drops::CartDrop)
+      cart.name.should == 'Cart'
+    end
+  end
+
   describe "#permalink" do
     it "should return the page by permalink" do
       page = pages.contact

data/spec/units/dugway/theme_spec.rb

@@ -118,7 +118,7 @@ describe Dugway::Theme do
     end
 
     it "should sprocketize and liquify theme.css" do
-      theme.file_content('theme.css').gsub(/\s+/, '').should == %{html,body{height:100%;}a{background:lime;color:red;}footercitea{background:url(/images/bc_badge.png)no-repeat;}/**/}
+      theme.file_content('theme.css').gsub(/\s+/, '').should == %{html,body{height:100%;}a{background:#0f0;color:red;}footercitea{background:url(/images/bc_badge.png)no-repeat;}/**/}
     end
   end
 
@@ -134,7 +134,7 @@ describe Dugway::Theme do
     end
 
     it "should sprocketize and not liquify theme.css" do
-      theme.build_file('theme.css').gsub(/\s+/, '').should == %{html,body{height:100%;}a{background:lime;color:{{theme.link_color}};}footercitea{background:url({{'bc_badge.png'|theme_image_url}})no-repeat;}/**/}
+      theme.build_file('theme.css').gsub(/\s+/, '').should == %{html,body{height:100%;}a{background:#0f0;color:{{theme.link_color}};}footercitea{background:url({{'bc_badge.png'|theme_image_url}})no-repeat;}/**/}
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dugway
 version: !ruby/object:Gem::Version
-  version: 0.12.2
+  version: 1.0.3
 platform: ruby
 authors:
 - Big Cartel
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-03-23 00:00:00.000000000 Z
+date: 2021-01-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -100,14 +100,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.2.5
+        version: 3.4.25
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.2.5
+        version: 3.4.25
 - !ruby/object:Gem::Dependency
   name: sprockets
   requirement: !ruby/object:Gem::Requirement
@@ -142,14 +142,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.12.2
+        version: 1.0.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.12.2
+        version: 1.0.3
 - !ruby/object:Gem::Dependency
   name: httparty
   requirement: !ruby/object:Gem::Requirement
@@ -268,14 +268,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.7.2
+        version: 1.8.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.7.2
+        version: 1.8.0
 - !ruby/object:Gem::Dependency
   name: bigcartel-theme-fonts
   requirement: !ruby/object:Gem::Requirement
@@ -545,7 +545,7 @@ homepage: https://github.com/bigcartel/dugway
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -560,9 +560,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
-signing_key: 
+rubygems_version: 3.0.6
+signing_key:
 specification_version: 4
 summary: Easily build and test Big Cartel themes.
 test_files: