ui_bibz 2.4.0 → 2.5.5

data/lib/ui_bibz/ui/ux/tables/table_card.rb

@@ -1,6 +1,7 @@
-require "ui_bibz/ui/ux/tables/components/store"
-module UiBibz::Ui::Ux::Tables
+# frozen_string_literal: true
 
+require 'ui_bibz/ui/ux/tables/components/store'
+module UiBibz::Ui::Ux::Tables
   # Create a TableCard
   #
   # This element is an extend of UiBibz::Ui::Core::Boxes::card.
@@ -104,11 +105,8 @@ module UiBibz::Ui::Ux::Tables
   #     end
   #   end
   class TableCard < UiBibz::Ui::Core::Boxes::Card
-
-    attr_accessor :columns
-
     # See UiBibz::Ui::Core::Boxes::Card.initialize
-    def initialize content = nil, options = nil, html_options = nil, &block
+    def initialize(content = nil, options = nil, html_options = nil, &block)
       super
       table_options = (@options[:table_options] || {}).merge({ store: store })
       @table        = UiBibz::Ui::Ux::Tables::Table.new(table_options, @options[:table_html_options])
@@ -120,7 +118,7 @@ module UiBibz::Ui::Ux::Tables
 
       content_tag :div, html_options do
         form_tag(url_for(url_parameters), method: :get) do
-          store.parameters.with_indifferent_access.reject{ |k,v|  default_parameters?(k) || v.blank? }.each do |k,v|
+          store.parameters.with_indifferent_access.reject { |k, v| default_parameters?(k) || v.blank? }.each do |k, v|
             concat tag(:input, type: 'hidden', name: k, value: v)
           end
           concat tag(:input, type: 'hidden', name: 'store_id', value: store.id) unless store.id.nil? # if there is more one table in html page
@@ -130,13 +128,13 @@ module UiBibz::Ui::Ux::Tables
     end
 
     # Add table columns item
-    def columns &block
-      @table.columns &block
+    def columns(&block)
+      @table.columns(&block)
     end
 
     # Add table actions item
-    def actions &block
-      @table.actions &block
+    def actions(&block)
+      @table.actions(&block)
     end
 
     # for test
@@ -144,20 +142,17 @@ module UiBibz::Ui::Ux::Tables
       @table.actions_list
     end
 
-  private
+    private
 
     def component_html_classes
-      %w(card table-card)
+      %w[card table-card]
     end
 
     def store
-      @store ||= if @options[:store].nil?
-        raise 'Store is nil!'
-      elsif @options[:store].try(:records).nil?
-        raise 'Store can be created only with "table_search_pagination" method!'
-      else
-        Store.new @options.delete(:store) if @options[:store]
-      end
+      raise 'Store is nil!' if @options[:store].nil?
+      raise 'Store can be created only with "table_search_pagination" method!' if @options[:store].try(:records).nil?
+
+      @store ||= Store.new @options[:store]
     end
 
     def init_components
@@ -166,17 +161,15 @@ module UiBibz::Ui::Ux::Tables
       @items << pagination.render if pagination.paginable?
     end
 
-    def default_parameters?(k)
-      %w(store_id per_page page search utf8 search controller action link_type).include?(k)
+    def default_parameters?(key)
+      %w[store_id per_page page search utf8 search controller action link_type].include?(key)
     end
 
     def url_parameters
       { controller: store.controller, action: store.action, id: store.param_id }
     end
 
-    def table
-      @table
-    end
+    attr_reader :table
 
     def table_html
       content_tag :div, @table.render, class: 'card-table'
@@ -189,6 +182,5 @@ module UiBibz::Ui::Ux::Tables
     def pagination
       @pagination ||= Paginable.new store, @options.merge({ wrap_form: false }), { class: 'card-footer' }
     end
-
   end
 end

data/lib/ui_bibz/ui/ux/tables/table_pagination.rb

@@ -1,8 +1,9 @@
-#require 'will_paginate'
-require "will_paginate-bootstrap4"
-require "ui_bibz/ui/ux/tables/components/store"
-module UiBibz::Ui::Ux::Tables
+# frozen_string_literal: true
 
+# require 'will_paginate'
+require 'will_paginate-bootstrap4'
+require 'ui_bibz/ui/ux/tables/components/store'
+module UiBibz::Ui::Ux::Tables
   # Create a pagination for table
   #
   # ==== Attributes
@@ -27,11 +28,11 @@ module UiBibz::Ui::Ux::Tables
   class TablePagination < UiBibz::Ui::Base
     include WillPaginate::ActionView
     include ActionView
-    #include WillPaginate::ActionView::BootstrapLinkRenderer
+    # include WillPaginate::ActionView::BootstrapLinkRenderer
 
     # Initialize pagination with component item
     # pagination require WillPaginate gem
-    def initialize options = nil, html_options = nil
+    def initialize(options = nil, html_options = nil)
       @pagination = UiBibz::Ui::Core::Component.new nil, options, html_options
     end
 
@@ -41,25 +42,21 @@ module UiBibz::Ui::Ux::Tables
       paginate_parameters = paginate_parameters.merge({ store_id: store.id }) unless store.id.nil?
       paginate_parameters = paginate_parameters.merge(store.parameters)
       paginate_parameters = paginate_parameters.merge({ link_type: 'pagination' })
-      will_paginate(store.records, params: paginate_parameters.with_indifferent_access.reject{|k,v| default_parameters?(k) || v.blank? }, renderer: WillPaginate::ActionView::BootstrapLinkRenderer)
+      will_paginate(store.records, params: paginate_parameters.with_indifferent_access.reject { |k, v| default_parameters?(k) || v.blank? }, renderer: WillPaginate::ActionView::BootstrapLinkRenderer)
     end
 
-  private
+    private
 
-    def default_parameters?(k)
-      %w(per_page page link_type).include?(k)
+    def default_parameters?(key)
+      %w[per_page page link_type].include?(key)
     end
 
     # Store must be generated by *table_search_pagination* method
     def store
-      @store ||= if @pagination.options[:store].nil?
-        raise 'Store is nil!'
-      elsif @pagination.options[:store].try(:records).nil?
-        raise 'Store can be created only with "table_search_pagination" method!'
-      else
-        Store.new @pagination.options.delete :store
-      end
-    end
+      raise 'Store is nil!' if @pagination.options[:store].nil?
+      raise 'Store can be created only with "table_search_pagination" method!' if @pagination.options[:store].try(:records).nil?
 
+      @store ||= Store.new @pagination.options[:store]
+    end
   end
 end

data/lib/ui_bibz/ui/ux/tables/table_pagination_per_page.rb

@@ -1,6 +1,7 @@
-require "ui_bibz/ui/ux/tables/components/store"
-module UiBibz::Ui::Ux::Tables
+# frozen_string_literal: true
 
+require 'ui_bibz/ui/ux/tables/components/store'
+module UiBibz::Ui::Ux::Tables
   # Create a select pagination per page
   #
   # ==== Attributes
@@ -27,7 +28,7 @@ module UiBibz::Ui::Ux::Tables
 
     # Initialize per_page_field with component item
     # per_page_field require WillPaginate gem
-    def initialize options = nil, html_options = nil
+    def initialize(options = nil, html_options = nil)
       @per_page_field = UiBibz::Ui::Core::Component.new nil, options, html_options
     end
 
@@ -42,16 +43,13 @@ module UiBibz::Ui::Ux::Tables
       end
     end
 
-  private
+    private
 
     def store
-      @store ||= if @per_page_field.options[:store].nil?
-        raise 'Store is nil!'
-      elsif @per_page_field.options[:store].try(:records).nil?
-        raise 'Store can be created only with "table_search_pagination" method!'
-      else
-        Store.new @per_page_field.options.delete :store
-      end
+      raise 'Store is nil!' if @per_page_field.options[:store].nil?
+      raise 'Store can be created only with "table_search_pagination" method!' if @per_page_field.options[:store].try(:records).nil?
+
+      @store ||= Store.new @per_page_field.options[:store]
     end
 
     def url_parameters
@@ -61,7 +59,7 @@ module UiBibz::Ui::Ux::Tables
     def per_page_html
       content_tag :div, class: 'table-pagination-per-page' do
         concat results_count_html
-        concat UiBibz::Utils::Internationalization.new("ui_bibz.table.pagination.per_page", default: "Per page: ").translate
+        concat UiBibz::Utils::Internationalization.new('ui_bibz.table.pagination.per_page', default: 'Per page: ').translate
         concat select_tag('per_page', options_for_select([5, 10, 20, 30, 50, 100, 200, 500], store.per_page), class: 'form-control')
       end
     end
@@ -69,8 +67,8 @@ module UiBibz::Ui::Ux::Tables
     def per_page_html_in_wrap
       content_tag :div, class: 'table-pagination-per-page' do
         concat results_count_html
-        concat UiBibz::Utils::Internationalization.new("ui_bibz.table.pagination.per_page", default: "Per page: ").translate
-        store.parameters.with_indifferent_access.reject{ |k, v| default_parameters?(k) || v.blank? }.each do |k,v|
+        concat UiBibz::Utils::Internationalization.new('ui_bibz.table.pagination.per_page', default: 'Per page: ').translate
+        store.parameters.with_indifferent_access.reject { |k, v| default_parameters?(k) || v.blank? }.each do |k, v|
           concat tag(:input, type: 'hidden', name: k, value: v)
         end
         concat select_tag('per_page', options_for_select([5, 10, 20, 30, 50, 100, 200, 500], store.per_page), class: 'form-control')
@@ -79,7 +77,7 @@ module UiBibz::Ui::Ux::Tables
     end
 
     def results_count_html
-      "#{ page_entries_info store.records } | ".html_safe
+      "#{page_entries_info store.records} | ".html_safe
     end
 
     def from_current_results
@@ -90,9 +88,8 @@ module UiBibz::Ui::Ux::Tables
       store.limit_value * store.current_page
     end
 
-    def default_parameters?(k)
-      %w(store_id search controller action page link_type utf8).include?(k)
+    def default_parameters?(key)
+      %w[store_id search controller action page link_type utf8].include?(key)
     end
-
   end
 end

data/lib/ui_bibz/ui/ux/tables/table_search_field.rb

@@ -1,11 +1,11 @@
-require "ui_bibz/ui/ux/tables/components/store"
-module UiBibz::Ui::Ux::Tables
+# frozen_string_literal: true
 
+require 'ui_bibz/ui/ux/tables/components/store'
+module UiBibz::Ui::Ux::Tables
   class TableSearchField < UiBibz::Ui::Core::Component
-
     # Initialize search_field with component item
-    def initialize content = nil, options = nil, html_options = nil
-      html_options = options
+    def initialize(content = nil, options = nil, html_options = nil)
+      html_options = options # rubocop:disable Lint/ShadowedArgument
       options = content
       content = nil
       super
@@ -22,22 +22,19 @@ module UiBibz::Ui::Ux::Tables
       end
     end
 
-  private
+    private
 
     def url_parameters
-      #{ controller: store.controller, action: store.action, id: store.param_id }
+      # { controller: store.controller, action: store.action, id: store.param_id }
       store.parameters
     end
 
     # Store must be generated by *table_search_pagination* method
     def store
-      @store ||= if @options[:store].nil?
-        raise 'Store is nil!'
-      elsif @options[:store].try(:records).nil?
-        raise 'Store can be created only with "table_search_pagination" method!'
-      else
-        Store.new @options.delete :store
-      end
+      raise 'Store is nil!' if @options[:store].nil?
+      raise 'Store can be created only with "table_search_pagination" method!' if @options[:store].try(:records).nil?
+
+      @store ||= Store.new @options[:store]
     end
 
     def search_field_html
@@ -53,7 +50,7 @@ module UiBibz::Ui::Ux::Tables
     def search_field_html_in_wrap
       content_tag :div, html_options do
         concat content_tag(:span, UiBibz::Ui::Core::Icons::Glyph.new('search').render, class: 'input-group-addon')
-        store.parameters.with_indifferent_access.reject{ |k,_|  default_parameters?(k) }.each do |k,v|
+        store.parameters.with_indifferent_access.reject { |k, _| default_parameters?(k) }.each do |k, v|
           concat tag(:input, type: 'hidden', name: k, value: v)
         end
         concat tag(:input, type: 'hidden', name: 'store_id', value: store.id) unless store.id.nil? # if there is more one table in html page
@@ -76,7 +73,7 @@ module UiBibz::Ui::Ux::Tables
     end
 
     def component_html_classes
-      %w(input-group input-group-sm table-search-field)
+      %w[input-group input-group-sm table-search-field]
     end
 
     def search_placeholder_field
@@ -86,7 +83,7 @@ module UiBibz::Ui::Ux::Tables
     def searchable_attributes_sentence
       store.searchable_attributes.map do |i|
         attribute_name = underscorize_hash(i)
-        UiBibz::Utils::Internationalization.new("ui_bibz.grid.searchable.field.searchable_attributes.#{ model_name }.#{ attribute_name }", default: [translate_searchable_attributes_by_active_record(attribute_name), attribute_name.to_s.humanize]).translate
+        UiBibz::Utils::Internationalization.new("ui_bibz.grid.searchable.field.searchable_attributes.#{model_name}.#{attribute_name}", default: [translate_searchable_attributes_by_active_record(attribute_name), attribute_name.to_s.humanize]).translate
       end.to_sentence(locale: I18n.locale)
     end
 
@@ -94,17 +91,16 @@ module UiBibz::Ui::Ux::Tables
       store.model.to_s.underscore
     end
 
-    def translate_searchable_attributes_by_active_record attr
+    def translate_searchable_attributes_by_active_record(attr)
       store.model.human_attribute_name(attr)
     end
 
-    def underscorize_hash attr
-      attr.kind_of?(Hash) ? "#{ attr.keys.first }_#{ attr.values.first }".to_sym : attr
+    def underscorize_hash(attr)
+      attr.is_a?(Hash) ? "#{attr.keys.first}_#{attr.values.first}".to_sym : attr
     end
 
-    def default_parameters?(k)
-      %w(store_id search controller action utf8 link_type).include?(k)
+    def default_parameters?(key)
+      %w[store_id search controller action utf8 link_type].include?(key)
     end
-
   end
 end

data/lib/ui_bibz/utils/glyph_changer.rb

@@ -1,16 +1,17 @@
-# Glyph changer
-#
-#  => ui_link 'My link', url: '#', glyph: { type: :group, items: [{ name: 'circle', stataus: :danger },{ name: 'times', inverse: true, transform:'shrink-6'}] }
+# frozen_string_literal: true
+
 module UiBibz::Utils
+  # Glyph changer
+  #
+  #  => ui_link 'My link', url: '#', glyph: { type: :group, items: [{ name: 'circle', stataus: :danger },{ name: 'times', inverse: true, transform:'shrink-6'}] }
   class GlyphChanger
-
-    def initialize glyph_options, options = {}
+    def initialize(glyph_options, options = {})
       @glyph_options = glyph_options
       @options       = options
     end
 
     def render
-      @glyph_options.kind_of?(Hash) ? glyph_or_glyph_group : glyph_by_hash
+      @glyph_options.is_a?(Hash) ? glyph_or_glyph_group : glyph_by_hash
     end
 
     private
@@ -23,21 +24,21 @@ module UiBibz::Utils
       glyph_name      = @glyph_options.try(:[], :name)
       glyph_opts      = @glyph_options
       glyph_html_opts = @options[:html_options] || {}
-      #glyph_html_opts = glyph_html_opts.merge(@options[:text] ? {} : { title: @options[:content] })
+      # glyph_html_opts = glyph_html_opts.merge(@options[:text] ? {} : { title: @options[:content] })
       glyph_items     = @glyph_options.try(:[], :items) || []
 
       [glyph_name, glyph_opts, glyph_html_opts, glyph_items]
     end
 
     def glyph_by_hash
-      if @glyph_options.kind_of?(Hash)
+      if @glyph_options.is_a?(Hash)
         glyph_name, glyph_opts, glyph_html_opts = format_opts
       else
         glyph_name      = @glyph_options
         glyph_opts      = @options
-        #title           = [(@options[:content] unless @options[:text]), ("<kbd>#{ @options[:shortcut] }</kbd>" unless @options[:shortcut].nil?)].compact.join(" ")
+        # title           = [(@options[:content] unless @options[:text]), ("<kbd>#{ @options[:shortcut] }</kbd>" unless @options[:shortcut].nil?)].compact.join(" ")
         glyph_html_opts = @options[:html_options] || {}
-       # glyph_html_opts = glyph_html_opts.merge(title.blank? ? {} : { title: title.html_safe })
+        # glyph_html_opts = glyph_html_opts.merge(title.blank? ? {} : { title: title.html_safe })
       end
 
       UiBibz::Ui::Core::Icons::Glyph.new(glyph_name, glyph_opts, glyph_html_opts).render unless glyph_name.nil?
@@ -47,11 +48,10 @@ module UiBibz::Utils
       _, glyph_opts, glyph_html_opts, glyph_items = format_opts
 
       UiBibz::Ui::Core::Icons::GlyphGroup.new(glyph_opts, glyph_html_opts).tap do |gg|
-        glyph_items.each do|item|
-          gg.send(item[:type] ||:glyph, item[:name] || item[:content], item, item[:html_options] || {})
+        glyph_items.each do |item|
+          gg.send(item[:type] || :glyph, item[:name] || item[:content], item, item[:html_options] || {})
         end
       end.render
     end
-
   end
 end

data/lib/ui_bibz/utils/internationalization.rb

@@ -1,10 +1,12 @@
+# frozen_string_literal: true
+
 # Transalte with array of translation
 #
 #   => UiBibz::Utils::Internationalization.new("ui_bibz.table.actions.#{ @store.model.to_s.underscore }.show", default: defaults).translate
 module UiBibz::Utils
+  # Internalizations methods to translate can be used outside of Ui Bibz
   class Internationalization
-
-    def initialize translation, options = {}
+    def initialize(translation, options = {})
       @translation = translation
       @options     = options
     end
@@ -13,7 +15,7 @@ module UiBibz::Utils
       I18n.t(@translation, options_with_default)
     end
 
-  private
+    private
 
     def options_with_default
       @options[:default] = translate_default unless @options[:default].nil?
@@ -21,13 +23,14 @@ module UiBibz::Utils
     end
 
     # To know if translation missing
-    def i18n_set? key
-      I18n.t key, :raise => true rescue false
+    def i18n_set?(key)
+      I18n.t key, raise: true
+    rescue StandardError
+      false
     end
 
     def translate_default
-      I18n.t([*@options[:default]].select{ |translation| i18n_set? translation }.first, default: [*@options[:default]].last)
+      I18n.t([*@options[:default]].select { |translation| i18n_set? translation }.first, default: [*@options[:default]].last)
     end
-
   end
 end

data/lib/ui_bibz/utils/screwdriver.rb

@@ -1,20 +1,22 @@
+# frozen_string_literal: true
+
 module UiBibz::Utils
+  # Utils methods can be used outside of Ui Bibz
   class Screwdriver
     include Singleton
 
-    def self.join_classes *classes
+    def self.join_classes(*classes)
       klasses = [*classes].flatten.map(&:to_s).compact.uniq.reject(&:blank?)
       klasses.empty? ? nil : klasses
     end
 
-    def self.exclude_classes html_classes, *classes
-      klasses = (html_classes || []).flatten.map(&:to_s).reject{ |klass_name| [*classes].flatten.include?(klass_name.to_s) || klass_name.blank? }
+    def self.exclude_classes(html_classes, *classes)
+      klasses = (html_classes || []).flatten.map(&:to_s).reject { |klass_name| [*classes].flatten.include?(klass_name.to_s) || klass_name.blank? }
       klasses.empty? ? nil : klasses
     end
 
-    def self.uniq_word_in_string str
+    def self.uniq_word_in_string(str)
       str.split(/\s/).uniq
     end
-
   end
 end