canvas_sync 0.18.6 → 0.18.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ef5adc65f2560567bff6cb71e817e646fe6f3d727cc0def5ec3b8cdee798131e
-  data.tar.gz: 8380125f3523e3f08861c1af47e4b81178955ed215a2c0b0f0f63a06921d6f45
+  metadata.gz: 33a5018a59c29aa95670c467beba015277c9991bc7767987ea6f29f1444f18fa
+  data.tar.gz: 585cb02b25db814d71f4b6626142e3b85965f6bdcf7547cf879f0094cfc3dad2
 SHA512:
-  metadata.gz: fe874e5e55c77e566f9cce71981e55c2482cd72740ae07c4b9dddeae5bd971df5b694f08bc9479d7096256da8eca9a5f169d4c964b4b733e6e872f069de563d8
-  data.tar.gz: ab9d9a36d9eae58eff042131b5f8166908e7690482a04de2e2f14f7086a09227c919c4e02f9b012ba3baccdd929c93c06c7029fc9494907c6ab55ea85899a88e
+  metadata.gz: '0618fa0863292e0b97a68d237037583d61da3e40c1f9ae4596b3f1198fc9d664ff49f9dc5d8eabc2c397dc84d7753bbed30a7cabda5b4588e1fbb89bbdb1d854'
+  data.tar.gz: 0d9b2156a11c1eb1eeadb894f81accf0f75bf92166ce1fec0c8eacdf2a657fb5d64dc8a20422fd649a99d81a7fc933c632376ac73dbec48b73a3a8a53c56c2c6

data/lib/canvas_sync/concerns/api_syncable.rb

@@ -90,10 +90,12 @@ module CanvasSync::Concerns
       private
 
       def api_sync_race_create!(inst, save: true)
-        inst = find_or_initialize_by(canvas_id: inst) unless inst.is_a?(self)
-        yield inst
-        inst.save! if save && inst.changed?
-        inst
+        transaction(requires_new: true) do
+          inst = find_or_initialize_by(canvas_id: inst) unless inst.is_a?(self)
+          yield inst
+          inst.save! if save && inst.changed?
+          inst
+        end
       rescue ActiveRecord::RecordNotUnique
         inst = find_by(canvas_id: inst.canvas_id)
         yield inst

data/lib/canvas_sync/job_batches/batch.rb

@@ -45,7 +45,7 @@ module CanvasSync
       redis_attr :created_at
       redis_attr :callback_queue, read_only: false
       redis_attr :callback_params, :json
-      redis_attr :allow_context_changes
+      redis_attr :allow_context_changes, :bool
 
       def context
         return @context if defined?(@context)

data/lib/canvas_sync/job_batches/jobs/managed_batch_job.rb

@@ -20,7 +20,7 @@ module CanvasSync
           Batch.redis do |r|
             r.multi do |r|
               r.hset("MNGBID-#{man_batch_id}", "root_bid", root_batch.bid)
-              r.hset("MNGBID-#{man_batch_id}", "ordered", ordered)
+              r.hset("MNGBID-#{man_batch_id}", "ordered", ordered ? 1 : 0)
               r.hset("MNGBID-#{man_batch_id}", "concurrency", concurrency)
               r.expire("MNGBID-#{man_batch_id}", Batch::BID_EXPIRE_TTL)
 
@@ -93,6 +93,8 @@ module CanvasSync
           end
         end
 
+        ordered = CanvasSync::MiscHelper.to_boolean(ordered)
+
         next_job_json = Batch.redis do |r|
           if ordered
             r.lpop("MNGBID-#{man_batch_id}-jobs")

data/lib/canvas_sync/job_batches/redis_model.rb

@@ -41,13 +41,13 @@ module CanvasSync
         if @initialized || @existing
           redis do |r|
             r.multi do |r|
-              r.hset(redis_key, attribute, value)
+              r.hset(redis_key, attribute, value.to_s)
               r.expire(redis_key, Batch::BID_EXPIRE_TTL)
             end
           end
         else
           @pending_attrs ||= {}
-          @pending_attrs[attribute] = value
+          @pending_attrs[attribute] = value.to_s
         end
       end
 

data/lib/canvas_sync/version.rb

@@ -1,3 +1,3 @@
 module CanvasSync
-  VERSION = "0.18.6".freeze
+  VERSION = "0.18.8".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: canvas_sync
 version: !ruby/object:Gem::Version
-  version: 0.18.6
+  version: 0.18.8
 platform: ruby
 authors:
-- Nate Collings
+- Instructure CustomDev
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-07-14 00:00:00.000000000 Z
+date: 2023-07-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -390,7 +390,7 @@ dependencies:
         version: '0'
 description:
 email:
-- ncollings@instructure.com
+- pseng@instructure.com
 executables: []
 extensions: []
 extra_rdoc_files: []