LTI-190: Update handler_legacy if passed and nil (#195)

app/controllers/rooms_controller.rb

@@ -258,9 +258,14 @@ def set_launch
     # Continue through happy path.
     @tenant = session_params['tenant']
     resource_handler = Digest::SHA1.hexdigest("rooms#{@tenant}#{launch_params['resource_link_id']}")
-    @room = Room.find_or_create_by(handler: resource_handler, tenant: @tenant) do |room|
-      room.update(launch_params_to_new_room_params(launch_params))
+    to_new_room_params = launch_params_to_new_room_params(launch_params)
+    @room = Room.find_by(handler: resource_handler, tenant: @tenant)
+    if @room
+      @room.update(to_new_room_params)
+    else
+      @room = Room.create(to_new_room_params)
     end
+
     user_params = launch_params_to_new_user_params(launch_params)
     session[@room.handler] = { user_params: user_params }
   end

config/environments/production.rb

@@ -17,7 +17,7 @@
   # Settings specified here will take precedence over those in config/application.rb.
 
   # Code is not reloaded between requests.
-  config.cache_classes = true
+  config.cache_classes = (ENV['DEVELOPER_MODE_ENABLED'] != 'true')
 
   # Eager load code on boot. This eager loads most of Rails and
   # your application in memory, allowing both threaded web servers

db/schema.rb

@@ -30,8 +30,8 @@
     t.string "tenant"
     t.boolean "hide_name"
     t.boolean "hide_description"
-    t.jsonb "settings", default: {}, null: false
     t.string "handler_legacy"
+    t.jsonb "settings", default: {}, null: false
     t.index ["tenant", "handler"], name: "index_rooms_on_tenant_and_handler", unique: true
     t.index ["tenant", "handler_legacy"], name: "index_rooms_on_tenant_and_handler_legacy", unique: true
   end