granite 0.9.4 → 0.9.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bfd220e19b901b3778ae1a5934da2298a28a5435536401606e05faabdbaa80ae
-  data.tar.gz: af299e280b01e76fca44edbb969b7e5d009424965367955c1ede05f24c10065b
+  metadata.gz: c916196599f14f9af76a9a550ffb51e7e4db5fd97deee1cf7cf04135c4bfb217
+  data.tar.gz: 6fc2f4a436734b34eb2096fa063a0d82e6fe6bf58afbb04af07b1a86d82d8727
 SHA512:
-  metadata.gz: d7db61043e8e3039bea436f9d43807aaa2363c2ab537c1d613e326beb92f305c86e3ca8cae5eed59717bb8dd38edef29b7b37494873f7ec265f6e1d5b060a0e3
-  data.tar.gz: a1360ff25d185e8c0848c1b385a5ba455025abfac7e691b1203e7b645bacd4aac5979436d096353ecbfb14aca6c7d66fef1c31374dbdf320ab7257e6e2193db8
+  metadata.gz: 0bd7b860deb8221df8be1e1227df3c71462862a47ba9243ec7ebc3be1972aeaa95b5c14334ab45aa372f7fe45fc89ed7b3af79a9fb52c3ed5f3e3908981611af
+  data.tar.gz: 6f164dc00ce5e56be2bff2bd01f22b49d37996decc05b74816a129f1598451d0dbbb184c1c7d02cac90e52eca80e947e8cba6932dba6842a030ce7801864ba0d

data/app/controllers/granite/controller.rb

@@ -1,9 +1,9 @@
-require 'granite/projector/translations/helper'
 require 'action_controller'
 
 module Granite
   class Controller < Granite.base_controller_class
-    include Granite::Projector::Translations::Helper
+    include Controller::Translations
+    helper Controller::Translations
 
     singleton_class.__send__(:attr_accessor, :projector_class)
     singleton_class.delegate :projector_path, :projector_name, :action_class, to: :projector_class

data/app/controllers/granite/controller/translations.rb

@@ -0,0 +1,15 @@
+module Granite
+  class Controller
+    module Translations
+      def i18n_scopes
+        Granite::Translations.combine_paths(projector.i18n_scopes, [*action_name, nil])
+      end
+
+      def translate(*args, **options)
+        super(*Granite::Translations.scope_translation_args(i18n_scopes, *args, **options))
+      end
+
+      alias t translate
+    end
+  end
+end

data/config/rubocop-default.yml

@@ -0,0 +1,3 @@
+Lint/UselessAccessModifier:
+  ContextCreatingMethods:
+    - projector

data/lib/granite.rb

@@ -4,6 +4,7 @@ require 'action_controller'
 require 'granite/version'
 require 'granite/config'
 require 'granite/context'
+require 'granite/util'
 
 module Granite
   def self.config

data/lib/granite/action.rb

@@ -7,10 +7,12 @@ require 'granite/action/types'
 require 'granite/action/error'
 require 'granite/action/performing'
 require 'granite/action/performer'
+require 'granite/action/precondition'
 require 'granite/action/preconditions'
 require 'granite/action/policies'
 require 'granite/action/projectors'
 require 'granite/action/subject'
+require 'granite/action/translations'
 
 module Granite
   class Action
@@ -38,6 +40,7 @@ module Granite
     end
 
     include Base
+    include Translations
     include Performing
     include Subject
     include Performer
@@ -64,10 +67,6 @@ module Granite
       end
     end
 
-    def self.i18n_scope
-      :granite_action
-    end
-
     # Almost the same as Dirty `#changed?` method, but
     # doesn't check subject reference key
     def attributes_changed?(except: [])

data/lib/granite/action/precondition.rb

@@ -0,0 +1,38 @@
+module Granite
+  class Action
+    class Precondition < BasicObject
+      UNDEFINED = ::Object.new.freeze
+
+      def self.description(text = UNDEFINED)
+        case text
+        when UNDEFINED
+          @description
+        else
+          @description = text
+        end
+      end
+
+      def initialize(context)
+        @context = context
+      end
+
+      def call(*)
+        fail NotImplementedError, "#call method must be implemented for #{self.class}"
+      end
+
+      def method_missing(method_name, *args, &blk)
+        super unless @context.respond_to?(method_name)
+
+        @context.__send__(method_name, *args, &blk)
+      end
+
+      def respond_to_missing?(method_name, _include_private = false)
+        @context.respond_to?(method_name)
+      end
+
+      private
+
+      attr_reader :context
+    end
+  end
+end

data/lib/granite/action/preconditions.rb

@@ -1,5 +1,6 @@
 require 'granite/action/preconditions/base_precondition'
 require 'granite/action/preconditions/embedded_precondition'
+require 'granite/action/preconditions/object_precondition'
 
 module Granite
   class Action
@@ -13,6 +14,10 @@ module Granite
       extend ActiveSupport::Concern
 
       class PreconditionsCollection
+        include Enumerable
+
+        delegate :each, to: :@preconditions
+
         def initialize(*preconditions)
           @preconditions = preconditions.flatten
         end
@@ -43,8 +48,10 @@ module Granite
           options = args.extract_options!
           if block
             add_precondition(BasePrecondition, options, &block)
+          elsif args.first.is_a?(Class)
+            add_precondition(ObjectPrecondition, *args, options)
           else
-            common_options = options.extract!(:if, :unless)
+            common_options = options.extract!(:if, :unless, :desc, :description)
             args.each do |type|
               precondition common_options.merge(type => {})
             end

data/lib/granite/action/preconditions/base_precondition.rb

@@ -9,10 +9,7 @@ module Granite
         end
 
         def execute!(context)
-          return if @options[:if] && !context.instance_exec(&@options[:if])
-          return if @options[:unless] && context.instance_exec(&@options[:unless])
-
-          _execute(context)
+          _execute(context) if context.conditions_satisfied?(@options)
         end
 
         private

data/lib/granite/action/preconditions/object_precondition.rb

@@ -0,0 +1,15 @@
+require 'granite/action/preconditions/base_precondition'
+
+module Granite
+  class Action
+    module Preconditions
+      class ObjectPrecondition < BasePrecondition
+        private
+
+        def _execute(context)
+          @args.first.new(context).call(**@options.except(:if, :unless))
+        end
+      end
+    end
+  end
+end

data/lib/granite/action/translations.rb

@@ -0,0 +1,24 @@
+module Granite
+  class Action
+    module Translations
+      extend ActiveSupport::Concern
+
+      module ClassMethods
+        def i18n_scope
+          :granite_action
+        end
+
+        def i18n_scopes
+          lookup_ancestors.flat_map do |klass|
+            :"#{klass.i18n_scope}.#{klass.model_name.i18n_key}"
+          end + [nil]
+        end
+      end
+
+      def translate(*args, **options)
+        I18n.translate(*Granite::Translations.scope_translation_args(self.class.i18n_scopes, *args, **options))
+      end
+      alias t translate
+    end
+  end
+end

data/lib/granite/base.rb

@@ -18,7 +18,7 @@ module Granite
     include ActiveData::Model::Primary
     include ActiveModel::Validations::Callbacks
 
-    include Granite::Translations
+    include Granite::Util
     include Granite::Represents
   end
 end

data/lib/granite/projector/translations.rb

@@ -1,52 +1,17 @@
 module Granite
   class Projector
     module Translations
-      extend ActiveSupport::Concern
+      include ActionView::Helpers::TranslationHelper
 
-      class TranslationsWrapper
-        include ActionView::Helpers::TranslationHelper
+      def i18n_scopes
+        Granite::Translations.combine_paths(action_class.i18n_scopes, [:"#{projector_name}"])
       end
 
-      def translate(*args)
-        TranslationsWrapper.new.translate(*self.class.scope_translation_args_by_projector(args))
+      def translate(*args, **options)
+        super(*Granite::Translations.scope_translation_args(i18n_scopes, *args, **options))
       end
-      alias t translate
-
-      module ClassMethods
-        def scope_translation_args_by_projector(args, action_name: nil)
-          options = args.extract_options!
-
-          lookups = expand_relative_key(args.first, action_name).map(&:to_sym)
-          lookups += [options[:default]]
-          lookups = lookups.flatten.compact
-
-          key = lookups.shift
-          options[:default] = lookups
-
-          [key, options]
-        end
-
-        private
 
-        def expand_relative_key(key, action_name = nil)
-          return [key] unless key.is_a?(String) && key.start_with?('.')
-
-          base_keys = extract_base_keys(key, action_name)
-
-          action_class.lookup_ancestors.map do |klass|
-            base_keys.map do |base_key|
-              :"#{klass.i18n_scope}.#{klass.model_name.i18n_key}.#{base_key}"
-            end
-          end.flatten + base_keys
-        end
-
-        def extract_base_keys(key, action_name)
-          undotted_key = key.sub(/^\./, '')
-          base_keys = [:"#{projector_name}.#{undotted_key}"]
-          base_keys.unshift :"#{projector_name}.#{action_name}.#{undotted_key}" if action_name
-          base_keys
-        end
-      end
+      alias t translate
     end
   end
 end

data/lib/granite/represents/attribute.rb

@@ -11,9 +11,7 @@ module Granite
       end
 
       def sync
-        return if reference.nil?
-
-        reference.public_send(writer, read)
+        reference.public_send(writer, read) if reference.respond_to?(writer)
       end
 
       def typecast(value)

data/lib/granite/routing/mapper.rb

@@ -5,8 +5,8 @@ module Granite
   module Routing
     module Mapper
       def granite(projector_path, **options)
-        route = Route.new(projector_path, options.extract!(:path, :as, :projector_prefix))
-        Declarer.declare(self, route, options)
+        route = Route.new(projector_path, **options.extract!(:path, :as, :projector_prefix))
+        Declarer.declare(self, route, **options)
       end
     end
   end

data/lib/granite/translations.rb

@@ -1,19 +1,14 @@
 module Granite
-  module Translations
-    extend ActiveSupport::Concern
-
-    def translate(*args)
-      I18n.translate(*self.class.scope_translation_args(args))
-    end
-    alias t translate
-
-    module ClassMethods
-      def scope_translation_args(args)
-        options = args.extract_options!
+  class Translations
+    class << self
+      def combine_paths(paths1, paths2)
+        paths1.flat_map do |path1|
+          paths2.map { |path2| [*path1, *path2].join('.') }
+        end
+      end
 
-        lookups = expand_relative_key(args.first).map(&:to_sym)
-        lookups += [options[:default]]
-        lookups = lookups.flatten.compact
+      def scope_translation_args(scopes, key, *, **options)
+        lookups = expand_relative_key(scopes, key) + Array(options[:default])
 
         key = lookups.shift
         options[:default] = lookups
@@ -23,14 +18,10 @@ module Granite
 
       private
 
-      def expand_relative_key(key)
+      def expand_relative_key(scopes, key)
         return [key] unless key.is_a?(String) && key.start_with?('.')
 
-        base_key = key.sub(/^\./, '')
-
-        lookup_ancestors.map do |klass|
-          :"#{klass.i18n_scope}.#{klass.model_name.i18n_key}.#{base_key}"
-        end.flatten + [base_key]
+        combine_paths(scopes, [key.sub(/^\./, '')]).map(&:to_sym)
       end
     end
   end

data/lib/granite/util.rb

@@ -0,0 +1,51 @@
+module Granite
+  module Util
+    extend ActiveSupport::Concern
+
+    # Evaluates value and returns result based on what was passed:
+    # - if Proc was passed, then executes it in context of self
+    # - if Symbol was passed, then calls a method with that name and returns result
+    # - otherwise just returns the value itself
+    # @param value [Object] value to evaluate
+    # @return [Object] result of evaluation
+    def evaluate(value)
+      case value
+      when Proc
+        evaluate_proc(value)
+      when Symbol
+        evaluate_symbol(value)
+      else
+        value
+      end
+    end
+
+    # Evaluates `if` or `unless` conditions present in the supplied
+    # `options` being it a symbol or callable.
+    #
+    # @param [Hash] options The method options to evaluate.
+    # @option options :if method name or callable
+    # @option options :unless method name or callable
+    # @return [Boolean] whether conditions are satisfied
+    def conditions_satisfied?(options)
+      fail ArgumentError, 'You cannot specify both if and unless' if options.key?(:if) && options.key?(:unless)
+
+      if options.key?(:if)
+        evaluate(options[:if])
+      elsif options.key?(:unless)
+        !evaluate(options[:unless])
+      else
+        true
+      end
+    end
+
+    private
+
+    def evaluate_proc(value)
+      instance_exec(&value)
+    end
+
+    def evaluate_symbol(value)
+      __send__(value)
+    end
+  end
+end

data/lib/granite/version.rb

@@ -1,3 +1,3 @@
 module Granite
-  VERSION = '0.9.4'.freeze
+  VERSION = '0.9.9'.freeze
 end

data/lib/rubocop-granite.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+require 'rubocop'
+
+require_relative 'rubocop/granite'
+require_relative 'rubocop/granite/inject'
+
+RuboCop::Granite::Inject.defaults!

data/lib/rubocop/granite.rb

@@ -0,0 +1,9 @@
+module RuboCop
+  module Granite
+    PROJECT_ROOT   = Pathname.new(__dir__).parent.parent.expand_path.freeze
+    CONFIG_DEFAULT = PROJECT_ROOT.join('config', 'rubocop-default.yml').freeze
+    CONFIG         = YAML.safe_load(CONFIG_DEFAULT.read).freeze
+
+    private_constant(:CONFIG_DEFAULT, :PROJECT_ROOT)
+  end
+end

data/lib/rubocop/granite/inject.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+# The original code is from https://github.com/rubocop-hq/rubocop-rspec/blob/master/lib/rubocop/rspec/inject.rb
+# See https://github.com/rubocop-hq/rubocop-rspec/blob/master/MIT-LICENSE.md
+module RuboCop
+  module Granite
+    # Because RuboCop doesn't yet support plugins, we have to monkey patch in a
+    # bit of our configuration.
+    module Inject
+      def self.defaults!
+        path = CONFIG_DEFAULT.to_s
+        hash = ConfigLoader.load_file(path)
+        config = Config.new(hash, path).tap(&:make_excludes_absolute)
+        puts "configuration from #{path}" if ConfigLoader.debug?
+        config = ConfigLoader.merge_with_default(config, path)
+        ConfigLoader.instance_variable_set(:@default_configuration, config)
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: granite
 version: !ruby/object:Gem::Version
-  version: 0.9.4
+  version: 0.9.9
 platform: ruby
 authors:
 - Arkadiy Zabazhanov & friends
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-12-31 00:00:00.000000000 Z
+date: 2021-06-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack
@@ -244,28 +244,42 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.52'
+        version: 0.78.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.52'
+        version: 0.78.0
+- !ruby/object:Gem::Dependency
+  name: rubocop-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.4.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.4.1
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.22'
+        version: 1.37.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.22'
+        version: 1.37.0
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -288,6 +302,8 @@ extra_rdoc_files: []
 files:
 - LICENSE
 - app/controllers/granite/controller.rb
+- app/controllers/granite/controller/translations.rb
+- config/rubocop-default.yml
 - lib/generators/USAGE
 - lib/generators/granite/install_controller_generator.rb
 - lib/generators/granite_generator.rb
@@ -305,14 +321,17 @@ files:
 - lib/granite/action/policies/always_allow_strategy.rb
 - lib/granite/action/policies/any_strategy.rb
 - lib/granite/action/policies/required_performer_strategy.rb
+- lib/granite/action/precondition.rb
 - lib/granite/action/preconditions.rb
 - lib/granite/action/preconditions/base_precondition.rb
 - lib/granite/action/preconditions/embedded_precondition.rb
+- lib/granite/action/preconditions/object_precondition.rb
 - lib/granite/action/projectors.rb
 - lib/granite/action/subject.rb
 - lib/granite/action/transaction.rb
 - lib/granite/action/transaction_manager.rb
 - lib/granite/action/transaction_manager/transactions_stack.rb
+- lib/granite/action/translations.rb
 - lib/granite/action/types.rb
 - lib/granite/action/types/collection.rb
 - lib/granite/base.rb
@@ -327,8 +346,6 @@ files:
 - lib/granite/projector/error.rb
 - lib/granite/projector/helpers.rb
 - lib/granite/projector/translations.rb
-- lib/granite/projector/translations/helper.rb
-- lib/granite/projector/translations/view_helper.rb
 - lib/granite/rails.rb
 - lib/granite/represents.rb
 - lib/granite/represents/attribute.rb
@@ -348,7 +365,11 @@ files:
 - lib/granite/rspec/satisfy_preconditions.rb
 - lib/granite/translations.rb
 - lib/granite/typecasters.rb
+- lib/granite/util.rb
 - lib/granite/version.rb
+- lib/rubocop-granite.rb
+- lib/rubocop/granite.rb
+- lib/rubocop/granite/inject.rb
 homepage: https://github.com/toptal/granite
 licenses:
 - MIT
@@ -368,7 +389,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.6
+rubygems_version: 3.0.9
 signing_key: 
 specification_version: 4
 summary: Another business actions architecture for Rails apps

data/lib/granite/projector/translations/helper.rb

@@ -1,22 +0,0 @@
-require 'granite/projector/translations/view_helper'
-
-module Granite
-  class Projector
-    module Translations
-      module Helper
-        extend ActiveSupport::Concern
-
-        included do
-          delegate :scope_translation_args_by_projector, to: :projector_class
-          helper_method :scope_translation_args_by_projector
-          helper ViewHelper
-        end
-
-        def translate(*args)
-          super(*scope_translation_args_by_projector(args, action_name: action_name))
-        end
-        alias t translate
-      end
-    end
-  end
-end

data/lib/granite/projector/translations/view_helper.rb

@@ -1,12 +0,0 @@
-module Granite
-  class Projector
-    module Translations
-      module ViewHelper
-        def translate(*args)
-          super(*scope_translation_args_by_projector(args, action_name: action_name))
-        end
-        alias t translate
-      end
-    end
-  end
-end