bandwidth-sdk 7.1.0 → 9.1.1

data/lib/bandwidth/messaging_lib/messaging/models/bandwidth_message.rb

@@ -6,6 +6,9 @@
 module Bandwidth
   # BandwidthMessage Model.
   class BandwidthMessage < BaseModel
+    SKIP = Object.new
+    private_constant :SKIP
+
     # The id of the message
     # @return [String]
     attr_accessor :id
@@ -39,7 +42,10 @@ module Bandwidth
     # @return [String]
     attr_accessor :from
 
-    # The list of media URLs sent in the message
+    # The list of media URLs sent in the message. Including a `filename` field
+    # in the `Content-Disposition` header of the media linked with a URL will
+    # set the displayed file name. This is a best practice to ensure that your
+    # media has a readable file name.
     # @return [List of String]
     attr_accessor :media
 
@@ -73,6 +79,29 @@ module Bandwidth
       @_hash
     end
 
+    # An array for optional fields
+    def optionals
+      %w[
+        id
+        owner
+        application_id
+        time
+        segment_count
+        direction
+        to
+        from
+        media
+        text
+        tag
+        priority
+      ]
+    end
+
+    # An array for nullable fields
+    def nullables
+      []
+    end
+
     def initialize(id = nil,
                    owner = nil,
                    application_id = nil,
@@ -85,18 +114,18 @@ module Bandwidth
                    text = nil,
                    tag = nil,
                    priority = nil)
-      @id = id
-      @owner = owner
-      @application_id = application_id
-      @time = time
-      @segment_count = segment_count
-      @direction = direction
-      @to = to
-      @from = from
-      @media = media
-      @text = text
-      @tag = tag
-      @priority = priority
+      @id = id unless id == SKIP
+      @owner = owner unless owner == SKIP
+      @application_id = application_id unless application_id == SKIP
+      @time = time unless time == SKIP
+      @segment_count = segment_count unless segment_count == SKIP
+      @direction = direction unless direction == SKIP
+      @to = to unless to == SKIP
+      @from = from unless from == SKIP
+      @media = media unless media == SKIP
+      @text = text unless text == SKIP
+      @tag = tag unless tag == SKIP
+      @priority = priority unless priority == SKIP
     end
 
     # Creates an instance of the object from a hash.
@@ -104,18 +133,18 @@ module Bandwidth
       return nil unless hash
 
       # Extract variables from the hash.
-      id = hash['id']
-      owner = hash['owner']
-      application_id = hash['applicationId']
-      time = hash['time']
-      segment_count = hash['segmentCount']
-      direction = hash['direction']
-      to = hash['to']
-      from = hash['from']
-      media = hash['media']
-      text = hash['text']
-      tag = hash['tag']
-      priority = hash['priority']
+      id = hash.key?('id') ? hash['id'] : SKIP
+      owner = hash.key?('owner') ? hash['owner'] : SKIP
+      application_id = hash.key?('applicationId') ? hash['applicationId'] : SKIP
+      time = hash.key?('time') ? hash['time'] : SKIP
+      segment_count = hash.key?('segmentCount') ? hash['segmentCount'] : SKIP
+      direction = hash.key?('direction') ? hash['direction'] : SKIP
+      to = hash.key?('to') ? hash['to'] : SKIP
+      from = hash.key?('from') ? hash['from'] : SKIP
+      media = hash.key?('media') ? hash['media'] : SKIP
+      text = hash.key?('text') ? hash['text'] : SKIP
+      tag = hash.key?('tag') ? hash['tag'] : SKIP
+      priority = hash.key?('priority') ? hash['priority'] : SKIP
 
       # Create object from extracted values.
       BandwidthMessage.new(id,

data/lib/bandwidth/messaging_lib/messaging/models/bandwidth_message_item.rb

@@ -6,6 +6,9 @@
 module Bandwidth
   # BandwidthMessageItem Model.
   class BandwidthMessageItem < BaseModel
+    SKIP = Object.new
+    private_constant :SKIP
+
     # The message id
     # @return [String]
     attr_accessor :message_id
@@ -92,6 +95,33 @@ module Bandwidth
       @_hash
     end
 
+    # An array for optional fields
+    def optionals
+      %w[
+        message_id
+        account_id
+        source_tn
+        destination_tn
+        message_status
+        message_direction
+        message_type
+        segment_count
+        error_code
+        receive_time
+        carrier_name
+        message_size
+        message_length
+        attachment_count
+        recipient_count
+        campaign_class
+      ]
+    end
+
+    # An array for nullable fields
+    def nullables
+      []
+    end
+
     def initialize(message_id = nil,
                    account_id = nil,
                    source_tn = nil,
@@ -108,22 +138,22 @@ module Bandwidth
                    attachment_count = nil,
                    recipient_count = nil,
                    campaign_class = nil)
-      @message_id = message_id
-      @account_id = account_id
-      @source_tn = source_tn
-      @destination_tn = destination_tn
-      @message_status = message_status
-      @message_direction = message_direction
-      @message_type = message_type
-      @segment_count = segment_count
-      @error_code = error_code
-      @receive_time = receive_time
-      @carrier_name = carrier_name
-      @message_size = message_size
-      @message_length = message_length
-      @attachment_count = attachment_count
-      @recipient_count = recipient_count
-      @campaign_class = campaign_class
+      @message_id = message_id unless message_id == SKIP
+      @account_id = account_id unless account_id == SKIP
+      @source_tn = source_tn unless source_tn == SKIP
+      @destination_tn = destination_tn unless destination_tn == SKIP
+      @message_status = message_status unless message_status == SKIP
+      @message_direction = message_direction unless message_direction == SKIP
+      @message_type = message_type unless message_type == SKIP
+      @segment_count = segment_count unless segment_count == SKIP
+      @error_code = error_code unless error_code == SKIP
+      @receive_time = receive_time unless receive_time == SKIP
+      @carrier_name = carrier_name unless carrier_name == SKIP
+      @message_size = message_size unless message_size == SKIP
+      @message_length = message_length unless message_length == SKIP
+      @attachment_count = attachment_count unless attachment_count == SKIP
+      @recipient_count = recipient_count unless recipient_count == SKIP
+      @campaign_class = campaign_class unless campaign_class == SKIP
     end
 
     # Creates an instance of the object from a hash.
@@ -131,22 +161,25 @@ module Bandwidth
       return nil unless hash
 
       # Extract variables from the hash.
-      message_id = hash['messageId']
-      account_id = hash['accountId']
-      source_tn = hash['sourceTn']
-      destination_tn = hash['destinationTn']
-      message_status = hash['messageStatus']
-      message_direction = hash['messageDirection']
-      message_type = hash['messageType']
-      segment_count = hash['segmentCount']
-      error_code = hash['errorCode']
-      receive_time = hash['receiveTime']
-      carrier_name = hash['carrierName']
-      message_size = hash['messageSize']
-      message_length = hash['messageLength']
-      attachment_count = hash['attachmentCount']
-      recipient_count = hash['recipientCount']
-      campaign_class = hash['campaignClass']
+      message_id = hash.key?('messageId') ? hash['messageId'] : SKIP
+      account_id = hash.key?('accountId') ? hash['accountId'] : SKIP
+      source_tn = hash.key?('sourceTn') ? hash['sourceTn'] : SKIP
+      destination_tn = hash.key?('destinationTn') ? hash['destinationTn'] : SKIP
+      message_status = hash.key?('messageStatus') ? hash['messageStatus'] : SKIP
+      message_direction =
+        hash.key?('messageDirection') ? hash['messageDirection'] : SKIP
+      message_type = hash.key?('messageType') ? hash['messageType'] : SKIP
+      segment_count = hash.key?('segmentCount') ? hash['segmentCount'] : SKIP
+      error_code = hash.key?('errorCode') ? hash['errorCode'] : SKIP
+      receive_time = hash.key?('receiveTime') ? hash['receiveTime'] : SKIP
+      carrier_name = hash.key?('carrierName') ? hash['carrierName'] : SKIP
+      message_size = hash.key?('messageSize') ? hash['messageSize'] : SKIP
+      message_length = hash.key?('messageLength') ? hash['messageLength'] : SKIP
+      attachment_count =
+        hash.key?('attachmentCount') ? hash['attachmentCount'] : SKIP
+      recipient_count =
+        hash.key?('recipientCount') ? hash['recipientCount'] : SKIP
+      campaign_class = hash.key?('campaignClass') ? hash['campaignClass'] : SKIP
 
       # Create object from extracted values.
       BandwidthMessageItem.new(message_id,

data/lib/bandwidth/messaging_lib/messaging/models/bandwidth_messages_list.rb

@@ -6,6 +6,9 @@
 module Bandwidth
   # BandwidthMessagesList Model.
   class BandwidthMessagesList < BaseModel
+    SKIP = Object.new
+    private_constant :SKIP
+
     # Total number of messages matched by the search
     # @return [Integer]
     attr_accessor :total_count
@@ -27,12 +30,26 @@ module Bandwidth
       @_hash
     end
 
+    # An array for optional fields
+    def optionals
+      %w[
+        total_count
+        page_info
+        messages
+      ]
+    end
+
+    # An array for nullable fields
+    def nullables
+      []
+    end
+
     def initialize(total_count = nil,
                    page_info = nil,
                    messages = nil)
-      @total_count = total_count
-      @page_info = page_info
-      @messages = messages
+      @total_count = total_count unless total_count == SKIP
+      @page_info = page_info unless page_info == SKIP
+      @messages = messages unless messages == SKIP
     end
 
     # Creates an instance of the object from a hash.
@@ -40,7 +57,7 @@ module Bandwidth
       return nil unless hash
 
       # Extract variables from the hash.
-      total_count = hash['totalCount']
+      total_count = hash.key?('totalCount') ? hash['totalCount'] : SKIP
       page_info = PageInfo.from_hash(hash['pageInfo']) if hash['pageInfo']
       # Parameter is an array, so we need to iterate through it
       messages = nil
@@ -51,6 +68,8 @@ module Bandwidth
         end
       end
 
+      messages = SKIP unless hash.key?('messages')
+
       # Create object from extracted values.
       BandwidthMessagesList.new(total_count,
                                 page_info,

data/lib/bandwidth/messaging_lib/messaging/models/deferred_result.rb

@@ -6,6 +6,9 @@
 module Bandwidth
   # DeferredResult Model.
   class DeferredResult < BaseModel
+    SKIP = Object.new
+    private_constant :SKIP
+
     # TODO: Write general description for this method
     # @return [Object]
     attr_accessor :result
@@ -22,10 +25,23 @@ module Bandwidth
       @_hash
     end
 
+    # An array for optional fields
+    def optionals
+      %w[
+        result
+        set_or_expired
+      ]
+    end
+
+    # An array for nullable fields
+    def nullables
+      []
+    end
+
     def initialize(result = nil,
                    set_or_expired = nil)
-      @result = result
-      @set_or_expired = set_or_expired
+      @result = result unless result == SKIP
+      @set_or_expired = set_or_expired unless set_or_expired == SKIP
     end
 
     # Creates an instance of the object from a hash.
@@ -33,8 +49,8 @@ module Bandwidth
       return nil unless hash
 
       # Extract variables from the hash.
-      result = hash['result']
-      set_or_expired = hash['setOrExpired']
+      result = hash.key?('result') ? hash['result'] : SKIP
+      set_or_expired = hash.key?('setOrExpired') ? hash['setOrExpired'] : SKIP
 
       # Create object from extracted values.
       DeferredResult.new(result,

data/lib/bandwidth/messaging_lib/messaging/models/media.rb

@@ -6,6 +6,9 @@
 module Bandwidth
   # Media Model.
   class Media < BaseModel
+    SKIP = Object.new
+    private_constant :SKIP
+
     # TODO: Write general description for this method
     # @return [String]
     attr_accessor :content
@@ -27,12 +30,26 @@ module Bandwidth
       @_hash
     end
 
+    # An array for optional fields
+    def optionals
+      %w[
+        content
+        content_length
+        media_name
+      ]
+    end
+
+    # An array for nullable fields
+    def nullables
+      []
+    end
+
     def initialize(content = nil,
                    content_length = nil,
                    media_name = nil)
-      @content = content
-      @content_length = content_length
-      @media_name = media_name
+      @content = content unless content == SKIP
+      @content_length = content_length unless content_length == SKIP
+      @media_name = media_name unless media_name == SKIP
     end
 
     # Creates an instance of the object from a hash.
@@ -40,9 +57,9 @@ module Bandwidth
       return nil unless hash
 
       # Extract variables from the hash.
-      content = hash['content']
-      content_length = hash['contentLength']
-      media_name = hash['mediaName']
+      content = hash.key?('content') ? hash['content'] : SKIP
+      content_length = hash.key?('contentLength') ? hash['contentLength'] : SKIP
+      media_name = hash.key?('mediaName') ? hash['mediaName'] : SKIP
 
       # Create object from extracted values.
       Media.new(content,

data/lib/bandwidth/messaging_lib/messaging/models/message_request.rb

@@ -6,6 +6,9 @@
 module Bandwidth
   # MessageRequest Model.
   class MessageRequest < BaseModel
+    SKIP = Object.new
+    private_constant :SKIP
+
     # The ID of the Application your from number is associated with in the
     # Bandwidth Phone Number Dashboard.
     # @return [String]
@@ -51,6 +54,21 @@ module Bandwidth
       @_hash
     end
 
+    # An array for optional fields
+    def optionals
+      %w[
+        text
+        media
+        tag
+        priority
+      ]
+    end
+
+    # An array for nullable fields
+    def nullables
+      []
+    end
+
     def initialize(application_id = nil,
                    to = nil,
                    from = nil,
@@ -58,13 +76,13 @@ module Bandwidth
                    media = nil,
                    tag = nil,
                    priority = nil)
-      @application_id = application_id
-      @to = to
-      @from = from
-      @text = text
-      @media = media
-      @tag = tag
-      @priority = priority
+      @application_id = application_id unless application_id == SKIP
+      @to = to unless to == SKIP
+      @from = from unless from == SKIP
+      @text = text unless text == SKIP
+      @media = media unless media == SKIP
+      @tag = tag unless tag == SKIP
+      @priority = priority unless priority == SKIP
     end
 
     # Creates an instance of the object from a hash.
@@ -72,13 +90,13 @@ module Bandwidth
       return nil unless hash
 
       # Extract variables from the hash.
-      application_id = hash['applicationId']
-      to = hash['to']
-      from = hash['from']
-      text = hash['text']
-      media = hash['media']
-      tag = hash['tag']
-      priority = hash['priority']
+      application_id = hash.key?('applicationId') ? hash['applicationId'] : SKIP
+      to = hash.key?('to') ? hash['to'] : SKIP
+      from = hash.key?('from') ? hash['from'] : SKIP
+      text = hash.key?('text') ? hash['text'] : SKIP
+      media = hash.key?('media') ? hash['media'] : SKIP
+      tag = hash.key?('tag') ? hash['tag'] : SKIP
+      priority = hash.key?('priority') ? hash['priority'] : SKIP
 
       # Create object from extracted values.
       MessageRequest.new(application_id,

data/lib/bandwidth/messaging_lib/messaging/models/page_info.rb

@@ -6,6 +6,9 @@
 module Bandwidth
   # PageInfo Model.
   class PageInfo < BaseModel
+    SKIP = Object.new
+    private_constant :SKIP
+
     # The link to the previous page for pagination
     # @return [String]
     attr_accessor :prev_page
@@ -32,14 +35,29 @@ module Bandwidth
       @_hash
     end
 
+    # An array for optional fields
+    def optionals
+      %w[
+        prev_page
+        next_page
+        prev_page_token
+        next_page_token
+      ]
+    end
+
+    # An array for nullable fields
+    def nullables
+      []
+    end
+
     def initialize(prev_page = nil,
                    next_page = nil,
                    prev_page_token = nil,
                    next_page_token = nil)
-      @prev_page = prev_page
-      @next_page = next_page
-      @prev_page_token = prev_page_token
-      @next_page_token = next_page_token
+      @prev_page = prev_page unless prev_page == SKIP
+      @next_page = next_page unless next_page == SKIP
+      @prev_page_token = prev_page_token unless prev_page_token == SKIP
+      @next_page_token = next_page_token unless next_page_token == SKIP
     end
 
     # Creates an instance of the object from a hash.
@@ -47,10 +65,12 @@ module Bandwidth
       return nil unless hash
 
       # Extract variables from the hash.
-      prev_page = hash['prevPage']
-      next_page = hash['nextPage']
-      prev_page_token = hash['prevPageToken']
-      next_page_token = hash['nextPageToken']
+      prev_page = hash.key?('prevPage') ? hash['prevPage'] : SKIP
+      next_page = hash.key?('nextPage') ? hash['nextPage'] : SKIP
+      prev_page_token =
+        hash.key?('prevPageToken') ? hash['prevPageToken'] : SKIP
+      next_page_token =
+        hash.key?('nextPageToken') ? hash['nextPageToken'] : SKIP
 
       # Create object from extracted values.
       PageInfo.new(prev_page,

data/lib/bandwidth/messaging_lib/messaging/models/tag.rb

@@ -6,6 +6,9 @@
 module Bandwidth
   # Tag Model.
   class Tag < BaseModel
+    SKIP = Object.new
+    private_constant :SKIP
+
     # TODO: Write general description for this method
     # @return [String]
     attr_accessor :key
@@ -22,10 +25,23 @@ module Bandwidth
       @_hash
     end
 
+    # An array for optional fields
+    def optionals
+      %w[
+        key
+        value
+      ]
+    end
+
+    # An array for nullable fields
+    def nullables
+      []
+    end
+
     def initialize(key = nil,
                    value = nil)
-      @key = key
-      @value = value
+      @key = key unless key == SKIP
+      @value = value unless value == SKIP
     end
 
     # Creates an instance of the object from a hash.
@@ -33,8 +49,8 @@ module Bandwidth
       return nil unless hash
 
       # Extract variables from the hash.
-      key = hash['key']
-      value = hash['value']
+      key = hash.key?('key') ? hash['key'] : SKIP
+      value = hash.key?('value') ? hash['value'] : SKIP
 
       # Create object from extracted values.
       Tag.new(key,

data/lib/bandwidth/models/base_model.rb

@@ -14,6 +14,17 @@ module Bandwidth
         name = name[1..-1]
         key = self.class.names.key?(name) ? self.class.names[name] : name
 
+        optional_fields = optionals if respond_to? 'optionals'
+        nullable_fields = nullables if respond_to? 'nullables'
+        if value.nil?
+          next unless nullable_fields.include?(name)
+
+          if !optional_fields.include?(name) && !nullable_fields.include?(name)
+            raise ArgumentError,
+                  "`#{name}` cannot be nil in `#{self.class}`. Please specify a valid value."
+          end
+        end
+
         hash[key] = nil
         unless value.nil?
           if respond_to?("to_#{name}")

data/lib/bandwidth/multi_factor_auth_lib/multi_factor_auth/controllers/base_controller.rb

@@ -19,14 +19,12 @@ module Bandwidth
 
     def validate_parameters(args)
       args.each do |_name, value|
-        if value.nil?
-          raise ArgumentError, "Required parameter #{_name} cannot be nil."
-        end
+        raise ArgumentError, "Required parameter #{_name} cannot be nil." if value.nil?
       end
     end
 
     def execute_request(request, binary: false)
-      @http_call_back.on_before_request(request) if @http_call_back
+      @http_call_back&.on_before_request(request)
 
       APIHelper.clean_hash(request.headers)
       request.headers.merge!(@global_headers)
@@ -36,7 +34,7 @@ module Bandwidth
                  else
                    config.http_client.execute_as_string(request)
                  end
-      @http_call_back.on_after_response(response) if @http_call_back
+      @http_call_back&.on_after_response(response)
 
       response
     end