params_keeper_rails 1.1.2 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fcba9e0da62cef22f6a10be854b3f87576ca42959a07e73ba6f5db15b4cc3832
-  data.tar.gz: 9f34f3aff5de4d3aaa00b8c1091ac4d577e5eaba0bbe68d35f59af9a8aa720bc
+  metadata.gz: 7fe873eb03f68ceb7c4a7f3e222f58e3b293f9a72866305707498c5a8f6cb778
+  data.tar.gz: b6f41e0ff74d1a44a165bdbf9ba9368cfb0cd98b4add0dcbe80e4d906add36b1
 SHA512:
-  metadata.gz: c0cd32d51cbd4ff13e175ce8f9ada37458325c9d51e464569fc7969661ef29f1966f155bba0850de004ad11a773fd2b646a2cfcf3aff3cc488c0dd10a0364fb9
-  data.tar.gz: bf5d57f3cb6c783a6af545c0d22b183556ad0280b4708b9521b286f00cc065ed1f36bd07c296757e4817c8fc8d948ecbbff542643e7dcc7f50762b33a0d5d102
+  metadata.gz: ff2288172a6d89a2bcecd4e8e7ba21375232478cc41d4488451f9e44ebcfb9bd29f5cf08a2418083d8699c78dfea18d72e0a7567ab788d09100f9c0fb5db6bd4
+  data.tar.gz: dc765ac84630960b6f0d70dcc3de0d1bedc7e21592053eadc9f6f6e34e13142ac46d097c3735ef55667788024fe77aaa35ef6620198b2c6ae0cc8a0af4acfde6

data/.travis.yml

@@ -4,6 +4,7 @@ rvm:
   - 2.4
   - 2.5
   - 2.6
+  - 2.7
 gemfile:
   - gemfiles/rails50.gemfile
   - gemfiles/rails51.gemfile

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 # CHANGELOG
 
+## 1.2.0
+
+* Support multiple configs.
+* Support hidden fields for `form_with` with GET method. (Rails => 5.1)
+
 ## 1.1.2
 
 * Fix url_for helper outside of controller.

data/README.md

@@ -29,59 +29,87 @@ class ExamplesController < ApplicationController
   keep_params :key1, :key2
 ```
 
-Parameters are kept if destination controller is same as current controller.
+Parameters are kept via `url_for` if destination controller is same as current controller.
 For example:
 
 ```ruby
 GET "/examples?key1=**&key2=**"
 
-# hash
-url_for(action: :show)  #=> '/examples/:id?key1=**&key2=**'
+# hash arg
+url_for(action: :index)  #=> '/examples?key1=**&key2=**'
 
-# string and active model don't keep parameters by defalut
+# parameters are not kept if string or model arg is specified
 url_for('/examples')  #=> '/examples'
 url_for(@example)  #=> '/examples/:id'
 
 # parameters are not kept if destination controller is different from current controller
 url_for(controller: 'examples2', action: :index)  #=> '/examples2'
 
-# parameters are not kept if you set keep_params: false
+# parameters are not kept if you put "keep_params: false" into args
 url_for(action: :show, keep_params: false)  #=> '/examples/:id'
 ```
 
-Enable only specific class of url_for:
+Parameters are kept for form with GET method via hidden fields.
+This feature is supported by `form_with` for rails >= 5.1.
 
 ```ruby
-# enable only hash like url_for(action: :show)
+<%= form_with url: { action: :index}, method: :get do %>
+  <%= submit_tag 'submit' %>
+<% end %>
+#=> ...<input type="hidden" name="key1" value="**" />
+#      <input type="hidden" name="key2" value="**" /></form>
+```
+
+## Options
+
+### Argument type
+
+Enable only specific argument type of url_for:
+
+```ruby
+# hash arg (same as default behaviour)
 keep_params :key1, :key2, for: :hash
+url_for(action: :index)  #=> '/examples?key1=**&key2=**'
 
-# enable only string like url_for('/examples')
+# string arg
 keep_params :key1, :key2, for: :string
+url_for('/examples')  #=> '/examples?key1=**&key2=**'
 
-# enable only model like url_for(@example)
+# model arg
 keep_params :key1, :key2, for: :model
+url_for(@example)  #=> '/examples/:id?key1=**&key2=**'
 ```
 
-Specify default options of url_for:
+`:for` allows to set multiple argument type as follows:
 
 ```ruby
-keep_params :key1, :key2, url_options: { fixed_param: :something }
+keep_params :key1, :key2, for: [:hash, :model]
 ```
 
+### Multiple controllers
+
 Keep parameters throught multiple controllers:
 
 ```ruby
 class ExamplesController < ApplicationController
   include ParamsKeeper::Controller
-  keep_params :key1, :key2, to: %w(examples nested_examples)
+  keep_params :key1, :key2, to: %w(examples examples2)
 end
 
-class NestedExamplesController < ApplicationController
+class Examples2Controller < ApplicationController
   include ParamsKeeper::Controller
-  keep_params :key1, :key2, to: %w(examples nested_examples)
+  keep_params :key1, :key2, to: %w(examples examples2)
 end
 ```
 
+### Default parameters
+
+Specify default parameters:
+
+```ruby
+keep_params :key1, :key2, url_options: { fixed_param: :something }
+```
+
 ## Contributing
 
 Bug reports and pull requests are welcome on GitHub at https://github.com/kanety/params_keeper_rails.

data/lib/params_keeper/config.rb

@@ -0,0 +1,12 @@
+module ParamsKeeper
+  class Config
+    attr_accessor :keys, :to, :for, :url_options
+
+    def initialize(keys, options = {})
+      @keys = Array(keys)
+      @to = Array(options[:to])
+      @for = Array(options[:for] || :hash)
+      @url_options = options[:url_options] || {}
+    end
+  end
+end

data/lib/params_keeper/controller.rb

@@ -1,25 +1,33 @@
-module ParamsKeeper::Controller
-  extend ActiveSupport::Concern
+module ParamsKeeper
+  module Controller
+    extend ActiveSupport::Concern
 
-  included do
-    class_attribute :keep_params_config
-    helper ParamsKeeper::Helper
-  end
+    included do
+      class_attribute :keep_params_configs
+      helper ParamsKeeper::Helper
+    end
 
-  def url_for(options = nil)
-    ParamsKeeper::Resolver.new(self, self, options).resolve || super
-  end
+    def url_for(options = nil)
+      ParamsKeeper::UrlFor.new(self, self, options).call || super
+    end
+
+    def redirect_to(options = {}, response_options = {})
+      return super unless options.is_a?(String)
+
+      url = ParamsKeeper::UrlFor.new(self, self, options).call
+      url ? super(url, response_options) : super
+    end
+
+    class_methods do
+      def keep_params(*args)
+        options = args.last.is_a?(Hash) ? args.pop : {}
+        config = ParamsKeeper::Config.new(args, options)
+        self.keep_params_configs = keep_params_configs.to_a + [config]
+      end
 
-  class_methods do
-    def keep_params(*args)
-      self.keep_params_config = {
-        keys: nil,
-        to: nil,
-        for: :hash,
-        url_options: nil
-      }
-      self.keep_params_config.merge!(args.last.is_a?(Hash) ? args.pop : {})
-      self.keep_params_config[:keys] = Array(args)
+      def clear_keep_params!
+        self.keep_params_configs = nil
+     end
     end
   end
 end

data/lib/params_keeper/helper.rb

@@ -1,9 +1,23 @@
-module ParamsKeeper::Helper
-  def url_for(options = nil)
-    if controller
-      ParamsKeeper::Resolver.new(self, controller, options).resolve || super
-    else
-      super
+module ParamsKeeper
+  module Helper
+    def url_for(url_options = nil)
+      return super unless controller
+
+      ParamsKeeper::UrlFor.new(self, controller, url_options).call || super
+    end
+
+    def form_with(**options, &block)
+      return super unless controller
+      return super if options[:method].to_s.downcase != 'get'
+
+      html = super
+      url_options = options[:model] || options[:url]
+      hidden_fields = ParamsKeeper::HiddenFields.new(controller, url_options).call
+      if hidden_fields.present?
+        html.sub('</form>') { "#{hidden_fields}</form>" }.html_safe
+      else
+        html
+      end
     end
   end
 end

data/lib/params_keeper/hidden_fields.rb

@@ -0,0 +1,19 @@
+module ParamsKeeper
+  class HiddenFields
+    def initialize(controller, url_options)
+      @controller = controller
+      @url_options = url_options
+    end
+
+    def call
+      return if @controller.class.keep_params_configs.blank?
+
+      params = ParamsKeeper::Resolver.new(@controller, @url_options).call
+      return if params.blank?
+
+      CGI.parse(params.to_query).flat_map do |key, values|
+        values.map { |value| @controller.view_context.hidden_field_tag(key, value, id: nil) }
+      end.join.html_safe
+    end
+  end
+end

data/lib/params_keeper/resolver.rb

@@ -1,85 +1,87 @@
-class ParamsKeeper::Resolver
-  def initialize(klass, controller, options)
-    @klass = klass
-    @controller = controller
-    @options = options
-  end
+module ParamsKeeper
+  class Resolver
+    def initialize(controller, url_options)
+      @controller = controller
+      @url_options = url_options
+      @cache = {}
+    end
 
-  def resolve
-    return if !configured? || !enable_options? || !target_options?
+    def call
+      return {} if configs.blank? || skip_url_options?
 
-    if @options.is_a?(Hash)
-      resolve_from_hash
-    else
-      resolve_from_routing
+      configs.each_with_object({}) do |config, params|
+        if target_config?(config)
+          params.merge!(extract_params(config))
+        end
+      end
     end
-  end
 
-  private
-
-  def resolve_from_hash
-    if link_to_target_controller?(@options)
-      base_url_for(self.class.merge_params(@options, @controller.params, config[:keys]))
-    else
-      base_url_for(@options)
+    def url_options_hash
+      if @url_options.is_a?(Hash)
+        @url_options
+      else
+        recognize_path(base_url_for(@url_options))
+      end
     end
-  end
 
-  def resolve_from_routing
-    url = base_url_for(@options)
-    url_opts = recognize_path(url)
+    private
 
-    if url_opts && link_to_target_controller?(url_opts)
-      base_url_for(self.class.merge_params(url_opts, @controller.params, config[:keys]))
-    else
-      url
+    def configs
+      @controller.class.keep_params_configs
     end
-  end
 
-  def link_to_target_controller?(options)
-    controller = options[:controller].to_s
-    if config[:to].present?
-      target = controller.present? ? [controller] : [@controller.controller_name, @controller.controller_path]
-      (Array(config[:to]).map(&:to_s) & target).present?
-    else
-      controller.blank? || controller.in?([@controller.controller_name, @controller.controller_path])
+    def skip_url_options?
+      if @url_options.is_a?(Hash)
+        @url_options.delete(:keep_params) == false
+      else
+        false
+      end
     end
-  end
 
-  def config
-    @controller.class.keep_params_config
-  end
+    def target_config?(config)
+      target_url_options?(config) && target_controller?(config)
+    end
 
-  def configured?
-    config && config[:keys].present?
-  end
+    def target_url_options?(config)
+      (config.for.include?(:hash) && @url_options.is_a?(Hash)) ||
+        (config.for.include?(:string) && @url_options.is_a?(String)) ||
+        (config.for.include?(:model) && @url_options.class.respond_to?(:model_name))
+    end
 
-  def enable_options?
-    !@options.is_a?(Hash) || @options.delete(:keep_params) != false
-  end
+    def target_controller?(config)
+      dests = destination_controllers(url_options_hash)
+      if config.to.present?
+        (dests & config.to.map(&:to_s)).present?
+      else
+        (dests & current_controllers).present?
+      end
+    end
 
-  def target_options?
-    fors = Array(config[:for] || :hash)
-    (fors.include?(:hash) && @options.is_a?(Hash)) ||
-      (fors.include?(:string) && @options.is_a?(String)) ||
-      (fors.include?(:model) && @options.class.respond_to?(:model_name))
-  end
+    def destination_controllers(url_options_hash)
+      if url_options_hash[:controller].present?
+        [url_options_hash[:controller].to_s]
+      else
+        current_controllers
+      end
+    end
 
-  def recognize_path(url)
-    Rails.application.routes.recognize_path(url)
-  rescue ActionController::RoutingError
-    nil
-  end
+    def current_controllers
+      [@controller.controller_name, @controller.controller_path]
+    end
 
-  def base_url_for(options)
-    options = options.merge(config[:url_options]) if options.is_a?(Hash) && config[:url_options]
-    @klass.method(:url_for).super_method.call(options)
-  end
+    def recognize_path(url)
+      @cache[url] ||= Rails.application.routes.recognize_path(url)
+    rescue ActionController::RoutingError
+      {}
+    end
+
+    def extract_params(config)
+      params = @controller.request.params.deep_symbolize_keys
+      params.slice(*config.keys).merge(config.url_options)
+    end
 
-  class << self
-    def merge_params(options, params, keys)
-      keeps = params.to_unsafe_h.deep_symbolize_keys.slice(*keys.to_a)
-      options.reverse_merge(keeps)
+    def base_url_for(url_options)
+      @controller.method(:url_for).super_method.call(url_options)
     end
   end
 end

data/lib/params_keeper/url_for.rb

@@ -0,0 +1,25 @@
+module ParamsKeeper
+  class UrlFor
+    def initialize(caller, controller, url_options)
+      @caller = caller
+      @controller = controller
+      @url_options = url_options
+    end
+
+    def call
+      return if @controller.class.keep_params_configs.blank?
+
+      resolver = ParamsKeeper::Resolver.new(@controller, @url_options)
+      params = resolver.call
+      return if params.blank?
+
+      base_url_for(resolver.url_options_hash.reverse_merge(params))
+    end
+
+    private
+
+    def base_url_for(url_options)
+      @caller.method(:url_for).super_method.call(url_options)
+    end
+  end
+end

data/lib/params_keeper/version.rb

@@ -1,3 +1,3 @@
 module ParamsKeeper
-  VERSION = "1.1.2"
+  VERSION = "1.2.0"
 end

data/lib/params_keeper_rails.rb

@@ -1,5 +1,8 @@
 require 'active_support'
 require 'params_keeper/version'
+require 'params_keeper/config'
 require 'params_keeper/resolver'
+require 'params_keeper/url_for'
+require 'params_keeper/hidden_fields'
 require 'params_keeper/helper'
 require 'params_keeper/controller'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: params_keeper_rails
 version: !ruby/object:Gem::Version
-  version: 1.1.2
+  version: 1.2.0
 platform: ruby
 authors:
 - Yoshikazu Kaneta
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-03-15 00:00:00.000000000 Z
+date: 2020-09-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -97,15 +97,16 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
-- bin/console
-- bin/setup
 - gemfiles/rails50.gemfile
 - gemfiles/rails51.gemfile
 - gemfiles/rails52.gemfile
 - gemfiles/rails60.gemfile
+- lib/params_keeper/config.rb
 - lib/params_keeper/controller.rb
 - lib/params_keeper/helper.rb
+- lib/params_keeper/hidden_fields.rb
 - lib/params_keeper/resolver.rb
+- lib/params_keeper/url_for.rb
 - lib/params_keeper/version.rb
 - lib/params_keeper_rails.rb
 - params_keeper_rails.gemspec
@@ -128,7 +129,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: keep specific parameters through links.

data/bin/console

@@ -1,14 +0,0 @@
-#!/usr/bin/env ruby
-
-require "bundler/setup"
-require "params_keeper/controller"
-
-# You can add fixtures and/or initialization code here to make experimenting
-# with your gem easier. You can also use a different console, if you like.
-
-# (If you use this, don't forget to add pry to your Gemfile!)
-# require "pry"
-# Pry.start
-
-require "irb"
-IRB.start

data/bin/setup

@@ -1,8 +0,0 @@
-#!/usr/bin/env bash
-set -euo pipefail
-IFS=$'\n\t'
-set -vx
-
-bundle install
-
-# Do any other automated setup that you need to do here