ecoportal-api 0.10.8 → 0.10.9

data/lib/ecoportal/api/internal/person_schema.rb

@@ -2,7 +2,7 @@ module Ecoportal
   module API
     class Internal
       class PersonSchema < V1::PersonSchema
-        class_resolver :schema_field_class, "Ecoportal::API::Internal::SchemaField"
+        class_resolver :schema_field_class, 'Ecoportal::API::Internal::SchemaField'
       end
     end
   end

data/lib/ecoportal/api/internal/person_schemas.rb

@@ -2,7 +2,7 @@ module Ecoportal
   module API
     class Internal
       class PersonSchemas < V1::PersonSchemas
-        class_resolver :person_schema_class,  "Ecoportal::API::Internal::PersonSchema"
+        class_resolver :person_schema_class, 'Ecoportal::API::Internal::PersonSchema'
       end
     end
   end

data/lib/ecoportal/api/internal/policy_groups.rb

@@ -15,13 +15,14 @@ module Ecoportal
 
         # Gets all the policy groups via api request.
         # @return [Enumerable<PolicyGroup>] an `Enumerable` with all the policy groups already wrapped as `PolicyGroup` objects.
-        def get_all
-          response = client.get("/policy_groups")
+        def get_all # rubocop:disable Naming/AccessorMethodName
+          response = client.get('/policy_groups')
           Common::WrappedResponse.new(response, Internal::PolicyGroup)
         end
 
         def each(&block)
           return to_enum(:each) unless block
+
           get_all.each(&block)
         end
       end

data/lib/ecoportal/api/internal/preferences.rb

@@ -2,17 +2,19 @@ module Ecoportal
   module API
     class Internal
       class Preferences < Common::BaseModel
-        passthrough  :kiosk_enabled, :kiosk_workflow_message, :kiosk_create_button_label,
-            :kiosk_create_button_help, :kiosk_return_button_label, :kiosk_return_button_help,
-            :kiosk_dashboard_button_label, :kiosk_dashboard_button_help
+        passthrough(
+          :kiosk_enabled, :kiosk_workflow_message, :kiosk_create_button_label,
+          :kiosk_create_button_help, :kiosk_return_button_label, :kiosk_return_button_help,
+          :kiosk_dashboard_button_label, :kiosk_dashboard_button_help
+        )
 
-        [
-          :show_sidebar,
-          :show_shortcuts,
-          :show_coming_soon,
-          :show_recently_visited_forms,
-          :show_tasks,
-          :show_task_bubbles
+        %i[
+          show_sidebar
+          show_shortcuts
+          show_coming_soon
+          show_recently_visited_forms
+          show_tasks
+          show_task_bubbles
         ].map(&:to_s).each do |field|
           define_method(field) do
             if doc.key?(field)
@@ -21,6 +23,7 @@ module Ecoportal
               true
             end
           end
+
           define_method("#{field}=") do |value|
             doc[field] = !!value
           end

data/lib/ecoportal/api/internal.rb

@@ -3,12 +3,12 @@ module Ecoportal
     class Internal < V1
       include Common::Logging
 
-      VERSION = "v0".freeze
-      class_resolver :people_class, "Ecoportal::API::Internal::People"
-      class_resolver :person_schemas_class, "Ecoportal::API::Internal::PersonSchemas"
+      VERSION = 'v0'.freeze
+      class_resolver :people_class, 'Ecoportal::API::Internal::People'
+      class_resolver :person_schemas_class, 'Ecoportal::API::Internal::PersonSchemas'
 
-      class_resolver :policy_groups_class, "Ecoportal::API::Internal::PolicyGroups"
-      class_resolver :login_providers_class, "Ecoportal::API::Internal::LoginProviders"
+      class_resolver :policy_groups_class, 'Ecoportal::API::Internal::PolicyGroups'
+      class_resolver :login_providers_class, 'Ecoportal::API::Internal::LoginProviders'
 
       # Obtain specific object for policy groups api requests.
       # @return [PolicyGroups] an instance object ready to make policy groups api requests.

data/lib/ecoportal/api/logger.rb

@@ -1,26 +1,27 @@
 module Ecoportal
   module API
     class Logger
-      TIMESTAMP_PATTERN  = "%Y-%m-%dT%H:%M:%S".freeze
+      TIMESTAMP_PATTERN  = '%Y-%m-%dT%H:%M:%S'.freeze
 
       STDOUT_FORMAT_PROC = proc do |severity, _datetime, _progname, msg|
         prefix = "%5s > " % severity # rubocop:disable Style/FormatString
-        msg.lines.map.with_index do |line, idx|
+        msg.lines.map.with_index do |line, idx| # rubocop:disable Style/StringConcatenation
           if idx.zero?
             prefix + line.chomp
           else
-            (" " * prefix.length) + line.chomp
+            (' ' * prefix.length) + line.chomp
           end
         end.join("\n")+"\n"
       end
 
       FILE_FORMAT_PROC = proc do |severity, datetime, _progname, msg|
         prefix = "%5s(%s) > " % [severity, datetime.strftime(TIMESTAMP_PATTERN)] # rubocop:disable Style/FormatString, Style/FormatStringToken
-        msg.lines.map.with_index do |line, idx|
+
+        msg.lines.map.with_index do |line, idx| # rubocop:disable Style/StringConcatenation
           if idx.zero?
             prefix + line.chomp
           else
-            (" " * prefix.length) + line.chomp
+            (' ' * prefix.length) + line.chomp
           end
         end.join("\n")+"\n"
       end
@@ -30,7 +31,7 @@ module Ecoportal
       def initialize(
         console_level: ::Logger::INFO,
         file_level:   ::Logger::DEBUG,
-        output_file:  File.join(Dir.getwd, "API_Log-#{Time.now.strftime("%Y-%m-%dT%H%M")}.txt")
+        output_file:  File.join(Dir.getwd, "API_Log-#{Time.now.strftime('%Y-%m-%dT%H%M')}.txt")
       )
         @console = make_stdout_logger(console_level)
         @file    = make_file_logger(file_level, output_file)
@@ -54,7 +55,7 @@ module Ecoportal
 
       def make_file_logger(level, output_file)
         return unless output_file
-        ::Logger.new(File.open(output_file, "a")).tap do |logger|
+        ::Logger.new(File.open(output_file, 'a')).tap do |logger|
           logger.formatter = FILE_FORMAT_PROC
           logger.level     = level
         end

data/lib/ecoportal/api/v1/job/awaiter/timer.rb

@@ -9,6 +9,7 @@ module Ecoportal
             status lstatus
           ].freeze
 
+          # The class
           Timer = Struct.new(*TIMER_ARGS) do
             self::MAX_START_DELAY = 60
 

data/lib/ecoportal/api/v1/job/status.rb

@@ -44,8 +44,8 @@ module Ecoportal
           end
 
           def to_s
-            msg = complete? ? "Completed" : "In progress"
-            msg = "Errored" if errored?
+            msg = complete? ? 'Completed' : 'In progress'
+            msg = 'Errored' if errored?
             msg << " with #{progress} done."
             msg
           end

data/lib/ecoportal/api/v1/job.rb

@@ -31,7 +31,7 @@ module Ecoportal
           raise_if_already_launched! unless recover
 
           @operation ||= Common::BatchOperation.new(
-            "/people",
+            '/people',
             person_class,
             logger: client.logger
           )
@@ -84,7 +84,7 @@ module Ecoportal
         def raise_if_already_launched!
           return unless created?
 
-          msg  = "Missusage: job was already created."
+          msg  = 'Missusage: job was already created.'
           msg << " Can't call batch more than once"
           raise msg
         end
@@ -102,8 +102,8 @@ module Ecoportal
 
           job_id = nil
 
-          client.post("/people/job", data: operation.as_json).tap do |response|
-            job_id = body_data(response.body)["id"] if response.success?
+          client.post('/people/job', data: operation.as_json).tap do |response|
+            job_id = body_data(response.body)['id'] if response.success?
 
             next if job_id
 

data/lib/ecoportal/api/v1/people.rb

@@ -8,7 +8,7 @@ module Ecoportal
         include Common::DocHelpers
         include Enumerable
 
-        class_resolver :person_class, "Ecoportal::API::V1::Person"
+        class_resolver :person_class, 'Ecoportal::API::V1::Person'
 
         attr_reader :client
 
@@ -44,7 +44,7 @@ module Ecoportal
             count    = 5
 
             loop do
-              response = client.get("/people", params: params)
+              response = client.get('/people', params: params)
               body     = response && body_data(response.body)
               break if response.success? || count <= 0
 
@@ -56,22 +56,22 @@ module Ecoportal
 
             raise "Request failed - Status #{response.status}: #{body}" unless response.success?
 
-            unless silent || (total = body["total_results"])&.zero?
-              results += body["results"].length
+            unless silent || (total = body['total_results'])&.zero?
+              results += body['results'].length
               percent  = results * 100 / total
 
-              msg      = "People GET"
+              msg      = 'People GET'
               msg     << " (search=#{params[:q]})" if params.key?(:q)
 
               print "#{msg}: #{percent.round}% (of #{total}): #{results}\r"
               $stdout.flush
             end
 
-            body["results"].each do |person|
+            body['results'].each do |person|
               yield person_class.new(person)
             end
 
-            break unless (cursor_id = body["cursor_id"])
+            break unless (cursor_id = body['cursor_id'])
           end
           self
         end
@@ -106,6 +106,7 @@ module Ecoportal
         def update(doc)
           body = get_body(doc)
           id   = get_id(doc)
+
           client.patch("/people/#{CGI.escape(id)}", data: body)
         end
 
@@ -114,7 +115,8 @@ module Ecoportal
         # @return [Response] an object with the api response.
         def create(doc)
           body = get_body(doc)
-          client.post("/people", data: body)
+
+          client.post('/people', data: body)
         end
 
         # Requests to update an existing person or if it does not exist, to create it, via api.
@@ -143,7 +145,7 @@ module Ecoportal
           return job.batch(&block) if job_mode
 
           operation = Common::BatchOperation.new(
-            "/people",
+            '/people',
             person_class,
             logger: client.logger
           )
@@ -151,7 +153,7 @@ module Ecoportal
           yield operation
 
           # The batch operation is responsible for logging the output
-          client.post("/people/batch", data: operation.as_json).tap do |response|
+          client.post('/people/batch', data: operation.as_json).tap do |response|
             operation.process_response(response)
           end
         end

data/lib/ecoportal/api/v1/person.rb

@@ -10,25 +10,35 @@ module Ecoportal
       class Person < Common::BaseModel
         passthrough :id, :external_id, :name, :email, :filter_tags
         passthrough :archived
+        alias_method :archived?, :archived
+
         passthrough :supervisor_id, :contractor_organization_id
         passthrough :brand_id
         passthrough :freemium
 
-        class_resolver :person_schema_class,  "Ecoportal::API::V1::PersonSchema"
-        class_resolver :person_details_class, "Ecoportal::API::V1::PersonDetails"
+        class_resolver :person_schema_class,  'Ecoportal::API::V1::PersonSchema'
+        class_resolver :person_details_class, 'Ecoportal::API::V1::PersonDetails'
         embeds_one :details, nullable: true, klass: :person_details_class
 
         VALID_TAG_REGEX   = /^[A-Za-z0-9 &_'\/.-]+$/
         VALID_EMAIL_REGEX = /^[^@\s]+@[^@\s]+\.[^@\s]+$/
 
+        def unarchive!
+          self.archived = false
+        end
+
+        def archive!
+          self.archived = true
+        end
+
         # Gets the supervisor (`Person`) of this person, with given his `supervisor_id`.
         #
         # **Example Usage**:
         # ```ruby
         # API_KEY = 'some-private-api-key-version'
-        # HOST    = "live.ecoportal.com"
+        # HOST    = 'live.ecoportal.com'
         # api     = Ecoportal::API::Internal.new(API_KEY, host: HOST)
-        # person  = api.people.get({"id": "my-dummy-user"})
+        # person  = api.people.get({'id': 'my-dummy-user'})
         # super   = person.supervisor(api.client)
         # pp "#{person.name}'s supervisor is #{super.name}."
         # ```
@@ -37,6 +47,7 @@ module Ecoportal
         def supervisor(client)
           return @supervisor if defined?(@supervisor)
           return @supervisor = nil if supervisor_id.nil?
+
           @supervisor = client.people.get(supervisor_id).result
         end
 
@@ -51,7 +62,7 @@ module Ecoportal
         def email=(value)
           raise "Invalid email #{value.inspect}" if value && !value.match(VALID_EMAIL_REGEX)
 
-          doc["email"] = value&.downcase
+          doc['email'] = value&.downcase
         end
 
         # Validates the string tags of the array, and sets the `filter_tags` property of the account.
@@ -69,20 +80,20 @@ module Ecoportal
             tag.upcase
           end
 
-          set_uniq_array_keep_order("filter_tags", end_tags)
+          set_uniq_array_keep_order('filter_tags', end_tags)
         end
 
         # @return [Array<String>] the filter tags of this person.
         def filter_tags
-          doc["filter_tags"] ||= []
+          doc['filter_tags'] ||= []
         end
 
         def as_json
-          super.merge "details" => details&.as_json
+          super.merge 'details' => details&.as_json
         end
 
         def as_update(ref = :last, ignore: [])
-          super(ref, ignore: ignore)
+          super
         end
 
         # Sets the PersonDetails to the person, depending on the paramter received:
@@ -95,15 +106,15 @@ module Ecoportal
         def details=(value)
           case value
           when NilClass
-            doc["details"] = nil
+            doc['details'] = nil
           when person_details_class
-            doc["details"] = value.as_json
+            doc['details'] = value.as_json
           when Hash
-            doc["details"] = value.slice("schema_id", "fields")
+            doc['details'] = value.slice('schema_id', 'fields')
           else
             raise "Invalid type set on details. Required nil, PersonDetails or Hash; got #{value.class}"
           end
-          remove_instance_variable("@details") if defined?(@details)
+          remove_instance_variable('@details') if defined?(@details)
         end
 
         # Sets the PersonDetails to the person, depending on the parameter received:
@@ -127,10 +138,11 @@ module Ecoportal
             else
               raise "Invalid set on details: Requierd PersonSchema or String; got #{schema_or_id.class}"
             end
+
             self.details = new_details
             # Patch out static data from as_update
-            original_doc["details"] = {
-              "fields" => JSON.parse(doc["details"]["fields"].to_json)
+            original_doc['details'] = {
+              'fields' => JSON.parse(doc['details']['fields'].to_json)
             }
           end
         end

data/lib/ecoportal/api/v1/person_details.rb

@@ -7,10 +7,10 @@ module Ecoportal
 
         passthrough :schema_id
 
-        class_resolver :schema_field_value_class, "Ecoportal::API::V1::SchemaFieldValue"
+        class_resolver :schema_field_value_class, 'Ecoportal::API::V1::SchemaFieldValue'
 
         def as_json
-          super.merge "fields" => fields.map(&:as_json)
+          super.merge 'fields' => fields.map(&:as_json)
         end
 
         # Sets the `id` of the PersonDetails.
@@ -18,15 +18,15 @@ module Ecoportal
         # @param value [nil, String] the id of the schema.
         def schema_id=(value)
           @fields          = [] if value.nil?
-          doc["schema_id"] = value
+          doc['schema_id'] = value
         end
 
         # Gets all the fields of the PersonDetails.
         # @return [Array<SchemaFieldValue>] the array of fields of the schema.
         def fields
           return @fields if defined?(@fields)
-          @fields = (doc["fields"] || []).each_with_index.map do |field, i|
-            schema_field_value_class.new(field, parent: self, key: ["fields", i])
+          @fields = (doc['fields'] || []).each_with_index.map do |field, i|
+            schema_field_value_class.new(field, parent: self, key: ['fields', i])
           end
         end
 
@@ -69,13 +69,13 @@ module Ecoportal
         def changed?(id, doc = :original)
           return false unless (field = get_field(id))
 
-          field.as_update.key?("value")
+          field.as_update.key?('value')
         end
 
         def original_value(id)
           return nil unless (field = get_field(id))
 
-          field.original_doc["value"]
+          field.original_doc['value']
         end
 
         protected

data/lib/ecoportal/api/v1/person_schema.rb

@@ -5,7 +5,7 @@ module Ecoportal
         passthrough :id, :name
         passthrough :enable_tags, :tags
 
-        class_resolver :schema_field_class, "Ecoportal::API::V1::SchemaField"
+        class_resolver :schema_field_class, 'Ecoportal::API::V1::SchemaField'
 
         def fields
           @fields_by_id or index_fields
@@ -31,7 +31,7 @@ module Ecoportal
           @fields_by_id     = {}
           @fields_by_alt_id = {}
 
-          doc["fields"].each do |field|
+          doc['fields'].each do |field|
             wrapped = schema_field_class.new(field)
             @fields_by_id[wrapped.id] = wrapped
             @fields_by_alt_id[wrapped.alt_id] = wrapped
@@ -40,9 +40,9 @@ module Ecoportal
 
         def initialize_details(details)
           details.schema_id = id
-          details.doc["fields"] = fields.map do |field|
+          details.doc['fields'] = fields.map do |field|
             field.doc.slice(*%w[id alt_id name multiple type shared]).merge(
-              "value" => field.multiple ? [] : nil
+              'value' => field.multiple ? [] : nil
             )
           end
         end
@@ -50,4 +50,5 @@ module Ecoportal
     end
   end
 end
+
 require 'ecoportal/api/v1/schema_field'

data/lib/ecoportal/api/v1/person_schemas.rb

@@ -1,12 +1,13 @@
 module Ecoportal
   module API
     class V1
-      # @attr_reader client [Common::Client] a `Common::Client` object that holds the configuration of the api connection.
+      # @attr_reader client [Common::Client] a `Common::Client` object that holds
+      #   the configuration of the api connection.
       class PersonSchemas
         extend Common::BaseClass
         include Enumerable
 
-        class_resolver :person_schema_class,  "Ecoportal::API::V1::PersonSchema"
+        class_resolver :person_schema_class, 'Ecoportal::API::V1::PersonSchema'
 
         attr_reader :client
 
@@ -18,8 +19,8 @@ module Ecoportal
 
         # Gets all the schemas via api request.
         # @return [Enumerable<PersonSchema>] an `Enumerable` with all schemas already wrapped as `PersonSchema` objects.
-        def get_all
-          response = client.get("/person_schemas")
+        def get_all # rubocop:disable Naming/AccessorMethodName
+          response = client.get('/person_schemas')
           Common::WrappedResponse.new(response, person_schema_class)
         end
 
@@ -29,13 +30,15 @@ module Ecoportal
         #   - `:params` doesn't really do anything.
         #   - same as #get_all but with block :)
         #   - `to_a` will call `each` (see this [detailed explanation](https://stackoverflow.com/a/45201663/4352306))
-        #     - however, as far as you have an iterator, such as `each`, `to_a` should be last resource (see [this explanation](https://stackoverflow.com/a/44186921/4352306))
+        #     - however, as far as you have an iterator, such as `each`,
+        #       `to_a` should be last resource (see [this explanation](https://stackoverflow.com/a/44186921/4352306))
         # @yield [schema] does some stuff with the schema.
         # @yieldparam schema [PersonSchema]
         # @yieldreturn [PersonSchema]
         # @return [Enumerable<PersonSchema>] an `Enumerable` with all the person schema objects.
         def each(params: {}, &block)
-          return to_enum(:each) unless block
+          return to_enum(:each, params: params) unless block
+
           get_all.each(&block)
         end
       end

data/lib/ecoportal/api/v1/schema_field.rb

@@ -7,19 +7,19 @@ module Ecoportal
         def parse_text(value)
           values = [*value.to_s.lines].map do |line|
             line = line.chomp
-            next if line == ""
+            next if line == ''
             case type
-            when "text", "phone_number"
+            when 'text', 'phone_number'
               line
-            when "number"
-              Float(line) rescue return nil, false
-            when "boolean"
+            when 'number'
+              Float(line) rescue return nil, false # rubocop:disable Style/RescueModifier
+            when 'boolean'
               %w[true TRUE True Y y YES X x].include?(line)
-            when "select"
+            when 'select'
               return nil, false unless options.include?(line)
               line
-            when "date"
-              Date.parse(line) rescue return nil, false
+            when 'date'
+              Date.parse(line) rescue return nil, false # rubocop:disable Style/RescueModifier
             end
           end.compact
 

data/lib/ecoportal/api/v1/schema_field_value.rb

@@ -6,18 +6,18 @@ module Ecoportal
         passthrough :id, :alt_id, :type, :name, :shared, :multiple
 
         def clear
-          return doc["value"] = [] if multiple
+          return doc['value'] = [] if multiple
 
-          doc["value"] = nil
+          doc['value'] = nil
         end
 
         def value
           case type
-          when "text", "phone_number", "number", "boolean", "select"
-            doc["value"]
-          when "date"
-            if doc["value"]
-              maybe_multiple(doc["value"]) do |v|
+          when 'text', 'phone_number', 'number', 'boolean', 'select'
+            doc['value']
+          when 'date'
+            if doc['value']
+              maybe_multiple(doc['value']) do |v|
                 Date.iso8601(v)
               end
             end
@@ -28,28 +28,28 @@ module Ecoportal
 
         def value=(value) # rubocop:disable Metrics/AbcSize
           case type
-          when "text", "phone_number", "select"
-            doc["value"] = maybe_multiple(value) do |v|
+          when 'text', 'phone_number', 'select'
+            doc['value'] = maybe_multiple(value) do |v|
               v&.to_s
             end
-          when "number"
+          when 'number'
             maybe_multiple(value) do |v|
               next if v.nil? || v.is_a?(Numeric)
 
               raise "Invalid number type #{v.class}"
             end
 
-            doc["value"] = value
-          when "boolean"
-            doc["value"] = !!value
-          when "date"
+            doc['value'] = value
+          when 'boolean'
+            doc['value'] = !!value
+          when 'date'
             maybe_multiple(value) do |v|
               next if v.nil? || v.respond_to?(:to_date)
 
               raise "Invalid date type #{v.class}"
             end
 
-            doc["value"] = maybe_multiple(value) do |v|
+            doc['value'] = maybe_multiple(value) do |v|
               v&.to_date&.to_s
             end
           else

data/lib/ecoportal/api/v1.rb

@@ -6,10 +6,10 @@ module Ecoportal
       extend Common::BaseClass
       include Common::Logging
 
-      VERSION = "v1".freeze
+      VERSION = 'v1'.freeze
 
-      class_resolver :people_class, "Ecoportal::API::V1::People"
-      class_resolver :person_schemas_class, "Ecoportal::API::V1::PersonSchemas"
+      class_resolver :people_class, 'Ecoportal::API::V1::People'
+      class_resolver :person_schemas_class, 'Ecoportal::API::V1::PersonSchemas'
 
       attr_reader :client, :logger
 
@@ -21,7 +21,7 @@ module Ecoportal
       # @param host [String] api server domain.
       # @param logger [Logger] an object with `Logger` interface to generate logs.
       # @return [V1] an object with the api version suit.
-      def initialize(api_key, host: "live.ecoportal.com", logger: default_logger)
+      def initialize(api_key, host: 'live.ecoportal.com', logger: default_logger)
         @logger = logger
         @client = Common::Client.new(
           api_key: api_key,

data/lib/ecoportal/api/version.rb

@@ -1,5 +1,5 @@
 module Ecoportal
   module API
-    VERSION = '0.10.8'.freeze
+    VERSION = '0.10.9'.freeze
   end
 end