system_settings 0.7.0 → 0.9.2

data/app/models/system_settings/errors/error.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-module SystemSettings
-  module Errors
-    class Error < StandardError
-    end
+module SystemSettings::Errors
+  class Error < StandardError
   end
 end

data/app/models/system_settings/errors/not_found_error.rb

@@ -1,8 +1,13 @@
 # frozen_string_literal: true
 
-module SystemSettings
-  module Errors
-    class NotFoundError < SystemSettings::Errors::Error
+module SystemSettings::Errors
+  class NotFoundError < SystemSettings::Errors::Error
+    if ActiveSupport.const_defined?("ActionableError")
+      include ActiveSupport::ActionableError
+
+      action "Load all settings" do
+        SystemSettings.load
+      end
     end
   end
 end

data/app/models/system_settings/errors/not_loaded_error.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-module SystemSettings
-  module Errors
-    class NotLoadedError < SystemSettings::Errors::Error
-    end
+module SystemSettings::Errors
+  class NotLoadedError < SystemSettings::Errors::Error
   end
 end

data/app/models/system_settings/errors/settings_read_error.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-module SystemSettings
-  module Errors
-    class SettingsReadError < SystemSettings::Errors::Error
-    end
+module SystemSettings::Errors
+  class SettingsReadError < SystemSettings::Errors::Error
   end
 end

data/app/models/system_settings/integer_list_setting.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-module SystemSettings
-  class IntegerListSetting < SystemSettings::Setting
-    attribute :value, SystemSettings::Type::IntegerList.new
-    validates :value, "system_settings/list_of_integers": true
-  end
+class SystemSettings::IntegerListSetting < SystemSettings::Setting
+  attribute :value, SystemSettings::Type::IntegerList.new(limit: 8)
+  validates :value, "system_settings/list_of_integers": true
 end

data/app/models/system_settings/integer_setting.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-module SystemSettings
-  class IntegerSetting < SystemSettings::Setting
-    attribute :value, :integer
-    validates :value, numericality: { only_integer: true }
-  end
+class SystemSettings::IntegerSetting < SystemSettings::Setting
+  attribute :value, :integer, limit: 8
+  validates :value, numericality: { only_integer: true }
 end

data/app/models/system_settings/list_of_decimals_validator.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+class SystemSettings::ListOfDecimalsValidator < ActiveModel::EachValidator
+  LIST_REGEXP = /\A[+-]?\d+(?:\.\d+){0,1}(?:; *[+-]?\d+(?:\.\d+){0,1})*\z/.freeze
+  SINGLE_REGEXP = /\A[+-]?\d+(?:\.\d+){0,1}\z/.freeze
+
+  def validate_each(record, attr_name, value)
+    came_from_user = :"#{attr_name}_came_from_user?"
+
+    raw_value = record.read_attribute_before_type_cast(attr_name) if record.respond_to?(came_from_user) && record.public_send(came_from_user)
+    raw_value ||= value
+
+    raw_value = value if record_attribute_changed_in_place?(record, attr_name)
+
+    record.errors.add(attr_name, :not_a_list_of_decimals) unless matches_list_of_decimals_regexp?(raw_value)
+  end
+
+  private
+
+  def record_attribute_changed_in_place?(record, attr_name)
+    record.respond_to?(:attribute_changed_in_place?) &&
+      record.attribute_changed_in_place?(attr_name.to_s)
+  end
+
+  def matches_list_of_decimals_regexp?(raw_value)
+    case raw_value
+    when String
+      LIST_REGEXP.match?(raw_value)
+    when Array
+      raw_value.all? { |v| SINGLE_REGEXP.match?(v.to_s) }
+    else
+      false
+    end
+  end
+end

data/app/models/system_settings/list_of_integers_validator.rb

@@ -1,37 +1,35 @@
 # frozen_string_literal: true
 
-module SystemSettings
-  class ListOfIntegersValidator < ActiveModel::EachValidator
-    LIST_REGEXP = /\A[+-]?\d+(?:; *[+-]?\d+)*\z/.freeze
-    SINGLE_REGEXP = /\A[+-]?\d+\z/.freeze
+class SystemSettings::ListOfIntegersValidator < ActiveModel::EachValidator
+  LIST_REGEXP = /\A[+-]?\d+(?:; *[+-]?\d+)*\z/.freeze
+  SINGLE_REGEXP = /\A[+-]?\d+\z/.freeze
 
-    def validate_each(record, attr_name, value)
-      came_from_user = :"#{attr_name}_came_from_user?"
+  def validate_each(record, attr_name, value)
+    came_from_user = :"#{attr_name}_came_from_user?"
 
-      raw_value = record.read_attribute_before_type_cast(attr_name) if record.respond_to?(came_from_user) && record.public_send(came_from_user)
-      raw_value ||= value
+    raw_value = record.read_attribute_before_type_cast(attr_name) if record.respond_to?(came_from_user) && record.public_send(came_from_user)
+    raw_value ||= value
 
-      raw_value = value if record_attribute_changed_in_place?(record, attr_name)
+    raw_value = value if record_attribute_changed_in_place?(record, attr_name)
 
-      record.errors.add(attr_name, :not_a_list_of_integers) unless matches_list_of_integers_regexp?(raw_value)
-    end
+    record.errors.add(attr_name, :not_a_list_of_integers) unless matches_list_of_integers_regexp?(raw_value)
+  end
 
-    private
+  private
 
-    def record_attribute_changed_in_place?(record, attr_name)
-      record.respond_to?(:attribute_changed_in_place?) &&
-        record.attribute_changed_in_place?(attr_name.to_s)
-    end
+  def record_attribute_changed_in_place?(record, attr_name)
+    record.respond_to?(:attribute_changed_in_place?) &&
+      record.attribute_changed_in_place?(attr_name.to_s)
+  end
 
-    def matches_list_of_integers_regexp?(raw_value)
-      case raw_value
-      when String
-        LIST_REGEXP.match?(raw_value)
-      when Array
-        raw_value.all? { |v| SINGLE_REGEXP.match?(v.to_s) }
-      else
-        false
-      end
+  def matches_list_of_integers_regexp?(raw_value)
+    case raw_value
+    when String
+      LIST_REGEXP.match?(raw_value)
+    when Array
+      raw_value.all? { |v| SINGLE_REGEXP.match?(v.to_s) }
+    else
+      false
     end
   end
 end

data/app/models/system_settings/list_of_strings_validator.rb

@@ -1,38 +1,36 @@
 # frozen_string_literal: true
 
-module SystemSettings
-  class ListOfStringsValidator < ActiveModel::EachValidator
-    NON_WHITESPACE_REGEXP = /[^[:space:]]/.freeze
+class SystemSettings::ListOfStringsValidator < ActiveModel::EachValidator
+  NON_WHITESPACE_REGEXP = /[^[:space:]]/.freeze
 
-    def validate_each(record, attr_name, value)
-      came_from_user = :"#{attr_name}_came_from_user?"
+  def validate_each(record, attr_name, value)
+    came_from_user = :"#{attr_name}_came_from_user?"
 
-      raw_value = record.read_attribute_before_type_cast(attr_name) if record.respond_to?(came_from_user) && record.public_send(came_from_user)
-      raw_value ||= value
+    raw_value = record.read_attribute_before_type_cast(attr_name) if record.respond_to?(came_from_user) && record.public_send(came_from_user)
+    raw_value ||= value
 
-      raw_value = value if record_attribute_changed_in_place?(record, attr_name)
+    raw_value = value if record_attribute_changed_in_place?(record, attr_name)
 
-      record.errors.add(attr_name, :not_a_list_of_strings) unless matches_list_of_strings_regexp?(raw_value)
-    end
+    record.errors.add(attr_name, :not_a_list_of_strings) unless matches_list_of_strings_regexp?(raw_value)
+  end
 
-    private
+  private
 
-    def record_attribute_changed_in_place?(record, attr_name)
-      record.respond_to?(:attribute_changed_in_place?) &&
-        record.attribute_changed_in_place?(attr_name.to_s)
-    end
+  def record_attribute_changed_in_place?(record, attr_name)
+    record.respond_to?(:attribute_changed_in_place?) &&
+      record.attribute_changed_in_place?(attr_name.to_s)
+  end
 
-    def matches_list_of_strings_regexp?(raw_value)
-      case raw_value
-      when String
-        raw_value.split(SystemSettings::Type::StringList::DELIMITER_REGEXP).all? do |value|
-          NON_WHITESPACE_REGEXP.match?(value)
-        end
-      when Array
-        raw_value.all? { |v| NON_WHITESPACE_REGEXP.match?(v.to_s) }
-      else
-        false
+  def matches_list_of_strings_regexp?(raw_value)
+    case raw_value
+    when String
+      raw_value.split(SystemSettings::Type::StringList::DELIMITER_REGEXP).all? do |value|
+        NON_WHITESPACE_REGEXP.match?(value)
       end
+    when Array
+      raw_value.all? { |v| NON_WHITESPACE_REGEXP.match?(v.to_s) }
+    else
+      false
     end
   end
 end

data/app/models/system_settings/setting.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-module SystemSettings
-  class Setting < SystemSettings::ApplicationRecord
-    validates :type, presence: true
-    validates :name, presence: true, uniqueness: true
-  end
+class SystemSettings::Setting < SystemSettings::ApplicationRecord
+  validates :type, presence: true
+  validates :name, presence: true, uniqueness: true
 end

data/app/models/system_settings/string_list_setting.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-module SystemSettings
-  class StringListSetting < SystemSettings::Setting
-    attribute :value, SystemSettings::Type::StringList.new
-    validates :value, "system_settings/list_of_strings": true
-  end
+class SystemSettings::StringListSetting < SystemSettings::Setting
+  attribute :value, SystemSettings::Type::StringList.new
+  validates :value, "system_settings/list_of_strings": true
 end

data/app/models/system_settings/string_setting.rb

@@ -1,8 +1,11 @@
 # frozen_string_literal: true
 
-module SystemSettings
-  class StringSetting < SystemSettings::Setting
-    attribute :value, :string
-    validates :value, presence: true
+class SystemSettings::StringSetting < SystemSettings::Setting
+  attribute :value, :string
+  validates :value, presence: true
+
+  def value=(original_value)
+    next_value = original_value.to_s.blank? ? nil : original_value.to_s.strip
+    super(next_value)
   end
 end

data/app/models/system_settings/type/decimal_list.rb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+module SystemSettings::Type
+  class DecimalList < ActiveModel::Type::Value
+    SEPARATOR = ";"
+
+    def initialize(precision: nil, limit: nil, scale: nil)
+      super
+      @single_type = ActiveModel::Type::Decimal.new(precision: precision, limit: limit, scale: scale)
+    end
+
+    def type
+      :decimal_list
+    end
+
+    def deserialize(value)
+      result = value.presence && JSON.parse(value)
+      if result.is_a?(Array)
+        result.map { |v| @single_type.cast(v) }
+      else
+        result
+      end
+    end
+
+    def serialize(value)
+      JSON.dump(value) unless value.nil?
+    end
+
+    private
+
+    def cast_value(value)
+      case value
+      when Array
+        value.map { |v| @single_type.cast(v) }
+      when String
+        value.split(SEPARATOR).map { |v| @single_type.cast(v) }
+      end
+    end
+  end
+end

data/app/models/system_settings/type/integer_list.rb

@@ -1,43 +1,34 @@
 # frozen_string_literal: true
 
-module SystemSettings
-  module Type
-    class IntegerList < ActiveModel::Type::Value
-      SEPARATOR = ";"
+module SystemSettings::Type
+  class IntegerList < ActiveModel::Type::Value
+    SEPARATOR = ";"
 
-      def type
-        :integer_list
-      end
+    def initialize(precision: nil, limit: nil, scale: nil)
+      super
+      @single_type = ActiveModel::Type::Integer.new(precision: precision, limit: limit, scale: scale)
+    end
 
-      def deserialize(value)
-        value.presence && JSON.parse(value)
-      end
+    def type
+      :integer_list
+    end
 
-      def serialize(value)
-        JSON.dump(value) unless value.nil?
-      end
+    def deserialize(value)
+      value.presence && JSON.parse(value)
+    end
+
+    def serialize(value)
+      JSON.dump(value) unless value.nil?
+    end
 
-      private
+    private
 
-      def cast_value(value)
-        case value
-        when Array
-          value.map do |v|
-            begin
-              v.to_i
-            rescue StandardError
-              nil
-            end
-          end
-        when String
-          value.split(SEPARATOR).map do |v|
-            begin
-              v.to_i
-            rescue StandardError
-              nil
-            end
-          end
-        end
+    def cast_value(value)
+      case value
+      when Array
+        value.map { |v| @single_type.cast(v) }
+      when String
+        value.split(SEPARATOR).map { |v| @single_type.cast(v) }
       end
     end
   end

data/app/models/system_settings/type/string_list.rb

@@ -1,30 +1,28 @@
 # frozen_string_literal: true
 
-module SystemSettings
-  module Type
-    class StringList < ActiveModel::Type::Value
-      DELIMITER_REGEXP = /(?<=[^\\]);/.freeze
-      def type
-        :string_list
-      end
+module SystemSettings::Type
+  class StringList < ActiveModel::Type::Value
+    DELIMITER_REGEXP = /(?<=[^\\]);/.freeze
+    def type
+      :string_list
+    end
 
-      def deserialize(value)
-        value.presence && JSON.parse(value)
-      end
+    def deserialize(value)
+      value.presence && JSON.parse(value)
+    end
 
-      def serialize(value)
-        JSON.dump(value) unless value.nil?
-      end
+    def serialize(value)
+      JSON.dump(value) unless value.nil?
+    end
 
-      private
+    private
 
-      def cast_value(value)
-        case value
-        when Array
-          value.map { |v| String(v).strip }
-        when String
-          value.split(DELIMITER_REGEXP).map(&:strip).map { |str| str.gsub("\\;", ";") }
-        end
+    def cast_value(value)
+      case value
+      when Array
+        value.map { |v| String(v).strip }
+      when String
+        value.split(DELIMITER_REGEXP).map(&:strip).map { |str| str.gsub("\\;", ";") }
       end
     end
   end

data/app/views/layouts/system_settings/application.html.erb

@@ -0,0 +1,21 @@
+<!DOCTYPE html>
+<html>
+  <head>
+    <meta charset="utf-8" />
+    <title><%= SystemSettings::Setting.model_name.human(count: 2) %> - <%= SystemSettings::VERSION %></title>
+    <%= csrf_meta_tags %>
+    <%= stylesheet_link_tag 'system_settings/application', media: 'all' %>
+  </head>
+  <body>
+    <header>
+      <div class="container">
+        <%= link_to SystemSettings::Setting.model_name.human(count: 2), root_path, class: "app-name" %>
+      </div>
+    </header>
+    <main>
+      <div class="container">
+        <%= yield %>
+      </div>
+    </main>
+  </body>
+</html>

data/app/views/system_settings/settings/_common_attributes.html.erb

@@ -0,0 +1,14 @@
+<div class="attribute">
+  <div class="name"><%= setting.class.human_attribute_name(:name) %></div>
+  <div class="value">
+    <span class="sys-name"><%= setting.name %></span>
+  </div>
+</div>
+<div class="attribute">
+  <div class="name"><%= setting.class.human_attribute_name(:type) %></div>
+  <div class="value"><%= setting.model_name.human %></div>
+</div>
+<div class="attribute">
+  <div class="name"><%= setting.class.human_attribute_name(:description) %></div>
+  <div class="value"><%= setting.description %></div>
+</div>

data/app/views/system_settings/settings/_form.html.erb

@@ -0,0 +1,18 @@
+<%= form_for setting, as: :setting, url: setting_path(setting), method: "post" do |f| %>
+  <div class="attribute">
+    <%= f.label :value, class: "name" %>
+    <% if f.object.is_a?(SystemSettings::BooleanSetting) %>
+      <%= f.check_box :value, value: f.object.value %>
+    <% else %>
+      <%= f.text_field :value, value: format_value_for_form(f.object), class: "value" %>
+    <% end %>
+    <% if f.object.errors.messages[:value].any? %>
+      <div class="errors"><%= f.object.errors.messages[:value].join("; ") %></div>
+    <% end %>
+    <div class="hint"><%= t(f.object.type, scope: "system_settings.settings.form.hints_by_type", default: "").html_safe %></div>
+  </div>
+  <div class="buttons">
+  <%= f.submit t(".save"), class: "primary" %>
+  <%= link_to t(".back"), setting_path(setting), class: "button" %>
+  </div>
+<% end %>

data/app/views/system_settings/settings/edit.html.erb

@@ -0,0 +1,2 @@
+<%= render "common_attributes", setting: @setting %>
+<%= render "form", setting: @setting %>

data/app/views/system_settings/settings/index.html.erb

@@ -0,0 +1,29 @@
+<table class="settings-table">
+  <thead>
+    <tr>
+      <th class="name"><%= SystemSettings::Setting.human_attribute_name(:name) %></th>
+      <th class="description"><%= SystemSettings::Setting.human_attribute_name(:description) %></th>
+      <th class="value"><%= SystemSettings::Setting.human_attribute_name(:value) %></th>
+    </tr>
+  </thead>
+  <tbody>
+  <% if @settings.any? %>
+    <% @settings.each do |setting| %>
+      <tr>
+        <td class="name"><%= link_to setting.name, setting_path(setting), class: "sys-name" %></td>
+        <td class="description"><%= setting.description %></td>
+        <td class="value"><%= format_value setting.value %></td>
+      </tr>
+    <% end %>
+  <% else %>
+    <tr>
+      <td colspan="3" class="no-entries"><%= t(".no_settings").html_safe %></td>
+    </tr>
+  <% end %>
+  </tbody>
+  <tfoot>
+    <tr>
+      <td colspan="3"><%= t(".add_new", path: display_settings_file_path).html_safe %></td>
+    </tr>
+  </tfoot>
+</table>

data/app/views/system_settings/settings/show.html.erb

@@ -0,0 +1,9 @@
+<%= render "common_attributes", setting: @setting %>
+<div class="attribute">
+  <div class="name"><%= @setting.class.human_attribute_name(:value) %></div>
+  <div class="value"><%= format_value @setting.value %></div>
+</div>
+<div class="buttons">
+  <%= link_to t(".edit"), edit_setting_path(@setting), class: "button primary" %>
+  <%= link_to t(".back"), root_path, class: "button" %>
+</div>

data/config/locales/system_settings.en.yml

@@ -16,11 +16,21 @@ en:
       system_settings/string_setting:
         one: "System Setting (String)"
         other: "System Settings (String)"
+      system_settings/decimal_setting:
+        one: "System Setting (Decimal)"
+        other: "System Settings (Decimal)"
+      system_settings/decimal_list_setting:
+        one: "System Setting (Decimal list)"
+        other: "System Settings (Decimal list)"
+      system_settings/boolean_setting:
+        one: "System Setting (Boolean)"
+        other: "System Settings (Boolean)"
     attributes:
       system_settings/setting:
         id: "ID"
         name: "Name"
         type: "Type"
+        value: "Value"
         description: "Description"
         created_at: "Created at"
         uploaded_at: "Updated at"
@@ -31,7 +41,23 @@ en:
             value:
               not_a_list_of_integers: "not a list of integers"
               not_a_list_of_strings: "not a list of strings"
+              not_a_list_of_decimals: "not a list of decimals"
         system_settings/boolean_setting:
           attributes:
             value:
-              inclusion: "must be set to true or false"
+              inclusion: "must be set to true or false"
+  system_settings:
+    settings:
+      index:
+        no_settings: "No settings"
+        add_new: "Add new entries in <code>%{path}</code>"
+      show:
+        edit: Edit
+        back: Back
+      form:
+        save: Save
+        back: Back
+        hints_by_type:
+          "SystemSettings::StringListSetting": "Separate values by <code>;</code> character."
+          "SystemSettings::IntegerListSetting": "Separate values by <code>;</code> character."
+          "SystemSettings::DecimalListSetting": "Decimal separator is the <code>.</code> character. Separate multiple values by <code>;</code> character."

data/config/locales/system_settings.lv.yml

@@ -0,0 +1,65 @@
+lv:
+  activerecord:
+    models:
+      system_settings/setting:
+        one: "Sistēmas iestatījums"
+        other: "Sistēmas iestatījumi"
+      system_settings/integer_list_setting:
+        one: "Sistēmas iestatījums (Veselu skaitļu virkne)"
+        other: "Sistēmas iestatījumi (Veselu skaitļu virkne)"
+      system_settings/integer_setting:
+        one: "Sistēmas iestatījums (Vesels skaitlis)"
+        other: "Sistēmas iestatījumi (Veseli skaitļi)"
+      system_settings/string_list_setting:
+        one: "Sistēmas iestatījums (Vārdu virkne)"
+        other: "Sistēmas iestatījumi (Vārdu virkne)"
+      system_settings/string_setting:
+        one: "Sistēmas iestatījums (Vārds)"
+        other: "Sistēmas iestatījumi (Vārds)"
+      system_settings/decimal_setting:
+        one: "Sistēmas iestatījums (Decimālskaitlis)"
+        other: "Sistēmas iestatījumi (Decimālskaitļi)"
+      system_settings/decimal_list_setting:
+        one: "Sistēmas iestatījums (Decimālskaitļu virkne)"
+        other: "Sistēmas iestatījumi (Decimālskaitļu virkne)"
+      system_settings/boolean_setting:
+        one: "Sistēmas iestatījums (Patiesumvērtība)"
+        other: "Sistēmas iestatījumi (Patiesumvērtības)"
+    attributes:
+      system_settings/setting:
+        id: "ID"
+        name: "Nosaukums"
+        type: "Tips"
+        value: "Vērtība"
+        description: "Apraksts"
+        created_at: "Izveides datums"
+        uploaded_at: "Atjaunošanas datums"
+    errors:
+      models:
+        system_settings/setting:
+          attributes:
+            value:
+              blank: "vērtībai ir jābūt aizpildītai"
+              not_a_number: "vērtība nav skaitlis"
+              not_a_list_of_integers: "vērtība nav virkne ar veseliem skaitļiem"
+              not_a_list_of_strings: "vērtība nav virke ar vārdiem"
+              not_a_list_of_decimals: "vērtība nav virkne ar decimālskaitļiem"
+        system_settings/boolean_setting:
+          attributes:
+            value:
+              inclusion: "ir jābūt aizpildītam"
+  system_settings:
+    settings:
+      index:
+        no_settings: "Nav ierakstu"
+        add_new: "Jaunus ierakstus iespējams pievienot: <code>%{path}</code>"
+      show:
+        edit: Labot
+        back: Atpakaļ
+      form:
+        save: Saglabāt
+        back: Atpakaļ
+        hints_by_type:
+          "SystemSettings::StringListSetting": "Vairākas vērtības atdala ar <code>;</code> simbolu."
+          "SystemSettings::IntegerListSetting": "Vairākas vērtības atdala ar <code>;</code> simbolu."
+          "SystemSettings::DecimalListSetting": "Decimālskaitļu veselo skaitli no decimāldaļas atdala ar <code>.</code> simbolu. Vairākas vērtības atdala ar <code>;</code> simbolu."