active_recall 1.2.1 → 1.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9fd2908666f48d08173c20837e851db9ebd7a7a8b10d2f565deb16d84b013d60
-  data.tar.gz: e5028ad1e24d606adbd87aa0b568b9325f9a1cf33c68969a1b3ec188934ae15e
+  metadata.gz: 1784eaa3c18a644a3b7b16935626de436851355d6ec1f0ac19367ef2d789a873
+  data.tar.gz: '08e4a478783a55852dbefa1d0f6703ab2e44b4c02cd493e7091ff8cd715f4f43'
 SHA512:
-  metadata.gz: 2bba9aa5339b214a51255a04d0c63c2d518f130f376166c742a7c0267923c778279b760dd0292cb80157d0fad3f7ade2155f52cef2403f79f53c418681ac273c
-  data.tar.gz: '09d5aa80737619a70d387527bf214ad39dcaaf1def5c5c28b3db4384fee1be5f855454137463dbdb19aabbdfab63b785ec3ce20daf82bceed647221b0105b719'
+  metadata.gz: d80c727f175317bb7173721981dba6eaea93e8e6a13793e3fc38ffebf0215b998c58904b936c6fb9cbd9784a3a6641ce35016402e6b85ef335f8df3c8bf622a5
+  data.tar.gz: e645b4e3cd9ee513885f3a766d7d0566efa12bae1be14e0c3593c130bc6a10083544ffdb1e71a22e9a0cd313f1415811aa9fec3c734c9839c677a367f21e91be

data/.github/workflows/tests.yml

@@ -0,0 +1,46 @@
+name: Tests 
+
+on:
+  pull_request:
+    branches:
+      - master
+  push:
+    branches:
+      - master
+
+jobs:
+  unit_tests:
+    name: Unit Tests
+    if: "contains(github.event.commits[0].message, '[ci skip]') == false"
+    strategy:
+      fail-fast: false
+      matrix:
+        os:
+          - macos 
+          - ubuntu
+        ruby:
+          - 2.5
+          - 2.6
+          - 2.7
+          - 3.0
+        allow_failures:
+          - false
+        include:
+          - os: ubuntu
+            ruby: ruby-head
+            allow_failures: false
+    env:
+      BUNDLE_GEMFILE: "${{ matrix.gemfile }}"
+      ALLOW_FAILURES: "${{ matrix.allow_failures }}"
+    runs-on: ${{ matrix.os }}-latest
+    continue-on-error: ${{ endsWith(matrix.ruby, 'head') || matrix.ruby == 'debug' }}
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2
+      - name: Setup Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+          bundler-cache: true
+      - name: Test
+        run: bundle exec rake spec || $ALLOW_FAILURES

data/.ruby-version

@@ -1 +1 @@
-2.5.3
+3.0.0

data/Gemfile

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 source 'https://rubygems.org'
-ruby '2.5.3'
 
 git_source(:github) { |repo_name| "https://github.com/#{repo_name}" }
 

data/Gemfile.lock

@@ -1,30 +1,30 @@
 PATH
   remote: .
   specs:
-    active_recall (1.2.1)
-      activerecord (>= 5.2.3)
-      activesupport (>= 5.2.3)
+    active_recall (1.6.0)
+      activerecord (>= 5.2.3, <= 6.1)
+      activesupport (>= 5.2.3, <= 6.1)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    activemodel (6.0.0)
-      activesupport (= 6.0.0)
-    activerecord (6.0.0)
-      activemodel (= 6.0.0)
-      activesupport (= 6.0.0)
-    activesupport (6.0.0)
+    activemodel (6.1.0)
+      activesupport (= 6.1.0)
+    activerecord (6.1.0)
+      activemodel (= 6.1.0)
+      activesupport (= 6.1.0)
+    activesupport (6.1.0)
       concurrent-ruby (~> 1.0, >= 1.0.2)
-      i18n (>= 0.7, < 2)
-      minitest (~> 5.1)
-      tzinfo (~> 1.1)
-      zeitwerk (~> 2.1, >= 2.1.8)
-    concurrent-ruby (1.1.5)
+      i18n (>= 1.6, < 2)
+      minitest (>= 5.1)
+      tzinfo (~> 2.0)
+      zeitwerk (~> 2.3)
+    concurrent-ruby (1.1.8)
     diff-lcs (1.3)
-    i18n (1.7.0)
+    i18n (1.8.9)
       concurrent-ruby (~> 1.0)
-    minitest (5.12.2)
-    rake (10.5.0)
+    minitest (5.14.4)
+    rake (12.3.3)
     rdoc (6.2.0)
     rspec (3.8.0)
       rspec-core (~> 3.8.0)
@@ -39,25 +39,20 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.8.0)
     rspec-support (3.8.2)
-    sqlite3 (1.4.1)
-    thread_safe (0.3.6)
-    tzinfo (1.2.5)
-      thread_safe (~> 0.1)
-    zeitwerk (2.2.0)
+    sqlite3 (1.4.2)
+    tzinfo (2.0.4)
+      concurrent-ruby (~> 1.0)
+    zeitwerk (2.4.2)
 
 PLATFORMS
   ruby
 
 DEPENDENCIES
   active_recall!
-  bundler (>= 1.16)
-  rake (~> 10.0)
+  rake (~> 12.0)
   rdoc
   rspec (~> 3.0)
   sqlite3
 
-RUBY VERSION
-   ruby 2.5.3p105
-
 BUNDLED WITH
-   1.17.3
+   2.2.3

data/README.md

@@ -1,8 +1,8 @@
-# ActiveRecall [![Build Status](https://travis-ci.org/jaysonvirissimo/active_recall.svg?branch=master)](https://travis-ci.org/jaysonvirissimo/active_recall)
+# ActiveRecall
 
 **ActiveRecall** is a spaced-repetition system that allows you to treat arbitrary [ActiveRecord](https://github.com/rails/rails/tree/master/activerecord) models as if they were flashcards to be learned and reviewed.
 It it based on, and is intended to be backwards compatible with, the [okubo](https://github.com/rgravina/okubo) gem.
-The primary differentiating features are that it lets the user specify the scheduling algorithm and is fully compatible with Rails 6.
+The primary differentiating features are that it lets the user specify the scheduling algorithm and is fully compatible with Rails 6.0 and Ruby 3.0.
 
 ## Installation
 

data/active_recall.gemspec

@@ -32,11 +32,10 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ['lib']
 
-  spec.add_development_dependency 'bundler', '>= 1.16'
-  spec.add_development_dependency 'rake', '~> 10.0'
+  spec.add_development_dependency 'rake', '~> 12.0'
   spec.add_development_dependency 'rdoc'
   spec.add_development_dependency 'rspec', '~> 3.0'
   spec.add_development_dependency 'sqlite3'
-  spec.add_runtime_dependency 'activerecord', '>= 5.2.3'
-  spec.add_runtime_dependency 'activesupport', '>= 5.2.3'
+  spec.add_runtime_dependency 'activerecord', '>= 5.2.3', '<= 6.1'
+  spec.add_runtime_dependency 'activesupport', '>= 5.2.3', '<= 6.1'
 end

data/lib/active_recall.rb

@@ -10,7 +10,7 @@ require 'active_recall/models/deck'
 require 'active_recall/models/item'
 require 'active_recall/version'
 
-ActiveRecord::Base.send(:include, ActiveRecall::Base)
+ActiveRecord::Base.include ActiveRecall::Base
 
 module ActiveRecall
   class << self

data/lib/active_recall/algorithms/fibonacci_sequence.rb

@@ -2,7 +2,7 @@
 
 module ActiveRecall
   class FibonacciSequence
-    def self.right(box:, current_time: Time.current, times_right:, times_wrong:)
+    def self.right(box:, times_right:, times_wrong:, current_time: Time.current)
       new(
         box: box,
         current_time: current_time,
@@ -11,7 +11,7 @@ module ActiveRecall
       ).right
     end
 
-    def self.wrong(box:, current_time: Time.current, times_right:, times_wrong:)
+    def self.wrong(box:, times_right:, times_wrong:, current_time: Time.current)
       new(
         box: box,
         current_time: current_time,
@@ -20,7 +20,7 @@ module ActiveRecall
       ).wrong
     end
 
-    def initialize(box:, current_time: Time.current, times_right:, times_wrong:)
+    def initialize(box:, times_right:, times_wrong:, current_time: Time.current)
       @box = box
       @current_time = current_time
       @times_right = times_right

data/lib/active_recall/algorithms/leitner_system.rb

@@ -4,7 +4,7 @@ module ActiveRecall
   class LeitnerSystem
     DELAYS = [3, 7, 14, 30, 60, 120, 240].freeze
 
-    def self.right(box:, current_time: Time.current, times_right:, times_wrong:)
+    def self.right(box:, times_right:, times_wrong:, current_time: Time.current)
       new(
         box: box,
         current_time: current_time,
@@ -13,7 +13,7 @@ module ActiveRecall
       ).right
     end
 
-    def self.wrong(box:, current_time: Time.current, times_right:, times_wrong:)
+    def self.wrong(box:, times_right:, times_wrong:, current_time: Time.current)
       new(
         box: box,
         current_time: current_time,
@@ -22,7 +22,7 @@ module ActiveRecall
       ).wrong
     end
 
-    def initialize(box:, current_time: Time.current, times_right:, times_wrong:)
+    def initialize(box:, times_right:, times_wrong:, current_time: Time.current)
       @box = box
       @current_time = current_time
       @times_right = times_right

data/lib/active_recall/deck_methods.rb

@@ -3,10 +3,10 @@
 module ActiveRecall
   module DeckMethods
     def deck
-      d = ActiveRecall::Deck.where(user_id: id, user_type: self.class.name).first_or_create
+      d = ActiveRecall::Deck.find_or_create_by(user_id: id, user_type: self.class.name)
       d.source_class.module_eval do
         def stats
-          ActiveRecall::Item.where(source_id: id, source_type: self.class.name).first
+          ActiveRecall::Item.find_by(source_id: id, source_type: self.class.name)
         end
       end
       d

data/lib/active_recall/item_methods.rb

@@ -3,13 +3,13 @@
 module ActiveRecall
   module ItemMethods
     def right_answer_for!(item)
-      i = deck.items.where(source_id: item.id).first
+      i = deck.items.find_by(source_id: item.id)
       i.right!
       i.save!
     end
 
     def wrong_answer_for!(item)
-      i = deck.items.where(source_id: item.id).first
+      i = deck.items.find_by(source_id: item.id)
       i.wrong!
       i.save!
     end

data/lib/active_recall/models/deck.rb

@@ -37,8 +37,9 @@ module ActiveRecall
     end
 
     def delete(source)
-      item = ActiveRecall::Item.new(deck: self, source_id: source.id, source_type: source.class.name)
-      item.destroy
+      ActiveRecall::Item
+        .find_by(deck: self, source_id: source.id, source_type: source.class.name)
+        .destroy
     end
 
     def review

data/lib/active_recall/models/item.rb

@@ -5,7 +5,6 @@ module ActiveRecall
     self.table_name = 'active_recall_items'
 
     belongs_to :deck
-    belongs_to :source, polymorphic: true
 
     scope :failed, -> { where(['box = ? and last_reviewed is not null', 0]) }
     scope :untested, -> { where(['box = ? and last_reviewed is null', 0]) }
@@ -18,12 +17,16 @@ module ActiveRecall
       where(['box > ? and next_review > ?', 0, current_time])
     end
 
+    def source
+      source_type.constantize.find(source_id)
+    end
+
     def right!
-      update!(algorithm_class.right(scoring_attributes))
+      update!(algorithm_class.right(**scoring_attributes))
     end
 
     def wrong!
-      update!(algorithm_class.wrong(scoring_attributes))
+      update!(algorithm_class.wrong(**scoring_attributes))
     end
 
     private

data/lib/active_recall/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ActiveRecall
-  VERSION = '1.2.1'
+  VERSION = '1.6.0'
 end

data/lib/generators/active_recall/active_recall_generator.rb

@@ -20,9 +20,7 @@ class ActiveRecallGenerator < Rails::Generators::Base
   def create_migration_files
     create_migration_file_if_not_exist 'create_active_recall_tables'
     create_migration_file_if_not_exist 'add_active_recall_item_answer_counts'
-    if options['migrate_data']
-      create_migration_file_if_not_exist 'migrate_okubo_to_active_recall'
-    end
+    create_migration_file_if_not_exist 'migrate_okubo_to_active_recall' if options['migrate_data']
   end
 
   private

metadata

@@ -1,44 +1,30 @@
 --- !ruby/object:Gem::Specification
 name: active_recall
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 1.6.0
 platform: ruby
 authors:
 - Robert Gravina
 - Jayson Virissimo
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2019-12-16 00:00:00.000000000 Z
+date: 2021-02-28 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: bundler
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '1.16'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '1.16'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '12.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '12.0'
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement
@@ -88,6 +74,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 5.2.3
+    - - "<="
+      - !ruby/object:Gem::Version
+        version: '6.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -95,6 +84,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 5.2.3
+    - - "<="
+      - !ruby/object:Gem::Version
+        version: '6.1'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
@@ -102,6 +94,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 5.2.3
+    - - "<="
+      - !ruby/object:Gem::Version
+        version: '6.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -109,6 +104,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 5.2.3
+    - - "<="
+      - !ruby/object:Gem::Version
+        version: '6.1'
 description: A spaced-repetition system to be used with ActiveRecord models
 email:
 - robert.gravina@gmail.com
@@ -117,10 +115,10 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/tests.yml"
 - ".gitignore"
 - ".rspec"
 - ".ruby-version"
-- ".travis.yml"
 - Gemfile
 - Gemfile.lock
 - LICENSE
@@ -148,7 +146,7 @@ licenses:
 - MIT
 metadata:
   allowed_push_host: https://rubygems.org/
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -163,9 +161,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
-signing_key: 
+rubygems_version: 3.2.3
+signing_key:
 specification_version: 4
 summary: A spaced-repetition system
 test_files: []

data/.travis.yml

@@ -1,5 +0,0 @@
-language: ruby
-cache: bundler
-rvm:
-  - 2.5.3
-before_install: gem install bundler -v 1.17.3