render_turbo_stream 1.4.10 → 1.4.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 64b60c79e4190078560e1f08d175acfe17563316c6bf6079401841e9ac316198
-  data.tar.gz: 42d0f14d62ad08acb537658b2b83b113297e464da77ac95a33d41b9667e6da6a
+  metadata.gz: 6aaffdee6cb866e03e20f1b6d46e00d187e9f7a63b0d5f26c29b4615fae7b483
+  data.tar.gz: b031dbf2eac68543ddf75de146c58715abaf3e6c6e9852119e3beb4f049832cc
 SHA512:
-  metadata.gz: b47e40f983bff09126f2307f0b3f40e64a16132899cdb032573b4fcfa6ab1f9fb01256a8237eb15b17ce0c1e250a25e2e9f18af7efbdc14fd94f2ca4882c5b5c
-  data.tar.gz: 4e0c932e70b68297d69aecdda5259abc5bcd11e0c355c9c12b27dcda204d656f90680b97b5bff2472cda64d9828d0b51c9cadf3e58415acf155f034407a9a95d
+  metadata.gz: cec6a8701b17aabe2d76006771bd0441f28617a80bc16e9987d5334c84742d5df93471a7fb89d474b071fa2bd4f2391c78ea96abc3d8da8bc206198cf0268f64
+  data.tar.gz: 77afb83d8a217eca87b53cfa0aee768cdd8272052540758976c266d72f7d532119e9260473d761743b57de64e252ba41c48ef927685ee90122a0ee0300749a46

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.11"
 end

data/lib/render_turbo_stream.rb

@@ -159,10 +159,9 @@ module RenderTurboStream
 
     ary = []
     array.each do |pr|
-      if !pr.present?
-        a = 1
-      elsif pr.is_a?(Hash)
+      if 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 +169,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 +289,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.11
 platform: ruby
 authors:
 - christian