activeadmin_select_many 0.3.4 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: d326b4fd06dc88911b5ec9da5b3790ad277c0753
-  data.tar.gz: 5a0e1c2f07992dee3c23806b6ce14e074eaf039f
+SHA256:
+  metadata.gz: 125c98dc457c8e510aa9e35d0c03c9d86341a42c6395cbd5da501f32a0782ab3
+  data.tar.gz: aa9ab407d84de81df360209eba4c868da7232805fe6606d19d2e3b3c74d00279
 SHA512:
-  metadata.gz: 3bfc12b5ed28f0094ef5e8ead2b839df85b7bff88c791c8c34471ebc080ab8396b0c2d12971764d392d3405c85e190818ec7993c54c026e5db0af071c608ea8d
-  data.tar.gz: 47e1eb447e3ea4ca6f62101f8608d30d80c83044e20812074631d8475dec71de5b0bf9e79c81cee4b92c5bce1d78f06c10cb38ab659c31abf48199ad0a349005
+  metadata.gz: f40eda600327c6a7262548e7052ae0662c65b88a40cdd21d4138d7b2ce9e4df37e9884384d23dc34e34236f778e876e2a7d48f575d11af8d361598189e7b3ffa
+  data.tar.gz: 5e0d0b7b131174658ad5dfde9c235dedb5ddfb09c703efed47ea4141b4b0ac06fec1f3591c4ab6b790167f1793570f70a5d2b3a557b7196bed4bb993f4ed9035

data/README.md

@@ -1,22 +1,22 @@
 # ActiveAdmin Select Many [![Gem Version](https://badge.fury.io/rb/activeadmin_select_many.svg)](https://badge.fury.io/rb/activeadmin_select_many)
 
-An Active Admin plugin which improves one-to-many / many-to-many / many-to-one associations selection using 2 new inputs: **select_many** and **select_one** (jQuery required)
+An Active Admin plugin which improves one-to-many / many-to-many / many-to-one associations selection using 2 new inputs: **select_many** and **select_one** (jQuery required).
 
 Features for *select_many*:
-- search box
-- available items on the left, selected items on the right
-- local/remote collections
-- double click to add/remove items
-- sortable (with up/down buttons)
-- key bindings to improve accessibility
+- search box;
+- available items on the left, selected items on the right;
+- local/remote collections;
+- double click to add/remove items;
+- sortable (with up/down buttons);
+- key bindings to improve accessibility.
 
 Features for *select_one*:
-- search box
-- selected items on the right
-- remote collections
-- counter of items found
-- can be used as filter
-- key bindings to improve accessibility
+- search box;
+- selected items on the right;
+- remote collections;
+- counter of items found;
+- can be used as filter;
+- key bindings to improve accessibility.
 
 ![screenshot](screenshot.png)
 
@@ -47,7 +47,7 @@ Features for *select_one*:
 - **size**: number of rows of both the selects (default: 4)
 - **sortable**: set to true to enable sortable buttons (default: not set)
 
-## Example with select_many
+## Examples with select_many
 
 Add to ActiveAdmin model config, in *form* block.
 
@@ -85,7 +85,7 @@ ActiveAdmin.register Tag do
 end
 ```
 
-## Example with select_one
+## Examples with select_one
 
 In a form:
 
@@ -95,6 +95,10 @@ As filter:
 
 `filter :article_id_eq, as: :select_one, filter_form: true, placeholder: 'Search...', search_param: 'title_contains', member_label: 'title', remote_collection: '/admin/articles.json'`
 
+## Notes
+
+- To use this plugins with ActiveAdmin 1.x please use the version 0.3.4
+
 ## Do you like it? Star it!
 
 If you use this component just star it. A developer is more motivated to improve a project when there is some interest.

data/app/assets/javascripts/activeadmin/select_many.js

@@ -1,15 +1,15 @@
-function smActivate( target ) {
-  if( target.tagName.toLowerCase() == 'option' ) {
-    var parent = $(this).closest( '.select_many' );
+function smActivate(target) {
+  if(target.tagName.toLowerCase() == 'option') {
+    var parent = $(this).closest('.select_many');
     var opt = $(target);
-    var dst = parent.find( $(this).data( 'select' ) == 'src' ? '[data-select="dst"]' : '[data-select="src"]' );
-    dst.append( $('<option>', { value: opt.val(), text: opt.text() }) );
+    var dst = parent.find($(this).data('select') == 'src' ? '[data-select="dst"]' : '[data-select="src"]');
+    dst.append($('<option>', { value: opt.val(), text: opt.text() }));
     opt.remove();
-    smUpdateValues( parent );
+    smUpdateValues(parent);
   }
 }
 
-function smDebounce( func, wait, immediate ) {
+function smDebounce(func, wait, immediate) {
   var timeout;
   return function() {
     var context = this, args = arguments;
@@ -24,197 +24,198 @@ function smDebounce( func, wait, immediate ) {
   };
 };
 
-function smUpdateValues( parent ) {
-  var cnt = 0, values = parent.find( '.values' );
+function smUpdateValues(parent) {
+  var cnt = 0, values = parent.find('.values');
   values.empty();
-  parent.find( '[data-select="dst"] option' ).each( function() {
-    values.append( $('<input>', { type: 'hidden', name: values.data( 'name' ), value: $(this).val() }) );
+  parent.find('[data-select="dst"] option').each(function() {
+    values.append($('<input>', { type: 'hidden', name: values.data('name'), value: $(this).val() }));
     cnt++;
   });
-  if( cnt == 0 ) values.append( $('<input>', { type: 'hidden', name: values.data( 'name' ) }) );
-  parent.find( '.selected span' ).text( ' [' + cnt + ']' );
+  if(cnt == 0) values.append($('<input>', { type: 'hidden', name: values.data('name') }));
+  parent.find('.selected span').text(' [' + cnt + ']');
+  parent.find('.available span').text(' [' + parent.find('[data-select="src"] option').length + ']');
 }
 
-$(document).ready( function() {
-  $('.select_many.input select').on( 'dblclick', function( event ) {
-    $.proxy( smActivate, $(this) )( event.target );
+$(document).ready(function() {
+  $('.select_many.input select').on('dblclick', function(event) {
+    $.proxy(smActivate, $(this))(event.target);
   });
 
   // --- select_many ----------------------------------------------------------
-  var onLocalSelect = smDebounce( function() {
+  var onLocalSelect = smDebounce(function() {
     var cnt = 0, search = $(this).val().toLowerCase();
-    $(this).closest( '.select_many' ).find( '[data-select="src"] option' ).each( function() {
-      var found = $(this).text().toLowerCase().indexOf( search ) >= 0;
-      $(this).toggle( found );
-      if( found ) cnt++;
+    $(this).closest('.select_many').find('[data-select="src"] option').each(function() {
+      var found = $(this).text().toLowerCase().indexOf(search) >= 0;
+      $(this).toggle(found);
+      if(found) cnt++;
     });
-    $(this).parent().find( '.available span' ).text( ' [' + cnt + ']' );
-  }, 250 );
+    $(this).parent().find('.available span').text(' [' + cnt + ']');
+  }, 250);
 
-  var onRemoteSelect = smDebounce( function( event ) {
+  var onRemoteSelect = smDebounce(function(event) {
     var search = $(this).val().trim();
-    if( $(this).data( 'searching' ) != '1' && search && $(this).data( 'last-search' ) != search ) {
-      $(this).data( 'searching', '1' );
-      $(this).data( 'last-search', search );
+    if($(this).data('searching') != '1' && search && $(this).data('last-search') != search) {
+      $(this).data('searching', '1');
+      $(this).data('last-search', search);
       var _this = $(this);
       var data = {}
       var text_key = $(this).data('text');
       var value_key = $(this).data('value');
-      var counter_limit = $(this).data('counter-limit') ? Number( $(this).data('counter-limit') ) : 0;
+      var counter_limit = $(this).data('counter-limit') ? Number($(this).data('counter-limit')) : 0;
       data['q['+$(this).data('search')+']'] = search;
       $.ajax({
         context: _this,
         data: data,
-        url: $(this).data( 'remote-collection' ),
-        complete: function( req, status ) {
-          $(this).data( 'searching', '' );
+        url: $(this).data('remote-collection'),
+        complete: function(req, status) {
+          $(this).data('searching', '');
         },
-        success: function( data, status, req ) {
-          var select = $(this).closest( '.select_many' ).find( '[data-select="src"]' );
+        success: function(data, status, req) {
+          var select = $(this).closest('.select_many').find('[data-select="src"]');
           select.empty();
-          data.forEach( function( item ) {
-            select.append( $('<option>', { value: item[value_key], text: item[text_key] }) );
+          data.forEach(function(item) {
+            select.append($('<option>', { value: item[value_key], text: item[text_key] }));
           });
-          $(this).parent().find( '.available span' ).text( ' [' + ( ( counter_limit > 0 && data.length >= counter_limit ) ? ( counter_limit + '+' ) : data.length ) + ']' );
+          $(this).parent().find('.available span').text(' [' + ((counter_limit > 0 && data.length >= counter_limit) ? (counter_limit + '+') : data.length) + ']');
         },
       });
     }
-  }, 400 );
+  }, 400);
 
-  $('.select_many.input .search-select').each( function() {
+  $('.select_many.input .search-select').each(function() {
     var parent = $(this).parent();
-    parent.find( '.available' ).append( '<span> [' + parent.find( '[data-select="src"] option' ).length + ']</span>' );
-    parent.find( '.selected' ).append( '<span> [' + parent.find( '[data-select="dst"] option' ).length + ']</span>' );
-    $(this).on( 'keydown', function( event ) {
-      if( event.which == 13 || event.which == 40 ) {  // enter or arrow down
+    parent.find('.available').append('<span> [' + parent.find('[data-select="src"] option').length + ']</span>');
+    parent.find('.selected').append('<span> [' + parent.find('[data-select="dst"] option').length + ']</span>');
+    $(this).on('keydown', function(event) {
+      if(event.which == 13 || event.which == 40) {  // enter or arrow down
         event.preventDefault();
-        $(this).closest( '.select_many' ).find( '[data-select="src"]' ).focus();
+        $(this).closest('.select_many').find('[data-select="src"]').focus();
       }
-      else $.proxy( $(this).data( 'remote-collection' ) ? onRemoteSelect : onLocalSelect, $(this) )( event );
+      else $.proxy($(this).data('remote-collection') ? onRemoteSelect : onLocalSelect, $(this))(event);
     });
 
     // --- key bindings -------------------------------------------------------
-    parent.find( '[data-select="src"]' ).on( 'keydown', function( event ) {
-      if( event.which == 13 ) {  // enter
+    parent.find('[data-select="src"]').on('keydown', function(event) {
+      if(event.which == 13) {  // enter
         event.preventDefault();
-        var opts = $(this).find( ':selected' );
-        if( opts.length > 0 ) {
-          var next = $( opts[0] ).next();
-          $.proxy( smActivate, $(this) )( opts[0] );
-          if( next.length > 0 ) $(this).val( next.val() );
+        var opts = $(this).find(':selected');
+        if(opts.length > 0) {
+          var next = $(opts[0]).next();
+          $.proxy(smActivate, $(this))(opts[0]);
+          if(next.length > 0) $(this).val(next.val());
         }
       }
-      else if( event.which == 9 || event.which == 39 ) {  // tab or right arrow
+      else if(event.which == 9 || event.which == 39) {  // tab or right arrow
         event.preventDefault();
-        parent.find( '[data-select="dst"]' ).focus();
+        parent.find('[data-select="dst"]').focus();
       }
-      else if( event.which == 38 ) {  // up arrow
-        if( $(this).find('option')[0] == $(this).find(':selected')[0] ) {
+      else if(event.which == 38) {  // up arrow
+        if($(this).find('option')[0] == $(this).find(':selected')[0]) {
           event.preventDefault();
-          parent.find( '.search-select' ).focus();
+          parent.find('.search-select').focus();
         }
       }
     });
-    parent.find( '[data-select="dst"]' ).on( 'keydown', function( event ) {
-      if( event.which == 13 ) {  // enter
+    parent.find('[data-select="dst"]').on('keydown', function(event) {
+      if(event.which == 13) {  // enter
         event.preventDefault();
-        var opts = $(this).find( ':selected' );
-        if( opts.length > 0 ) {
-          var next = $( opts[0] ).next();
-          $.proxy( smActivate, $(this) )( opts[0] );
-          if( next.length > 0 ) $(this).val( next.val() );
+        var opts = $(this).find(':selected');
+        if(opts.length > 0) {
+          var next = $(opts[0]).next();
+          $.proxy(smActivate, $(this))(opts[0]);
+          if(next.length > 0) $(this).val(next.val());
         }
       }
-      else if( event.which == 37 ) {  // left arrow
+      else if(event.which == 37) {  // left arrow
         event.preventDefault();
-        parent.find( '[data-select="src"]' ).focus();
+        parent.find('[data-select="src"]').focus();
       }
     });
   });
 
   // --- buttons --------------------------------------------------------------
-  $('.select_many .add').on( 'click', function() {
+  $('.select_many .add').on('click', function() {
     var select = $(this).parent().prev();
-    var current = select.find( 'option:selected' )[0];
-    if( current ) $.proxy( smActivate, select )( current );
+    var current = select.find('option:selected')[0];
+    if(current) $.proxy(smActivate, select)(current);
   });
-  $('.select_many .remove').on( 'click', function() {
+  $('.select_many .remove').on('click', function() {
     var select = $(this).parent().next();
-    var current = select.find( 'option:selected' )[0];
-    if( current ) $.proxy( smActivate, select )( current );
+    var current = select.find('option:selected')[0];
+    if(current) $.proxy(smActivate, select)(current);
   });
-  $('.select_many [sortable] .move_up').on( 'click', function() {
+  $('.select_many [sortable] .move_up').on('click', function() {
     var select = $(this).parent().next();
-    var current = select.find( 'option:selected' )[0];
-    if( current ) {
-      $(current).prev().before( current );
-      smUpdateValues( $(this).closest( '.select_many' ) );
+    var current = select.find('option:selected')[0];
+    if(current) {
+      $(current).prev().before(current);
+      smUpdateValues($(this).closest('.select_many'));
     }
   });
-  $('.select_many [sortable] .move_down').on( 'click', function() {
+  $('.select_many [sortable] .move_down').on('click', function() {
     var select = $(this).parent().next();
-    var current = select.find( 'option:selected' )[0];
-    if( current ) {
-      $(current).next().after( current );
-      smUpdateValues( $(this).closest( '.select_many' ) );
+    var current = select.find('option:selected')[0];
+    if(current) {
+      $(current).next().after(current);
+      smUpdateValues($(this).closest('.select_many'));
     }
   });
 
   // --- select one -----------------------------------------------------------
-  var onRemoteSelectOne = smDebounce( function( event ) {
+  var onRemoteSelectOne = smDebounce(function(event) {
     var search = $(this).val().trim();
-    var select = $(this).closest( '.select_one' ).find( '[data-select="src"]' );;
-    if( select.data( 'searching' ) != '1' && search && select.data( 'last-search' ) != search ) {
-      select.data( 'searching', '1' );
-      select.data( 'last-search', search );
+    var select = $(this).closest('.select_one').find('[data-select="src"]');;
+    if(select.data('searching') != '1' && search && select.data('last-search') != search) {
+      select.data('searching', '1');
+      select.data('last-search', search);
       var data = {}
       var search_key = $(this).data('search') ? $(this).data('search') : 'name_contains';
       var value_key = $(this).data('value') ? $(this).data('value') : 'id';
       var text_key = $(this).data('text') ? $(this).data('text') : 'name';
-      var counter_limit = $(this).data('counter-limit') ? Number( $(this).data('counter-limit') ) : 0;
+      var counter_limit = $(this).data('counter-limit') ? Number($(this).data('counter-limit')) : 0;
       data['q['+search_key+']'] = search;
       $.ajax({
         context: select,
         data: data,
-        url: $(this).data( 'remote-collection' ),
-        complete: function( req, status ) {
-          $(this).data( 'searching', '' );
+        url: $(this).data('remote-collection'),
+        complete: function(req, status) {
+          $(this).data('searching', '');
         },
-        success: function( data, status, req ) {
+        success: function(data, status, req) {
           var first = false, sel = $(this);
           sel.empty();
-          if( sel.data('include-blank') ) sel.append( $('<option>', { value: '', text: '' }) );
-          data.forEach( function( item ) {
-            sel.append( $('<option>', { value: item[value_key], text: item[text_key] }) );
-            if( !first ) first = item[value_key];
+          if(sel.data('include-blank')) sel.append($('<option>', { value: '', text: '' }));
+          data.forEach(function(item) {
+            sel.append($('<option>', { value: item[value_key], text: item[text_key] }));
+            if(!first) first = item[value_key];
           });
-          sel.parent().find( '.status' ).text( '[' + ( ( counter_limit > 0 && data.length >= counter_limit ) ? ( counter_limit + '+' ) : data.length ) + ']' );
-          if( first ) sel.val( first );
+          sel.parent().find('.status').text('[' + ((counter_limit > 0 && data.length >= counter_limit) ? (counter_limit + '+') : data.length) + ']');
+          if(first) sel.val(first);
         },
       });
     }
-  }, 500 );
+  }, 500);
 
-  $('.select-one-inputs').each( function() {
-    $(this).find( '.search-select' ).on( 'keydown', function( event ) {
-      if( event.which == 38 ) {  // up arrow
+  $('.select-one-inputs').each(function() {
+    $(this).find('.search-select').on('keydown', function(event) {
+      if(event.which == 38) {  // up arrow
         event.preventDefault();
-        var select = $(this).closest( '.select-one-inputs' ).find( '[data-select="src"]' );
-        var prev = select.find( ':selected' ).prev();
-        if( prev.length ) select.val( prev.val() );
+        var select = $(this).closest('.select-one-inputs').find('[data-select="src"]');
+        var prev = select.find(':selected').prev();
+        if(prev.length) select.val(prev.val());
       }
-      else if( event.which == 40 ) {  // down arrow
+      else if(event.which == 40) {  // down arrow
         event.preventDefault();
-        var select = $(this).closest( '.select-one-inputs' ).find( '[data-select="src"]' );
-        var next = select.find( ':selected' ).next();
-        if( next.length ) select.val( next.val() );
+        var select = $(this).closest('.select-one-inputs').find('[data-select="src"]');
+        var next = select.find(':selected').next();
+        if(next.length) select.val(next.val());
       }
-      else $.proxy( onRemoteSelectOne, $(this) )( event );
+      else $.proxy(onRemoteSelectOne, $(this))(event);
     });
-    $(this).find( '[data-select="src"]' ).on( 'keydown', function( event ) {
-      if( event.which == 37 ) {  // left arrow
+    $(this).find('[data-select="src"]').on('keydown', function(event) {
+      if(event.which == 37) {  // left arrow
         event.preventDefault();
-        $(this).closest( '.select-one-inputs' ).find( '.search-select' ).focus();
+        $(this).closest('.select-one-inputs').find('.search-select').focus();
       }
     });
   });

data/lib/activeadmin/select_many/version.rb

@@ -1,5 +1,5 @@
 module ActiveAdmin
   module SelectMany
-    VERSION = '0.3.4'
+    VERSION = '0.4.0'
   end
 end

data/lib/formtastic/inputs/select_many_input.rb

@@ -1,18 +1,20 @@
+# frozen_string_literal: true
+
 module Formtastic
   module Inputs
     class SelectManyInput < SelectInput
       def to_html
-        options[:'data-remote-collection'] = options.delete( :remote_collection )
+        options[:'data-remote-collection'] = options.delete(:remote_collection)
         opts = { class: 'select-many-inputs' }
-        opts[:sortable] = options.delete( :sortable ) if options[:sortable]
+        opts[:sortable] = options.delete(:sortable) if options[:sortable]
         input_wrapping do
           label_html <<
-          template.content_tag( :div, opts ) do
+          template.content_tag(:div, opts) do
             hidden_input <<
             search_box_html <<
-            template.content_tag( :span, '', class: 'empty' ) <<
-            template.content_tag( :span, ::I18n.t( 'inputs.select_many.available' ), class: 'available' ) <<
-            template.content_tag( :span, ::I18n.t( 'inputs.select_many.selected' ), class: 'selected' ) <<
+            template.content_tag(:span, '', class: 'empty') <<
+            template.content_tag(:span, ::I18n.t('inputs.select_many.available'), class: 'available') <<
+            template.content_tag(:span, ::I18n.t('inputs.select_many.selected'), class: 'selected') <<
             select_src_html <<
             buttons_html <<
             select_dst_html
@@ -21,21 +23,21 @@ module Formtastic
       end
 
       def hidden_input
-        template.content_tag( :div, class: 'values', 'data-name': input_html_options[:name] ) do
-          values = object.send( input_name )
+        template.content_tag(:div, class: 'values', 'data-name': input_html_options[:name]) do
+          values = object.send(input_name)
           values = [values] if values.is_a? Integer
           values.each do |value|
-            template.concat template.hidden_field_tag( input_html_options[:name], value, {id: nil} )
+            template.concat template.hidden_field_tag(input_html_options[:name], value, {id: nil})
           end if values
         end
       end
 
       def buttons_html
-        template.content_tag( :div, class: 'buttons' ) do
-          template.link_to( '&rarr;'.html_safe, 'Javascript:void(0)', class: 'add' ) +
-          template.link_to( '&larr;'.html_safe, 'Javascript:void(0)', class: 'remove' ) +
-          template.link_to( '&uarr;'.html_safe, 'Javascript:void(0)', class: 'move_up' ) +
-          template.link_to( '&darr;'.html_safe, 'Javascript:void(0)', class: 'move_down' )
+        template.content_tag(:div, class: 'buttons') do
+          template.link_to('&rarr;'.html_safe, 'Javascript:void(0)', class: 'add') +
+          template.link_to('&larr;'.html_safe, 'Javascript:void(0)', class: 'remove') +
+          template.link_to('&uarr;'.html_safe, 'Javascript:void(0)', class: 'move_up') +
+          template.link_to('&darr;'.html_safe, 'Javascript:void(0)', class: 'move_down')
         end
       end
 
@@ -43,35 +45,50 @@ module Formtastic
         opts = {
           id: nil,
           class: 'search-select',
-          placeholder: options.delete( :placeholder ),
+          placeholder: options.delete(:placeholder),
           'data-counter-limit': options[:counter_limit].to_i,
           'data-remote-collection': options[:'data-remote-collection'],
           'data-search': options[:search_param] ? options[:search_param] : 'name_contains',
-          'data-text': options[:member_label] ? options[:member_label] : ( options[:text_key] ? options[:text_key] : 'name' ),
+          'data-text': options[:member_label] ? options[:member_label] : (options[:text_key] ? options[:text_key] : 'name'),
           'data-value': options[:value_key] ? options[:value_key] : 'id',
         }
-        template.text_field_tag( nil, '', opts )
+        template.text_field_tag(nil, '', opts)
       end
 
       def select_src_html
-        coll = if options[:'data-remote-collection']
-          []
-        else
-          # TODO: add option unique ?
-          selected = object.send( input_name )
-          selected = [selected] if selected.is_a? Integer
-          selected ? collection.select { |option| !selected.include?( option[1] ) } : collection
-        end
-        opts = input_options.merge( name: nil, id: nil, multiple: true, 'data-select': 'src', size: options[:size] ? options[:size] : 4 )
-        template.select_tag nil, template.options_for_select( coll ), opts
+        coll =
+          if options[:'data-remote-collection']
+            []
+          else
+            # TODO: add option unique ?
+            selected = object.send(input_name)
+            selected = [selected] if selected.is_a? Integer
+            selected ? collection.select { |option| !selected.include?(option[1]) } : collection
+          end
+        opts = {
+          id: nil,
+          include_blank: false,
+          multiple: true,
+          name: nil,
+          size: options[:size] || 4,
+          'data-select': 'src'
+        }
+        template.select_tag nil, template.options_for_select(coll), input_options.merge(opts)
       end
 
       def select_dst_html
-        selected = options[:selected] ? options[:selected] : object.send( input_name )
+        selected = options[:selected] || object.send(input_name)
         selected = [selected] if selected.is_a? Integer
-        coll = selected ? collection.select { |option| selected.include?( option[1] ) } : collection
-        opts = input_options.merge( name: nil, id: nil, multiple: true, 'data-select': 'dst', size: options[:size] ? options[:size] : 4 )
-        template.select_tag nil, template.options_for_select( coll ), opts
+        coll = selected ? collection.select { |option| selected.include?(option[1]) } : []
+        opts = {
+          id: nil,
+          include_blank: false,
+          multiple: true,
+          name: nil,
+          size: options[:size] || 4,
+          'data-select': 'dst'
+        }
+        template.select_tag nil, template.options_for_select(coll), input_options.merge(opts)
       end
     end
   end

data/lib/formtastic/inputs/select_one_input.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Formtastic
   module Inputs
     class SelectOneInput < SelectInput
@@ -16,10 +18,10 @@ module Formtastic
         opts = { class: 'select-one-inputs' }
         input_wrapping do
           label_html <<
-          template.content_tag( :div, opts ) do
+          template.content_tag(:div, opts) do
             search_box <<
             select_html <<
-            template.content_tag( :span, '', class: 'status' )
+            template.content_tag(:span, '', class: 'status')
           end
         end
       end
@@ -33,25 +35,18 @@ module Formtastic
           'data-counter-limit': options[:counter_limit].to_i,
           'data-msg': options[:msg_items],
           'data-remote-collection': options[:remote_collection],
-          'data-search': options[:search_param] ? options[:search_param] : 'name_contains',
-          'data-text': options[:member_label] ? options[:member_label] : ( options[:text_key] ? options[:text_key] : 'name' ),
-          'data-value': options[:value_key] ? options[:value_key] : 'id',
+          'data-search': options[:search_param] || 'name_contains',
+          'data-text': options[:member_label] || options[:text_key] || 'name',
+          'data-value': options[:value_key] || 'id'
         }
-        template.text_field_tag( nil, '', opts )
+        template.text_field_tag(nil, '', opts)
       end
 
       def select_html
-        selected = options[:selected] ? options[:selected] : object.send( input_name )
-        sel = ''
-        collection.each do |item|
-          if item[1] == selected
-            sel = template.options_for_select( [item], selected ) if item[1] == selected
-            break
-          end
-        end
-        opts = input_html_options.merge( 'data-select': 'src' )
+        selected = options[:selected] || object.send(input_name)
+        opts = input_html_options.merge('data-select': 'src')
         opts['data-include-blank'] = '1' if input_options[:include_blank]
-        builder.select(input_name, sel, input_options, opts )
+        template.select_tag input_name, template.options_for_select(collection, selected), input_options.merge(opts)
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: activeadmin_select_many
 version: !ruby/object:Gem::Version
-  version: 0.3.4
+  version: 0.4.0
 platform: ruby
 authors:
 - Mattia Roccoberton
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2018-06-10 00:00:00.000000000 Z
+date: 2020-08-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activeadmin
@@ -16,14 +16,140 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '2.0'
+- !ruby/object:Gem::Dependency
+  name: activestorage
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 6.0.3.2
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 6.0.3.2
+- !ruby/object:Gem::Dependency
+  name: capybara
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 3.33.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 3.33.0
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.13.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.13.1
+- !ruby/object:Gem::Dependency
+  name: puma
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 4.3.5
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 4.3.5
+- !ruby/object:Gem::Dependency
+  name: rspec_junit_formatter
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.4.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.4.1
+- !ruby/object:Gem::Dependency
+  name: rspec-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 4.0.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 4.0.1
+- !ruby/object:Gem::Dependency
+  name: selenium-webdriver
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 3.142.7
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 3.142.7
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.19.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.19.0
+- !ruby/object:Gem::Dependency
+  name: sqlite3
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.4.2
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.4.2
 description: An Active Admin plugin which improves one-to-many and many-to-many associations
   selection (jQuery required)
 email: mat@blocknot.es
@@ -31,12 +157,9 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".gitignore"
-- Gemfile
 - LICENSE.txt
 - README.md
 - Rakefile
-- activeadmin_select_many.gemspec
 - app/assets/javascripts/activeadmin/select_many.js
 - app/assets/stylesheets/activeadmin/_select_many.sass
 - config/locales/en.yml
@@ -46,12 +169,11 @@ files:
 - lib/activeadmin_select_many.rb
 - lib/formtastic/inputs/select_many_input.rb
 - lib/formtastic/inputs/select_one_input.rb
-- screenshot.png
 homepage: https://github.com/blocknotes/activeadmin_select_many
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -66,9 +188,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.13
-signing_key: 
+rubygems_version: 3.0.3
+signing_key:
 specification_version: 4
 summary: SelectMany plugin for ActiveAdmin
 test_files: []

data/.gitignore

@@ -1,3 +0,0 @@
-_misc/
-
-*.orig

data/Gemfile

@@ -1,4 +0,0 @@
-source 'https://rubygems.org'
-
-gemspec
-

data/activeadmin_select_many.gemspec

@@ -1,19 +0,0 @@
-lib = File.expand_path('../lib', __FILE__)
-$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'activeadmin/select_many/version'
-
-Gem::Specification.new do |spec|
-  spec.name          = 'activeadmin_select_many'
-  spec.version       = ActiveAdmin::SelectMany::VERSION
-  spec.summary       = 'SelectMany plugin for ActiveAdmin'
-  spec.description   = 'An Active Admin plugin which improves one-to-many and many-to-many associations selection (jQuery required)'
-  spec.license       = 'MIT'
-  spec.authors       = ['Mattia Roccoberton']
-  spec.email         = 'mat@blocknot.es'
-  spec.homepage      = 'https://github.com/blocknotes/activeadmin_select_many'
-
-  spec.files         = `git ls-files -z`.split("\x0")
-  spec.require_paths = ['lib']
-
-  spec.add_runtime_dependency 'activeadmin', '~> 1.0'
-end