eco-helpers 3.0.25 → 3.0.27

data/lib/eco/api/usecases/default/people/treat/analyse_people_case.rb

@@ -1,5 +1,5 @@
 class Eco::API::UseCases::Default::People::Treat::Analyse < Eco::API::Common::Loaders::UseCase
-  name "analyse-people"
+  name 'analyse-people'
   type :export
 
   require_relative 'cli/analyse_people_cli'
@@ -12,7 +12,7 @@ class Eco::API::UseCases::Default::People::Treat::Analyse < Eco::API::Common::Lo
     elsif case_options[:identify_unnamed]
       identify_unnamed
     else
-      log(:info) { "No analysis operation was specified" }
+      log(:info) { 'No analysis operation was specified' }
     end.tap do |people_involved|
       next unless people_involved
 
@@ -27,7 +27,7 @@ class Eco::API::UseCases::Default::People::Treat::Analyse < Eco::API::Common::Lo
     similarity_analytics.unnamed.tap do |unnamed|
       next unless unnamed.empty?
 
-      log(:info) { "There were no people with no name!!" }
+      log(:info) { 'There were no people with no name!!' }
     end
   end
 
@@ -35,13 +35,13 @@ class Eco::API::UseCases::Default::People::Treat::Analyse < Eco::API::Common::Lo
     analysed = similarity_screening
 
     if case_options[:ignore_matching_words]
-      log(:info) { "Fine tune results by ignoring matching words..." }
+      log(:info) { 'Fine tune results by ignoring matching words...' }
       analysed = strict_similarity(analysed)
     end
 
     similarity_analytics.newSimilarity(analysed).tap do |related_people|
       if related_people.empty?
-        log(:info) { "There were no possible duplicates identified!!" }
+        log(:info) { 'There were no possible duplicates identified!!' }
       else
         report = similarity_analytics.report(analysed, format: :txt)
         save!(report)
@@ -157,23 +157,23 @@ class Eco::API::UseCases::Default::People::Treat::Analyse < Eco::API::Common::Lo
   end
 
   def output_file
-    @output_file ||= options.dig(:output, :file) || "analytics.txt"
+    @output_file ||= options.dig(:output, :file) || 'analytics.txt'
   end
 
   def save!(data)
-    ext = File.extname(output_file).downcase.delete(".")
+    ext = File.extname(output_file).downcase.delete('.')
 
     log(:info) { "Generating file '#{output_file}'" }
 
-    File.open(output_file, "w") do |fd|
+    File.open(output_file, 'w') do |fd|
       case ext
-      when "txt"
+      when 'txt'
         fd << data
-      when "html"
-        puts "html is still not supported"
+      when 'html'
+        puts 'html is still not supported'
         exit(1)
-      when "json"
-        puts "json is still not supported"
+      when 'json'
+        puts 'json is still not supported'
         exit(1)
       end
     end
@@ -182,11 +182,11 @@ class Eco::API::UseCases::Default::People::Treat::Analyse < Eco::API::Common::Lo
   # A way to use command line to specify part
   # => i.e. details[first-name] AND details[surname]
   def proc_value_access(expression)
-    #return expression.to_sym if expression.start_with?(":")
-    subexpressions = expression.split(" AND ")
+    #return expression.to_sym if expression.start_with?(':')
+    subexpressions = expression.split(' AND ')
     proc do |person|
       values = subexpressions.map {|exp| attribute_access(person, exp)}
-      values.compact.join(" ")
+      values.compact.join(' ')
     end
   end
 
@@ -216,9 +216,9 @@ class Eco::API::UseCases::Default::People::Treat::Analyse < Eco::API::Common::Lo
 
     is_method = part.is_a?(Symbol) || obj.respond_to?(part.to_sym)
     return obj.send(part.to_sym)    if is_method
-    return get_attr(obj, part[1..]) if part.start_with?(":")
+    return get_attr(obj, part[1..]) if part.start_with?(':')
 
-    if part.start_with?("details[")
+    if part.start_with?('details[')
       via_details = obj.respond_to?(:details) && (details = obj.details)
       return unless via_details
 
@@ -228,11 +228,11 @@ class Eco::API::UseCases::Default::People::Treat::Analyse < Eco::API::Common::Lo
       raise "Review your -use-field expression. It should read: person.details[target-alt_id]"
     end
 
-    if part.start_with?("account")
+    if part.start_with?('account')
       return obj.respond_to?(:account) ? obj.account : nil
     end
 
-    return obj if part.start_with?("person")
+    return obj if part.start_with?('person')
 
     raise "Review your expression. Cannot recognize '#{part}' as part of '#{obj.class}'"
   end

data/lib/eco/api/usecases/default.rb

@@ -2,11 +2,11 @@ module Eco
   module API
     class UseCases
       class Default < Eco::API::UseCases
-        # autoloads_children_of "Eco::API::Common::Loaders::UseCase"
-        # autoload_namespace    "Eco::API::UseCases::Default"
-        # autoload_namespace    "Eco::API::UseCases::DefaultCases"
-        # autoload_namespace    "Eco::API::UseCases::OozeCases"
-        # autoload_namespace    "Eco::API::UseCases::GraphQL::Cases"
+        # autoloads_children_of 'Eco::API::Common::Loaders::UseCase'
+        # autoload_namespace    'Eco::API::UseCases::Default'
+        # autoload_namespace    'Eco::API::UseCases::DefaultCases'
+        # autoload_namespace    'Eco::API::UseCases::OozeCases'
+        # autoload_namespace    'Eco::API::UseCases::GraphQL::Cases'
       end
     end
   end

data/lib/eco/api/usecases/default_cases/to_csv_case.rb

@@ -1,5 +1,5 @@
 class Eco::API::UseCases::DefaultCases::ToCsvCase < Eco::API::Common::Loaders::UseCase
-  name "to-csv"
+  name 'to-csv'
   type :export
 
   OUT_FILENAME = 'pm'.freeze
@@ -8,14 +8,14 @@ class Eco::API::UseCases::DefaultCases::ToCsvCase < Eco::API::Common::Loaders::U
     options[:end_get] = false
 
     unless people && !people.empty?
-      log(:warn) { "No source people to create the file... aborting!" }
+      log(:warn) { 'No source people to create the file... aborting!' }
       return false
     end
 
     if options.dig(:export, :options, :split_schemas)
       by_schema.each do |id, people|
         sch_name = schemas.to_name(id)
-        prefix   = sch_name ? sch_name.gsub(" ", "_").downcase : "no_schema"
+        prefix   = sch_name ? sch_name.gsub(' ', '_').downcase : 'no_schema'
         filename = in_folder("#{prefix}_#{File.basename(file)}")
         create_file!(filename, people)
       end
@@ -29,7 +29,7 @@ class Eco::API::UseCases::DefaultCases::ToCsvCase < Eco::API::Common::Loaders::U
   def create_file!(filename = file, data = people)
     session.log(:info) { "going to create file: #{filename}" }
 
-    CSV.open(filename, "w") do |csv|
+    CSV.open(filename, 'w') do |csv|
       csv << spot_header(data.first)
       data.each do |person|
         csv << to_row(person)
@@ -40,13 +40,13 @@ class Eco::API::UseCases::DefaultCases::ToCsvCase < Eco::API::Common::Loaders::U
   def to_row(person)
     entry = to_entry_type(person)
     entry.values_at(*keys(entry)).tap do |row|
-      row << (schemas.to_name(person.details&.schema_id) || "No Schema")
+      row << (schemas.to_name(person.details&.schema_id) || 'No Schema')
     end
   end
 
   def spot_header(person = people.first)
     header = keys(to_entry_type(person))
-    header << "Schema"
+    header << 'Schema'
     header = yield(header) if block_given?
     header = nice_header_names(header, schema: schema(person.details)) if nice_header_names?
     header
@@ -73,14 +73,14 @@ class Eco::API::UseCases::DefaultCases::ToCsvCase < Eco::API::Common::Loaders::U
   end
 
   def deps
-    @deps ||= {"supervisor_id" => {people: people}}
+    @deps ||= {'supervisor_id' => {people: people}}
   end
 
   def file
     @file ||= out_filename.tap do |filename|
       next if filename
 
-      log(:error) { "Destination file not specified" }
+      log(:error) { 'Destination file not specified' }
       return false
     end
   end

data/lib/eco/api/usecases/default_cases/upsert_case.rb

@@ -1,12 +1,12 @@
 class Eco::API::UseCases::DefaultCases::UpsertCase < Eco::API::Common::Loaders::UseCase
-  name "upsert"
+  name 'upsert'
   type :sync
 
   def main(entries, people, session, options, usecase)
     micro    = session.micro
-    creation = session.new_job("main", "create", :create, usecase)
-    update   = session.new_job("main", "update", :update, usecase)
-    supers   = session.new_job("post", "supers", :update, usecase, :core)
+    creation = session.new_job('main', 'create', :create, usecase)
+    update   = session.new_job('main', 'update', :update, usecase)
+    supers   = session.new_job('post', 'supers', :update, usecase, :core)
 
     micro.with_each(entries, people, options, append_created: append_created) do |entry, person|
       person.new? ? creation.add(person) : update.add(person)

data/lib/eco/api/usecases/graphql/helpers/base.rb

@@ -11,7 +11,7 @@ module Eco::API::UseCases::GraphQL::Helpers
 
     # Keep a copy of the requests/responses for future reference
     def backup(data, type:)
-      dry_run   = simulate? ? "_dry_run" : ""
+      dry_run   = simulate? ? '_dry_run' : ''
       dir       = config.people.requests_folder
       file      = File.join(dir, "#{type}#{dry_run}.json")
       session.file_manager.save_json(data, file, :timestamp)

data/lib/eco/api/usecases/graphql/helpers/location/base/tree_tracking.rb

@@ -35,7 +35,11 @@ module Eco::API::UseCases::GraphQL::Helpers::Location
 
         latest_tree = tree if tree.is_a?(Eco::API::Organization::TagTree)
         if tree.respond_to?(:treeify) && track_current_tree?
-          latest_tree ||= Eco::API::Organization::TagTree.new(tree.treeify, id: tree.id, name: tree.name)
+          latest_tree ||= Eco::API::Organization::TagTree.new(
+            tree.treeify,
+            id:   tree.id,
+            name: tree.name
+          )
         end
 
         latest_tree.tap do

data/lib/eco/api/usecases/graphql/helpers/location/command/diffs/stages/commandable.rb

@@ -37,7 +37,8 @@ class Eco::API::UseCases::GraphQL::Helpers::Location::Command::Diffs
       end
 
       def stage_diffs(stage)
-        raise "There is NO support for stage '#{stage}'. Please review" unless respond_to?(stage)
+        msg = "There is NO support for stage '#{stage}'. Please review"
+        raise ArgumentError, msg unless respond_to?(stage)
 
         sort_diffs(stage) do
           send(stage)

data/lib/eco/api/usecases/graphql/helpers/location/command/result.rb

@@ -60,13 +60,13 @@ module Eco::API::UseCases::GraphQL::Helpers::Location::Command
     def command_id
       return unless result
 
-      command_result["id"]
+      command_result['id']
     end
 
     def as_json
       {
-        "request"  => input,
-        "response" => result.doc
+        'request'  => input,
+        'response' => result.doc
       }
     end
   end

data/lib/eco/api/usecases/graphql/helpers/location/command.rb

@@ -72,13 +72,16 @@ module Eco::API::UseCases::GraphQL::Helpers::Location
       pages = (total.to_f / size).ceil.to_i
       page  = 1
       out   = []
+
       comms.each_slice(size) do |comms_slice|
         sliced_input = input_data.slice(:clientMutationId, :id).merge(commands: comms_slice)
+
         yield(sliced_input, page, pages, comms_slice.count, total).tap do |response|
           out.push([sliced_input, response])
           page += 1
         end
       end
+
       out
     end
   end

data/lib/eco/api/usecases/graphql/samples/location/command/dsl.rb

@@ -17,8 +17,8 @@ class Eco::API::UseCases::GraphQL::Samples::Location
     #       end
     #     end
     def inputs(*_args, force_continue: force_continue?, **_kargs, &_block) # rubocop:disable Lint/UnusedMethodArgument
-      msg  = "You should implement this method in your child class.\n"
-      msg << "Which should yield the input Hash and the stage or descriptor."
+      msg  = "You should implement this method in your child class.\n'
+      msg << 'Which should yield the input Hash and the stage or descriptor."
       raise Eco::API::UseCases::GraphQL::Base::NotImplementedMethod, msg
     end
 

data/lib/eco/api/usecases/graphql/samples/location/command/service/tree_update.rb

@@ -8,6 +8,7 @@ class Eco::API::UseCases::GraphQL::Samples::Location
       class << self
         def included(base)
           super
+
           base.send :include, Eco::API::UseCases::GraphQL::Samples::Location::Service::TreeDiff
           base.send :include, Eco::API::UseCases::GraphQL::Samples::Location::Command::DSL
           base.send :include, Eco::API::UseCases::GraphQL::Utils::Sftp
@@ -40,7 +41,11 @@ class Eco::API::UseCases::GraphQL::Samples::Location
             original_tree: current_tree,
             logger:        logger
           ).tap do |nodes_diff|
-            archive_input = input(nodes_diff.stage_commands(:archive), force_continue: true)
+            archive_input = input(
+              nodes_diff.stage_commands(:archive),
+              force_continue: true
+            )
+
             sliced_batches(
               archive_input,
               desc:            stage,
@@ -48,6 +53,7 @@ class Eco::API::UseCases::GraphQL::Samples::Location
             ) do |sliced_input, response, page, pages, count, total| # rubocop:disable Metrics/ParameterLists
               page_results = request_results_class.new(sliced_input, response)
               (results[stage] ||= []) << page_results
+
               self.error ||= page_errors?(page_results, page, pages, count, total, stage: stage)
             end
           end
@@ -89,8 +95,8 @@ class Eco::API::UseCases::GraphQL::Samples::Location
           return if options.dig(:workflow, :no_email)
 
           digest_msgs   = logger.cache.logs(level: %i[info error warn])
-          str_exception = exception ? " - Exception!" : ''
-          str_exception = " - Aborted!" if exiting?
+          str_exception = exception ? ' - Exception!' : ''
+          str_exception = ' - Aborted!' if exiting?
 
           subject       = "#{config.active_enviro} - #{title}#{str_exception}"
 

data/lib/eco/api/usecases/graphql/samples/location/service/tree_diff/convertible/inputable.rb

@@ -4,14 +4,15 @@ module Eco::API::UseCases::GraphQL::Samples::Location::Service
       module Inputable
         class << self
           def included(base)
-            super(base)
+            super
+
             base.send :include, Eco::Data::Files
           end
         end
 
         include Eco::API::UseCases::GraphQL::Helpers::Base::CaseEnv
 
-        IN_FILENAME  = /requested_tree\.csv/.freeze
+        IN_FILENAME  = /requested_tree\.csv/
         IN_FOLDER    = 'sftp'.freeze
 
         private
@@ -25,7 +26,7 @@ module Eco::API::UseCases::GraphQL::Samples::Location::Service
             if file
               log(:info) { "Using input file '#{file}'" }
             else
-              log(:warn) { "Could not find any input file" }
+              log(:warn) { 'Could not find any input file' }
               exit(1)
             end
           end
@@ -45,7 +46,7 @@ module Eco::API::UseCases::GraphQL::Samples::Location::Service
         end
 
         def input_folder
-          "#{config.active_enviro}/#{self.class::IN_FOLDER}"
+          File.join(config.active_enviro, self.class::IN_FOLDER)
         end
       end
     end

data/lib/eco/api/usecases/graphql/samples/location/service/tree_diff/convertible/parsing.rb

@@ -26,18 +26,21 @@ module Eco::API::UseCases::GraphQL::Samples::Location::Service
             row_transform&.call(row)
             transform_classifications(row)
           end
+
           csv
         end
 
         # The user define block for row transformation
         def custom_input_row_transform_block
-          return nil unless respond_to?(:custom_input_row_transform, true)
+          return unless respond_to?(:custom_input_row_transform, true)
+
           method(:custom_input_row_transform)
         end
 
         # The user define block for row transformation
         def custom_input_csv_transform_block
-          return nil unless respond_to?(:custom_input_csv_transform, true)
+          return unless respond_to?(:custom_input_csv_transform, true)
+
           method(:custom_input_csv_transform)
         end
       end

data/lib/eco/api/usecases/graphql/samples/location/service/tree_diff.rb

@@ -5,11 +5,11 @@ module Eco::API::UseCases::GraphQL::Samples
     #    class Custom::UseCase::TagtreeDiff < Eco::API::UseCases::GraphQL::Samples::Location
     #      name 'tagtree-diff'
     #      type :other
-
+    #
     #      require_relative 'cli/10_tagtree_diff_cli'
-
+    #
     #      include Eco::API::UseCases::GraphQL::Samples::Location::Service::TreeDiff
-
+    #
     #      IN_FILENAME  = /requested_tree\.csv/.freeze
     #      IN_FOLDER    = 'sftp'.freeze
     #      HEADER_MAPS  = {

data/lib/eco/api/usecases/graphql/utils/sftp.rb

@@ -1,6 +1,6 @@
 module Eco::API::UseCases::GraphQL::Utils
   module Sftp
     include Eco::API::UseCases::GraphQL::Helpers::Base::CaseEnv
-    include Eco::API::UseCases::Lib::Sftp
+    include Eco::API::UseCases::Lib::Files::Sftp
   end
 end

data/lib/eco/api/usecases/lib/{file_pattern.rb → files/file_pattern.rb}

@@ -1,4 +1,4 @@
-module Eco::API::UseCases::Lib
+module Eco::API::UseCases::Lib::Files
   module FilePattern
     class WrongConst < ArgumentError; end
 
@@ -29,6 +29,10 @@ module Eco::API::UseCases::Lib
         rescue NameError
           self.class.const_get(fpc)
         end
+      elsif respond_to?(:file_pattern, true)
+        file_pattern
+      elsif self.class.const_defined?(:FILE_PATTERN)
+        self.class::FILE_PATTERN
       end
     rescue NameError
       raise WrongConst, "(#{self.class}) Unknown constant: #{fpc}"

data/lib/eco/api/usecases/lib/{sftp.rb → files/sftp.rb}

@@ -1,4 +1,4 @@
-module Eco::API::UseCases::Lib
+module Eco::API::UseCases::Lib::Files
   module Sftp
     class UnspecifiedRemoteFolder < ArgumentError; end
 
@@ -18,10 +18,15 @@ module Eco::API::UseCases::Lib
     alias_method :sftp_upload, :upload
 
     def sftp_group_id
-      if self.class.const_defined?(:SFTP_GROUP)
-        self.class.const_get(:SFTP_GROUP)
-      elsif (group_id = options.dig(:sftp, :group))
+      return super    if defined?(super) && !super.nil?
+      return group_id if respond_to?(:group_id, true)
+
+      if (group_id = options.dig(:sftp, :group))
         group_id
+      elsif self.class.const_defined?(:SFTP_GROUP)
+        self.class.const_get(:SFTP_GROUP)
+      elsif self.class.const_defined?(:GROUP_ID)
+        self.class.const_get(:GROUP_ID)
       end
     end
 
@@ -78,7 +83,7 @@ module Eco::API::UseCases::Lib
       sftp_archive_file!(filename)
     end
 
-    def ensure_remote_empty(subfolder: remote_subfolder, pattern: nil)
+    def ensure_remote_empty(subfolder: remote_subfolder, pattern: nil, timeout: 3)
       files = with_remote_files(subfolder: subfolder, pattern: pattern)
       return if files.empty?
 
@@ -92,7 +97,7 @@ module Eco::API::UseCases::Lib
         'Do you want to proceed to delete? (Y/n):',
         explanation: msg,
         default:     'Y',
-        timeout:     3
+        timeout:     timeout
       ) do |response|
         next unless response.upcase.start_with?('Y')
 
@@ -106,6 +111,18 @@ module Eco::API::UseCases::Lib
       end
     end
 
+    def local_folder
+      return super if defined?(super) && !super.nil?
+
+      if (local_dir = options.dig(:sftp, :local_folder))
+        local_dir
+      elsif self.class.const_defined?(:LOCAL_FOLDER)
+        self.class::LOCAL_FOLDER
+      else
+        '.'
+      end
+    end
+
     def to_remote_path(file, subfolder: remote_subfolder)
       [remote_folder(subfolder), file].join('/')
     end
@@ -113,6 +130,8 @@ module Eco::API::UseCases::Lib
     # `remote_target_folder` overrides `sftp_config.remote_folder` as well as `remote_subfolder`
     # `remote_folder` overrides `sftp_config.remote_folder` but NOT `remote_subfolder`
     def remote_folder(subfolder = remote_subfolder)
+      return super if defined?(super) && !super.nil?
+
       rm_tf = options.dig(:sftp, :remote_target_folder)
       return rm_tf if rm_tf
 
@@ -121,6 +140,8 @@ module Eco::API::UseCases::Lib
     end
 
     def remote_subfolder(required: true)
+      return super if defined?(super) && !super.nil?
+
       remote_subdir   = options.dig(:sftp, :remote_subfolder)
       remote_subdir ||= self.class::REMOTE_FOLDER    if self.class.const_defined?(:REMOTE_FOLDER)
       remote_subdir ||= self.class::REMOTE_SUBFOLDER if self.class.const_defined?(:REMOTE_SUBFOLDER)
@@ -132,16 +153,6 @@ module Eco::API::UseCases::Lib
       raise UnspecifiedRemoteFolder, msg
     end
 
-    def local_folder
-      if (local_dir = options.dig(:sftp, :local_folder))
-        local_dir
-      elsif self.class.const_defined?(:LOCAL_FOLDER)
-        self.class::LOCAL_FOLDER
-      else
-        '.'
-      end
-    end
-
     # @param subfolder [String] the subfolder.
     def with_remote_files(subfolder: remote_subfolder, pattern: nil)
       sftp.files(remote_folder(subfolder), pattern: pattern).each do |remote_file|

data/lib/eco/api/usecases/lib/files.rb

@@ -0,0 +1,7 @@
+module Eco::API::UseCases::Lib
+  module Files
+  end
+end
+
+require_relative 'files/sftp'
+require_relative 'files/file_pattern'

data/lib/eco/api/usecases/lib.rb

@@ -8,5 +8,4 @@ module Eco
 end
 
 require_relative 'lib/error_handling'
-require_relative 'lib/sftp'
-require_relative 'lib/file_pattern'
+require_relative 'lib/files'

data/lib/eco/api/usecases/ooze_cases/export_register_case.rb

@@ -1,13 +1,13 @@
 # Use case to export a register into a CSV
 class Eco::API::UseCases::OozeCases::ExportRegisterCase < Eco::API::UseCases::OozeSamples::RegisterExportCase
-  name "export-register"
+  name 'export-register'
   type :other
   batch_size 5
 
   def main(session, options, usecase)
-    super(session, options, usecase) do
+    super do
       # Save the File
-      CSV.open(filename, "w") do |csv|
+      CSV.open(filename, 'w') do |csv|
         csv << exportable_register.header(refresh: true)
         exportable_register.each(as_values: true) do |values|
           csv << values
@@ -79,7 +79,7 @@ class Eco::API::UseCases::OozeCases::ExportRegisterCase < Eco::API::UseCases::Oo
   end
 
   def options_delimiter
-    options.dig(:export, :options, :delimiter) || "|"
+    options.dig(:export, :options, :delimiter) || '|'
   end
 
   def include_deindexed?
@@ -94,7 +94,7 @@ class Eco::API::UseCases::OozeCases::ExportRegisterCase < Eco::API::UseCases::Oo
     @filename ||= (options[:file] || options.dig(:export, :file, :name)).tap do |filename|
       next if filename
 
-      log(:error) { "Destination file not specified" }
+      log(:error) { 'Destination file not specified' }
       return false
     end
   end

data/lib/eco/api/usecases/ooze_samples/helpers/exportable_ooze.rb

@@ -5,16 +5,16 @@ module Eco
         module Helpers
           # Class to ease the export process
           class ExportableOoze
-            MERGE_DELIMITER = "#:#".freeze
+            MERGE_DELIMITER = '#:#'.freeze
             META_FIELDS     = {
-              "id"         => "Internal ID",
-              "uid"        => "Unique ID",
-              "name"       => "Name of Page",
-              "state"      => "State of Page",
-              "time_zone"  => "Time Zone",
-              "created_at" => "Page Created",
-              "updated_at" => "Last updated",
-              "tags"       => "Location Tags"
+              'id'         => 'Internal ID',
+              'uid'        => 'Unique ID',
+              'name'       => 'Name of Page',
+              'state'      => 'State of Page',
+              'time_zone'  => 'Time Zone',
+              'created_at' => 'Page Created',
+              'updated_at' => 'Last updated',
+              'tags'       => 'Location Tags'
             }.freeze
             DEFAULT_OPTIONS = {
               delimiter:     "\n",
@@ -31,9 +31,10 @@ module Eco
               def label(field, default: nil)
                 value   = nil
                 value ||= field.label.to_s.strip if field.respond_to?(:label)
-                value   = nil unless value && !value.empty?
+                value   = nil                    unless value && !value.empty?
                 return value if value || !default
-                default || "Unnamed field:"
+
+                default || 'Unnamed field:'
               end
 
               def key_ref_label(field, default: nil)
@@ -157,6 +158,7 @@ module Eco
               return false if opts[:only_indexed]  && !self.class.indexed?(field)
               return false if opts[:only_labeled]  && !self.class.label?(field)
               return false if opts[:only_with_ref] && !self.class.with_ref?(field)
+
               true
             end
 
@@ -167,6 +169,7 @@ module Eco
             def to_value(field, delimiter: self.delimiter)
               return nil unless field.respond_to?(:to_s)
               return field.to_s unless offers_delimiter?(field.method(:to_s))
+
               field.to_s(delimiter: delimiter)
             end