agilibox 1.9.5 → 1.9.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 70cf28216c3454232717705c0f856cbbcc9c7afe64a64940031e37c244e087ca
-  data.tar.gz: 4b69d768141760b3aa70b2bbecad849a359fe6e5c4e3d312971f5f648a7187b4
+  metadata.gz: 910653d569b2435173e10da0415be0a35356421d5e6fcd112b8653c0b39f2854
+  data.tar.gz: 2d67a98fe71852d8746be01fe140826183eaf12d42a6099ce5175bfac15e296d
 SHA512:
-  metadata.gz: b895a6a5d99124b9a8fa3e7185e949b426bbe90c134441dbf1ee7e5329f8b10609ab40a3420f2067941158e9821c08a1ea1145fe8394506f7306bd309946c8ed
-  data.tar.gz: fc304783878fcfe3bfaad9d452d299ace65059046244cbe5ec0aa526107b23fe957869d26c1c39dbf306bcc755d8a1d71cea297dd42123d522b8b15b4a56bca9
+  metadata.gz: 2b5db5518d98aa662b3bef973bce64adb438beff835276b02d679aa9cba8714f71178fa3510792589b602439fd162f26f161679b2c5063ebd375b3b72a2618ce
+  data.tar.gz: 37e954d5f9c4e1a767c91e999327ac32b64f9a984153c1cccf4273423d6b1818e89b3dcab08a056889decd6e72abd84672bfd94bb05c1b2207cc7a640efa44de

data/CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ## Next version
 
+## v1.9.6
+- Fix info helper with false
+- Update Rubocop + fix offences
+
 ## 1.9.5
 - Fix warnings on Rails 6
 

data/app/controllers/concerns/agilibox/api_controller_concern.rb

@@ -37,7 +37,7 @@ module Agilibox::ApiControllerConcern
   end
 
   def json_error_string_for_model(object)
-    json_errors_hash_for_model(object).values.map { |e| e[:full_message] }.join(", ")
+    json_errors_hash_for_model(object).values.pluck(:full_message).join(", ")
   end
 
   def render_not_found

data/app/filters/agilibox/small_data/filter.rb

@@ -43,7 +43,7 @@ class Agilibox::SmallData::Filter
   end
 
   def respond_to_missing?(method, *)
-    strategies.key?(method.to_s) || strategies.key?(method.to_s.chomp("="))
+    super || strategies.key?(method.to_s) || strategies.key?(method.to_s.chomp("="))
   end
 
   def read

data/app/filters/agilibox/small_data/filter_strategy_by_key_value.rb

@@ -2,6 +2,7 @@ class Agilibox::SmallData::FilterStrategyByKeyValue < ::Agilibox::SmallData::Fil
   attr_reader :key
 
   def initialize(key = nil)
+    super()
     @key = key
   end
 

data/app/helpers/agilibox/bootstrap_helper.rb

@@ -13,8 +13,8 @@ module Agilibox::BootstrapHelper
   end
 
   def bs_progress_bar(percentage)
-    content_tag(:div, class: "progress") do
-      content_tag(:div, class: "progress-bar", style: "width:#{percentage}%") do
+    tag.div(class: "progress") do
+      tag.div(class: "progress-bar", style: "width:#{percentage}%") do
         "#{percentage}%"
       end
     end

data/app/helpers/agilibox/filters_helper.rb

@@ -7,7 +7,7 @@ module Agilibox::FiltersHelper
     text = options.delete(:text) || t("actions.filter")
     icon = options.delete(:icon) || :filter
 
-    content_tag(:button, options) do
+    tag.button(options) do
       icon(icon) + " " + text
     end
   end
@@ -20,7 +20,7 @@ module Agilibox::FiltersHelper
     text = options.delete(:text) || t("actions.reset")
     icon = options.delete(:icon) || :undo
 
-    content_tag(:button, options) do
+    tag.button(options) do
       icon(icon) + " " + text
     end
   end

data/app/helpers/agilibox/font_awesome_helper.rb

@@ -15,7 +15,7 @@ module Agilibox::FontAwesomeHelper
 
     attributes = options.merge(class: css_classes.sort.join(" ")).sort.to_h
 
-    content_tag(:span, attributes) {}
+    tag.span(attributes) {}
   end
 
   class << self

data/app/helpers/agilibox/form_helper.rb

@@ -21,13 +21,13 @@ module Agilibox::FormHelper
       submit_action = :save
     end
 
-    content_tag("div", class: "actions form-actions") do
-      submit = content_tag(:button, type: :submit, class: "btn btn-sm btn-success form-submit") do
-        content_tag(:span, class: "fa fa-save") {} + " " + ta(submit_action)
+    tag.div(class: "actions form-actions") do
+      submit = tag.button(type: :submit, class: "btn btn-sm btn-success form-submit") do
+        tag.span(class: "fa fa-save") {} + " " + ta(submit_action)
       end
 
-      cancel = content_tag("a", href: back_url, class: "btn btn-primary btn-sm form-cancel") do
-        content_tag(:span, class: "fa fa-times") {} + " " + ta(:cancel)
+      cancel = tag.a(href: back_url, class: "btn btn-primary btn-sm form-cancel") do
+        tag.span(class: "fa fa-times") {} + " " + ta(:cancel)
       end
 
       cancel = "" if back_url == false

data/app/helpers/agilibox/pagination_helper.rb

@@ -5,7 +5,7 @@ module Agilibox::PaginationHelper
   end
 
   def pagination_infos(collection)
-    content_tag(:p, class: "pagination-infos") { page_entries_info(collection) }
+    tag.p(class: "pagination-infos") { page_entries_info(collection) }
   end
 
   def pagination_and_infos(collection)

data/app/helpers/agilibox/text_helper.rb

@@ -93,13 +93,16 @@ module Agilibox::TextHelper
       value = value_or_options
     end
 
+    value = t("yes") if value == true
+    value = t("no")  if value == false
+
     if value.blank?
       value = options[:default]
       return if value == :hide
     end
 
     label       = options[:label]     || object.t(attribute)
-    tag         = options[:tag]       || :div
+    info_tag    = options[:tag]       || :div
     separator   = options[:separator] || " : "
     separator   = " :<br/>".html_safe if separator == :br
     helper      = options[:helper]
@@ -108,8 +111,6 @@ module Agilibox::TextHelper
     klass       = object.is_a?(Module) ? object : object.class
     object_type = klass.to_s.split("::").last.underscore
 
-    value = t("yes")             if value == true
-    value = t("no")              if value == false
     value = object.tv(attribute) if nested
     value = send(helper, value)  if helper
     value = number(value)        if value.is_a?(Numeric)
@@ -117,10 +118,10 @@ module Agilibox::TextHelper
     value = l(value)             if value.is_a?(Date)
     value = value.to_s
 
-    html_label     = content_tag(:strong, class: "info-label") { label }
+    html_label     = tag.strong(class: "info-label") { label }
     span_css_class = "info-value #{object_type}-#{attribute}"
-    html_value     = content_tag(:span, class: span_css_class) { value }
-    separator_html = content_tag(:span, class: "info-separator") { separator }
+    html_value     = tag.span(class: span_css_class) { value }
+    separator_html = tag.span(class: "info-separator") { separator }
 
     if value.blank?
       container_css_class = "info blank"
@@ -128,7 +129,7 @@ module Agilibox::TextHelper
       container_css_class = "info"
     end
 
-    content_tag(tag, class: container_css_class) do
+    content_tag(info_tag, class: container_css_class) do
       [html_label, separator_html, html_value].join.html_safe
     end
   end # def info
@@ -138,7 +139,7 @@ module Agilibox::TextHelper
     return "" if object.tag_list.empty?
 
     object.tag_list.map { |tag|
-      content_tag(:span, class: "tag label label-primary") {
+      tag.span(class: "tag label label-primary") {
         "#{icon :tag} #{tag}".html_safe
       }
     }.join(" ").html_safe

data/app/libs/agilibox/fcm/notifier.rb

@@ -3,6 +3,7 @@ class Agilibox::FCM::Notifier < Agilibox::Service
 
   # rubocop:disable Metrics/ParameterLists
   def initialize(to:, title: nil, body:, badge: 0, sound: "default", data: {})
+    super()
     @to    = to
     @title = title
     @body  = body

data/app/libs/agilibox/fcm/request.rb

@@ -39,7 +39,7 @@ class Agilibox::FCM::Request < Agilibox::Service
   end
 
   def errors
-    response_json[:results].map { |r| r[:error] }.compact
+    response_json[:results].pluck(:error).compact
   end
 
   def invalid_token?

data/app/libs/agilibox/mini_model_serializer/serializer.rb

@@ -18,6 +18,6 @@ class Agilibox::MiniModelSerializer::Serializer < Agilibox::MiniModelSerializer:
   end
 
   def respond_to_missing?(m, *)
-    object.respond_to?(m, true)
+    super || object.respond_to?(m, true)
   end
 end

data/app/libs/agilibox/phone_number_sanitizer.rb

@@ -6,7 +6,7 @@ class Agilibox::PhoneNumberSanitizer
   end
 
   def call
-    phone_number.to_s.gsub(/[^0-9\+]+/, "").presence
+    phone_number.to_s.gsub(/[^0-9+]+/, "").presence
   end
 
   def self.call(*args)

data/app/sms/agilibox/sms/application_sms.rb

@@ -36,7 +36,7 @@ class Agilibox::SMS::ApplicationSMS
     end
 
     def respond_to_missing?(m, *)
-      public_instance_methods.include?(m)
+      super || public_instance_methods.include?(m)
     end
   end # class << self
 end

data/lib/agilibox/active_model_type_cast.rb

@@ -2,7 +2,7 @@ module Agilibox::ActiveModelTypeCast
   module Decimal
     def cast_value(value)
       if value.is_a?(String)
-        super value.tr(",", ".").gsub(/[^-0-9\.]/, "")
+        super value.tr(",", ".").gsub(/[^-0-9.]/, "")
       else
         super value
       end

data/lib/agilibox/version.rb

@@ -1,3 +1,3 @@
 module Agilibox
-  VERSION = "1.9.5"
+  VERSION = "1.9.6"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: agilibox
 version: !ruby/object:Gem::Version
-  version: 1.9.5
+  version: 1.9.6
 platform: ruby
 authors:
 - agilidée
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-06-04 00:00:00.000000000 Z
+date: 2020-09-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails-i18n
@@ -228,7 +228,7 @@ homepage: https://github.com/agilidee/agilibox
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -244,7 +244,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.0.3
-signing_key: 
+signing_key:
 specification_version: 4
 summary: Agilibox
 test_files: []