merit 2.4.0 → 4.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: c1d70d0ff56841cebd7bfb79bf17a152ba0a8156
-  data.tar.gz: c602f5a16bbe014cd78ba2fc5d5eeec1ee9ac362
+SHA256:
+  metadata.gz: f0ee557f98d45fe9a83d5de23d407e2b36f85f6a5d4204b1ed89fec8bf5c90fb
+  data.tar.gz: ae393531c2a569a8aece0d533523aaa6fa6e407c4fe4aa3efe96f49d349b3d37
 SHA512:
-  metadata.gz: 5a8a44e81bddc1a740ebfb860eabefa2314f136c19851f5011113b1e25e4f539bb86affa4c9d447c6768f77b5c0a6b107258563a7bc41bd0b16ec86b24bd7a72
-  data.tar.gz: 4e8523ab4e5c04583184da68b1aa5ff9937f053ad39c3756d10526bf9fde5f1e93c54d2d209c65b3f8c811e8d1bc21c15a475faa688eef5021ab363ee97bfe3c
+  metadata.gz: c4b48c33d57774076e00d9732ccac73dbb71b1ee1efe87329ba9ee542593e1a8c3823bef1c5c783a45ed46b28cc3eda4626ddefe9683682341ccc8d51bcf9889
+  data.tar.gz: 0e886c1d895343875d06c537015d4ae14b0723f96d8ddac2a264892b7a3aee0d821f7e6ccb404f68371ac9ca9d5adad78f211b1943747a9bc6dc53361efd6cb6

data/Gemfile

@@ -2,42 +2,10 @@ source 'https://rubygems.org'
 
 gemspec
 
-version = ENV['RAILS_VERSION'] || '4.2'
-rails = case version
-when 'master'
-  { github: 'rails/rails' }
-when '4.0-protected-attributes'
-  gem 'protected_attributes'
-  "~> 4.0.0"
-when /4\.0|4\.1/
-  "~> #{version}.0"
-when /4\.2/
-  "~> #{version}.0.rc3"
-when '3.2'
-  gem 'strong_parameters'
-  "~> #{version}.0"
-end
-
-gem 'rails', rails
-
-case ENV['ORM']
-when 'active_record'
-  gem 'activerecord'
-when 'mongoid'
-  gem 'mongoid'
-end
+gem 'rails', "~> #{ENV.fetch('RAILS_VERSION', 6.0)}"
 
 group :development, :test do
-  gem 'activerecord-jdbcsqlite3-adapter', :platforms => [:jruby]
-  gem 'sqlite3', '~> 1.3.8', :platforms => [:ruby, :mswin, :mingw]
-end
-
-platforms :rbx do
-  gem 'rubysl', '~> 2.0'
-  gem 'psych'
-  gem 'racc'
-  gem 'minitest'
-  gem 'rubinius-developer_tools'
+  gem 'sqlite3', '~> 1.4'
 end
 
-gem 'coveralls', require: false
+gem 'coveralls', '~> 0.8.23', require: false

data/NEWS.md

@@ -2,6 +2,37 @@
 
 User-visible changes worth mentioning.
 
+## 4.0.0
+
+- Stop testing on Rails 5.1
+- Drop support for mongoid
+- Drop v2 upgrade generators
+- Enable Zeitwerk gem mode (and renames internal merit constants)
+
+## 3.0.3
+
+- Test against Ruby 2.6 and Rails 6
+- Use `find_each` in favor of `map` in `Merit::Action.check_unprocessed`
+- Allow finding badges defined with symbol names
+- Remove `Badge#last_granted` method (deprecated in merit <2.0)
+
+## 3.0.2
+
+- [#287] Make Sash an optional dependency, fixing issues in Rails 5.1.
+- [#297] Initialize merit once both for API or non-API apps
+- Updates dependencies
+
+## 3.0.1
+
+- [#282] Run action controller load hook only once
+- [#284] Add migration version to active record migrations
+- Add RELEASING.md document
+
+## 3.0.0
+
+- [#276] Drops Rails <5 version support. Drops deprecated `action_filter` call.
+- Fix deprecation warnings
+
 ## 2.4.0
 
 - [#260] Works with Rails 5

data/README.md

@@ -4,7 +4,7 @@ Merit adds reputation behavior to Rails apps in the form of Badges, Points,
 and Rankings.
 
 [![Build Status](https://travis-ci.org/merit-gem/merit.svg?branch=master)](http://travis-ci.org/merit-gem/merit)
-[![Coverage Status](https://img.shields.io/coveralls/tute/merit.svg)](https://coveralls.io/r/tute/merit?branch=master)
+[![Coverage Status](https://coveralls.io/repos/github/merit-gem/merit/badge.svg?branch=master)](https://coveralls.io/github/merit-gem/merit?branch=master)
 [![Code Climate](https://codeclimate.com/github/tute/merit/badges/gpa.svg)](https://codeclimate.com/github/tute/merit)
 
 # Table of Contents
@@ -38,8 +38,7 @@ and Rankings.
 2. Run `rails g merit:install`. This creates several migrations.
 3. Run `rails g merit MODEL_NAME` (e.g. `user`). This creates a migration and adds `has_merit` to MODEL_NAME.
 4. Run `rake db:migrate`
-5. Define badges in `config/initializers/merit.rb`. You can also define ORM:
-   `:active_record` (default) or `:mongoid`.
+5. Define badges in `config/initializers/merit.rb`
 6. Configure reputation rules for your application in `app/models/merit/*`
 
 
@@ -114,6 +113,19 @@ grant_on ['users#create', 'users#update'], badge: 'autobiographer', temporary: t
 end
 ```
 
+If your controller is under a namespace other than root (example:
+`Api::ModelController`) then for merit to find your object automatically you
+must specify the model class and not forget that your action is of the form
+`namespace/models#action`.
+
+See an example of a `Post` model that belongs to user:
+
+```ruby
+grant_on 'api/posts#create', badge: 'first-post', model_name: 'Post', to: :user do |post|
+  post.user.posts.count >= 1
+end
+```
+
 ## Other Actions
 
 ```ruby

data/RELEASING.md

@@ -0,0 +1,16 @@
+# Releasing
+
+1. Update `NEWS.md` to reflect the changes since last release.
+1. Tag the release: `git tag vVERSION -a -s`. The tag message should contain the
+   appropriate `NEWS.md` subsection.
+1. Push changes: `git push --tags`
+1. Build and publish to rubygems:
+   ```bash
+   gem build merit.gemspec
+   gem push merit-*.gem
+   ```
+
+1. Add a new GitHub release:
+   https://github.com/merit-gem/merit/releases/new?tag=vVERSION
+1. Announce the new release, making sure to say "thank you" to the contributors
+   who helped shape this version!

data/Rakefile

@@ -8,7 +8,11 @@ end
 require 'rake/testtask'
 
 desc 'Default: run tests for all ORMs.'
-task default: :test
+task default: [:setup, :test, :api_test]
+
+task :setup do
+  system "cd test/dummy && rake db:migrate && rake db:test:prepare"
+end
 
 Rake::TestTask.new(:test) do |t|
   t.libs << 'lib'
@@ -17,6 +21,14 @@ Rake::TestTask.new(:test) do |t|
   t.verbose = true
 end
 
+Rake::TestTask.new(:api_test) do |t|
+  t.libs << 'lib'
+  t.libs << 'test'
+  t.test_files = FileList['test/**/*_test.rb'].exclude(/navigation_test.rb/)
+  t.verbose = true
+  t.options = '-- -api-only'
+end
+
 begin
   require 'rdoc/task'
   Rake::RDocTask.new(:rdoc) do |rdoc|

data/app/models/merit/badge.rb

@@ -18,7 +18,7 @@ module Merit
       end
 
       def by_name(name)
-        find { |b| b.name == name.to_s }
+        find { |b| b.name.to_s == name.to_s }
       end
 
       def by_level(level)
@@ -26,14 +26,6 @@ module Merit
       end
     end
 
-    def _mongoid_sash_in(sashes)
-      {:sash_id.in => sashes}
-    end
-
-    def _active_record_sash_in(sashes)
-      {sash_id: sashes}
-    end
-
     class << self
       def find_by_name_and_level(name, level)
         badges = Merit::Badge.by_name(name)
@@ -45,22 +37,13 @@ module Merit
         badge
       end
 
-      # DEPRECATED: `last_granted` will be removed from merit, please refer to:
-      # https://github.com/tute/merit/wiki/How-to-show-last-granted-badges
-      def last_granted(options = {})
-        warn '[merit] [DEPRECATION] `last_granted` will be removed from merit, please refer to: https://github.com/tute/merit/wiki/How-to-show-last-granted-badges'
-        options[:since_date] ||= 1.month.ago
-        options[:limit]      ||= 10
-        BadgesSash.last_granted(options)
-      end
-
       # Defines Badge#meritable_models method, to get related
       # entries with certain badge. For instance, Badge.find(3).users
       # orm-specified
       def _define_related_entries_method(meritable_class_name)
         define_method(:"#{meritable_class_name.underscore.pluralize}") do
           sashes = BadgesSash.where(badge_id: id).pluck(:sash_id)
-          meritable_class_name.constantize.where(send "_#{Merit.orm}_sash_in", sashes)
+          meritable_class_name.constantize.where(sash_id: sashes)
         end
       end
     end

data/lib/merit.rb

@@ -1,17 +1,4 @@
-require 'merit/rule'
-require 'merit/rules_badge_methods'
-require 'merit/rules_points_methods'
-require 'merit/rules_rank_methods'
-require 'merit/rules_matcher'
-require 'merit/controller_extensions'
-require 'merit/model_additions'
-require 'merit/judge'
-require 'merit/reputation_change_observer'
-require 'merit/sash_finder'
-require 'merit/base_target_finder'
-require 'merit/target_finder'
-require 'merit/models/base/sash'
-require 'merit/models/base/badges_sash'
+require 'zeitwerk'
 
 module Merit
   def self.setup
@@ -24,7 +11,6 @@ module Merit
     @config.checks_on_each_request
   end
 
-  # # Define ORM
   def self.orm
     @config.orm || :active_record
   end
@@ -49,11 +35,6 @@ module Merit
     @config.add_observer(class_name)
   end
 
-  def self.upgrade_target_data_warning
-    Rails.logger.warn '[merit] Missing column: target_data. Run `rails ' \
-                      'generate merit:upgrade` and `rake db:migrate` to add it.'
-  end
-
   class Configuration
     attr_accessor :checks_on_each_request, :orm, :user_model_name, :observers,
                   :current_user_method
@@ -81,8 +62,7 @@ module Merit
 
     initializer 'merit.controller' do |app|
       extend_orm_with_has_merit
-      require_models
-      ActiveSupport.on_load(:action_controller) do
+      ActiveSupport.on_load(action_controller_hook) do
         begin
           # Load app rules on boot up
           Merit::AppBadgeRules = Merit::BadgeRules.new.defined_rules
@@ -96,22 +76,20 @@ module Merit
       end
     end
 
-    def require_models
-      require 'merit/models/base/sash'
-      require 'merit/models/base/badges_sash'
-      require "merit/models/#{Merit.orm}/merit/activity_log"
-      require "merit/models/#{Merit.orm}/merit/badges_sash"
-      require "merit/models/#{Merit.orm}/merit/sash"
-      require "merit/models/#{Merit.orm}/merit/score"
+    def extend_orm_with_has_merit
+      ActiveRecord::Base.include(Merit)
     end
 
-    def extend_orm_with_has_merit
-      if Object.const_defined?('ActiveRecord')
-        ActiveRecord::Base.send :include, Merit
-      end
-      if Object.const_defined?('Mongoid')
-        Mongoid::Document.send :include, Merit
+    def action_controller_hook
+      if Rails.application.config.api_only
+        :action_controller_api
+      else
+        :action_controller_base
       end
     end
   end
 end
+
+loader = Zeitwerk::Loader.for_gem
+loader.setup
+loader.eager_load

data/lib/merit/base_target_finder.rb

@@ -25,12 +25,7 @@ module Merit
     end
 
     def reanimate_target_from_action
-      if @action.respond_to? :target_data
-        YAML.load(@action.target_data)
-      else
-        Merit.upgrade_target_data_warning
-        nil
-      end
+      YAML.load(@action.target_data)
     end
   end
 end

data/lib/merit/{model_additions.rb → class_methods.rb}

@@ -7,10 +7,8 @@ module Merit
       # That's why MeritableModel belongs_to Sash. Can't use
       # dependent: destroy as it may raise FK constraint exceptions. See:
       # https://rails.lighthouseapp.com/projects/8994-ruby-on-rails/tickets/1079-belongs_to-dependent-destroy-should-destroy-self-before-assocation
-      belongs_to :sash, class_name: 'Merit::Sash', inverse_of: nil
-      attr_accessible :sash if show_attr_accessible?
+      belongs_to :sash, class_name: 'Merit::Sash', inverse_of: nil, optional: true
 
-      send :"_merit_#{Merit.orm}_specific_config"
       _merit_delegate_methods_to_sash
       _merit_define_badge_related_entries_method
       _merit_sash_initializer
@@ -23,26 +21,11 @@ module Merit
       methods.each { |method| delegate method, to: :_sash }
     end
 
-    def _merit_active_record_specific_config
-    end
-
-    def _merit_mongoid_specific_config
-      field :level, type: Integer, default: 0
-      def find_by_id(id)
-        where(_id: id).first
-      end
-    end
-
     def _merit_define_badge_related_entries_method
       meritable_class_name = name.demodulize
       Merit::Badge._define_related_entries_method(meritable_class_name)
     end
 
-    def show_attr_accessible?
-      defined?(ProtectedAttributes) ||
-        !defined?(ActionController::StrongParameters)
-    end
-
     # _sash initializes a sash if doesn't have one yet.
     # From Rails 3.2 we can override association methods to do so
     # transparently, but merit supports Rails ~> 3.0.0. See:
@@ -50,7 +33,7 @@ module Merit
     def _merit_sash_initializer
       define_method(:_sash) do
         # TODO: reload.sash is not regression tested
-        sash || reload.sash || update_attributes(sash: Sash.create)
+        sash || reload.sash || update(sash: Sash.create)
         sash
       end
     end

data/lib/merit/controller_extensions.rb

@@ -4,18 +4,19 @@ module Merit
   # 'controller_path#action_name'
   module ControllerExtensions
     def self.included(base)
-      base.after_filter do |controller|
-        if rules_defined?
-          log_merit_action
-          Merit::Action.check_unprocessed if Merit.checks_on_each_request
-        end
-      end
+      base.after_action :log_and_process
     end
 
     private
 
-    def log_merit_action
-      Merit::Action.create(merit_action_hash)
+    def log_and_process
+      if rules_defined?
+        Merit::Action.create(merit_action_hash)
+
+        if Merit.checks_on_each_request
+          Merit::Action.check_unprocessed
+        end
+      end
     end
 
     def merit_action_hash
@@ -26,16 +27,8 @@ module Merit
         had_errors:    had_errors?,
         target_model:  controller_path,
         target_id:     target_id,
-      }.merge(target_data_hash)
-    end
-
-    def target_data_hash
-      if Merit::Action.new.respond_to?(:target_data)
-        { target_data: target_object.to_yaml }
-      else
-        Merit.upgrade_target_data_warning
-        {}
-      end
+        target_data:   target_object.to_yaml,
+      }
     end
 
     def rules_defined?
@@ -68,21 +61,11 @@ module Merit
 
     def target_id
       target_id = target_object.try(:id)
-      # If target_id is nil
-      # then use params[:id].
-      if target_id.nil? && send("check_#{Merit.orm}_id", params[:id])
+      # If target_id is nil use (only digits of) params[:id]
+      if target_id.nil? && params[:id].to_s =~ /^[0-9]+$/
         target_id = params[:id]
       end
       target_id
     end
-
-    # This check avoids trying to set a slug as integer FK
-    def check_active_record_id(id)
-      id.to_s =~ /^[0-9]+$/
-    end
-
-    def check_mongoid_id(id)
-      id.to_s =~ /^[0-9a-fA-F]{24}$/
-    end
   end
 end

data/lib/{generators → merit/generators}/active_record/install_generator.rb

@@ -1,8 +1,8 @@
 require 'rails/generators/active_record'
 
-module ActiveRecord
-  module Generators
-    class InstallGenerator < Rails::Generators::Base
+module Merit
+  module Generators::ActiveRecord
+    class InstallGenerator < ::Rails::Generators::Base
       include Rails::Generators::Migration
 
       source_root File.expand_path('../templates', __FILE__)
@@ -13,21 +13,25 @@ module ActiveRecord
       end
 
       def copy_migrations_and_model
-        migration_template 'create_merit_actions.rb',
+        migration_template 'create_merit_actions.erb',
                            'db/migrate/create_merit_actions.rb'
 
-        migration_template 'create_merit_activity_logs.rb',
+        migration_template 'create_merit_activity_logs.erb',
                            'db/migrate/create_merit_activity_logs.rb'
 
-        migration_template 'create_sashes.rb',
+        migration_template 'create_sashes.erb',
                            'db/migrate/create_sashes.rb'
 
-        migration_template 'create_badges_sashes.rb',
+        migration_template 'create_badges_sashes.erb',
                            'db/migrate/create_badges_sashes.rb'
 
-        migration_template 'create_scores_and_points.rb',
+        migration_template 'create_scores_and_points.erb',
                            'db/migrate/create_scores_and_points.rb'
       end
+
+      def migration_version
+        "[#{Rails::VERSION::MAJOR}.#{Rails::VERSION::MINOR}]"
+      end
     end
   end
 end