agilibox 1.9.4 → 1.9.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dc669342f537268da38524e9f81e53596461c9f1bebe3c041d50b61c566c67c3
-  data.tar.gz: 7858d8529558a1578128cf2c4dc2626fd0d19c48f8166eec2c0c31308c23b3df
+  metadata.gz: '01460916eeacaa37c86a5d34195d8020658a4de645d0e5e6eb231d52eb884880'
+  data.tar.gz: 87920f596a3e90b353e7413f1bbf3c90ad44ed12ccaa7fb1b6ed515d03b17c8b
 SHA512:
-  metadata.gz: 1299f5905563c84ebd1ab668e24c3da052fbda8073a9e551d0d070cce7c1a06af2e3e53a189918f7ab6e29450b7532f5fd1b5a76dd5dcb3415ec579681cfd8c0
-  data.tar.gz: 11cc2e97f14993636d36f2ce183c30915f532edcf5420465b575846b0d5f79545ac497606f89b0e5683e23f565017efe3634738c166b97a74176a37dd7213e83
+  metadata.gz: 73de451eed71b4ef4c7f2497125d8bc73fac702210814b5769915a86c80262a85a0f32e4b81e74c7ddd6526c300ff283325a2cf8a209fb3f37916e4e9b514f7e
+  data.tar.gz: f16fbc412ce7ab063bb58addbb642a9f03dbe4f810c2b7f6032b796f3d366cb935070428214281ea4c7d8a007adc0108ea47ea317a829ea1dcedb7ae2ba38993

data/CHANGELOG.md

@@ -2,6 +2,22 @@
 
 ## Next version
 
+## v1.9.9
+- Add error to ErrorsMiddleware
+
+## v1.9.8
+- Fix `tags` helper
+
+## v1.9.7
+- Change XLSX export
+
+## v1.9.6
+- Fix info helper with false
+- Update Rubocop + fix offences
+
+## 1.9.5
+- Fix warnings on Rails 6
+
 ## 1.9.4
 - Cucumber : disable Zonebie
 

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/routes_helper.rb

@@ -1,6 +1,10 @@
 module Agilibox::RoutesHelper
   def engine_polymorphic_path(obj, opts = {})
-    engine = obj.class.parents[-2]
+    if Rails::VERSION::STRING >= "6.0.0"
+      engine = obj.class.module_parents[-2]
+    else
+      engine = obj.class.parents[-2]
+    end
 
     if engine.nil?
       routes = main_app

data/app/helpers/agilibox/sorting_helper.rb

@@ -38,7 +38,7 @@ module Agilibox::SortingHelper
 
     if sort_param.present?
       if sort_param.start_with?("-")
-        column     = sort_param[1..-1].to_sym
+        column     = sort_param[1..].to_sym
         direction  = :desc
       else
         column     = sort_param.to_sym

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
@@ -137,9 +138,9 @@ module Agilibox::TextHelper
   def tags(object)
     return "" if object.tag_list.empty?
 
-    object.tag_list.map { |tag|
-      content_tag(:span, class: "tag label label-primary") {
-        "#{icon :tag} #{tag}".html_safe
+    object.tag_list.map { |tag_name|
+      tag.span(class: "tag label label-primary") {
+        "#{icon :tag} #{tag_name}".html_safe
       }
     }.join(" ").html_safe
   end

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/serializers/agilibox/serializers/xlsx.rb

@@ -1,6 +1,7 @@
 class Agilibox::Serializers::XLSX < Agilibox::Serializers::Base
   def render_inline
-    SpreadsheetArchitect.to_xlsx(data: formatted_data)
+    headers, *data = formatted_data
+    SpreadsheetArchitect.to_xlsx(headers: headers, data: data, freeze_headers: true)
   end
 
   def render_file(file_path)

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_custom_error_messages.rb

@@ -1,6 +1,6 @@
 module Agilibox::ActiveModelCustomErrorMessages
   def full_message(attribute, message)
-    return message[1..-1] if message[0] == "^"
+    return message[1..] if message[0] == "^"
     super(attribute, message)
   end
 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/errors_middleware.rb

@@ -10,6 +10,7 @@ class Agilibox::ErrorsMiddleware
     "ActionController::UnknownFormat",
     "ActionController::UnknownHttpMethod",
     "ActionView::MissingTemplate",
+    "Mime::Type::InvalidMimeType",
   ]
 
   def initialize(app)

data/lib/agilibox/version.rb

@@ -1,3 +1,3 @@
 module Agilibox
-  VERSION = "1.9.4"
+  VERSION = "1.9.9"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: agilibox
 version: !ruby/object:Gem::Version
-  version: 1.9.4
+  version: 1.9.9
 platform: ruby
 authors:
 - agilidée
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-01-27 00:00:00.000000000 Z
+date: 2020-10-22 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: []