tessitura_rest 1.1.6 → 1.1.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b6b2fcc55c832163daaada90d305305c168190bc90d8ff8320b87d16dbf5e3dd
-  data.tar.gz: 54423f5d268a2ca8595288500edfd521c8774826279b0851b1d4331d24932645
+  metadata.gz: 29001154bd7cb717ba0cdf2dc395a99d7fc6dad99a362c59ee656cb83f50e07d
+  data.tar.gz: '084cda4d0a4019127badad64017a862d3514ce726b0aa9a3fe19f78c3ffedda2'
 SHA512:
-  metadata.gz: fd82b97aec4322cdd20a48f1db2ed9b03611bf175d1ee376c8ab3e0dc050303f950d14d4c3668e3b3e7b67da909196d58588bd30ad2eb6341c1dc2246cf59853
-  data.tar.gz: d2e4457f4fbedcf849b25635df5a91202621c41b25f5f881a6f5e6155be6acca84af4e0118fbc1689d2e33c07d59222c3e01ae6a22ab531102ab47090ae2bec6
+  metadata.gz: 3b27cb8a505f4c72934c5b835e794d006cfb6a7c7a1d1c1f3b4661e32673cedf988d9e3eb835b382ef06395235a66579982a30dd59fc560ec292a81aa7e9af81
+  data.tar.gz: 7e0c2d304a6ef951bf8234fed18c1ab439751536f2726a3450d4cea3df35ecd463913e740d43b381e6dd5cdc8401277fe597cc579c798e2456e30f1c8e411e4a

data/.codeclimate.yml

@@ -28,7 +28,7 @@ plugins:
         enabled: false
   rubocop:
     enabled: true
-    channel: rubocop-0-88
+    channel: rubocop-1-21-0
   stylelint:
     enabled: false
     checks:

data/.rubocop.yml

@@ -1,6 +1,9 @@
+
 AllCops:
+  NewCops: enable
   Exclude:
     - spec/**/*
+  TargetRubyVersion: 3.0.1
 
 Naming/AccessorMethodName:
   Description: Check the naming of accessor methods for get_/set_.
@@ -196,10 +199,10 @@ Style/LineEndConcatenation:
                  line end.
   Enabled: false
 
-# Metrics/LineLength:
-#   Description: 'Limit lines to 80 characters.'
-#   StyleGuide: 'https://github.com/bbatsov/ruby-style-guide#80-character-limits'
-#   Max: 80
+Layout/LineLength:
+  Description: 'Limit lines to 80 characters.'
+  StyleGuide: 'https://github.com/bbatsov/ruby-style-guide#80-character-limits'
+  Max: 180
 
 Metrics/MethodLength:
   Description: 'Avoid methods longer than 10 lines of code.'
@@ -543,11 +546,24 @@ Lint/Void:
   Description: 'Possible use of operator/literal/variable in void context.'
   Enabled: false
 
-# Performance
-
-Performance/Sample:
+Style/Sample:
   Description: >-
                   Use `sample` instead of `shuffle.first`,
                   `shuffle.last`, and `shuffle[Fixnum]`.
   Reference: 'https://github.com/JuanitoFatas/fast-ruby#arrayshufflefirst-vs-arraysample-code'
+  Enabled: false
+
+Style/OptionHash:
+  Enabled: false
+
+Style/OptionalBooleanParameter:
+  Enabled: false
+
+Lint/SymbolConversion:
+  Enabled: false
+
+Style/HashSyntax:
+  EnforcedStyle: no_mixed_keys
+
+Naming/VariableNumber:
   Enabled: false

data/.travis.yml

@@ -1,7 +1,7 @@
 sudo: false
 language: ruby
 rvm:
-  - 2.5.3
+  - 2.6.3
 before_install:
   - gem install bundler -v 2.2.15
   - openssl aes-256-cbc -K $encrypted_3c84dcdc6bbe_key -iv $encrypted_3c84dcdc6bbe_iv -in .env.enc -out .env -d

data/lib/tessitura_rest/crm/accounts.rb

@@ -1,8 +1,9 @@
 module Accounts
-
-  def get_all_accounts(constituent_id, inactive=false, include_affiliate=false, options={})
+  def get_all_accounts(constituent_id, inactive = false, include_affiliate = false, options = {})
     options.merge!(basic_auth: @auth, headers: @headers)
-    response = self.class.get(base_api_endpoint("CRM/Accounts?constituentId=#{constituent_id}&includeInactive=#{inactive}&includeAffiliates=#{include_affiliate}"), options)
+    response = self.class.get(
+      base_api_endpoint("CRM/Accounts?constituentId=#{constituent_id}&includeInactive=#{inactive}&includeAffiliates=#{include_affiliate}"), options
+    )
     JSON.parse(response.body)
   end
 end

data/lib/tessitura_rest/crm/actions.rb

@@ -1,36 +1,34 @@
 module Actions
-
-  def get_action(id, options={})
+  def get_action(id, options = {})
     options.merge!(basic_auth: @auth, headers: @headers)
     response = self.class.get(base_api_endpoint("CRM/Actions/#{id}"), options)
     JSON.parse(response.body)
   end
 
-  def get_actions(constituent_id, issue_id, options={})
+  def get_actions(constituent_id, issue_id, options = {})
     options.merge!(basic_auth: @auth, headers: @headers)
     response = self.class.get(base_api_endpoint("CRM/Actions?constituentId=#{constituent_id}&issueId=#{issue_id}"), options)
     JSON.parse(response.body)
   end
 
-  def create_action(action_type, constituent_id, issue_id, notes = '', options={})
+  def create_action(action_type, constituent_id, issue_id, notes = '', options = {})
     parameters =
-    {
-      'ActionType': {
-        'Id': action_type
-      },
-      'Issue': {
-        'Id': issue_id
-      },
-      'Constituent': {
-        'Id': constituent_id
-      },
-      'Notes': notes,
-      'Resolved': true,
-    }
+      {
+        'ActionType': {
+          'Id': action_type,
+        },
+        'Issue': {
+          'Id': issue_id,
+        },
+        'Constituent': {
+          'Id': constituent_id,
+        },
+        'Notes': notes,
+        'Resolved': true,
+      }
     options.merge!(basic_auth: @auth, headers: @headers)
     options.merge!(:body => parameters)
     response = self.class.post(base_api_endpoint('CRM/Actions'), options)
     JSON.parse(response.body)
   end
-
 end

data/lib/tessitura_rest/crm/addresses.rb

@@ -1,121 +1,120 @@
 module Addresses
-
-  def get_address(address_id, options={})
+  def get_address(address_id, options = {})
     options.merge!(basic_auth: @auth, headers: @headers)
     response = self.class.get(base_api_endpoint("CRM/Addresses/#{address_id}"), options)
     JSON.parse(response.body)
   end
 
-  def get_all_addresses(constituent_id, options={})
+  def get_all_addresses(constituent_id, options = {})
     options.merge!(basic_auth: @auth, headers: @headers)
     response = self.class.get(base_api_endpoint("CRM/Addresses?constituentId=#{constituent_id}"), options)
     JSON.parse(response.body)
   end
 
-  def create_address(constituent, address_type, city, postal_code, state, country, street1, street2, primary, options={})
+  def create_address(constituent, address_type, city, postal_code, state, country, street1, street2, primary, options = {})
     parameters =
-    {
-      'AddressType': {
-        'Id': address_type
-      },
-      'City': city,
-      'Constituent': {
-        'Id': constituent
-      },
-      'Inactive': false,
-      'Label': true,
-      'Months': 'YYYYYYYYYYYY',
-      'PostalCode': postal_code,
-      'PrimaryIndicator': primary,
-      'State': {
-        'Id': state,
+      {
+        'AddressType': {
+          'Id': address_type,
+        },
+        'City': city,
+        'Constituent': {
+          'Id': constituent,
+        },
+        'Inactive': false,
+        'Label': true,
+        'Months': 'YYYYYYYYYYYY',
+        'PostalCode': postal_code,
+        'PrimaryIndicator': primary,
+        'State': {
+          'Id': state,
+          'Country': {
+            'Id': country,
+          },
+        },
+        'Street1': street1,
+        'Street2': street2,
         'Country': {
-          'Id': country
-        }
-      },
-      'Street1': street1,
-      'Street2': street2,
-      'Country': {
-        'Id': country
+          'Id': country,
+        },
       }
-    }
     options.merge!(basic_auth: @auth, headers: @headers)
     options.merge!(:body => parameters)
-    response = self.class.post(base_api_endpoint("CRM/Addresses"), options)
+    response = self.class.post(base_api_endpoint('CRM/Addresses'), options)
     JSON.parse(response.body)
   end
 
-  def update_address(id, address_type, city, postal_code, state, country, street1, street2, primary, options={})
+  def update_address(id, address_type, city, postal_code, state, country, street1, street2, primary, options = {})
     current = get_address(id)
     parameters =
-    {
-      'Id': id,
-      'UpdatedDateTime': current['UpdatedDateTime'],
-      'AddressType': {
-        'Id': address_type
-      },
-      'City': city,
-      'Constituent': {
-        'Id': current['Constituent']['Id']
-      },
-      'Inactive': false,
-      'Label': true,
-      'Months': 'YYYYYYYYYYYY',
-      'PostalCode': postal_code,
-      'PrimaryIndicator': primary,
-      'State': {
-        'Id': state,
+      {
+        'Id': id,
+        'UpdatedDateTime': current['UpdatedDateTime'],
+        'AddressType': {
+          'Id': address_type,
+        },
+        'City': city,
+        'Constituent': {
+          'Id': current['Constituent']['Id'],
+        },
+        'Inactive': false,
+        'Label': true,
+        'Months': 'YYYYYYYYYYYY',
+        'PostalCode': postal_code,
+        'PrimaryIndicator': primary,
+        'State': {
+          'Id': state,
+          'Country': {
+            'Id': country,
+          },
+        },
+        'Street1': street1,
+        'Street2': street2,
         'Country': {
-          'Id': country
-        }
-      },
-      'Street1': street1,
-      'Street2': street2,
-      'Country': {
-        'Id': country
+          'Id': country,
+        },
       }
-    }
     options.merge!(basic_auth: @auth, headers: @headers)
     options.merge!(:body => parameters)
     response = self.class.put(base_api_endpoint("CRM/Addresses/#{id}"), options)
     JSON.parse(response.body)
   end
 
-  def deactivate_address(id, options={})
+  def deactivate_address(id, options = {})
     current = get_address(id)
     parameters =
-    {
-      'Id': id,
-      'UpdatedDateTime': current['UpdatedDateTime'],
-      'AddressType': {
-        'Id': current['AddressType']['Id']
-      },
-      'City': current['City'],
-      'Constituent': {
-        'Id': current['Constituent']['Id']
-      },
-      'Inactive': true,
-      'Label': true,
-      'Months': 'YYYYYYYYYYYY',
-      'PostalCode': current['PostalCode'],
-      'State': {
-        'Id': current['State']['Id'],
+      {
+        'Id': id,
+        'UpdatedDateTime': current['UpdatedDateTime'],
+        'AddressType': {
+          'Id': current['AddressType']['Id'],
+        },
+        'City': current['City'],
+        'Constituent': {
+          'Id': current['Constituent']['Id'],
+        },
+        'Inactive': true,
+        'Label': true,
+        'Months': 'YYYYYYYYYYYY',
+        'PostalCode': current['PostalCode'],
+        'State': {
+          'Id': current['State']['Id'],
+          'Country': {
+            'Id': current['Country']['Id'],
+          },
+        },
+        'Street1': current['Street1'],
+        'Street2': current['Street2'],
         'Country': {
-          'Id': current['Country']['Id']
-        }
-      },
-      'Street1': current['Street1'],
-      'Street2': current['Street2'],
-      'Country': {
-        'Id': current['Country']['Id']
+          'Id': current['Country']['Id'],
+        },
       }
-    }
     options.merge!(basic_auth: @auth, headers: @headers)
     options.merge!(:body => parameters)
     self.class.put(base_api_endpoint("CRM/Addresses/#{id}"), options)
   end
 
-  def delete_address(address_id, options={})
+  def delete_address(address_id, options = {})
     options.merge!(basic_auth: @auth, headers: @headers)
     self.class.delete(base_api_endpoint("CRM/Addresses/#{address_id}"), options)
   end

data/lib/tessitura_rest/crm/attributes.rb

@@ -1,16 +1,15 @@
 module Attributes
-
-  def add_attribute(constituent_id, attribute_id, value, options={})
+  def add_attribute(constituent_id, attribute_id, value, options = {})
     parameters =
       {
-          'Keyword' => {
-              'Id'=> attribute_id
-          },
-          'Constituent'=> {
-              'Id'=> constituent_id
-          },
-          'Value'=> value,
-          'UpdatedDateTime'=> DateTime.now.to_s
+        'Keyword' => {
+          'Id' => attribute_id,
+        },
+        'Constituent' => {
+          'Id' => constituent_id,
+        },
+        'Value' => value,
+        'UpdatedDateTime' => DateTime.now.to_s,
       }
     options.merge!(basic_auth: @auth, headers: @headers)
     options.merge!(:body => parameters)

data/lib/tessitura_rest/crm/constituencies.rb

@@ -1,20 +1,19 @@
 module Constituencies
-
-  def get_constituencies(id, options={})
+  def get_constituencies(id, options = {})
     options.merge!(basic_auth: @auth, headers: @headers)
     response = self.class.get(base_api_endpoint("CRM/Constituencies?constituentId=#{id}"), options)
     JSON.parse(response.body)
   end
 
-  def create_constituencies(constituency, id, options={})
+  def create_constituencies(constituency, id, options = {})
     parameters =
       {
-          'ConstituencyType': {
-              'Id': constituency
-          },
-          'Constituent': {
-              'Id': id
-          }
+        'ConstituencyType': {
+          'Id': constituency,
+        },
+        'Constituent': {
+          'Id': id,
+        },
       }
     options.merge!(basic_auth: @auth, headers: @headers)
     options.merge!(:body => parameters)

data/lib/tessitura_rest/crm/constituents.rb

@@ -1,59 +1,58 @@
 module Constituents
-
-  def get_constituent_snapshot(id, options={})
+  def get_constituent_snapshot(id, options = {})
     options.merge!(basic_auth: @auth, headers: @headers)
     response = self.class.get(base_api_endpoint("CRM/Constituents/#{id}/Snapshot"), options)
     JSON.parse(response.body)
   end
 
-  def create_constituent_by_snapshot(first_name, last_name, email, phone, constituent_type, source, street1, street2, city, state, zip, country, options={})
+  def create_constituent_by_snapshot(first_name, last_name, email, phone, constituent_type, source, street1, street2, city, state, zip, country, options = {})
     parameters =
       {
-        "ConstituentType": {
-          "Id": constituent_type,
-          "Inactive": false,
+        'ConstituentType': {
+          'Id': constituent_type,
+          'Inactive': false,
         },
-        "FirstName": first_name,
-        "LastName": last_name,
-        "ElectronicAddress": {
-          "Address": email,
-          "ElectronicAddressType": {
-            "Id": 1,
+        'FirstName': first_name,
+        'LastName': last_name,
+        'ElectronicAddress': {
+          'Address': email,
+          'ElectronicAddressType': {
+            'Id': 1,
           },
-          "AllowHtmlFormat": true,
-          "Inactive": false,
-          "AllowMarketing": false,
-          "Months": 'YYYYYYYYYYYY',
-          "PrimaryIndicator": true,
+          'AllowHtmlFormat': true,
+          'Inactive': false,
+          'AllowMarketing': false,
+          'Months': 'YYYYYYYYYYYY',
+          'PrimaryIndicator': true,
         },
-        "PrimaryPhoneNumbers":
+        'PrimaryPhoneNumbers':
           [
             {
-              "PhoneNumber": phone,
-              "PhoneType":
+              'PhoneNumber': phone,
+              'PhoneType':
                 {
-                  "Id": 1
+                  'Id': 1,
                 },
             },
           ],
-        "OriginalSource": {
-          "Id": source,
+        'OriginalSource': {
+          'Id': source,
         },
-        "Address": {
-          "AddressType": {
-            "Id": 3,
+        'Address': {
+          'AddressType': {
+            'Id': 3,
           },
-          "City": city,
-          "PostalCode": zip,
-          "State": {
-            "Id": state,
+          'City': city,
+          'PostalCode': zip,
+          'State': {
+            'Id': state,
           },
-          "Street1": street1,
-          "Street2": street2,
-          "Country": {
-            "Id": country,
+          'Street1': street1,
+          'Street2': street2,
+          'Country': {
+            'Id': country,
           },
-        }
+        },
       }
     parameters.delete(:PrimaryPhoneNumbers) unless phone.present?
     options.merge!(basic_auth: @auth, headers: @headers)
@@ -61,34 +60,34 @@ module Constituents
     self.class.post(base_api_endpoint('CRM/Constituents/Snapshot'), options)
   end
 
-  def update_constituent(constituent_id, params, options={})
+  def update_constituent(constituent_id, params, options = {})
     current = get_constituent_snapshot(constituent_id)
     if is_household?(current)
       update_household(params, current)
     else
       parameters =
         {
-          "ConstituentType": {
-            "Id": current['ConstituentType']['Id'],
-            "Inactive": false,
+          'ConstituentType': {
+            'Id': current['ConstituentType']['Id'],
+            'Inactive': false,
           },
-          "Id": constituent_id,
-          "UpdatedDateTime": current['UpdatedDateTime'],
-          "FirstName": params[:first_name],
-          "LastName": params[:last_name],
-          "MiddleName": params[:middle_name],
-          "SortName": current['SortName'],
-          "Prefix": {
-            "Id": params[:prefix],
-            "Inactive": false
+          'Id': constituent_id,
+          'UpdatedDateTime': current['UpdatedDateTime'],
+          'FirstName': params[:first_name],
+          'LastName': params[:last_name],
+          'MiddleName': params[:middle_name],
+          'SortName': current['SortName'],
+          'Prefix': {
+            'Id': params[:prefix],
+            'Inactive': false,
           },
-          "Suffix": {
-            "Id": params[:suffix],
-            "Inactive": false
+          'Suffix': {
+            'Id': params[:suffix],
+            'Inactive': false,
           },
-          "OriginalSource": {
-            "Id": current['OriginalSource']['Id'],
-            "Inactive": false
+          'OriginalSource': {
+            'Id': current['OriginalSource']['Id'],
+            'Inactive': false,
           },
         }
       parameters.delete(:Prefix) unless params[:prefix].present?
@@ -103,33 +102,33 @@ module Constituents
     snapshot['ConstituentType']['Id'] == 9
   end
 
-  def update_household(params, current, options={})
-    constituents = current['Affiliates'].map{|c| c['RelatedConstituentId']}
+  def update_household(params, current, options = {})
+    constituents = current['Affiliates'].map { |c| c['RelatedConstituentId'] }
     update = []
     constituents.each_with_index do |constituent, i|
       snapshot = get_constituent_snapshot(constituent)
       parameters =
         {
-          "ConstituentType": {
-            "Id": snapshot['ConstituentType']['Id'],
-            "Inactive": false,
+          'ConstituentType': {
+            'Id': snapshot['ConstituentType']['Id'],
+            'Inactive': false,
           },
-          "Id": constituent,
-          "FirstName": i == 0 ? params[:first_name] : params[:first_name_2],
-          "UpdatedDateTime": snapshot['UpdatedDateTime'],
-          "LastName": i == 0 ? params[:last_name] : params[:last_name_2],
-          "MiddleName": i == 0 ? params[:middle_name] : params[:middle_name_2],
-          "Prefix": {
-            "Id": i == 0 ? params[:prefix] : params[:prefix_2],
-            "Inactive": false
+          'Id': constituent,
+          'FirstName': i == 0 ? params[:first_name] : params[:first_name_2],
+          'UpdatedDateTime': snapshot['UpdatedDateTime'],
+          'LastName': i == 0 ? params[:last_name] : params[:last_name_2],
+          'MiddleName': i == 0 ? params[:middle_name] : params[:middle_name_2],
+          'Prefix': {
+            'Id': i == 0 ? params[:prefix] : params[:prefix_2],
+            'Inactive': false,
           },
-          "Suffix": {
-            "Id": i == 0 ? params[:suffix] : params[:suffix_2],
-            "Inactive": false
+          'Suffix': {
+            'Id': i == 0 ? params[:suffix] : params[:suffix_2],
+            'Inactive': false,
           },
-          "OriginalSource": {
-            "Id": snapshot['OriginalSource']['Id'],
-            "Inactive": false
+          'OriginalSource': {
+            'Id': snapshot['OriginalSource']['Id'],
+            'Inactive': false,
           },
         }
       options.merge!(basic_auth: @auth, headers: @headers)
@@ -138,6 +137,4 @@ module Constituents
     end
     update.last if update.present?
   end
-
-
 end