solidus_static_content 1.0.0 → 2.1.0

data/app/views/spree/admin/pages/_form.html.erb

@@ -1,98 +0,0 @@
-<div data-hook="admin_page_form_fields">
-  <div class="left eight columns alpha" data-hook="admin_page_form_left">
-
-    <%= f.field_container :title do %>
-      <%= f.label :title %> <span class="required">*</span><br />
-      <%= f.text_field :title, :class => 'fullwidth title' %>
-      <%= f.error_message_on :title %>
-    <% end %>
-
-    <%= f.field_container :slug do %>
-      <%= f.label :slug %> <span class="required">*</span><br />
-      <%= f.text_field :slug, :class => 'fullwidth title' %>
-      <%= f.error_message_on :slug %>
-    <% end %>
-
-    <%= f.field_container :body do %>
-      <%= f.label :body %> <span class="required">*</span><br />
-      <%= f.text_area :body, {rows: 12, :class => 'fullwidth'} %>
-      <%= f.error_message_on :body %>
-    <% end %>
-
-  </div>
-  <div class="right four columns omega" data-hook="admin_page_form_right">
-
-    <%= f.field_container :layout do %>
-      <%= f.label :layout %><br />
-      <%= f.text_field :layout, :class => 'fullwidth title' %>
-    <% end %>
-
-    <%= f.field_container :foreign_link do %>
-      <%= f.label :foreign_link %><br />
-      <%= f.text_field :foreign_link, :class => 'fullwidth title' %>
-    <% end %>
-
-    <%= f.field_container :position do %>
-      <%= f.label :position %><br />
-      <%= f.number_field :position, :min => 0, :class => 'fullwidth title' %>
-    <% end %>
-
-    <%= f.field_container :show_in_sidebar do %>
-      <%= f.check_box :show_in_sidebar %>
-      <%= f.label :show_in_sidebar %>
-    <% end %>
-
-    <%= f.field_container :show_in_heade do %>
-      <%= f.check_box :show_in_header %>
-      <%= f.label :show_in_header %>
-    <% end %>
-
-    <%= f.field_container :show_in_footer do %>
-      <%= f.check_box :show_in_footer %>
-      <%= f.label :show_in_footer %>
-    <% end %>
-
-    <%= f.field_container :visible do %>
-      <%= f.check_box :visible %>
-      <%= f.label :visible %>
-    <% end %>
-
-    <%= f.field_container :render_layout_as_partial do %>
-      <%= f.check_box :render_layout_as_partial %>
-      <%= f.label :render_layout_as_partial %>
-    <% end %>
-
-    <%= f.field_container :stores do %>
-      <%= f.label :stores, Spree.t(:stores)%><br />
-      <% Spree::Store.all.each do |store| %>
-        <%= check_box_tag "page[store_ids][]", store.id, @page.stores.include?(store) %> <%= store.name %>
-      <% end %>
-    <% end %>
-    
-  </div>
-
-  <div data-hook="admin_page_form_meta" class="alpha omega twelve columns">
-
-    <%= f.field_container :meta_title do %>
-      <%= f.label :meta_title %><br />
-      <%= f.text_field :meta_title, :class => 'fullwidth title' %>
-      <%= f.error_message_on :meta_title %>
-    <% end %>
-
-    <%= f.field_container :meta_keywords do %>
-      <%= f.label :meta_keywords %><br />
-      <%= f.text_field :meta_keywords, :class => 'fullwidth title' %>
-      <%= f.error_message_on :meta_keywords %>
-    <% end %>
-
-    <%= f.field_container :meta_description do %>
-      <%= f.label :meta_description %><br />
-      <%= f.text_field :meta_description, :class => 'fullwidth title' %>
-      <%= f.error_message_on :meta_description %>
-    <% end %>
-
-  </div>
-  <div class="clear"></div>
-  <div data-hook="admin_page_form_additional_fields"></div>
-  <div class="clear"></div>
-</div>

data/app/views/spree/admin/pages/edit.html.erb

@@ -1,18 +0,0 @@
-<%= render 'spree/admin/shared/configuration_menu' %>
-
-<% content_for :page_title do %>
-   <%= Spree.t("static_content.static_pages") %> <i class="icon-arrow-right"></i> <%= Spree.t("static_content.editing_page") %> "<%= @page.title%>"
-<% end %>
-
-<% content_for :page_actions do %>
-  <li><%= button_link_to Spree.t(:back_to_static_pages_list), spree.admin_pages_url, :icon => 'icon-arrow-left' %></li>
-<% end %>
-
-<%= render "spree/shared/error_messages", :target => @page %>
-
-<%= form_for [:admin, @page] do |f| %>
-  <fieldset class="no-border-top">
-    <%= render :partial => 'form', :locals => { :f => f } %>
-    <%= render :partial => 'spree/admin/shared/edit_resource_links' %>
-  </fieldset>
-<% end %>

data/app/views/spree/admin/pages/index.html.erb

@@ -1,51 +0,0 @@
-<% content_for :page_actions do %>
-  <li><%= button_link_to Spree.t("static_content.new_page"), new_object_url, :icon => 'icon-plus', :id => 'admin_new_page' %></li>
-<% end %>
-
-<% content_for :page_title do %>
-   <%= Spree.t("static_content.static_pages") %>
-<% end %>
-
-<% if @pages.any? %>
-<table class="index responsive" id="listing_pages" data-hook>
-  <colgroup>
-    <col style="width: 40%">
-    <col style="width: 35%">
-    <col style="width: 10%">
-    <col style="width: 15%">
-  </colgroup>
-  <thead>
-    <tr data-hook="admin_pages_index_headers">
-      <th><%= Spree::Page.human_attribute_name(:title) %></th>
-      <th><%= Spree.t("static_content.link") %></th>
-      <th><%= Spree::Page.human_attribute_name(:visible) %></th>
-      <th data-hook="admin_pages_index_header_actions" class="actions"></th>
-    </tr>
-  </thead>
-
-  <tbody>
-    <% @pages.each do |page| %>
-      <tr data-hook="admin_pages_index_rows" class="<%= cycle('odd', 'even') %>" id="<%= dom_id page %>">
-        <td>
-          <%= page.title %>
-        </td>
-        <td>
-          <%= link_to page.link, page.link, :target => '_blank' %>
-        </td>
-        <td class="align-center">
-          <%= content_tag(:i, '', :class => 'icon-ok green') if page.visible %>
-        </td>
-        <td class="actions" data-hook="admin_pages_index_row_actions">
-          <%= link_to_edit page, :no_text => true %>
-          <%= link_to_delete page, :no_text => true %>
-        </td>
-      </tr>
-    <% end %>
-  </tbody>
-</table>
-<% else %>
-  <div class="alpha twelve columns no-objects-found">
-    <%= Spree.t(:no_resource_found, resource: I18n.t(:other, scope: 'activerecord.models.spree/page')) %>,
-    <%= link_to Spree.t(:add_one), spree.new_admin_page_path %>!
-  </div>
-<% end %>

data/app/views/spree/admin/pages/new.html.erb

@@ -1,18 +0,0 @@
-<%= render 'spree/admin/shared/configuration_menu' %>
-
-<% content_for :page_title do %>
-   <%= Spree.t("static_content.static_pages") %> <i class="icon-arrow-right"></i> <%= Spree.t("static_content.new_page") %>
-<% end %>
-
-<% content_for :page_actions do %>
-  <li><%= button_link_to Spree.t(:back_to_static_pages_list), spree.admin_pages_url, :icon => 'icon-arrow-left' %></li>
-<% end %>
-
-<%= render "spree/shared/error_messages", :target => @page %>
-
-<%= form_for [:admin, @page] do |f| %>
-  <fieldset class="no-border-top">
-    <%= render :partial => 'form', :locals => { :f => f } %>
-    <%= render :partial => 'spree/admin/shared/new_resource_links' %>
-  </fieldset>
-<% end %>

data/app/views/spree/static_content/show.html.erb

@@ -1,26 +0,0 @@
-<% if @page.layout.present? and @page.render_layout_as_partial? %>
-  <%= render :partial => @page.layout %>
-<% else %>
-  <% content_for :head do -%>
-    <%- if @page.meta_title.present? -%>
-      <meta name="title" content="<%=@page.meta_title%>">
-    <%- else -%>
-      <meta name="title" content="<%=@page.title%>">
-    <%- end -%>
-    <meta name="keywords" content="<%=@page.meta_keywords%>">
-    <meta name="description" content="<%=@page.meta_description%>">
-  <% end -%>
-
-  <% content_for :sidebar do %>
-    <% if defined? @products && defined? @taxon %>
-      <%= render :partial => "spree/shared/filters" %>
-    <% elsif defined? @taxonomies %>
-      <%= render :partial => "spree/shared/taxonomies" %>
-    <% end %>
-  <% end %>
-
-  <h1><%= @page.title %></h1>
-  <div id="page_content">
-    <%= raw @page.body %>
-  </div>
-<% end %>

data/lib/spree_static_content/engine.rb

@@ -1,17 +0,0 @@
-module SpreeStaticContent
-  class Engine < Rails::Engine
-    require 'spree/core'
-    isolate_namespace Spree
-    engine_name 'spree_static_content'
-
-    config.autoload_paths += %W(#{config.root}/lib)
-
-    def self.activate
-      Dir.glob(File.join(File.dirname(__FILE__), "../../app/overrides/*.rb")) do |c|
-        Rails.configuration.cache_classes ? require(c) : load(c)
-      end
-    end
-
-    config.to_prepare &method(:activate).to_proc
-  end
-end

data/spec/controllers/static_content_controller_spec.rb

@@ -1,37 +0,0 @@
-require 'spec_helper'
-
-describe Spree::StaticContentController, type: :controller do
-  before do
-    controller.stub spree_current_user: nil
-  end
-
-  let!(:store) { create(:store, default: true) }
-
-  context '#show' do
-    it 'accepts path as root' do
-      page = create(:page, slug: '/', stores: [store])
-      request.path = page.slug
-      spree_get :show, path: page.slug
-      expect(response).to be_success
-    end
-
-    it 'accepts path as string' do
-      page = create(:page, slug: 'hello', stores: [store])
-      request.path = page.slug
-      spree_get :show, path: page.slug
-      expect(response).to be_success
-    end
-
-    it 'accepts path as nested' do
-      page = create(:page, slug: 'aa/bb/cc', stores: [store])
-      request.path = page.slug
-      spree_get :show, path: page.slug
-      expect(response).to be_success
-    end
-
-    it 'respond with a 404 when no page exists' do
-      spree_get :show
-      expect(response.response_code).to eq(404)
-    end
-  end
-end

data/spec/factories/page_factory.rb

@@ -1,18 +0,0 @@
-FactoryGirl.define do
-  factory :page, class: Spree::Page do
-    title { generate(:random_string) }
-    body  { generate(:random_description) }
-
-    sequence(:slug) { |n| "/page#{n}" }
-
-    trait :with_foreign_link do
-      foreign_link do
-        if defined?(FFaker)
-          FFaker::Internet.http_url
-        else
-          Faker::Internet.http_url
-        end
-      end
-    end
-  end
-end

data/spec/features/admin/pages_spec.rb

@@ -1,67 +0,0 @@
-require 'spec_helper'
-
-RSpec.feature 'Admin Static Content', js: true do
-  stub_authorization!
-
-  context 'when no page exists' do
-    background do
-      visit spree.admin_path
-      click_link 'Pages'
-    end
-
-    scenario 'can create new page' do
-      expect(page).to have_text /No Pages found, Add One!/i
-
-      click_link 'New page'
-      expect(page).to have_text 'Static pages'
-
-      fill_in 'page_title', with: 'Contact us'
-      fill_in 'page_slug', with: 'contact-us'
-      fill_in 'page_body', with: 'hello..'
-
-      fill_in 'page_meta_title', with: 'Contact us'
-      fill_in 'page_meta_keywords', with: 'spree, contact'
-      fill_in 'page_meta_description', with: 'Short description for contact page.'
-
-      check 'page_show_in_sidebar'
-      check 'page_show_in_header'
-      check 'page_show_in_footer'
-
-      expect(find('#page_visible')).to be_checked
-
-      click_button 'Create'
-      expect(page).to have_text 'successfully created!'
-    end
-  end
-
-  context 'when page exists' do
-    given!(:static_page) { create(:page) }
-
-    background do
-      visit spree.admin_path
-      click_link 'Pages'
-    end
-
-    scenario 'can be updated' do
-      within_row(1) do
-        click_icon :edit
-      end
-
-      fill_in 'page_title', with: 'Contact'
-      fill_in 'page_slug', with: 'contact'
-
-      click_button 'Update'
-      expect(page).to have_text 'successfully updated!'
-    end
-
-    scenario 'can be deleted' do
-      within_row(1) do
-        click_icon :trash
-      end
-
-      page.driver.browser.switch_to.alert.accept unless Capybara.javascript_driver == :poltergeist
-
-      expect(page).not_to have_text static_page.title
-    end
-  end
-end

data/spec/features/static_content_spec.rb

@@ -1,53 +0,0 @@
-require 'spec_helper'
-
-feature 'Static Content Page', js: true do
-
-  let!(:store) { create(:store, default: true) }
-
-  context 'render page' do
-    scenario 'is a query string' do
-      create(:page, slug: '/page', title: 'Query Test', stores: [store])
-      visit '/page?test'
-      expect(page).to have_text 'Query Test'
-    end
-
-    scenario 'can have slug not starting by /' do
-      create(:page, slug: 'page2', title: 'No Slash Prefix Test', stores: [store])
-      visit '/page2'
-      expect(page).to have_text 'No Slash Prefix Test'
-    end
-
-    scenario 'can have slug with multiple /' do
-      create(:page, slug: '/hello/shoppers/page3', title: 'Multiple Slash Test', stores: [store])
-      visit '/hello/shoppers/page3'
-      expect(page).to have_text 'Multiple Slash Test'
-    end
-
-    scenario 'can be a custom root page' do
-      create(:page, slug: '/', title: 'Root Page Test', stores: [store])
-      visit '/'
-      expect(page).to have_text 'Root Page Test'
-    end
-
-    scenario 'is limited within its own constraints' do
-      create(:page, slug: '/t/categories/page3', title: 'Constraint Test', stores: [store])
-      visit '/t/categories/page3'
-      expect(page).not_to have_text 'Constraint Test'
-    end
-
-    scenario 'fetch correct page' do
-      create(:page, slug: '/', stores: [store])
-      create(:page, slug: 'hello', title: 'Hello', stores: [store])
-      create(:page, slug: 'somwhere', stores: [store])
-      create(:page, :with_foreign_link, slug: 'whatever', stores: [store])
-      visit '/hello'
-      expect(page).to have_text 'Hello'
-    end
-
-    scenario 'do not effect the rendering of the rest of the site' do
-      product = create(:product)
-      visit spree.product_path(product)
-      expect(page).to have_text product.name
-    end
-  end
-end

data/spec/helpers/page_helper_spec.rb

@@ -1,14 +0,0 @@
-require 'spec_helper'
-
-RSpec.describe Spree::PagesHelper, type: :helper do
-  context '.render_snippet' do
-    it 'return body when page exists' do
-      page = create(:page)
-      expect(render_snippet(page.slug)).to eq page.body
-    end
-
-    it 'return nil when no page exist' do
-      expect(render_snippet('hello')).to be_nil
-    end
-  end
-end

data/spec/lib/spree_static.content_spec.rb

@@ -1,34 +0,0 @@
-require 'spec_helper'
-
-describe StaticPage do
-  context '.remove_spree_mount_point' do
-    specify do
-      path = '/hello'
-      expect(subject.remove_spree_mount_point(path)).to eq 'hello'
-    end
-  end
-end
-
-describe Spree::StaticPage do
-  subject { described_class }
-
-  context '.matches?' do
-    it 'is true when valid page' do
-      page = create(:page, slug: 'hello', visible: true)
-      request = double('request', path: page.slug)
-      expect(subject.matches?(request)).to be true
-    end
-
-    it 'is false when using reserved slug name' do
-      page = create(:page, slug: 'login', visible: true)
-      request = double('request', path: page.slug)
-      expect(subject.matches?(request)).to be false
-    end
-
-    it 'is false when page is not accessible' do
-      page = create(:page, slug: 'hello', visible: false)
-      request = double('request', path: page.slug)
-      expect(subject.matches?(request)).to be false
-    end
-  end
-end

data/spec/models/page_spec.rb

@@ -1,46 +0,0 @@
-require 'spec_helper'
-
-describe Spree::Page do
-  let!(:page) { create(:page) }
-
-  context 'factory' do
-    it 'is valid' do
-      expect(page).to be_valid
-    end
-  end
-
-  it 'always add / prefix to slug' do
-    page = create(:page, slug: 'hello')
-    expect(page.slug).to eq '/hello'
-  end
-
-  context '.link' do
-    it 'return slug if foreign_link blank' do
-      page = create(:page, slug: 'hello')
-      expect(page.link).to eq page.slug
-    end
-
-    it 'return foreign_link if set' do
-      page = create(:page, :with_foreign_link, slug: 'hello')
-      expect(page.link).to eq page.foreign_link
-    end
-  end
-
-
-  context "pages in stores" do
-
-    before(:each) do
-      @store = create(:store)
-      @page = create(:page, :stores => [@store])
-      @page2 = create(:page)
-    end
-
-    it 'should correctly find pages by store' do
-      pages_by_store = Spree::Page.by_store(@store)
-      expect(pages_by_store).to include(@page)
-      expect(pages_by_store).to_not include(@page2)
-    end
-
-  end
-
-end

data/spec/spec_helper.rb

@@ -1,50 +0,0 @@
-require 'simplecov'
-SimpleCov.start 'rails'
-
-ENV["RAILS_ENV"] = "test"
-
-require File.expand_path("../dummy/config/environment.rb",  __FILE__)
-
-require 'ffaker'
-require 'rspec/rails'
-require 'database_cleaner'
-require 'capybara/rspec'
-require 'capybara/rails'
-require 'capybara/poltergeist'
-
-Dir[Rails.root.join("spec/support/**/*.rb")].each {|f| require f}
-
-require 'spree/testing_support/factories'
-require 'spree/testing_support/url_helpers'
-require 'spree/testing_support/controller_requests'
-require 'spree/testing_support/authorization_helpers'
-require 'spree/testing_support/capybara_ext'
-
-FactoryGirl.find_definitions
-
-RSpec.configure do |config|
-  config.mock_with :rspec
-  config.use_transactional_fixtures = false
-
-  config.include Spree::TestingSupport::ControllerRequests, type: :controller
-  config.include Spree::TestingSupport::UrlHelpers
-  config.include FactoryGirl::Syntax::Methods
-
-  config.extend Spree::TestingSupport::AuthorizationHelpers::Request, type: :feature
-
-  config.before(:suite) do
-    DatabaseCleaner.strategy = :transaction
-    DatabaseCleaner.clean_with(:truncation)
-  end
-
-  config.before(:each) do
-    DatabaseCleaner.strategy = RSpec.current_example.metadata[:js] ? :truncation : :transaction
-    DatabaseCleaner.start
-  end
-
-  config.after(:each) do
-    DatabaseCleaner.clean
-  end
-
-  Capybara.javascript_driver = :poltergeist
-end