workarea-reviews 3.0.10 → 3.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cbc95c26a6972a59da6d0daa765491622783a60cfc8dd0961109b82839b0af77
-  data.tar.gz: 9f62f3d6484cdf8002172106970954fa053784844a6fa6084b2aee3baadf8e7b
+  metadata.gz: 7135d1659f6d6564addc0f472fbf72a6225098a29a9de3e9a3c16af4724b364e
+  data.tar.gz: fdfe10e7e4e59e7bfb4b4c08f05dc2a8190acc36c2bd5602277c2298a2f4072e
 SHA512:
-  metadata.gz: e6913aca804eeb31ab4bb877b74b2826b50596e7b3a087eb162c49d46dc6df2a162ab8ca683e8251a1c056c798d12597294125acea2f102a91150baef4b31a4c
-  data.tar.gz: ab487315635024cb2fcbb3b6cea388188b01ad31e0cfbcaf6e9e9513f35f1ad53c035eeb67927bda4796265bf6459eed6b14028bec6faf22066c3257907df5a1
+  metadata.gz: 294e2feacac1639da7c4b927fd9580f8c5e000bbd93885893a1c647cd980327ff8dd6d01d83db044cfc56da5dc71f2c62825deb07f71f85e042eeb59a54babb9
+  data.tar.gz: 014a4986c50d89dad1f932d8506d3171c776c91aedf83e959d6bab7e9a0d365255d31ce99aa960229189a959c1e744bb2c2ba889f46863677dd3abe4acfb2bb0

data/.eslintrc.json

@@ -0,0 +1,35 @@
+{
+    "extends": "eslint:recommended",
+    "rules": {
+      "semi": ["error", "always"],
+      "eqeqeq": ["error", "always"]
+    },
+    "globals": {
+      "window": true,
+      "document": true,
+      "WORKAREA": true,
+      "$": true,
+      "jQuery": true,
+      "_": true,
+      "feature": true,
+      "JST": true,
+      "Turbolinks": true,
+      "I18n": true,
+      "Chart": true,
+      "Dropzone": true,
+      "strftime": true,
+      "Waypoint": true,
+      "wysihtml": true,
+      "LocalTime": true,
+      "describe": true,
+      "after": true,
+      "afterEach": true,
+      "before": true,
+      "beforeEach": true,
+      "it": true,
+      "expect": true,
+      "sinon": true,
+      "fixture": true,
+      "chai": true
+    }
+}

data/.github/workflows/ci.yml

@@ -0,0 +1,60 @@
+name: CI
+on: [push]
+
+jobs:
+  static_analysis:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v1
+    - uses: workarea-commerce/ci/bundler-audit@v1
+    - uses: workarea-commerce/ci/rubocop@v1
+    - uses: workarea-commerce/ci/eslint@v1
+      with:
+        args: '**/*.js'
+    - uses: workarea-commerce/ci/stylelint@v1
+      with:
+        args: '**/*.scss'
+
+  admin_tests:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v1
+    - uses: actions/setup-ruby@v1
+      with:
+        ruby-version: 2.6.x
+    - uses: workarea-commerce/ci/test@v1
+      with:
+        command: bin/rails app:workarea:test:admin
+
+  core_tests:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v1
+    - uses: actions/setup-ruby@v1
+      with:
+        ruby-version: 2.6.x
+    - uses: workarea-commerce/ci/test@v1
+      with:
+        command: bin/rails app:workarea:test:core
+
+  storefront_tests:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v1
+    - uses: actions/setup-ruby@v1
+      with:
+        ruby-version: 2.6.x
+    - uses: workarea-commerce/ci/test@v1
+      with:
+        command: bin/rails app:workarea:test:storefront
+
+  plugins_tests:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v1
+    - uses: actions/setup-ruby@v1
+      with:
+        ruby-version: 2.6.x
+    - uses: workarea-commerce/ci/test@v1
+      with:
+        command: bin/rails app:workarea:test:plugins

data/.gitignore

@@ -12,5 +12,4 @@ test/dummy/tmp/
 test/dummy/public/
 node_modules
 test/reports
-package.json
-yarn.lock
+.rubocop-http*

data/.rubocop.yml

@@ -0,0 +1,3 @@
+inherit_from:
+  - https://raw.githubusercontent.com/workarea-commerce/workarea/master/.rubocop.yml
+

data/.stylelintrc.json

@@ -0,0 +1,8 @@
+{
+  "extends": "stylelint-config-recommended-scss",
+  "rules": {
+    "block-no-empty": null,
+    "no-descending-specificity": null,
+    "property-no-unknown": [true, { "ignoreProperties": ["mso-hide"] }]
+  }
+}

data/CHANGELOG.md

@@ -1,31 +1,13 @@
-Workarea Reviews 3.0.9 (2019-08-26)
+Workarea Reviews 3.1.0 (2019-11-26)
 --------------------------------------------------------------------------------
 
-* Only try to add schedule job if service connections are not skipped
+*   Updates for v3.5 compatibility
 
+    Ben Crouse
 
+*   Initial commit on master
 
-Workarea Reviews 3.0.8 (2019-08-21)
---------------------------------------------------------------------------------
-
-*   Open Source!
-
-
-
-Workarea Reviews 3.0.7 (2019-06-11)
---------------------------------------------------------------------------------
-
-*   Add Rake Task for Reconciling Verified Purchasers
-
-    Reviews can have a `:verified` badge associated with the content, but
-    for those upgrading to a newer version some data needs to be changed in
-    order to make this happen retroactively for older reviews. Add a Rake
-    task for adding the `:verified` field to reviews where the user actually
-    bought the product. For any reviews that are made after the upgrade,
-    this will be automatically assigned as the review is being created.
-
-    REVIEWS-146
-    Tom Scott
+    Curt Howard
 
 
 

data/Gemfile

@@ -1,6 +1,8 @@
 source 'https://rubygems.org'
-git_source(:github) { |repo| "git@github.com:#{repo}.git" }
+git_source(:github) { |repo| "https://github.com/#{repo}.git" }
 
 gemspec
 
-gem 'workarea'
+gem 'byebug'
+
+gem 'workarea', github: 'workarea-commerce/workarea'

data/Rakefile

@@ -28,16 +28,15 @@ desc "Release version #{Workarea::Reviews::VERSION} of the gem"
 task :release do
   host = "https://#{ENV['BUNDLE_GEMS__WEBLINC__COM']}@gems.weblinc.com"
 
-  #Rake::Task['workarea:changelog'].execute
-  #system 'git add CHANGELOG.md'
-  #system 'git commit -m "Update CHANGELOG"'
-  #system 'git push origin HEAD'
+  Rake::Task['workarea:changelog'].execute
+  system 'git add CHANGELOG.md'
+  system 'git commit -m "Update CHANGELOG"'
+  system 'git push origin HEAD'
 
   system "git tag -a v#{Workarea::Reviews::VERSION} -m 'Tagging #{Workarea::Reviews::VERSION}'"
   system 'git push --tags'
 
   system 'gem build workarea-reviews.gemspec'
-  system "gem push workarea-reviews-#{Workarea::Reviews::VERSION}.gem"
   system "gem push workarea-reviews-#{Workarea::Reviews::VERSION}.gem --host #{host}"
   system "rm workarea-reviews-#{Workarea::Reviews::VERSION}.gem"
 end

data/app/controllers/workarea/admin/reports_controller.decorator

@@ -0,0 +1,10 @@
+module Workarea
+  decorate Admin::ReportsController, with: :reviews do
+    def reviews_by_product
+      @report = Admin::Reports::ReviewsByProductViewModel.wrap(
+        Workarea::Reports::ReviewsByProduct.new(params),
+        view_model_options
+      )
+    end
+  end
+end

data/app/helpers/workarea/admin/reviews_helper.rb

@@ -1,11 +1,12 @@
 module Workarea
   module Admin
     module ReviewsHelper
-      def reviewer_info(model)
-        return model.user_info unless model.user_id.present?
-
-        link_to model.user_info,
-                edit_user_path(model.user_id)
+      def reviews_report_filter_options
+        [
+          [t('workarea.admin.reports.reviews_by_product.filters.all'), nil],
+          [t('workarea.admin.reports.reviews_by_product.filters.approved'), 'approved'],
+          [t('workarea.admin.reports.reviews_by_product.filters.unapproved'), 'unapproved']
+        ]
       end
     end
   end

data/app/helpers/workarea/storefront/reviews_helper.rb

@@ -6,10 +6,8 @@ module Workarea
         empty_star_count = 5 - rating.ceil
         half_star_size = (rating % 1).round(2) * 100
         half_star_width = 20 + (half_star_size - 0) * (80.0 - 20) / (100.0 - 0)
-        itemprop = options[:aggregate] ? 'aggregateRating' : 'reviewRating'
-        itemtype = options[:aggregate] ? 'http://schema.org/AggregateRating' : 'http://schema.org/Rating'
 
-        render 'workarea/storefront/products/rating', rating: rating, full_star_count: full_star_count, empty_star_count: empty_star_count, half_star_width: half_star_width, half_star_size: half_star_size, itemprop: itemprop, itemtype: itemtype
+        render 'workarea/storefront/products/rating', rating: rating, full_star_count: full_star_count, empty_star_count: empty_star_count, half_star_width: half_star_width, half_star_size: half_star_size
       end
 
       def display_purchase_requirement_message

data/app/helpers/workarea/storefront/reviews_schema_org_helper.rb

@@ -0,0 +1,34 @@
+module Workarea
+  module Storefront
+    module ReviewsSchemaOrgHelper
+      def product_schema(product, related_products: nil)
+        schema = super.merge(
+          'review': product.reviews.map do |review|
+            {
+              '@type': 'Review',
+              'author': review.user_info,
+              'datePublished': review.created_at.strftime('%Y-%m-%d'),
+              'description': review.body,
+              'name': review.title,
+              'reviewRating': {
+                '@type': 'Rating',
+                'bestRating': '5',
+                'worstRating': '1',
+                'ratingValue': review.rating.round(2).to_s
+              }
+            }
+          end
+        )
+
+        if product.total_reviews > 0 && product.average_rating.present?
+          schema['aggregateRating'] = {
+            'reviewCount': product.total_reviews.to_s,
+            'ratingValue': product.average_rating.round(2).to_s
+          }
+        end
+
+        schema
+      end
+    end
+  end
+end

data/app/mailers/workarea/admin/status_report_mailer.decorator

@@ -1,7 +1,7 @@
 module Workarea
   decorate Admin::StatusReportMailer, with: :reviews do
-    def report(email, date)
-      @reviews_summary =  ReviewSummary.new
+    def report(*)
+      @reviews_summary = ReviewSummary.new
       super
     end
   end

data/app/models/workarea/insights/most_active_reviewers.rb

@@ -0,0 +1,34 @@
+module Workarea
+  module Insights
+    class MostActiveReviewers < Base
+      class << self
+        def dashboards
+          %w(people marketing)
+        end
+
+        def generate_monthly!
+          results = generate_results
+          create!(results: results) if results.present?
+        end
+
+        def generate_results
+          report
+            .results
+            .select { |r| r['reviews'] > 0 }
+            .take(Workarea.config.insights_users_list_max_results)
+            .map { |result| result.merge(email: result['_id']) }
+        end
+
+        def report
+          Reports::ReviewsByUser.new(
+            starts_at: beginning_of_last_month,
+            ends_at: end_of_last_month,
+            sort_by: 'activity_score',
+            sort_direction: 'desc',
+            results_filter: 'approved'
+          )
+        end
+      end
+    end
+  end
+end

data/app/models/workarea/insights/most_reviewed_products.rb

@@ -0,0 +1,33 @@
+module Workarea
+  module Insights
+    class MostReviewedProducts < Base
+      class << self
+        def dashboards
+          %w(marketing)
+        end
+
+        def generate_monthly!
+          results = generate_results
+          create!(results: results) if results.present?
+        end
+
+        def generate_results
+          report
+            .results
+            .select { |r| r['reviews'] > 0 }
+            .take(Workarea.config.insights_products_list_max_results)
+            .map { |result| result.merge(product_id: result['_id']) }
+        end
+
+        def report
+          Reports::ReviewsByProduct.new(
+            starts_at: beginning_of_last_month,
+            ends_at: end_of_last_month,
+            sort_by: 'reviews',
+            sort_direction: 'desc'
+          )
+        end
+      end
+    end
+  end
+end

data/app/models/workarea/insights/top_rated_products.rb

@@ -0,0 +1,33 @@
+module Workarea
+  module Insights
+    class TopRatedProducts < Base
+      class << self
+        def dashboards
+          %w(marketing)
+        end
+
+        def generate_monthly!
+          results = generate_results
+          create!(results: results) if results.present?
+        end
+
+        def generate_results
+          report
+            .results
+            .select { |r| r['reviews'] > 0 }
+            .take(Workarea.config.insights_products_list_max_results)
+            .map { |result| result.merge(product_id: result['_id']) }
+        end
+
+        def report
+          Reports::ReviewsByProduct.new(
+            starts_at: beginning_of_last_month,
+            ends_at: end_of_last_month,
+            sort_by: 'weighted_average_rating',
+            sort_direction: 'desc'
+          )
+        end
+      end
+    end
+  end
+end

data/app/models/workarea/review.rb

@@ -95,23 +95,12 @@ module Workarea
     # @return [Float]
     #
     def self.find_sorting_score(product_id)
-      reviews = find_for_product(product_id)
-
-      votes = [ reviews.select { |r| r.rating == 1 }.length,
-                reviews.select { |r| r.rating == 2 }.length,
-                reviews.select { |r| r.rating == 3 }.length,
-                reviews.select { |r| r.rating == 4 }.length,
-                reviews.select { |r| r.rating == 5 }.length ]
-
-      prior = [2, 2, 2, 2, 2]
-
-      posterior = votes.zip(prior).map { |a, b| a + b }
-      sum = posterior.inject { |a, b| a + b }
+      reviews = find_for_product(product_id).to_a
+      count = (1..5).each_with_object({}) do |i, memo|
+        memo[i] = reviews.select { |r| r.rating == i }.length + 2
+      end
 
-      posterior.
-        map.with_index { |v, i| (i + 1) * v }.
-        inject { |a, b| a + b }.
-        to_f / sum
+      count.sum { |rating, count| rating * count }.to_f / count.values.sum
     end
 
     def anonymous?

data/app/queries/workarea/reports/reviews_by_product.rb

@@ -0,0 +1,86 @@
+module Workarea
+  module Reports
+    class ReviewsByProduct
+      include Report
+
+      self.reporting_class = Review
+      self.sort_fields = %w(reviews verified average_rating weighted_average_rating)
+
+      def aggregation
+        [filter_results, project_used_fields, group_by_product, project_averages]
+      end
+
+      def filter_results
+        {
+          '$match' => {
+            'created_at' => { '$gte' => starts_at, '$lte' => ends_at },
+            **approval_query
+          }
+        }
+      end
+
+      def project_used_fields
+        {
+          '$project' => {
+            'product_id' => 1,
+            'rating' => 1,
+            'verified' => 1
+          }
+        }
+      end
+
+      def group_by_product
+        {
+          '$group' => {
+            '_id' => '$product_id',
+            'reviews' => { '$sum' => 1 },
+            'verified' => { '$sum' => { '$cond' => { 'if' => '$verified', 'then' => 1, 'else' => 0 } } },
+            'rating_tally' => { '$sum' => '$rating' },
+            'rated_5' => { '$sum' => { '$cond' => { 'if' => { '$eq' => ['$rating', 5] }, 'then' => 1, 'else' => 0 } } },
+            'rated_4' => { '$sum' => { '$cond' => { 'if' => { '$eq' => ['$rating', 4] }, 'then' => 1, 'else' => 0 } } },
+            'rated_3' => { '$sum' => { '$cond' => { 'if' => { '$eq' => ['$rating', 3] }, 'then' => 1, 'else' => 0 } } },
+            'rated_2' => { '$sum' => { '$cond' => { 'if' => { '$eq' => ['$rating', 2] }, 'then' => 1, 'else' => 0 } } },
+            'rated_1' => { '$sum' => { '$cond' => { 'if' => { '$eq' => ['$rating', 1] }, 'then' => 1, 'else' => 0 } } }
+          }
+        }
+      end
+
+      def project_averages
+        {
+          '$project' => {
+            'product_id' => 1,
+            'reviews' => 1,
+            'verified' => 1,
+            'average_rating' => { '$divide' => ['$rating_tally', '$reviews'] },
+            'weighted_average_rating' => {
+              '$divide' => [
+                {
+                  '$sum' => [
+                    { '$multiply' => [{ '$sum' => [2, '$rated_5'] }, 5] },
+                    { '$multiply' => [{ '$sum' => [2, '$rated_4'] }, 4] },
+                    { '$multiply' => [{ '$sum' => [2, '$rated_3'] }, 3] },
+                    { '$multiply' => [{ '$sum' => [2, '$rated_2'] }, 2] },
+                    { '$multiply' => [{ '$sum' => [2, '$rated_1'] }, 1] }
+                  ]
+                },
+                { '$sum' => [10, '$rated_5', '$rated_4', '$rated_3', '$rated_2', '$rated_1'] }
+              ]
+            }
+          }
+        }
+      end
+
+      private
+
+      def approval_query
+        if params[:results_filter] == 'approved'
+          { approved: true }
+        elsif params[:results_filter] == 'unapproved'
+          { approved: false }
+        else
+          {}
+        end
+      end
+    end
+  end
+end