system_settings 0.8.0 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '0342639e1515fcf5d530d0561009f375199df5b86d00ee80cd1517bd9522fbf9'
-  data.tar.gz: ce20dab7409179611cdf5c223df76d5b9627cdb17ee10e90e0bff6dfbdb794c5
+  metadata.gz: 321c73f1b71d88e67c98191025ac69b6fa58ea6dbbaa838cb5a6eaea6622337a
+  data.tar.gz: 17bd42a54230f37063f618fe2cfebc3875410d3879757e9ed38138990c6a952d
 SHA512:
-  metadata.gz: 49e264aa4aee4f8451362b1687fbd3ac8a73b3584fc012b3d2f20c2bc5da931f780dace23589bcbc587d1ddef7f42fe5f2a4d92b33a96d66e8c8c9ff55cfa636
-  data.tar.gz: a2a95984c03860e6249007cc17739a445f5208c7ef985c49f0151565c682bd3510fedf3338fcc56985b2c582ff8508a5c1d840c5739811303a3a6dfaca8aaeda
+  metadata.gz: 4231c986165f29d944aec79b4cc5239ccc9f7d614ef710ec77e452a238f8579ec3afcf20c09c0701ec49b3a89547ebbfcc3e4f395a71d024b1349fadb8df1cfc
+  data.tar.gz: bab2dd0fbf7b7f5f38ea38e4ebdcb1d732eb0254a2e3278925ac16408cca8d3c05c898137c92e7916c2b4f85c6105a9ad1e1ec5be0e51eb9d8a0fa617ff3fd47

data/README.md

@@ -105,13 +105,8 @@ And if you modify settings values in test example you can reset to defaults with
 
 ## Development
 
-Required development dependencies:
-* [Node.js](https://nodejs.org/) - JavaScript runtime
-* [Yarn](https://yarnpkg.com/) - package manager
-
 Optional development tools:
-* [overmind](https://github.com/DarthSim/overmind) - Process manager for Procfile-based applications and tmux
-* [direnv](https://direnv.net/) - Unclutter your .profile 
+* [direnv](https://direnv.net/) - Unclutter your .profile
 
 Required environment variables:
 * `RAILS_VERSION`
@@ -126,16 +121,13 @@ Getting started with development:
 1) `RAILS_VERSION=5.2.3 SQLITE3_VERSION=1.4.1 bundle`
 2) `RAILS_VERSION=5.2.3 SQLITE3_VERSION=1.4.1 ./bin/rails db:create db:migrate`
 3) `RAILS_VERSION=5.2.3 SQLITE3_VERSION=1.4.1 ./bin/rails test`
-4) `RAILS_VERSION=5.2.3 SQLITE3_VERSION=1.4.1 ./bin/rails frontend:install`
-4) `RAILS_VERSION=5.2.3 SQLITE3_VERSION=1.4.1 ./bin/rails app:system_settings:load`
-4) `RAILS_VERSION=5.2.3 SQLITE3_VERSION=1.4.1 overmind start`
-
 
 ## Build status
 System Settings is being tested with Rails versions - `5.0`, `5.1`, `5.2`, `6.0`, `rails repo master branch`
 
 [![Build Status](https://dev.azure.com/kristsozols/System%20Settings/_apis/build/status/krists.system_settings?branchName=master)](https://dev.azure.com/kristsozols/System%20Settings/_build/latest?definitionId=1&branchName=master)
-
+[![Maintainability](https://api.codeclimate.com/v1/badges/3ad889ca36f62bad04dc/maintainability)](https://codeclimate.com/github/krists/system_settings/maintainability)
+[![Test Coverage](https://api.codeclimate.com/v1/badges/3ad889ca36f62bad04dc/test_coverage)](https://codeclimate.com/github/krists/system_settings/test_coverage)
 
 ## License
 The gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).

data/app/assets/config/manifest.js

@@ -0,0 +1 @@
+//= link_directory ../stylesheets .css

data/app/assets/stylesheets/system_settings/application.css

@@ -0,0 +1,227 @@
+html {
+    box-sizing: border-box;
+    font-size: 14px;
+    font-family: Arial, Helvetica, sans-serif;
+    overflow-y: scroll;
+}
+
+*, *:before, *:after {box-sizing: inherit;}
+
+body {
+    margin: 0;
+    padding: 0;
+    font-weight: normal;
+    min-height: 100vh;
+    position: relative;
+    display: flex;
+    flex-direction: column;
+    color: #233656;
+    background: white;
+    font-size: 1rem;
+    line-height: 1.5rem;
+}
+
+header {
+    border-bottom: 1px solid #e1e1e1;
+    height: 2rem;
+    padding: 0.5rem 0;
+    box-sizing: content-box;
+    background: white;
+    position: sticky;
+    top: 0;
+    width: 100%;
+}
+
+main {
+    padding: 3rem 0;
+}
+
+header .container,
+main .container {
+    max-width: 1020px;
+    margin: 0 auto;
+}
+
+a { color: #233656; text-decoration: none; }
+a:hover { text-decoration: underline; }
+
+.app-name, .app-name:hover, .app-name:active {
+    font-size: 1.5rem;
+    line-height: 2rem;
+    font-weight: bold;
+    user-select: none;
+    text-decoration: none;
+}
+
+.sys-name {
+    display: inline-block;
+    background: #f2f2f2;
+    border-radius: 3px;
+    padding: 0 0.5em;
+    font-family: "Courier New", Courier, monospace;
+    font-size: 0.75em;
+    max-width: 100%;
+}
+
+table th {
+    text-align: left;
+}
+
+.settings-table {
+    width: 100%;
+    table-layout: fixed;
+    white-space: nowrap;
+    user-select: none;
+}
+
+.settings-table td:empty:after {
+    content: "-";
+}
+
+.settings-table td {
+    overflow: hidden;
+    text-overflow: ellipsis;
+}
+
+.settings-table .description { width: 35% }
+.settings-table .value { width: 25% }
+
+.settings-table tbody td.no-entries {
+    color: #737373;
+    padding-top: 1rem;
+    margin-bottom: 1rem;
+}
+
+.settings-table tfoot td {
+    font-size: 0.8em;
+    padding-top: 1rem;
+    margin-bottom: 1rem;
+}
+
+.settings-table .value .value-part, .attribute .value .value-part {
+    max-width: 100%;
+    margin-right: 0.25em;
+}
+
+.settings-table .value .value-part:last-child, .attribute .value .value-part:last-child {
+    margin-right: 0;
+}
+
+.attribute {
+    display: flex;
+    flex-direction: column;
+    margin-bottom: 1rem;
+}
+
+.attribute .name {
+    font-size: 0.8em;
+    color: #737373;
+    user-select: none;
+}
+
+.attribute .value {
+    align-self: flex-start;
+}
+.attribute .value:empty:after {
+    content: "-";
+}
+
+.attribute .field_with_errors .name, .attribute .errors {
+    color: #d01d1d;
+}
+.attribute .hint {
+    font-size: 0.8em;
+    display: block;
+}
+
+.buttons {
+    border-top: 1px dotted #f2f2f2;
+    padding-top: 1rem;
+    margin-bottom: 1rem;
+}
+.buttons > * {
+    margin-right: 0.5rem;
+}
+
+button, input[type=submit], input[type=button], a.button {
+    display: inline-block;
+    cursor: pointer;
+    border: none;
+    border-radius: 3px;
+    padding: 0.375rem 0.75rem;
+    font-size: 1rem;
+    line-height: 1.5;
+    user-select: none;
+    white-space: nowrap;
+}
+
+button, input[type=submit], input[type=button] {
+    color: #4d4d4d;
+    background-color: #d9d9d9;
+    transition: background-color 0.1s ease-in-out;
+    outline: none;
+}
+
+button:focus, button:hover, input[type=submit]:focus, input[type=submit]:hover, input[type=button]:focus, input[type=button]:hover {
+    background-color: #b8b8b8;
+}
+
+button:active, input[type=submit]:active, input[type=button]:active {
+    background-color: #828282;
+}
+
+a.button {
+    color: #4d4d4d;
+    background-color: transparent;
+    transition: background-color 0.1s ease-in-out;
+    outline: none;
+    text-decoration: none;
+}
+
+a.button:focus, a.button:hover {
+    background-color: rgba(0, 0, 0, 0.15);
+}
+
+a.button:active {
+    background-color: rgba(0, 0, 0, 0.4);
+}
+
+button.primary, input[type=submit].primary, input[type=button].primary, a.button.primary {
+    color: #4d4d4d;
+    background-color: #a6a6a6;
+    transition: background-color 0.1s ease-in-out;
+    outline: none;
+}
+
+button.primary:focus, button.primary:hover, input[type=submit].primary:focus, input[type=submit].primary:hover, input[type=button].primary:focus, input[type=button].primary:hover, a.button.primary:focus, a.button.primary:hover {
+    background-color: #8d8d8d;
+}
+
+button.primary:active, input[type=submit].primary:active, input[type=button].primary:active, a.button.primary:active {
+    background-color: #646464;
+}
+
+input[type=text] {
+    border: none;
+    border-radius: 3px;
+    padding: 0.5em;
+    font-size: 1rem;
+    line-height: normal;
+    color: #4d4d4d;
+    background-color: #e6e6e6;
+    transition: background-color 0.1s ease-in-out;
+    outline: none;
+    box-sizing: content-box;
+    width: auto;
+    min-width: 12ch;
+    max-width: calc(100% - 2em);
+    will-change: width;
+}
+
+input[type=text]:focus {
+    background-color: #c4c4c4;
+}
+
+input[type=text]:active {
+    background-color: #bfbfbf;
+}

data/app/controllers/system_settings/application_controller.rb

@@ -3,11 +3,5 @@
 module SystemSettings
   class ApplicationController < ActionController::Base
     protect_from_forgery with: :exception
-
-    private
-
-    def add_authenticity_token
-      response.headers["Authenticity-Token"] = form_authenticity_token if protect_against_forgery?
-    end
   end
 end

data/app/controllers/system_settings/settings_controller.rb

@@ -2,27 +2,33 @@
 
 module SystemSettings
   class SettingsController < SystemSettings::ApplicationController
-    RETURN_ATTRIBUTES = ["id", "name", "type", "value", "description"].freeze
+    RETURN_ATTRIBUTES = %w[id name type value description].freeze
+    before_action :set_setting, only: [:edit, :show, :update]
 
     def index
-      @total_count = SystemSettings::Setting.count
-      @settings = SystemSettings::Setting.order(:name).extending(SystemSettings::Pagination).page(params[:page], per_page: params[:per])
-      render json: {items: @settings.map { |s| s.as_json(only: RETURN_ATTRIBUTES) }, total_count: @total_count}
+      @settings = SystemSettings::Setting.order(:name)
     end
 
-    def show
-      @setting = SystemSettings::Setting.find(params[:id])
-      add_authenticity_token
-      render json: @setting.as_json(only: RETURN_ATTRIBUTES)
-    end
+    def edit; end
+
+    def show; end
 
     def update
-      @setting = SystemSettings::Setting.find(params[:id])
-      if @setting.update(value: params[:value])
-        render json: @setting.as_json(only: RETURN_ATTRIBUTES)
+      if @setting.update(setting_params)
+        redirect_to setting_path(@setting)
       else
-        render json: {errors: @setting.errors.as_json}, status: :unprocessable_entity
+        render :edit
       end
     end
+
+    private
+
+    def set_setting
+      @setting = SystemSettings::Setting.find(params[:id])
+    end
+
+    def setting_params
+      params.require(:setting).permit(:value)
+    end
   end
 end

data/app/helpers/system_settings/application_helper.rb

@@ -0,0 +1,34 @@
+module SystemSettings::ApplicationHelper
+  SEPARATOR = ";".freeze
+
+  def format_value(value)
+    if value.respond_to?(:each)
+      capture do
+        value.each do |v|
+          concat content_tag(:span, v, class: "value-part")
+        end
+      end
+    else
+      value
+    end
+  end
+
+  def format_value_for_form(record)
+    case record
+    when SystemSettings::StringListSetting
+      record.value.map { |v| v.gsub(SEPARATOR, "\\#{SEPARATOR}") }.join(SEPARATOR)
+    when SystemSettings::IntegerListSetting, SystemSettings::DecimalListSetting
+      record.value.join(SEPARATOR)
+    else
+      record.value
+    end
+  end
+
+  def display_settings_file_path
+    if SystemSettings.settings_file_path.to_s.include?(Rails.root.to_s)
+      Pathname.new(SystemSettings.settings_file_path).relative_path_from(Rails.root)
+    else
+      SystemSettings.settings_file_path
+    end
+  end
+end

data/app/models/system_settings/list_of_decimals_validator.rb

@@ -20,7 +20,7 @@ module SystemSettings
 
     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)
+        record.attribute_changed_in_place?(attr_name.to_s)
     end
 
     def matches_list_of_decimals_regexp?(raw_value)

data/app/models/system_settings/string_setting.rb

@@ -4,5 +4,10 @@ module SystemSettings
   class 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
 end

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

@@ -0,0 +1,21 @@
+<!DOCTYPE html>
+<html>
+  <head>
+    <meta charset="utf-8" />
+    <title>System Settings - <%= SystemSettings::VERSION %></title>
+    <%= csrf_meta_tags %>
+    <%= stylesheet_link_tag 'system_settings/application', media: 'all' %>
+  </head>
+  <body>
+    <header>
+      <div class="container">
+        <%= link_to "System Settings", 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 %>