spree_zaez_variants_labels 3.0.1

data/app/assets/stylesheets/spree/backend/spectrum_overrides.css

@@ -0,0 +1,29 @@
+.sp-container {
+    background-color: white;
+    border: solid 1px #e0e0e0;
+}
+
+.sp-replacer {
+    background-color: white;
+    border-color: #e0e0e0;
+    width: 100%;
+    line-height: 1.42857143;
+    padding: 8px 16px;
+}
+
+.sp-replacer:hover, .sp-replacer.sp-active {
+    border-color: #e0e0e0;
+}
+
+.sp-preview {
+    width: 80%;
+}
+
+.sp-dd {
+    width: 10%;
+    float: right;
+}
+
+.sp-palette-button-container{
+    margin-top: 8px;
+}

data/app/assets/stylesheets/spree/backend/spree_zaez_variants_labels.css

@@ -0,0 +1,4 @@
+/*
+*= require ./spectrum
+*= require ./spectrum_overrides
+*/

data/app/assets/stylesheets/spree/frontend/spree_zaez_variants_labels.css

@@ -0,0 +1,4 @@
+/*
+Placeholder manifest file.
+the installer will append this file to the app vendored assets here: 'vendor/assets/stylesheets/spree/frontend/all.css'
+*/

data/app/controllers/spree/admin/labels_controller.rb

@@ -0,0 +1,23 @@
+module Spree
+  module Admin
+    class LabelsController < ResourceController
+      def index
+        session[:return_to] = request.url
+        respond_with @collection
+      end
+
+      def collection
+        return @collection if @collection.present?
+        params[:q] ||= {}
+        @collection = super
+        # @search needs to be defined as this is passed to search_form_for
+        Ransack.options[:ignore_unknown_conditions] = false
+        @search = @collection.ransack(params[:q])
+        @collection = @search.result.
+            page(params[:page]).
+            per(params[:per_page] || Spree::Config[:admin_products_per_page])
+        @collection
+      end
+    end
+  end
+end

data/app/models/spree/label.rb

@@ -0,0 +1,8 @@
+class Spree::Label < Spree::Base
+
+  has_and_belongs_to_many :variants, class_name: 'Spree::Variant'
+
+  validates_presence_of :name, :color
+  validates_uniqueness_of :name
+
+end

data/app/models/spree/variant_decorator.rb

@@ -0,0 +1,5 @@
+Spree::Variant.class_eval do
+
+  has_and_belongs_to_many :labels, class_name: 'Spree::Label'
+
+end

data/app/overrides/add_label_management_to_variants.rb

@@ -0,0 +1,4 @@
+Deface::Override.new virtual_path: 'spree/admin/variants/_form',
+                     name: 'add_label_management_to_variants',
+                     insert_top: '[data-hook="admin_variant_form_fields"]',
+                     partial: 'spree/admin/variants/labels'

data/app/overrides/add_labels_to_menu.rb

@@ -0,0 +1,4 @@
+Deface::Override.new virtual_path: 'spree/admin/shared/sub_menu/_product',
+                     name: 'add_labels_to_menu',
+                     insert_bottom: '[data-hook="admin_product_sub_tabs"]',
+                     text: '<%= tab :labels %>'

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

@@ -0,0 +1,26 @@
+<div class="row" data-hook="new_label_attrs">
+  <div class="col-md-6">
+    <%= f.field_container :name, :class => ['form-group'] do %>
+      <%= f.label :name, Spree.t(:name) %> <span class="required">*</span>
+      <%= f.text_field :name, class: 'form-control title' %>
+      <% f.error_message_on :name %>
+    <% end %>
+  </div>
+  <div class="col-md-6">
+    <%= f.field_container :color, :class => ['form-group'] do %>
+      <%= f.label :color, Spree.t(:color) %>
+      <div class="row">
+        <div class="col-md-3">
+          <input type="text" id="colorpicker" class="form-control" />
+        </div>
+        <div class="col-md-9">
+          <%= f.text_field :color, class: 'form-control color-text' %>
+        </div>
+      </div>
+      <%= f.error_message_on :color %>
+    <% end %>
+  </div>
+</div>
+<script type="text/javascript">
+  new window.LabelController('<%= @label.color %>');
+</script>

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

@@ -0,0 +1,17 @@
+<% content_for :page_title do %>
+  <%= link_to Spree.t(:labels), admin_labels_path %> /
+  <%= @label.name %>
+<% end %>
+
+<% content_for :page_actions do %>
+  <%= button_link_to Spree.t(:new_label), new_object_url, {class: 'btn-success', icon: 'add', id: 'admin_new_label'} %>
+<% end %>
+
+<%= render partial: 'spree/admin/shared/error_messages', locals: {target: @label } %>
+
+<%= form_for [:admin, @label], method: :put do |f| %>
+  <fieldset>
+    <%= render partial: 'form', locals: {f: f } %>
+    <%= render partial: 'spree/admin/shared/edit_resource_links' %>
+  </fieldset>
+<% end %>

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

@@ -0,0 +1,59 @@
+<% content_for :page_title do %>
+  <%= plural_resource_name(Spree::Label) %>
+<% end %>
+
+<% content_for :page_actions do %>
+    <%= button_link_to Spree.t(:new_label), new_object_url, {class: 'btn-success', icon: 'add', id: 'admin_new_label'} %>
+<% end %>
+
+<% content_for :table_filter do %>
+  <div class="" data-hook="admin_labels_sidebar">
+    <%= search_form_for [:admin, @search] do |f| %>
+      <div class="row" data-hook="admin_labels_index_search">
+        <div class="col-md-6">
+          <div class="form-group">
+            <%= f.label :name_cont, Spree.t(:name) %>
+            <%= f.text_field :name_cont, size: 15, class: 'form-control js-quick-search-target' %>
+          </div>
+        </div>
+      </div>
+      <div class="form-actions" data-hook="admin_labels_index_search_buttons">
+        <%= button Spree.t(:search), 'search' %>
+      </div>
+    <% end %>
+  </div>
+<% end %>
+
+
+<%= render partial: 'spree/admin/shared/index_table_options', locals: {collection: @collection} %>
+
+<% if @collection.any? %>
+  <table class="table" id="listing_labels">
+    <thead>
+    <tr data-hook="admin_labels_headers">
+      <th><%= Spree.t(:name) %></th>
+      <th><%= Spree.t(:color) %></th>
+      <th data-hook="admin_labels_index_header_actions" class="actions"></th>
+    </tr>
+    </thead>
+    <tbody>
+    <% @collection.each do |label| %>
+      <tr data-hook="admin_labels_index_rows" class="<%= cycle('odd', 'even') %>">
+        <td><%= label.name %></td>
+        <td><%= label.color %></td>
+        <td class="actions actions-2 text-right" data-hook="admin_labels_index_row_actions">
+          <%= link_to_edit label, no_text: true, class: 'edit' %>
+          <%= link_to_delete label, no_text: true %>
+        </td>
+      </tr>
+    <% end %>
+    </tbody>
+  </table>
+<% else %>
+  <div class="alert alert-info no-objects-found">
+    <%= Spree.t(:no_resource_found, resource: plural_resource_name(Spree::Label)) %>,
+    <%= link_to Spree.t(:add_one), new_object_url %>!
+  </div>
+<% end %>
+
+<%= render partial: 'spree/admin/shared/index_table_options', locals: {collection: @collection} %>

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

@@ -0,0 +1,14 @@
+<%= render partial: 'spree/admin/shared/error_messages', locals: {target: @label} %>
+
+<% content_for :page_title do %>
+  <%= Spree.t(:new_label) %>
+<% end %>
+
+<%= form_for [:admin, @label] do  |f| %>
+  <fieldset data-hook="new_label">
+
+    <%= render partial: 'form', locals: {f: f} %>
+    <%= render partial: 'spree/admin/shared/new_resource_links' %>
+
+  </fieldset>
+<% end %>

data/app/views/spree/admin/variants/_labels.html.erb

@@ -0,0 +1,12 @@
+<div data-hook="labels">
+  <label><%= Spree.t(:label) %></label>
+  <ul>
+    <% Spree::Label.all.each do |label| %>
+      <li>
+        <label for="label-<%= label.name %>">
+          <%= check_box_tag 'variant[label_ids][]', label.id, @variant.labels.include?(label), {id: "label-#{label.name}"} %> <%= label.name %>
+        </label>
+      </li>
+    <% end %>
+  </ul>
+</div>

data/bin/rails

@@ -0,0 +1,7 @@
+# This command will automatically be run when you run "rails" with Rails 3 gems installed from the root of your application.
+
+ENGINE_ROOT = File.expand_path('../..', __FILE__)
+ENGINE_PATH = File.expand_path('../../lib/spree_zaez_variants_labels/engine', __FILE__)
+
+require 'rails/all'
+require 'rails/engine/commands'

data/config/locales/en.yml

@@ -0,0 +1,5 @@
+# Sample localization file for English. Add more files in this directory for other locales.
+# See https://github.com/svenfuchs/rails-i18n/tree/master/rails%2Flocale for starting points.
+
+en:
+  hello: "Hello world"

data/config/routes.rb

@@ -0,0 +1,10 @@
+Spree::Core::Engine.routes.draw do
+
+  namespace :admin do
+
+    resources :labels
+
+  end
+
+
+end

data/db/migrate/20150706192427_create_spree_labels.rb

@@ -0,0 +1,8 @@
+class CreateSpreeLabels < ActiveRecord::Migration
+  def change
+    create_table :spree_labels do |t|
+      t.string :name
+      t.string :color
+    end
+  end
+end

data/db/migrate/20150707130840_create_spree_labels_variants.rb

@@ -0,0 +1,8 @@
+class CreateSpreeLabelsVariants < ActiveRecord::Migration
+  def change
+    create_table :spree_labels_variants, :id => false do |t|
+      t.references :label
+      t.references :variant
+    end
+  end
+end

data/lib/generators/spree_zaez_variants_labels/install/install_generator.rb

@@ -0,0 +1,31 @@
+module SpreeZaezVariantsLabels
+  module Generators
+    class InstallGenerator < Rails::Generators::Base
+
+      class_option :auto_run_migrations, :type => :boolean, :default => false
+
+      def add_javascripts
+        append_file 'vendor/assets/javascripts/spree/frontend/all.js', "//= require spree/frontend/spree_zaez_variants_labels\n"
+        append_file 'vendor/assets/javascripts/spree/backend/all.js', "//= require spree/backend/spree_zaez_variants_labels\n"
+      end
+
+      def add_stylesheets
+        inject_into_file 'vendor/assets/stylesheets/spree/frontend/all.css', " *= require spree/frontend/spree_zaez_variants_labels\n", :before => /\*\//, :verbose => true
+        inject_into_file 'vendor/assets/stylesheets/spree/backend/all.css', " *= require spree/backend/spree_zaez_variants_labels\n", :before => /\*\//, :verbose => true
+      end
+
+      def add_migrations
+        run 'bundle exec rake railties:install:migrations FROM=spree_zaez_variants_labels'
+      end
+
+      def run_migrations
+        run_migrations = options[:auto_run_migrations] || ['', 'y', 'Y'].include?(ask 'Would you like to run the migrations now? [Y/n]')
+        if run_migrations
+          run 'bundle exec rake db:migrate'
+        else
+          puts 'Skipping rake db:migrate, don\'t forget to run it!'
+        end
+      end
+    end
+  end
+end

data/lib/spree_zaez_variants_labels.rb

@@ -0,0 +1,2 @@
+require 'spree_core'
+require 'spree_zaez_variants_labels/engine'

data/lib/spree_zaez_variants_labels/engine.rb

@@ -0,0 +1,20 @@
+module SpreeZaezVariantsLabels
+  class Engine < Rails::Engine
+    require 'spree/core'
+    isolate_namespace Spree
+    engine_name 'spree_zaez_variants_labels'
+
+    # use rspec for tests
+    config.generators do |g|
+      g.test_framework :rspec
+    end
+
+    def self.activate
+      Dir.glob(File.join(File.dirname(__FILE__), '../../app/**/*_decorator*.rb')) do |c|
+        Rails.configuration.cache_classes ? require(c) : load(c)
+      end
+    end
+
+    config.to_prepare &method(:activate).to_proc
+  end
+end

data/lib/spree_zaez_variants_labels/factories.rb

@@ -0,0 +1,13 @@
+FactoryGirl.define do
+
+
+  factory :label, class: Spree::Label do
+
+    colors = ['#5bb85b', '#777777', '#337ab7', '#d9534f']
+
+    sequence(:name) {|n| "Label-#{n}"}
+    color colors[Random.rand(0..3)]
+
+  end
+
+end

data/spec/features/spree/admin/labels_spec.rb

@@ -0,0 +1,174 @@
+require 'spec_helper'
+
+RSpec.describe 'Label CRUD', {type: :feature, js: true} do
+
+  let!(:user) {create(:admin_user)}
+  let!(:label) {create(:label)}
+
+  before do
+    sign_in_admin! user
+  end
+
+  describe 'index' do
+
+    before { 10.times{create(:label)} }
+
+    before :each do
+      visit spree.admin_labels_path
+    end
+
+    context 'listing' do
+
+      it 'must list all labels' do
+        expect(page).to have_selector('tr[data-hook="admin_labels_index_rows"]', count: 10)
+      end
+
+    end
+
+    context 'filtering' do
+
+      before do
+        create(:label, name: 'Filtering Test')
+      end
+
+      it 'must filter using a quick search' do
+        within find('#quick-search') do
+          fill_in 'quick_search', with: 'Filtering Test'
+          find('.js-quick-search').native.send_keys(:Enter)
+        end
+        expect(page).to have_selector('tr[data-hook="admin_labels_index_rows"]', count: 1)
+      end
+
+      it 'must be filtered by name' do
+        find('.js-show-index-filters').click
+        expect(page).to have_selector('#table-filter')
+        fill_in 'q[name_cont]', with: 'Filtering Test'
+        find('button[type="submit"]').click
+        expect(page).to have_selector('tr[data-hook="admin_labels_index_rows"]', count: 1)
+      end
+
+    end
+
+  end
+
+  describe 'create' do
+
+    before :each do
+      visit spree.new_admin_label_path
+    end
+
+    context 'without errors' do
+
+      it 'must create the label and redirect to the listing with a success message' do
+        within find('#new_label') do
+          fill_in 'label[name]', with: 'Label test'
+          fill_in 'label[color]', with: '#CC0000'
+          find('button[type="submit"]').click
+        end
+        expect(page).to have_selector('tr[data-hook="admin_labels_index_rows"]')
+        expect(page).to have_content 'successfully created!'
+      end
+
+    end
+
+    context 'with errors' do
+
+      before {create(:label, name: 'Label Test')}
+      before :each do
+        visit spree.new_admin_label_path
+      end
+
+      it 'must not be created if name is already taken' do
+        within find('#new_label') do
+          fill_in 'label[name]', with: 'Label Test'
+          fill_in 'label[color]', with: '#CC0000'
+          find('button[type="submit"]').click
+        end
+        expect(page).to have_content 'Name has already been taken'
+      end
+
+      it 'must not be created with invalid params' do
+        within find('#new_label') do
+          fill_in 'label[name]', with: 'Label Test2'
+          find('button[type="submit"]').click
+        end
+        expect(page).to have_content "Color can't be blank"
+      end
+
+    end
+
+  end
+
+  describe 'edit' do
+
+    let!(:label){create(:label, name: 'Label editing test')}
+    let!(:label_2){create(:label, name: 'Test')}
+
+    before :each do
+      visit spree.edit_admin_label_path(label.id)
+    end
+
+    context 'without errors' do
+
+      it 'must update the label attributes and redirect to listing with a success message' do
+        within find('.edit_label') do
+          fill_in 'label[name]', with: 'Name updated'
+          find('button[type="submit"]').click
+        end
+        expect(page).to have_selector('tr[data-hook="admin_labels_index_rows"]')
+        expect(page).to have_content 'successfully updated!'
+      end
+
+    end
+
+    context 'with errors' do
+
+      it 'must not be updated if the name is already taken' do
+        within find('.edit_label') do
+          fill_in 'label[name]', with: 'Test'
+          find('button[type="submit"]').click
+        end
+        expect(label.name).to be == 'Label editing test'
+        expect(page).to have_content 'Name has already been taken'
+      end
+
+      it 'must not be updated without valid parameters' do
+        within find('.edit_label') do
+          fill_in 'label[color]', with: ''
+          find('button[type="submit"]').click
+        end
+        expect(label.color).not_to be == ''
+        expect(page).to have_content "Color can't be blank"
+      end
+
+    end
+
+  end
+
+  describe 'destroy' do
+
+    let!(:label) {create(:label)}
+    let!(:variant) do
+      variant = create(:variant)
+      variant.labels << label
+      2.times do
+        l = create(:label)
+        variant.labels << l
+      end
+      variant.save
+      variant
+    end
+
+    it 'must remove the deleted label from any variants who has it' do
+      visit spree.admin_labels_path
+      within first('tr[data-hook="admin_labels_index_rows"]') do
+        find('a[data-action="remove"]').click
+      end
+      expect(page).to have_content 'successfully removed!'
+      expect(variant.labels.count).to be == 2
+    end
+
+  end
+
+
+end