render_turbo_stream 1.4.10 → 1.4.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 64b60c79e4190078560e1f08d175acfe17563316c6bf6079401841e9ac316198
-  data.tar.gz: 42d0f14d62ad08acb537658b2b83b113297e464da77ac95a33d41b9667e6da6a
+  metadata.gz: 344d9c5276c13acd3cbde921821be8d224befaca25d1492af766c45cb80f7993
+  data.tar.gz: b7a86cd99b8d5808590c5ed14032f8275c3ae284e90e9682c5b4c2b8dcec332e
 SHA512:
-  metadata.gz: b47e40f983bff09126f2307f0b3f40e64a16132899cdb032573b4fcfa6ab1f9fb01256a8237eb15b17ce0c1e250a25e2e9f18af7efbdc14fd94f2ca4882c5b5c
-  data.tar.gz: 4e0c932e70b68297d69aecdda5259abc5bcd11e0c355c9c12b27dcda204d656f90680b97b5bff2472cda64d9828d0b51c9cadf3e58415acf155f034407a9a95d
+  metadata.gz: ee892284af2df779af7051f5e8c3f8132595657997429b1ba0d6a8585460a5ed087620c1c5182a786fe5a96c56f6968fc9c192db826f95a00b76a2d6d3ca14a7
+  data.tar.gz: b74f6487ea7f28a854b98d088e547b80ab67cfb2bbfd1bb62dab24d9d36f2f6b78a42b7e1e78f95ff8c91930632a87fd37a3f3a9d8275b2a418b8a650931435e

data/README.md

@@ -48,7 +48,7 @@ config.x.render_turbo_stream.flash_id = 'flash-box'
 config.x.render_turbo_stream.flash_action = 'prepend'
 ```
 
-The corresponding partials for flashes could look [like this](https://gitlab.com/sedl/renderturbostream/-/wikis/flashes)
+The corresponding partials for flashes could look [like this](https://gitlab.com/sedl/renderturbostream/-/wikis/Flashes-example)
 
 **Translations**
 
@@ -142,13 +142,12 @@ Under the hood, inside a `*.turbo_stream.erb` template, it does the following: `
 
 **WORKAROUND for redirects inside frame**
 
-Suppose you have a CRUD controller that should do all its actions inside a turbo frame. Classic redirect_to, along with a Flash message, would raise «Render and/or redirect were called multiple times in this action». See [issue](https://gitlab.com/sedl/renderturbostream/-/issues/3).
+Suppose you have a CRUD controller that should do all its actions inside a turbo frame. Classic redirect_to, together with a turbo_stream action on the same response, would raise «Render and/or redirect were called multiple times in this action». See [issue](https://gitlab.com/sedl/renderturbostream/-/issues/3).
 
-You can handle this by reloading the parent frame's src attribute. The code might look like this:
+A workaround is to use the src attribute of the parent frame. `turbo_frame_set_src` comes from turbo_power. The code could look like this:
 
 ```ruby
 def update
-    @car = Car.find(params['id'])
     turbo_stream_save(
       @car.update(car_params),
       streams_on_success: [

data/lib/render_turbo_stream/version.rb

@@ -1,3 +1,3 @@
 module RenderTurboStream
-  VERSION = "1.4.10"
+  VERSION = "1.4.12"
 end

data/lib/render_turbo_stream.rb

@@ -160,9 +160,10 @@ module RenderTurboStream
     ary = []
     array.each do |pr|
       if !pr.present?
-        a = 1
+        Rails.logger.warn "  WARNING render_turbo_stream: Empty element inside attributes: «#{array}»"
       elsif pr.is_a?(Hash)
         props = pr.symbolize_keys
+        raise "missing attribute :id in #{props}" if !props[:id].present?
         part = (props[:partial].present? ? props[:partial] : props[:id]).gsub('-', '_')
         partial = (part.to_s.include?('/') ? part : [controller_path, part].join('/'))
         r = props
@@ -170,6 +171,7 @@ module RenderTurboStream
         r[:partial] = partial
         ary.push(r)
       elsif pr.is_a?(Array)
+        raise "array has to contain at least one element: #{pr}" unless pr.first.present?
         ary.push(pr)
       else
         raise "ERROR render_turbo_stream invalid type: Only hash or array allowed"
@@ -289,7 +291,7 @@ module RenderTurboStream
         :append,
         :prepend
       ]
-      (config.present? ? config : default).map{|m|m.to_sym}.include?(method.to_sym)
+      (config.present? ? config : default).map { |m| m.to_sym }.include?(method.to_sym)
     end
 
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: render_turbo_stream
 version: !ruby/object:Gem::Version
-  version: 1.4.10
+  version: 1.4.12
 platform: ruby
 authors:
 - christian