jiffyshirts_spree_reviews 2.3.1.1

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: cd4ee18548affb28ed0d9ee64e900d47ba704280
+  data.tar.gz: 5c248b18db06d3614be6afc6729e7c74422517bf
+SHA512:
+  metadata.gz: eb207aba2f483a65703670d677d54c35fba18a9acf3099627e392db1c2a165f14c42046cd30d9bd8794b14470ceb79bc934d2c0e0350299193a2b7b4f91be4a7
+  data.tar.gz: bd808587a7477ad999b9645c94bc8f75f99a50830d7d95797c3c0a63eb3c944926547ad2e7b04b53a2d79eb73e89d7741271bc19d53a8d2ab75568ee899f2667

data/.gitignore

@@ -0,0 +1,10 @@
+*.gem
+*.swp
+.bundle
+.idea
+.DS_Store
+Gemfile.lock
+pkg/*
+spec/dummy
+coverage
+.rvmrc

data/.rspec

@@ -0,0 +1 @@
+--color

data/.travis.yml

@@ -0,0 +1,11 @@
+before_script:
+  - "sh -e /etc/init.d/xvfb start"
+  - "bundle exec rake test_app"
+script:
+  - "export DISPLAY=:99.0"
+  - "bundle exec rspec spec"
+rvm:
+  - 1.9.3
+  - 2.0.0
+  - 2.1.0
+language: ruby

data/Gemfile

@@ -0,0 +1,6 @@
+source "https://rubygems.org"
+
+gem 'spree', github: 'spree/spree', branch: '2-3-stable'
+gem 'spree_auth_devise', github: 'spree/spree_auth_devise', branch: '2-3-stable'
+
+gemspec

data/Guardfile

@@ -0,0 +1,9 @@
+guard "rspec", cmd: "bundle exec rspec" do
+  watch("spec/spec_helper.rb")                        { "spec" }
+  watch("config/routes.rb")                           { "spec/controllers" }
+  watch("app/controllers/application_controller.rb")  { "spec/controllers" }
+  watch(%r{^spec/(.+)_spec\.rb$})                     { |m| "spec/#{m[1]}_spec.rb"}
+  watch(%r{^app/(.+)_decorator\.rb$})                 { |m| "spec/#{m[1]}_spec.rb" }
+  watch(%r{^(app|lib)/(.+)(\.rb|\.erb)$})             { |m| "spec/#{m[2]}_spec.rb" }
+  watch(%r{^app/controllers/(.+)_(controller)\.rb$})  { |m| "spec/#{m[2]}s/#{m[1]}_#{m[2]}_spec.rb" }
+end

data/LICENSE.md

@@ -0,0 +1,26 @@
+Copyright (c) 2014 Spree Commerce and contributors.
+All rights reserved.
+
+Redistribution and use in source and binary forms, with or without modification,
+are permitted provided that the following conditions are met:
+
+    * Redistributions of source code must retain the above copyright notice,
+      this list of conditions and the following disclaimer.
+    * Redistributions in binary form must reproduce the above copyright notice,
+      this list of conditions and the following disclaimer in the documentation
+      and/or other materials provided with the distribution.
+    * Neither the name Spree nor the names of its contributors may be used to
+      endorse or promote products derived from this software without specific
+      prior written permission.
+
+THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+"AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR
+CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
+EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
+PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
+PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
+LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
+NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
+SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.

data/README.md

@@ -0,0 +1,54 @@
+Reviews
+=======
+
+
+Straightforward review/rating functionality.
+
+Installation
+------------
+
+Add the following to your `Gemfile` to install from git:
+```ruby
+gem 'jiffyshirts_spree_review'
+```
+Now bundle up with:
+
+    bundle
+
+Next, run the rake task that copies the necessary migrations and assets to your project:
+
+    rails g jiffyshirts_spree_reviews:install
+
+And finish with a migrate:
+
+    rake db:migrate
+
+Now you should be able to boot up your server with:
+
+    rails s
+
+That's all!
+
+Usage
+-----
+
+Action "submit" in "reviews" controller - goes to review entry form
+
+Users must be logged in to submit a review
+
+Three partials:
+ - `app/views/spree/products/_rating.html.erb` -- display number of stars
+ - `app/views/spree/products/_shortrating.html.erb` -- shorter version of above
+ - `app/views/spree/products/_review.html.erb` -- display a single review
+
+Administrator can edit and/or approve and/or delete reviews.
+
+Implementation
+--------------
+
+Reviews table is quite obvious - and note the "approved" flag which is for the
+administrator to update.
+
+Ratings table holds current fractional value - avoids frequent recalc...
+
+

data/Rakefile

@@ -0,0 +1,15 @@
+require 'bundler'
+Bundler::GemHelper.install_tasks
+
+require 'rspec/core/rake_task'
+require 'spree/testing_support/common_rake'
+
+RSpec::Core::RakeTask.new
+
+task :default => [:spec]
+
+desc 'Generates a dummy app for testing'
+task :test_app do
+  ENV['LIB_NAME'] = 'jiffyshirts_spree_reviews'
+  Rake::Task['common:test_app'].invoke 'Spree::User'
+end

data/app/assets/javascripts/spree/backend/jiffyshirts_spree_reviews.js

@@ -0,0 +1,7 @@
+//= require jquery.rating
+//= require spree/backend
+
+// Navigating to a page with ratings via TurboLinks shows the radio buttons
+$(document).on('page:load', function () {
+  $('input[type=radio].star').rating();
+});

data/app/assets/javascripts/spree/frontend/jiffyshirts_spree_reviews.js

@@ -0,0 +1,7 @@
+//= require jquery.rating
+//= require spree/frontend
+
+// Navigating to a page with ratings via TurboLinks shows the radio buttons
+$(document).on('page:load', function () {
+  $('input[type=radio].star').rating();
+});

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

@@ -0,0 +1,3 @@
+/*
+ *= require spree/backend
+*/

data/app/assets/stylesheets/spree/frontend/jiffyshirts_spree_reviews.css.erb

@@ -0,0 +1,83 @@
+/*
+ *= require spree/frontend
+*/
+
+#reviews div.header {
+  margin-top: 10px;
+  margin-bottom: 10px;
+}
+
+.lit {
+  background-color: #aaaaaa;
+  color: #0000bb;
+  margin-right: 1px;
+  padding-left: 1px;
+}
+
+.unlit {
+  background-color: #aaaaaa;
+  color: #000000;
+  margin-right: 1px;
+  padding-left: 1px;
+}
+
+.star-rating-control {
+  display: inline-block;
+  height: 14px;
+}
+
+.review_hr {
+  size: 1px;
+  background: silver;
+  border: none;
+  margin: 1em 0;
+}
+
+/* jQuery.Rating Plugin CSS - http://www.fyneworks.com/jquery/star-rating/ */
+div.rating-cancel, div.star-rating {
+  float: left;
+  width: 17px;
+  height: 16px;
+  text-indent: -999em;
+  cursor: pointer;
+  display: block;
+  background: transparent;
+  overflow: hidden
+}
+
+div.rating-cancel, div.rating-cancel a {
+  background: transparent url(<%= asset_path('store/reviews/delete.gif') %>) no-repeat scroll 0 -1px;
+}
+
+div.star-rating, div.star-rating a {
+  background: url(<%= asset_path('store/reviews/star.gif') %>) no-repeat 0 0px
+}
+
+div.rating-cancel a, div.star-rating a {
+  display: block;
+  width: 16px;
+  height: 100%;
+  background-position: 0 0px;
+  border: 0
+}
+
+div.star-rating-on a {
+  background-position: 0 -32px !important
+}
+
+div.star-rating-hover a {
+  background-position: 0 -32px
+}
+
+/* Read Only CSS */
+div.star-rating-readonly a {
+  cursor: default !important
+}
+
+/* Partial Star CSS */
+div.star-rating {
+  background: transparent !important;
+  overflow: hidden !important
+}
+
+/* END jQuery.Rating Plugin CSS */

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

@@ -0,0 +1,6 @@
+class Spree::Admin::FeedbackReviewsController <  Spree::Admin::ResourceController
+  belongs_to 'spree/review'
+  def index
+    @collection = parent.feedback_reviews
+  end
+end

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

@@ -0,0 +1,21 @@
+class Spree::Admin::ReviewSettingsController < Spree::Admin::BaseController
+  before_filter :process_unset_checkboxes, only: [:update]
+
+  def update
+    Spree::Reviews::Config.set(params[:preferences])
+
+    respond_to do |format|
+      format.html do
+        redirect_to edit_admin_review_settings_path
+      end
+    end
+  end
+
+  def process_unset_checkboxes
+    # workaround for unset checkbox behaviour
+    params[:preferences] ||= {}
+    Spree::ReviewsConfiguration.boolean_preferences.each do |sym|
+      params[:preferences][sym] = false if params[:preferences][sym].blank?
+    end
+  end
+end

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

@@ -0,0 +1,34 @@
+class Spree::Admin::ReviewsController < Spree::Admin::ResourceController
+  helper Spree::ReviewsHelper
+
+  def index
+    @reviews = collection
+  end
+
+  def approve
+    r = Spree::Review.find(params[:id])
+
+    if r.update_attribute(:approved, true)
+       flash[:notice] = Spree.t("info_approve_review")
+    else
+       flash[:error] = Spree.t("error_approve_review")
+    end
+    redirect_to admin_reviews_path
+  end
+
+  def edit
+    if @review.product.nil?
+      flash[:error] = Spree.t("error_no_product")
+      redirect_to admin_reviews_path and return
+    end
+  end
+
+  private
+
+  def collection
+    params[:q] ||= {}
+
+    @search = Spree::Review.ransack(params[:q])
+    @collection = @search.result.includes([:product, :user, :feedback_reviews]).page(params[:page]).per(params[:per_page])
+  end
+end

data/app/controllers/spree/feedback_reviews_controller.rb

@@ -0,0 +1,41 @@
+class Spree::FeedbackReviewsController < Spree::StoreController
+  helper Spree::BaseHelper
+
+  before_filter :sanitize_rating, only: [:create]
+  before_filter :load_review, only: [:create]
+
+  def create
+
+    if @review.present?
+      @feedback_review = @review.feedback_reviews.new(feedback_review_params)
+      @feedback_review.user = spree_current_user
+      @feedback_review.locale = I18n.locale.to_s if Spree::Reviews::Config[:track_locale]
+      authorize! :create, @feedback_review
+      @feedback_review.save
+    end
+
+    respond_to do |format|
+      format.html { redirect_to :back  }
+      format.js   { render :action => :create }
+    end
+
+  end
+
+  protected
+    def load_review
+      @review ||= Spree::Review.find_by_id!(params[:review_id])
+    end
+
+    def permitted_feedback_review_attributes
+      [:rating, :comment]
+    end
+
+    def feedback_review_params
+      params.require(:feedback_review).permit(permitted_feedback_review_attributes)
+    end
+
+    def sanitize_rating
+      params[:feedback_review][:rating].to_s.sub!(/\s*[^0-9]*\z/,'') unless (params[:feedback_review] && params[:feedback_review][:rating].blank?)
+    end
+end
+

data/app/controllers/spree/products_controller_decorator.rb

@@ -0,0 +1,5 @@
+Spree::ProductsController.class_eval do
+  helper Spree::ReviewsHelper
+
+  [:avg_rating, :reviews_count].each { |attrib| Spree::PermittedAttributes.product_attributes << attrib }
+end

data/app/controllers/spree/reviews_controller.rb

@@ -0,0 +1,48 @@
+class Spree::ReviewsController < Spree::StoreController
+  helper Spree::BaseHelper
+  before_filter :load_product, :only => [:index, :new, :create]
+  rescue_from ActiveRecord::RecordNotFound, :with => :render_404
+
+  def index
+    @approved_reviews = Spree::Review.approved.where(product: @product)
+  end
+
+  def new
+    @review = Spree::Review.new(:product => @product)
+    authorize! :create, @review
+  end
+
+  # save if all ok
+  def create
+    params[:review][:rating].sub!(/\s*[^0-9]*\z/,'') unless params[:review][:rating].blank?
+
+    @review = Spree::Review.new(review_params)
+    @review.product = @product
+    @review.user = spree_current_user if spree_user_signed_in?
+    @review.ip_address = request.remote_ip
+    @review.locale = I18n.locale.to_s if Spree::Reviews::Config[:track_locale]
+
+    authorize! :create, @review
+    if @review.save
+      flash[:notice] = Spree.t('review_successfully_submitted')
+      redirect_to spree.product_path(@product)
+    else
+      render :new
+    end
+  end
+
+  private
+
+  def load_product
+    @product = Spree::Product.friendly.find(params[:product_id])
+  end
+
+  def permitted_review_attributes
+    [:rating, :title, :review, :name]
+  end
+
+  def review_params
+    params.require(:review).permit(permitted_review_attributes)
+  end
+
+end

data/app/helpers/spree/reviews_helper.rb

@@ -0,0 +1,17 @@
+module Spree::ReviewsHelper
+
+  def star(the_class)
+    content_tag(:span, " ✮ ".html_safe, :class => the_class)
+  end
+
+  def mk_stars(m)
+    (1..5).collect {|n| n <= m ? star("lit") : star("unlit") }.join
+  end
+
+  def txt_stars(n, show_out_of = true)
+    res = Spree.t('star', :count => n)
+    res += " #{Spree.t('out_of_5')}" if show_out_of
+    res
+  end
+
+end

data/app/models/spree/feedback_review.rb

@@ -0,0 +1,17 @@
+class Spree::FeedbackReview < ActiveRecord::Base
+  belongs_to :user, :class_name => Spree.user_class.to_s
+
+  belongs_to :review, dependent: :destroy
+  validates :review, presence: true
+
+  validates :rating, numericality: { only_integer: true,
+                                     greater_than_or_equal_to: 1, 
+                                     less_than_or_equal_to: 5,
+                                     message: Spree.t('you_must_enter_value_for_rating') }
+
+  scope :most_recent_first, -> { order("spree_feedback_reviews.created_at DESC") }
+  default_scope { most_recent_first }
+
+  scope :localized, lambda { |lc| where('spree_feedback_reviews.locale = ?', lc) }
+
+end

data/app/models/spree/product_decorator.rb

@@ -0,0 +1,21 @@
+# Add access to reviews/ratings to the product model
+Spree::Product.class_eval do
+  has_many :reviews
+
+  def stars
+    avg_rating.try(:round) || 0
+  end
+
+  def recalculate_rating
+    reviews_count = self.reviews.reload.approved.count
+
+    self.reviews_count = reviews_count
+    if reviews_count > 0
+      self.avg_rating = self.reviews.approved.sum(:rating).to_f / reviews_count
+    else
+      self.avg_rating = 0
+    end
+    self.save
+  end
+
+end

data/app/models/spree/review.rb

@@ -0,0 +1,36 @@
+class Spree::Review < ActiveRecord::Base
+  belongs_to :product, touch: true
+  belongs_to :user, :class_name => Spree.user_class.to_s
+  has_many   :feedback_reviews
+
+  after_save :recalculate_product_rating, :if => :approved?
+  after_destroy :recalculate_product_rating
+
+  validates :name, presence: true
+  validates :review, presence: true
+
+  validates :rating, numericality: { only_integer: false,
+                                     greater_than_or_equal_to: 0.5,
+                                     less_than_or_equal_to: 5.0,
+                                     message: Spree.t('you_must_enter_value_for_rating') }
+
+
+  default_scope { order("spree_reviews.created_at DESC") }
+
+  scope :localized, ->(lc) { where('spree_reviews.locale = ?', lc) }
+  scope :most_recent_first, -> { order('spree_reviews.created_at DESC') }
+  scope :oldest_first, -> { reorder('spree_reviews.created_at ASC') }
+  scope :preview, -> { limit(Spree::Reviews::Config[:preview_size]).oldest_first }
+  scope :approved, -> { where(approved: true) }
+  scope :not_approved, -> { where(approved: false) }
+  scope :default_approval_filter, -> { Spree::Reviews::Config[:include_unapproved_reviews] ? all : approved }
+
+  def feedback_stars
+    return 0 if feedback_reviews.size <= 0
+    ((feedback_reviews.sum(:rating) / feedback_reviews.size) + 0.5).floor
+  end
+
+  def recalculate_product_rating
+    self.product.recalculate_rating if product.present?
+  end
+end

data/app/models/spree/reviews_ability.rb

@@ -0,0 +1,17 @@
+class Spree::ReviewsAbility
+  include CanCan::Ability
+
+  def initialize user
+    review_ability_class = self.class
+    can :create, Spree::Review do |review|
+      review_ability_class.allow_anonymous_reviews? || !user.email.blank?
+    end
+    can :create, Spree::FeedbackReview do |review|
+      review_ability_class.allow_anonymous_reviews? || !user.email.blank?
+    end
+  end
+
+  def self.allow_anonymous_reviews?
+    !Spree::Reviews::Config[:require_login]
+  end
+end

data/app/models/spree/reviews_configuration.rb

@@ -0,0 +1,23 @@
+class Spree::ReviewsConfiguration < Spree::Preferences::Configuration
+  def self.boolean_preferences
+    %w(include_unapproved_reviews feedback_rating show_email require_login track_locale)
+  end
+
+  # include non-approved reviews in (public) listings
+  preference :include_unapproved_reviews, :boolean, :default => false
+
+  # control how many reviews are shown in summaries etc.
+  preference :preview_size, :integer, :default => 3
+
+  # show a reviewer's email address
+  preference :show_email, :boolean, :default => false
+
+  # show helpfullness rating form elements
+  preference :feedback_rating, :boolean, :default => false
+
+  # require login to post reviews
+  preference :require_login, :boolean, :default => true
+
+  # whether to keep track of the reviewer's locale
+  preference :track_locale, :boolean, :default => false
+end

data/app/overrides/add_reviews_after_product_properties.rb

@@ -0,0 +1,5 @@
+Deface::Override.new(:virtual_path => "spree/products/show",
+                     :name => "converted_product_properties_767643482",
+                     :insert_after => "[data-hook='product_properties']",
+                     :partial => "spree/shared/reviews",
+                     :disabled => false)

data/app/overrides/add_reviews_tab_to_admin.rb

@@ -0,0 +1,4 @@
+Deface::Override.new(:virtual_path => "spree/admin/shared/_product_sub_menu",
+                     :name => "reviews_admin_tab",
+                     :insert_bottom => "[data-hook='admin_product_sub_tabs']",
+                     :text => "<%= tab(:reviews, :label => 'review_management') %>")

data/app/overrides/add_reviews_to_admin_configuration_sidebar.rb

@@ -0,0 +1,5 @@
+Deface::Override.new(:virtual_path => "spree/admin/shared/_configuration_menu",
+                     :name => "converted_admin_configurations_menu",
+                     :insert_bottom => "[data-hook='admin_configurations_sidebar_menu'], #admin_configurations_sidebar_menu[data-hook]",
+                     :text => "<%= configurations_sidebar_menu_item Spree.t('spree_reviews.review_settings'), edit_admin_review_settings_path %>",
+                     :disabled => false)

data/app/overrides/fix_highlighting_of_products_tab.rb

@@ -0,0 +1,5 @@
+Deface::Override.new(:virtual_path => "spree/admin/shared/_tabs",
+                    :name => "reviews_admin_tab_root",
+                    :replace_contents => "erb[loud]:contains('tab :products')",
+                    :text => "tab :products, :option_types, :properties, :prototypes, :variants, :product_properties, :taxons, :reviews, :icon => 'icon-th-large'",
+                    :original => '0abc74602592114216d5a53bc095823ec6759895')

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

@@ -0,0 +1,44 @@
+<% content_for :page_title do %>
+	<%= Spree.t('feedback_review_for', :review => @review.title) %>
+<% end %>
+
+<% content_for :page_actions do %>
+	<li><%= button_link_to Spree.t(:back_to_reviews), admin_reviews_path, { :icon => 'icon-arrow-left' } %></li>
+<% end %>
+
+<% render 'spree/admin/shared/product_sub_menu' %>
+
+<% if @collection.any? %>
+	<table class="index">
+		<colgroup>
+			<col style="width: 30%;">
+			<col style="width: 20%;">
+			<col style="width: 20%;">
+			<col style="width: 17%;">
+		</colgroup>
+		<thead>
+			<tr>
+				<th><%= Spree.t('user') %></th>
+				<th><%= Spree.t('stars') %></th>
+				<th><%= Spree.t('date') %></th>
+				<th></th>
+			</tr>
+		</thead>
+		<tbody>
+		<%- @collection.each do |feedback| -%>
+			<tr id="<%= dom_id(feedback) %>">
+			  <td><%= l feedback.created_at %></td>
+			  <td><%= feedback.user.try(:login) || Spree.t(:anonymous) %></td>
+			  <td><%= feedback.rating %></td>
+			  <td class="actions">
+				  <%= link_to_delete feedback, :no_text => true  %>
+			  </td>
+			</tr>
+		<% end %>
+		</tbody>
+	</table>
+<% else %>
+	<div class="no-objects-found">
+		<%= Spree.t(:no_results) %>
+	</div>
+<% end %>