canvas_sync 0.17.23.beta2 → 0.17.23.beta7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0d5da2edfe978cc9fec69714ce7a1bd90cce3977023478473a7999143e78df90
-  data.tar.gz: 510f341eb1d090275b302e971a6f26c59e046da7699cfaee567a34c23ce751c8
+  metadata.gz: 4ffd1eae28ffc49cdba7673f1648fbf82853a57768697df38d2a100c05c7ab3f
+  data.tar.gz: 4ab9d6f8c7ce264b69fae4da982d16dfedf40e5b1d1f64fbfd40831f16465bdf
 SHA512:
-  metadata.gz: 891a38e3bd240de7c5c248d14bc21f84bc1336a19fb1509e74769076d7059bc4e3a2b7b8764cafe3aba8b6127d2278241a30b8f78f0735467408c269710dab31
-  data.tar.gz: fc71f424836699e98cfa39ee435e919e81cb9bcc12deef30f4597806746ec8047136efc180db31ad906dc309d62526584fbdd17db04d2fe7b9b7495402f869a9
+  metadata.gz: 9619fbc749887158498c00761798cd0b10018ce5962262f27c9ff69c0615c66ffe1ebfa004bdc2ca7374308e53598ee81175f2372e2cde69527ce4672c288626
+  data.tar.gz: a9459bd07489e9c68abd692977e6ca1bc01b06d7dd960560f85e7f38bff4240ac8334aff15b91fb7f7a84897c7fcbd828e623d3b098fa34126b1b4e9be255222

data/config/initializers/apartment.rb

@@ -5,6 +5,15 @@
 #   * https://github.com/influitive/apartment/issues/508
 
 Rails.application.config.after_initialize do
+  next unless defined?(Apartment)
+
+  # Newer versions of Apartment already solves this issue (and in a better way)
+  begin
+    require('apartment/version')
+    next if Gem::Version.new(Apartment::VERSION) >= Gem::Version.new('2.8.1')
+  rescue LoadError
+  end
+
   begin
     Rails.application.eager_load!
     ActiveRecord::Base.descendants.each do |model|

data/lib/canvas_sync/concerns/sync_mapping.rb

@@ -24,9 +24,11 @@ module CanvasSync::Concerns
     class Mapping
       attr_reader :map_def
 
-      def initialize(map_def = {}, model: nil)
-        @map_def = map_def
+      def initialize(map_def, model: nil)
         @model = model
+        @map_def = map_def
+        @map_def[:conflict_target] ||= []
+        @map_def[:report_columns] ||= {}
       end
 
       def self.normalize_model_name(model)
@@ -82,7 +84,7 @@ module CanvasSync::Concerns
       end
 
       def reset_links
-        @map_def = {}
+        @map_def[:report_columns] = {}.with_indifferent_access
       end
 
       def unlink_column(key)

data/lib/canvas_sync/generators/templates/migrations/create_grading_period_groups.rb

@@ -4,8 +4,6 @@ class CreateGradingPeriodGroups < ActiveRecord::Migration[5.1]
   def change
     create_table :grading_period_groups do |t|
       t.bigint :canvas_id, null: false
-      t.bigint :canvas_course_id
-      t.bigint :canvas_account_id
       t.string :title
       t.boolean :weighted
       t.boolean :display_totals_for_all_grading_periods
@@ -16,7 +14,5 @@ class CreateGradingPeriodGroups < ActiveRecord::Migration[5.1]
     end
 
     add_index :grading_period_groups, :canvas_id, unique: true
-    add_index :grading_period_groups, :canvas_course_id
-    add_index :grading_period_groups, :canvas_account_id
   end
 end

data/lib/canvas_sync/generators/templates/models/grading_period_group.rb

@@ -4,6 +4,4 @@ class GradingPeriodGroup < ApplicationRecord
   include CanvasSync::Record
 
   validates :canvas_id, uniqueness: true, presence: true
-  belongs_to :course, primary_key: :canvas_id, foreign_key: :canvas_course_id, optional: true
-  belongs_to :account, primary_key: :canvas_id, foreign_key: :canvas_account_id, optional: true
 end

data/lib/canvas_sync/generators/templates/models/term.rb

@@ -6,6 +6,7 @@ class Term < ApplicationRecord
 
   validates :canvas_id, uniqueness: true, presence: true
   has_many :courses, foreign_key: :canvas_term_id, primary_key: :canvas_id
+  belongs_to :grading_period_group, primary_key: :canvas_id, foreign_key: :grading_period_group_id, optional: true
 
   api_syncable({
     canvas_id: :id,

data/lib/canvas_sync/jobs/sync_terms_job.rb

@@ -9,7 +9,7 @@ module CanvasSync
       #   will be started for each term in that scope. :models should be an array of
       #   models to sync.
       def perform(options)
-        CanvasSync.get_canvas_sync_client(batch_context).terms("self").all_pages!.each do |term_params|
+        CanvasSync.get_canvas_sync_client(batch_context).terms("self", workflow_state: ['all']).all_pages!.each do |term_params|
           if account_id = batch_context[:account_id]
             # These branches are primarily to support Legacy apps
             if Term.respond_to?(:create_or_update) && Term.method(:create_or_update).arity.abs == 2

data/lib/canvas_sync/processors/model_mappings.yml

@@ -457,12 +457,6 @@ grading_period_groups:
     grading_period_group_id:
       database_column_name: canvas_id
       type: integer
-    canvas_course_id:
-      database_column_name: canvas_course_id
-      type: integer
-    canvas_account_id:
-      database_column_name: canvas_account_id
-      type: integer
     title:
       database_column_name: title
       type: string

data/lib/canvas_sync/processors/report_processor.rb

@@ -5,11 +5,11 @@ module CanvasSync
     # Base report processing class
     class ReportProcessor
       def mapping
-        SyncMapping::Mapping.default_mappings
+        CanvasSync::Concerns::SyncMapping::Mapping.default_mappings
       end
 
       def mapping_for(model, key = nil)
-        model.try(:get_sync_mapping, key) || mapping[key || SyncMapping::Mapping.normalize_model_name(model)]
+        model.try(:get_sync_mapping, key) || mapping[key || CanvasSync::Concerns::SyncMapping::Mapping.normalize_model_name(model)]
       end
 
       def do_bulk_import(report_file_path, model, options: {}, mapping_key: nil)

data/lib/canvas_sync/version.rb

@@ -1,3 +1,3 @@
 module CanvasSync
-  VERSION = "0.17.23.beta2".freeze
+  VERSION = "0.17.23.beta7".freeze
 end

data/lib/canvas_sync.rb

@@ -179,7 +179,7 @@ module CanvasSync
       # These Models use the provisioning report, but are not term-scoped,
       # so we sync them outside of the term scoping to ensure work is not duplicated
       if term_scope.present?
-        models -= (first_provisioning_models = models & ['users', 'pseudonyms', 'user_observers'])
+        models -= (first_provisioning_models = models & ['users', 'pseudonyms', 'user_observers', 'grading_periods', 'grading_period_groups'])
         current_chain.insert(generate_provisioning_jobs(first_provisioning_models, options))
       end
 

data/spec/dummy/app/models/grading_period_group.rb

@@ -10,6 +10,4 @@ class GradingPeriodGroup < ApplicationRecord
   include CanvasSync::Record
 
   validates :canvas_id, uniqueness: true, presence: true
-  belongs_to :course, primary_key: :canvas_id, foreign_key: :canvas_course_id, optional: true
-  belongs_to :account, primary_key: :canvas_id, foreign_key: :canvas_account_id, optional: true
 end

data/spec/dummy/app/models/term.rb

@@ -12,6 +12,7 @@ class Term < ApplicationRecord
 
   validates :canvas_id, uniqueness: true, presence: true
   has_many :courses, foreign_key: :canvas_term_id, primary_key: :canvas_id
+  belongs_to :grading_period_group, primary_key: :canvas_id, foreign_key: :grading_period_group_id, optional: true
 
   api_syncable({
     canvas_id: :id,

data/spec/dummy/db/migrate/{20210907233331_create_grading_period_groups.rb → 20211001184920_create_grading_period_groups.rb}

@@ -10,8 +10,6 @@ class CreateGradingPeriodGroups < ActiveRecord::Migration[5.1]
   def change
     create_table :grading_period_groups do |t|
       t.bigint :canvas_id, null: false
-      t.bigint :canvas_course_id
-      t.bigint :canvas_account_id
       t.string :title
       t.boolean :weighted
       t.boolean :display_totals_for_all_grading_periods
@@ -22,7 +20,5 @@ class CreateGradingPeriodGroups < ActiveRecord::Migration[5.1]
     end
 
     add_index :grading_period_groups, :canvas_id, unique: true
-    add_index :grading_period_groups, :canvas_course_id
-    add_index :grading_period_groups, :canvas_account_id
   end
 end

data/spec/dummy/db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2021_09_07_233331) do
+ActiveRecord::Schema.define(version: 2021_10_01_184920) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -175,16 +175,12 @@ ActiveRecord::Schema.define(version: 2021_09_07_233331) do
 
   create_table "grading_period_groups", force: :cascade do |t|
     t.bigint "canvas_id", null: false
-    t.bigint "canvas_course_id"
-    t.bigint "canvas_account_id"
     t.string "title"
     t.boolean "weighted"
     t.boolean "display_totals_for_all_grading_periods"
     t.string "workflow_state"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
-    t.index ["canvas_account_id"], name: "index_grading_period_groups_on_canvas_account_id"
-    t.index ["canvas_course_id"], name: "index_grading_period_groups_on_canvas_course_id"
     t.index ["canvas_id"], name: "index_grading_period_groups_on_canvas_id", unique: true
   end