panda_pal 5.9.8 → 5.9.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2b12b3b77cb30daf1aef73f0e798f1e6e4087ed55f26749252815ac7f6e79d36
-  data.tar.gz: 0f2d3408e91d3df0a20d16d6b35a302aa319c0dc2c1fb33dae38baf859d3dc5f
+  metadata.gz: 16bcf4f72e56b52fdec3f6e6b537e91225ea9effba4d091876a505be4a5a2743
+  data.tar.gz: 5de224cfbc6895b735a4e26e427e470b8f8fe42bc5c486a3837e0b2c79ba2ba7
 SHA512:
-  metadata.gz: 8b9d7af482b7a0a6db6a8403f7f23380e7939b7bc09a5312e8af255a4caf8ac8a7e011f08f65949c42aaa31c0e28977d37b04ec5e64090d36cb8f49ad6efafd2
-  data.tar.gz: e93bc45ce0ae92dadece7844a14c259baae692a4b10c112699af1f3129c26ca706da9ef75c0d26547f69b9b991604e6bc7897c39b42416472cc3758914c03e90
+  metadata.gz: d6cc863556e1acd683b74a765aef84493004fb3af25dade1f6dafac035523fa63009c929a58cd0129d5a7899673ded1fb5170dff56da81e3d628969675c7e58d
+  data.tar.gz: 5f0f6c2d59ed052b0494a7d983bfb13550a6dffa1a1b4e2410eaa828fd58c143337f58739423fecbbe023df4ff64542f8af42a0fca6090f64284d44bb55d05d8

data/app/models/panda_pal/session.rb

@@ -6,9 +6,25 @@ module PandaPal
       self.session_key ||= SecureRandom.urlsafe_base64(60)
     end
 
+    def [](key)
+      if column_names.include?(key.to_s)
+        super
+      else
+        data[key]
+      end
+    end
+
+    def []=(key, value)
+      if column_names.include?(key.to_s)
+        super
+      else
+        data[key] = value
+      end
+    end
+
     def cache(key, &blk)
-      panda_pal_session[:cache] ||= {}
-      panda_pal_session[:cache][key] ||= blk.call
+      data[:cache] ||= {}
+      data[:cache][key] ||= blk.call
     end
 
     def launch_params

data/lib/panda_pal/concerns/ability_helper.rb

@@ -8,8 +8,15 @@ module PandaPal::Concerns
     def panda_pal_session
       @panda_pal_session ||= begin
                                raise "Ability class needs to set @panda_pal_session or @controller to use this feature" unless @controller.present?
-                               @controller.current_session
+                               @controller.current_session(create_missing: false)
                              end
+
+      if @panda_pal_session.is_a?(Hash)
+        # This is a breaking-change to CanvasSync, but not to PandaPal
+        raise "Ability#panda_pal_session/@panda_pal_session should now return a PandaPal::Session object, not a Hash"
+      end
+
+      @panda_pal_session
     end
 
     def rails_session
@@ -19,16 +26,15 @@ module PandaPal::Concerns
                          end
     end
 
+    # Delegate legacy AbilityHelper methods to panda_pal_session. TODO Should such direct usage be considered deprecated?
+    delegate :launch_params, :get_lti_cust_param, :lti_launch_placement, :canvas_site_admin?, :lti_roles, :canvas_role_labels, :canvas_account_role_labels, to: :panda_pal_session
+
     def is_lti_launch?
       panda_pal_session.present?
     end
 
-    def launch_params
-      @launch_params ||= panda_pal_session[:launch_params] || {}
-    end
-
-    def canvas_site_admin?
-      panda_pal_session.canvas_site_admin?
+    def cache_on_session(*args, **kwargs, &blk)
+      panda_pal_session.cache(*args, **kwargs, &blk)
     end
   end
 end

data/lib/panda_pal/helpers/session_replacement.rb

@@ -60,6 +60,7 @@ module PandaPal::Helpers
       @current_session
     end
 
+    # @deprecated
     def current_session_data
       current_session.data
     end

data/lib/panda_pal/version.rb

@@ -1,3 +1,3 @@
 module PandaPal
-  VERSION = "5.9.8"
+  VERSION = "5.9.9"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: panda_pal
 version: !ruby/object:Gem::Version
-  version: 5.9.8
+  version: 5.9.9
 platform: ruby
 authors:
 - Instructure CustomDev
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-05-15 00:00:00.000000000 Z
+date: 2024-05-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -282,7 +282,7 @@ dependencies:
     - - '='
       - !ruby/object:Gem::Version
         version: 2.7.1
-description:
+description: 
 email:
 - pseng@instructure.com
 executables: []
@@ -394,7 +394,7 @@ homepage: http://instructure.com
 licenses:
 - MIT
 metadata: {}
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -409,52 +409,52 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.4
-signing_key:
+rubygems_version: 3.1.6
+signing_key: 
 specification_version: 4
 summary: LTI mountable engine
 test_files:
+- spec/rails_helper.rb
+- spec/models/panda_pal/api_call_spec.rb
+- spec/models/panda_pal/organization_spec.rb
+- spec/models/panda_pal/session_spec.rb
+- spec/models/panda_pal/organization/settings_validation_spec.rb
+- spec/models/panda_pal/organization/task_scheduling_spec.rb
 - spec/controllers/panda_pal/api_call_controller_spec.rb
-- spec/dummy/README.rdoc
+- spec/spec_helper.rb
+- spec/dummy/db/schema.rb
+- spec/dummy/public/422.html
+- spec/dummy/public/favicon.ico
+- spec/dummy/public/404.html
+- spec/dummy/public/500.html
 - spec/dummy/Rakefile
-- spec/dummy/app/assets/javascripts/application.js
-- spec/dummy/app/assets/stylesheets/application.css
+- spec/dummy/app/views/layouts/application.html.erb
 - spec/dummy/app/controllers/application_controller.rb
 - spec/dummy/app/helpers/application_helper.rb
-- spec/dummy/app/views/layouts/application.html.erb
-- spec/dummy/bin/bundle
-- spec/dummy/bin/rails
-- spec/dummy/bin/rake
-- spec/dummy/bin/setup
+- spec/dummy/app/assets/stylesheets/application.css
+- spec/dummy/app/assets/javascripts/application.js
+- spec/dummy/config/secrets.yml
 - spec/dummy/config/application.rb
-- spec/dummy/config/boot.rb
-- spec/dummy/config/database.yml
-- spec/dummy/config/environment.rb
-- spec/dummy/config/environments/development.rb
-- spec/dummy/config/environments/production.rb
-- spec/dummy/config/environments/test.rb
-- spec/dummy/config/initializers/backtrace_silencers.rb
-- spec/dummy/config/initializers/cookies_serializer.rb
-- spec/dummy/config/initializers/filter_parameter_logging.rb
-- spec/dummy/config/initializers/inflections.rb
-- spec/dummy/config/initializers/mime_types.rb
 - spec/dummy/config/initializers/session_store.rb
 - spec/dummy/config/initializers/wrap_parameters.rb
-- spec/dummy/config/locales/en.yml
+- spec/dummy/config/initializers/inflections.rb
+- spec/dummy/config/initializers/filter_parameter_logging.rb
+- spec/dummy/config/initializers/mime_types.rb
+- spec/dummy/config/initializers/backtrace_silencers.rb
+- spec/dummy/config/initializers/cookies_serializer.rb
+- spec/dummy/config/environments/development.rb
+- spec/dummy/config/environments/test.rb
+- spec/dummy/config/environments/production.rb
+- spec/dummy/config/database.yml
+- spec/dummy/config/boot.rb
 - spec/dummy/config/routes.rb
-- spec/dummy/config/secrets.yml
+- spec/dummy/config/environment.rb
+- spec/dummy/config/locales/en.yml
+- spec/dummy/README.rdoc
 - spec/dummy/config.ru
-- spec/dummy/db/schema.rb
-- spec/dummy/public/404.html
-- spec/dummy/public/422.html
-- spec/dummy/public/500.html
-- spec/dummy/public/favicon.ico
+- spec/dummy/bin/bundle
+- spec/dummy/bin/rake
+- spec/dummy/bin/setup
+- spec/dummy/bin/rails
 - spec/factories/panda_pal_organizations.rb
 - spec/factories/panda_pal_sessions.rb
-- spec/models/panda_pal/api_call_spec.rb
-- spec/models/panda_pal/organization/settings_validation_spec.rb
-- spec/models/panda_pal/organization/task_scheduling_spec.rb
-- spec/models/panda_pal/organization_spec.rb
-- spec/models/panda_pal/session_spec.rb
-- spec/rails_helper.rb
-- spec/spec_helper.rb