tramway-core 1.17.2.1 → 1.17.3

data/app/helpers/tramway/core/title_helper.rb

@@ -1,29 +1,24 @@
 # frozen_string_literal: true
 
-module Tramway
-  module Core
-    module TitleHelper
-      def title(page_title = default_title)
-        if @application.present?
-          title_text = "#{page_title} | #{@application.try(:title) || @application.public_name}"
-          content_for(:title) { title_text }
-        else
-          error = Tramway::Error.new(plugin: :core, method: :title, message: 'You should set Tramway::Core::Application class using `::Tramway::Core.initialize_application model_class: #{model_class_name}` in config/initializers/tramway.rb OR maybe you don\'t have any records of application model')
-          raise error.message
-        end
-      end
+module Tramway::Core::TitleHelper
+  def title(page_title = default_title)
+    if @application.present?
+      title_text = "#{page_title} | #{@application.try(:title) || @application.public_name}"
+      content_for(:title) { title_text }
+    else
+      Tramway::Error.raise_error(:tramway, :core, :title_helper, :title, :you_should_set_tramway_core_application)
+    end
+  end
 
-      def default_title
-        t('.title')
-      end
+  def default_title
+    t('.title')
+  end
 
-      def page_title(action, model_name)
-        if I18n.locale == :ru
-          t("helpers.actions.#{action}") + ' ' + genitive(model_name)
-        else
-          t("helpers.actions.#{action}") + ' ' + model_name.model_name.human.downcase
-        end
-      end
+  def page_title(action, model_name)
+    if I18n.locale == :ru
+      t("helpers.actions.#{action}") + ' ' + genitive(model_name)
+    else
+      t("helpers.actions.#{action}") + ' ' + model_name.model_name.human.downcase
     end
   end
 end

data/app/models/tramway/core/application_record.rb

@@ -2,52 +2,50 @@
 
 require 'carrierwave/orm/activerecord' if defined?(CarrierWave::Mount)
 
-module Tramway
-  module Core
-    class ApplicationRecord < ActiveRecord::Base
-      self.abstract_class = true
-      audited
-      extend ::Enumerize
-
-      state_machine :state, initial: :active do
-        state :active
-        state :removed
-
-        event :remove do
-          transition active: :remove
-        end
-      end
+class Tramway::Core::ApplicationRecord < ActiveRecord::Base
+  self.abstract_class = true
+  audited
+  extend ::Enumerize
 
-      scope :active, -> { where state: :active }
-      scope :created_by_user, ->(user_id) { joins(:audits).where('audits.action = \'create\' AND audits.user_id = ?', user_id) }
-      scope :admin_scope, ->(_arg) { all }
+  state_machine :state, initial: :active do
+    state :active
+    state :removed
 
-      include ::PgSearch::Model
+    event :remove do
+      transition active: :remove
+    end
+  end
 
-      def creator
-        audits.where(action: :create).first.user
-      end
+  scope :active, -> { where state: :active }
+  scope :created_by_user, lambda { |user_id|
+    joins(:audits).where('audits.action = \'create\' AND audits.user_id = ?', user_id)
+  }
+  scope :admin_scope, ->(_arg) { all }
 
-      # FIXME: detect inhertited locales
-      class << self
-        def human_attribute_name(attribute_name, *_args)
-          excepted_attributes = %w[created_at updated_at state]
-          if attribute_name.to_s.in? excepted_attributes
-            I18n.t "activerecord.attributes.tramway/core/application_record.#{attribute_name}"
-          else
-            super attribute_name
-          end
-        end
-
-        def search_by(*attributes, **associations)
-          pg_search_scope :full_text_search, against: attributes, associated_against: associations
-        end
-      end
+  include ::PgSearch::Model
 
-      # FIXME: detect inhertited locales
-      def human_state_name
-        I18n.t "activerecord.state_machines.tramway/core/application_record.state.states.#{state}"
+  def creator
+    audits.where(action: :create).first.user
+  end
+
+  # FIXME: detect inhertited locales
+  class << self
+    def human_attribute_name(attribute_name, *_args)
+      excepted_attributes = %w[created_at updated_at state]
+      if attribute_name.to_s.in? excepted_attributes
+        I18n.t "activerecord.attributes.tramway/core/application_record.#{attribute_name}"
+      else
+        super attribute_name
       end
     end
+
+    def search_by(*attributes, **associations)
+      pg_search_scope :full_text_search, against: attributes, associated_against: associations
+    end
+  end
+
+  # FIXME: detect inhertited locales
+  def human_state_name
+    I18n.t "activerecord.state_machines.tramway/core/application_record.state.states.#{state}"
   end
 end

data/app/uploaders/image_defaults.rb

@@ -4,7 +4,8 @@ module ImageDefaults
   include CarrierWave::MiniMagick
 
   def default_url
-    "/images/fallback/#{model.class.model_name.to_s.underscore}/" << [mounted_as, version_name].compact.join('_') << '.gif'
+    "/images/fallback/#{model.class.model_name.to_s.underscore}/" <<
+      [mounted_as, version_name].compact.join('_') << '.gif'
   end
 
   def extension_white_list

data/app/uploaders/photo_uploader.rb

@@ -44,14 +44,14 @@ class PhotoUploader < ApplicationUploader
   before :cache, :capture_size
 
   def capture_size(file)
-    if version_name.blank?
-      if file.path.nil?
-        img = ::MiniMagick::Image.read(file.file)
-        @width = img[:width]
-        @height = img[:height]
-      else
-        @width, @height = `identify -format "%wx %h" #{file.path}`.split(/x/).map(&:to_i)
-      end
+    return unless version_name.blank?
+
+    if file.path.nil?
+      img = ::MiniMagick::Image.read(file.file)
+      @width = img[:width]
+      @height = img[:height]
+    else
+      @width, @height = `identify -format "%wx %h" #{file.path}`.split(/x/).map(&:to_i)
     end
   end
 end

data/app/views/tramway/core/shared/_input.html.haml

@@ -0,0 +1,26 @@
+- model_class = defined?(record) ? record.model : model_class
+- value = defined?(value) ? value : value_from_params(model_class: model_class, property: property, type: type)
+- form_object = defined?(record) ? record : instance_variable_get("@#{object}_form")
+- input_params = { property: property, object: object, form_object: form_object, value: value }
+- if !type.class.in?([ Symbol, String ]) && type[:input_options]
+  - input_params.merge!(options: type[:input_options])
+  - type = type[:type]
+- if type.class.in?([ Symbol, String ]) || type&.dig(:input_options)
+  - type = type.to_sym
+  = form.label form_object.model.class.human_attribute_name property
+  - case type
+  - when :default
+    = form.input property, **default_params(**input_params)
+  - when :association
+    = form.association property, **association_params(**input_params)
+  - when :polymorphic_association
+    = form.input property, **polymorphic_association_params(**input_params.merge(value: value))
+  - else
+    = form.input property, **else_params(**input_params.merge(type: type))
+- else
+  - property_value = form_object.model.values.present? && form_object.model.values[property.to_s]
+  = render 'tramway/core/shared/input_extended', field: type[:extended_form_property], class_name: :record, value: property_value, f: form
+
+- if params[:errors].present? && params[:errors][property]&.first
+  .alert.alert-danger
+    = params[:errors][property]&.first

data/config/initializers/plurals.rb

@@ -6,7 +6,7 @@
       plural: {
         keys: %i[zero one few many],
         rule: lambda do |n|
-          if n == 0
+          if n.zero?
             :zero
           elsif n % 10 == 1 && n % 100 != 11
             # 1, 21, 31, 41, 51, 61...
@@ -14,7 +14,7 @@
           elsif [2, 3, 4].include?(n % 10) && ![12, 13, 14].include?(n % 100)
             # 2-4, 22-24, 32-34...
             :few
-          elsif n % 10 == 0 || ![5, 6, 7, 8, 9].include?(n % 10) || ![11, 12, 13, 14].include?(n % 100)
+          elsif n % 10.zero? || ![5, 6, 7, 8, 9].include?(n % 10) || ![11, 12, 13, 14].include?(n % 100)
             # 0, 5-20, 25-30, 35-40...
             :many
           end

data/lib/tramway/collection.rb

@@ -1,11 +1,9 @@
 # frozen_string_literal: true
 
-module Tramway
-  class Collection
-    class << self
-      def list
-        raise 'Please add collection to list method'
-      end
+class Tramway::Collection
+  class << self
+    def list
+      raise 'Please add collection to list method'
     end
   end
 end

data/lib/tramway/collections.rb

@@ -0,0 +1,4 @@
+# frozen_string_literal: true
+
+module Tramway::Collections
+end

data/lib/tramway/collections/helper.rb

@@ -1,20 +1,21 @@
 # frozen_string_literal: true
 
-module Tramway
-  module Collections
-    module Helper
-      def collection_list_by(name:)
-        begin
-          require name # needed to load class name with collection
-        rescue LoadError
-          raise "No such file #{name}. You should create file in the `lib/#{name}.rb` or elsewhere you want"
-        end
-        unless ::Tramway::Collection.descendants.map(&:to_s).include?(name.camelize)
-          raise "There no such collection named #{name.camelize}. Please create class with self method `list` and extended of `Tramway::Collection`. You should reload your server after creating this collection."
-        end
+require 'tramway/collections'
 
-        name.camelize.constantize.list
-      end
+module Tramway::Collections::Helper
+  def collection_list_by(name:)
+    begin
+      require name # needed to load class name with collection
+    rescue LoadError
+      raise "No such file #{name}. You should create file in the `lib/#{name}.rb` or elsewhere you want"
     end
+    unless ::Tramway::Collection.descendants.map(&:to_s).include?(name.camelize)
+      ::Tramway::Error.raise_error(
+        :tramway, :collections, :helper, :collection_list_by, :there_no_such_collection,
+        name_camelize: name.camelize
+      )
+    end
+
+    name.camelize.constantize.list
   end
 end

data/lib/tramway/core.rb

@@ -9,37 +9,37 @@ require 'reform'
 require 'pg_search'
 require 'validators/presence_validator'
 
-module Tramway
-  module Core
-    class << self
-      def initialize_application(**options)
-        @application ||= Tramway::Core::Application.new
-        options.each do |attr, value|
-          @application.send "#{attr}=", value
-        end
+module Tramway::Core
+  class << self
+    def initialize_application(**options)
+      @application ||= Tramway::Core::Application.new
+      options.each do |attr, value|
+        @application.send "#{attr}=", value
       end
+    end
 
-      def application_object
-        if @application&.model_class.present?
-          begin
-            @application.model_class.first
-          rescue StandardError
-            nil
-          end
-        else
-          @application
+    def application_object
+      if @application&.model_class.present?
+        begin
+          @application.model_class.first
+        rescue StandardError
+          nil
         end
+      else
+        @application
       end
+    end
 
-      attr_reader :application
+    def root
+      File.dirname __dir__
     end
+
+    attr_reader :application
   end
 end
 
 # HACK: FIXME
 
-module ActiveModel
-  class Errors
-    def merge!(*args); end
-  end
+class ActiveModel::Errors
+  def merge!(*args); end
 end

data/lib/tramway/core/engine.rb

@@ -5,14 +5,10 @@ require 'tramway/core/application'
 require 'simple_form'
 require 'enumerize'
 
-module Tramway
-  module Core
-    class Engine < ::Rails::Engine
-      isolate_namespace Tramway::Core
+class Tramway::Core::Engine < ::Rails::Engine
+  isolate_namespace Tramway::Core
 
-      config.before_initialize do
-        config.i18n.load_path += Dir["#{config.root}/config/locales/**/*.yml"]
-      end
-    end
+  config.before_initialize do
+    config.i18n.load_path += Dir["#{config.root}/config/locales/**/*.yml"]
   end
 end

data/lib/tramway/core/generators.rb

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+module Tramway::Core
+  module Generators
+  end
+end

data/lib/tramway/core/generators/install_generator.rb

@@ -1,27 +1,27 @@
 # frozen_string_literal: true
 
 require 'rails/generators'
+require 'tramway/core/generators'
 
-module Tramway
-  module Core
-    module Generators
-      class InstallGenerator < ::Rails::Generators::Base
-        source_root File.expand_path('templates', __dir__)
+class Tramway::Core::Generators::InstallGenerator < ::Rails::Generators::Base
+  source_root File.expand_path('templates', __dir__)
 
-        def run_other_generators
-          generate 'audited:install'
-        end
+  def run_other_generators
+    generate 'audited:install'
+  end
 
-        def self.next_migration_number(path)
-          next_migration_number = current_migration_number(path) + 1
-          ActiveRecord::Migration.next_migration_number next_migration_number
-        end
+  def self.next_migration_number(path)
+    next_migration_number = current_migration_number(path) + 1
+    ActiveRecord::Migration.next_migration_number next_migration_number
+  end
 
-        def copy_initializer
-          copy_file "/#{File.dirname __dir__}/generators/templates/initializers/simple_form.rb", 'config/initializers/simple_form.rb'
-          copy_file "/#{File.dirname __dir__}/generators/templates/initializers/simple_form_bootstrap.rb", 'config/initializers/simple_form_bootstrap.rb'
-        end
-      end
+  def copy_initializer
+    simple_form_files = %w[simple_form simple_form_bootstrap]
+    simple_form_files.each do |file|
+      copy_file(
+        "/#{File.dirname __dir__}/generators/templates/initializers/#{file}.rb",
+        "config/initializers/#{file}.rb"
+      )
     end
   end
 end

data/lib/tramway/core/version.rb

@@ -2,6 +2,6 @@
 
 module Tramway
   module Core
-    VERSION = '1.17.2.1'
+    VERSION = '1.17.3'
   end
 end

data/lib/tramway/error.rb

@@ -2,7 +2,7 @@
 
 class Tramway::Error < RuntimeError
   def initialize(*args, plugin: nil, method: nil, message: nil)
-    @properties ||= {}
+    @properties = {}
     @properties[:plugin] = plugin
     @properties[:method] = method
     @properties[:message] = message
@@ -16,4 +16,15 @@ class Tramway::Error < RuntimeError
   def properties
     @properties ||= {}
   end
+
+  class << self
+    def raise_error(*coordinates, **options)
+      @errors ||= YAML.load_file("#{Tramway::Core.root}/yaml/errors.yml").with_indifferent_access
+      error = @errors.dig(*coordinates)
+      options.each do |pair|
+        error.gsub!("%{#{pair[0]}}", pair[1].to_s)
+      end
+      raise error
+    end
+  end
 end

data/lib/yaml/errors.yml

@@ -0,0 +1,35 @@
+tramway:
+  collections:
+    helper:
+      collection_list_by:
+        there_no_such_collection: "There no such collection named %{name_camelize}. Please create class with self method `list` and extended of `Tramway::Collection`. You should reload your server after creating this collection."
+  core:
+    title_helper:
+      title:
+        you_should_set_tramway_core_application: "You should set Tramway::Core::Application class using `::Tramway::Core.initialize_application model_class: #{model_class_name}` in config/initializers/tramway.rb OR maybe you don't have any records of application model"
+    application_form:
+      initialize:
+        polymorphic_class_is_nil: "Polymorphic association class is nil. Maybe, you should write `assocation #{association_name}` after `properties #{association_name}_id, #{association_name}_type`"
+      validates:
+        you_need_to_set_model_class: 'You need to set `model_class` name while using validations. Just write `self.model_class = YOUR_MODEL_NAME` in the class area.'
+      submit:
+        looks_like_you_have_method: "Looks like you have method `%{method_name}` in %{model_class}. You should rename it or rename property in %{class_name}"
+        params_should_not_be_nil: "ApplicationForm::Params should not be nil"
+      model_class: 
+        there_is_not_model_class: "There is not model class name for %{name}. Should be %{model_class_name} or you can use another class to initialize form object or just initialize form with object."
+    concerns:
+      attribute_decorator_helper:
+        you_should_mount_photo_uploader: "You should mount PhotoUploader to your model. Just add `mount_uploader %{attribute_name}, PhotoUploader` to your model. %{message}"
+    associations:
+      object_helper:
+        please_specify_association_name: "Please, specify `%{association_name}` association class_name in %{object_class} model. For example: `has_many :%{association_name}, class_name: '%{association_class_name}'`"
+      class_helper:
+        model_does_not_have_association: "Model %{object_class} does not have association named `%{association_name}`"
+    application_decorator:
+      attributes:
+        method_is_reserved_word: "Method `%{attribute_name}` is reserved word. Please, create or delegate method in %{class_name} with another name."
+      title:
+        please_implement_title: "Please, implement `title` method in a %{class_name} or %{object_class}"
+      link:
+        method_link_uses_file_attribute: "Method `link` uses `file` attribute of the decorated object. If decorated object doesn't contain `file`, you shouldn't use `link` method."
+