aboutyou-sdk 0.0.24 → 0.0.25

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    NzgzNjUwZmE3ZTRhOTE4ZjdmMzdlYzQyZDhiN2JjMGEwODlkZjI5Mg==
+    ZWRiNTIyMzA5N2MxYjdhM2YyZThlMThmMWYxYTY3ODZlMDQ5MDI5Mw==
   data.tar.gz: !binary |-
-    YTljNGU5NWZhMWY1OTc3NTc0OGVmZWI2ZjEzODJjMWMzYTE4MzM4ZA==
+    ZTM5OGU3Yjk0NzU4NjFiNGQxZmYyZjdiMjE0MThmZjlkNzM0ZjUxZg==
 SHA512:
   metadata.gz: !binary |-
-    MWYzMWExMTNlOGEzOGMyYWUyMTY0NmIxOWFkNmI1ZDNkOTViZjBhZDgxMGI4
-    MDIyNGY0MDI3ZmI0NzMwODI2ZjY2ODExYTdhZWZmYTZiMzkyZjBlN2ZhZWUy
-    MjNiNDczZWU4NjZmNmNmN2RhYzU3YmRmMjcyZjc3ZmE4NGI5OWM=
+    MWNhNTBkYWRhNjNjZWM1NjdjZWE4ZWMzNTg4MDc2YjM4M2IxYTI3MzhmNjli
+    YzA3N2Q4ZDkwMmEzZTcwM2EwNmI0YmUwOGFkZjVlYjc4MDlmMzE4ZjQ2ZmVh
+    N2U2N2IyNDM5ZWFiYWZiNTJkOTdjMjIwODI5MzI5NzMxMzRkZTQ=
   data.tar.gz: !binary |-
-    YjBlMzFiNWZhNTJjZGYwY2YyYmI5NzhkNDUzNWEyM2Q4YjE5YmIyNjVlOTQ1
-    MTEzM2IyMGQ5ODViNWI4Y2NlY2NhYWM4MzM3MDdiYTYyYzdhOGZhOGI3NzI1
-    YWM1Y2VlNTI0YmI0ZDk1NmVhMzYxYTlmMDU5MWQzYjdmOTlkNjk=
+    MTIyNDgzNjkxYmU4NzlmZmU5ZGRkN2U3OWRlMTdlYTVlOGQ2ODgzMTZkYzMx
+    NmNhYmZjMGU2NjM2ZDg5NDA5YjZhZjRmZjc4NjIyY2E2ZDlmNDQyOTMwZDYw
+    ZGE0OTQwMjg5ZmEwMGU2N2VkODRkNWEwMTczNTJkYjk0MDk5YTI=

data/lib/{aboutyou-sdk/CacheProvider/Dalli.rb → AboutYou/CacheProvider/dalli.rb}

@@ -1,7 +1,6 @@
 module AboutYou
   module SDK
     module CacheProvider
-      
       ###
       # This class is used as an interface for cache operations.
       # It is used when caching with Dalli.
@@ -10,23 +9,23 @@ module AboutYou
       ###
       class Dalli
         require 'dalli'
-        
+
         # an instance of the cache client.
         attr_accessor :client
-        
+
         ###
         # the Constructor for the Dalli class
         #
         # * *Args*    :
         #   - +client+ -> an instance of Dalli::Client
-        #   
+        #
         # * *Returns* :
         #   - Instance of AboutYou::SDK::CacheProvider::Dalli
         ###
         def initialize(client)
           self.client = client
         end
-        
+
         ###
         # This method is used for setting new cache entries with Dalli.
         #
@@ -34,53 +33,53 @@ module AboutYou
         #   - +key+ -> The key of the cache entry
         #   - +value+ -> The value of the cache entry
         #   - +duration+ -> the duration of the cache entry
-        #   
+        #
         # * *Returns* :
-        #   - True/False determining whether the setting was successfull of not
+        #   - True/False determining whether the setting was successful of not
         ###
         def set(key, value, duration)
-          self.client.set(key, value, duration)
+          client.set(key, value, duration)
         end
-        
+
         ###
         # This method is used for getting cache entries with Dalli.
         #
         # * *Args*    :
         #   - +key+ -> The key of the cache entry
-        #   
+        #
         # * *Returns* :
         #   - Either the value for the given key or nil if the key was not found
         ###
         def get(key)
-          self.client.get(key)
+          client.get(key)
         end
-        
+
         ###
         # This method is used for deleting cache entries with Dalli.
         #
         # * *Args*    :
         #   - +key+ -> The key of the cache entry
-        #   
+        #
         # * *Returns* :
-        #   - True/False determining whether the deletion was successfull or not
+        #   - True/False determining whether the deletion was successful or not
         ###
         def delete(key)
-          self.client.delete(key)
+          client.delete(key)
         end
-        
+
         ###
         # This method is used for checking whether a cache entry exists or not with Dalli.
         #
         # * *Args*    :
         #   - +key+ -> The key of the cache entry
-        #   
+        #
         # * *Returns* :
         #   - True/False determining whether the key exists in the cache or not
         ###
         def exists(key)
-          self.client.get(key) != nil
+          !client.get(key).nil?
         end
       end
     end
   end
-end
+end

data/lib/{aboutyou-sdk/CacheProvider/Redis.rb → AboutYou/CacheProvider/redis.rb}

@@ -1,7 +1,6 @@
 module AboutYou
   module SDK
     module CacheProvider
-      
       ###
       # This class is used as an interface for cache operations.
       # It is used when caching with Redis.
@@ -10,23 +9,23 @@ module AboutYou
       ###
       class Redis
         require 'redis'
-        
+
         # an instance of the cache client.
         attr_accessor :client
-        
+
         ###
         # the Constructor for the Redis class
         #
         # * *Args*    :
         #   - +client+ -> an instance of Redis
-        #   
+        #
         # * *Returns* :
         #   - Instance of AboutYou::SDK::CacheProvider::Redis
         ###
         def initialize(client)
           self.client = client
         end
-        
+
         ###
         # This method is used for setting new cache entries with Redis.
         #
@@ -34,55 +33,55 @@ module AboutYou
         #   - +key+ -> The key of the cache entry
         #   - +value+ -> The value of the cache entry
         #   - +duration+ -> the duration of the cache entry
-        #   
+        #
         # * *Returns* :
-        #   - True/False determining whether the setting was successfull of not
+        #   - True/False determining whether the setting was successful of not
         ###
         def set(key, value, duration)
           value = value.to_json unless value.is_a?(String)
-          self.client.set(key, value)
-          self.client.expire(key, duration)
+          client.set(key, value)
+          client.expire(key, duration)
         end
-        
+
         ###
         # This method is used for getting cache entries with Redis.
         #
         # * *Args*    :
         #   - +key+ -> The key of the cache entry
-        #   
+        #
         # * *Returns* :
         #   - Either the value for the given key or nil if the key was not found
         ###
         def get(key)
-          JSON.parse(self.client.get(key))
+          JSON.parse(client.get(key))
         end
-        
+
         ###
         # This method is used for deleting cache entries with Redis.
         #
         # * *Args*    :
         #   - +key+ -> The key of the cache entry
-        #   
+        #
         # * *Returns* :
-        #   - True/False determining whether the deletion was successfull or not
+        #   - True/False determining whether the deletion was successful or not
         ###
         def delete(key)
-          self.client.del(key)
+          client.del(key)
         end
-        
+
         ###
         # This method is used for checking whether a cache entry exists or not with Redis.
         #
         # * *Args*    :
         #   - +key+ -> The key of the cache entry
-        #   
+        #
         # * *Returns* :
         #   - True/False determining whether the key exists in the cache or not
         ###
         def exists(key)
-          self.client.exists(key)
+          client.exists(key)
         end
       end
     end
   end
-end
+end

data/lib/{aboutyou-sdk/Criteria/ProductFields.rb → AboutYou/Criteria/product_fields.rb}

@@ -9,38 +9,37 @@ module AboutYou
       ###
       class ProductFields
         # id and name is set per default
-        
-        #product field "is_active"
+        # product field "is_active"
         IS_ACTIVE              = 'active'
-        #product field "brand_id"
+        # product field "brand_id"
         BRAND                  = 'brand_id'
-        #product field "description_long"
+        # product field "description_long"
         DESCRIPTION_LONG       = 'description_long'
-        #product field "description_short"
+        # product field "description_short"
         DESCRIPTION_SHORT      = 'description_short'
-        #product field "default_variant"
+        # product field "default_variant"
         DEFAULT_VARIANT        = 'default_variant'
-        #product field "variants"
+        # product field "variants"
         VARIANTS               = 'variants'
-        #product field "min_price"
+        # product field "min_price"
         MIN_PRICE              = 'min_price'
-        #product field "max_price"
+        # product field "max_price"
         MAX_PRICE              = 'max_price'
-        #product field "sale"
+        # product field "sale"
         IS_SALE                = 'sale'
-        #product field "default_image"
+        # product field "default_image"
         DEFAULT_IMAGE          = 'default_image'
-        #product field "attributes_merged"
+        # product field "attributes_merged"
         ATTRIBUTES_MERGED      = 'attributes_merged'
-        #product field "categories"
+        # product field "categories"
         CATEGORIES             = 'categories'
-        #product field "inactive_variants"
+        # product field "inactive_variants"
         INACTIVE_VARIANTS      = 'inactive_variants'
-        #product field "max_savings"
+        # product field "max_savings"
         MAX_SAVINGS            = 'max_savings'
-        #product field "max_savings_percentage"
+        # product field "max_savings_percentage"
         MAX_SAVINGS_PERCENTAGE = 'max_savings_percentage'
-        #product field "tags"
+        # product field "tags"
         TAGS                   = 'tags'
 
         ###
@@ -52,17 +51,16 @@ module AboutYou
         # * *Returns* :
         #   - a filtered Array of product fields
         ###
-        def self.filterFields(fields)
+        def self.filter_fields(fields)
           fields = fields.uniq
 
           # this simplifies parsing on (pre)fetching facets
-          if (fields.include? ATTRIBUTES_MERGED) && (
-              fields.include?(BRAND)  ||
-              fields.include?(VARIANTS) ||
-              fields.include?(DEFAULT_VARIANT)
-            )
-            fields.push(ATTRIBUTES_MERGED)
-          end
+          fields.push(ATTRIBUTES_MERGED) if
+          fields.include?(ATTRIBUTES_MERGED) &&
+          (fields.include?(BRAND)  ||
+          fields.include?(VARIANTS) ||
+          fields.include?(DEFAULT_VARIANT))
+
           fields
         end
 
@@ -75,7 +73,7 @@ module AboutYou
         # * *Returns* :
         #   - a boolean which is true when ap call has to require facets
         ###
-        def self.requiresFacets(fields)
+        def self.requires_facets(fields)
           ([BRAND,
             VARIANTS,
             DEFAULT_VARIANT,
@@ -92,7 +90,7 @@ module AboutYou
         # * *Returns* :
         #   - a boolean which is true when ap call has to require categories
         ###
-        def self.requiresCategories(fields)
+        def self.requires_categories(fields)
           fields.include?(CATEGORIES)
         end
       end

data/lib/{aboutyou-sdk/Criteria/ProductSearchCriteria.rb → AboutYou/Criteria/product_search_criteria.rb}

@@ -47,20 +47,20 @@ module AboutYou
         # a specification of the result which should be fetched from api
         attr_accessor :result
         # the session id of a user
-        attr_accessor :sessionId
+        attr_accessor :session_id
 
         ###
         # Constructor for AboutYou::SDK::Criteria::ProductSearchCriteria
         #
         # * *Args*    :
-        #   - +sessionId+ -> the session is of an user
+        #   - +session_id+ -> the session is of an user
         #
         # * *Returns* :
         #   - an instance of AboutYou::SDK::Criteria::ProductSearchCriteria
         ###
-        def initialize(sessionId)
+        def initialize(session_id)
           self.filter    = {}
-          self.sessionId = sessionId
+          self.session_id = session_id
           self.result    = {}
         end
 
@@ -74,9 +74,10 @@ module AboutYou
         # * *Returns* :
         #   - an instance of AboutYou::SDK::Criteria::ProductSearchCriteria
         ###
-        def filterBy(key, value)
+        def filter_by(key, value)
           @filter[key] = value
-          return self
+
+          self
         end
 
         ###
@@ -101,12 +102,10 @@ module AboutYou
         # * *Returns* :
         #   - an instance of AboutYou::SDK::Criteria::ProductSearchCriteria
         ###
-        def filterBySale(sale = true)
-          unless sale.is_a?(TrueClass) || sale.is_a?(FalseClass) 
-            sale = nil
-          end
+        def filter_by_sale(sale = true)
+          sale = nil unless sale.is_a?(TrueClass) || sale.is_a?(FalseClass)
 
-          self.filterBy(FILTER_SALE, sale)
+          filter_by(FILTER_SALE, sale)
         end
 
         ###
@@ -118,8 +117,8 @@ module AboutYou
         # * *Returns* :
         #   - an instance of AboutYou::SDK::Criteria::ProductSearchCriteria
         ###
-        def filterBySearchword(searchword)
-          self.filterBy(FILTER_SEARCHWORD, searchword)
+        def filter_by_searchword(searchword)
+          filter_by(FILTER_SEARCHWORD, searchword)
         end
 
         ###
@@ -132,15 +131,14 @@ module AboutYou
         # * *Returns* :
         #   - an instance of AboutYou::SDK::Criteria::ProductSearchCriteria
         ###
-        def filterByCategoryIds(categoryIds, append = false)
-          if append && self.filter[FILTER_CATEGORY_IDS]
-            categoryIds = self.filter[FILTER_CATEGORY_IDS] + categoryIds
-          end
-          categoryIds = categoryIds.uniq
+        def filter_by_category_ids(category_ids, append = false)
+          category_ids = filter(FILTER_CATEGORY_IDS) + category_ids if
+          append && filter(FILTER_CATEGORY_IDS)
 
-          self.filterBy(FILTER_CATEGORY_IDS, categoryIds);
-        end
+          category_ids = category_ids.uniq
 
+          filter_by(FILTER_CATEGORY_IDS, category_ids)
+        end
 
         ###
         # sets a filter for filtering by category ids
@@ -152,49 +150,48 @@ module AboutYou
         # * *Returns* :
         #   - an instance of AboutYou::SDK::Criteria::ProductSearchCriteria
         ###
-        def filterByFacetIds(attributes, append = false)
+        def filter_by_facet_ids(attributes, append = false)
           merged = attributes[0]
-          if append && self.filter[FILTER_ATTRIBUTES]
-            merged = self.filter[FILTER_ATTRIBUTES]
-            attributes.each do |groupId, facetIds|
-              if merged.key?(groupId)
-                merged[groupId] = (merged[groupId] + facetIds).uniq
+          if append && filter(FILTER_ATTRIBUTES)
+            merged = filter(FILTER_ATTRIBUTES)
+            attributes.each do |group_id, facet_ids|
+              if merged.key?(group_id)
+                merged[group_id] = (merged[group_id] + facet_ids).uniq
               else
-                merged[groupId] = facetIds
-              end 
+                merged[group_id] = facet_ids
+              end
             end
-            attributes = merged
           end
-          self.filterBy(FILTER_ATTRIBUTES, merged)
-        end
 
+          filter_by(FILTER_ATTRIBUTES, merged)
+        end
 
         ###
         # sets a filter for filtering by Facet-Group
         #
         # * *Args*    :
-        #   - +facetGroup+ -> Instance of AboutYou::SDK::Model::FacetGroup
+        #   - +facet_group+ -> Instance of AboutYou::SDK::Model::FacetGroup
         #   - +append+ -> determines whether to append the category ids to an already set value or not [optional]
         #
         # * *Returns* :
         #   - an instance of AboutYou::SDK::Criteria::ProductSearchCriteria
         ###
-        def filterByFacetGroup(facetGroup, append = false)
-          self.filterByFacetIds(facetGroup.ids, append)
+        def filter_by_facet_group(facet_group, append = false)
+          filter_by_facet_ids(facet_group.ids, append)
         end
 
         ###
         # sets a filter for filtering by Facet-Groupset
         #
         # * *Args*    :
-        #   - +facetGroupSet+ -> Instance of AboutYou::SDK::Model::FacetGroupSet
+        #   - +facet_group_set+ -> Instance of AboutYou::SDK::Model::FacetGroupSet
         #   - +append+ -> determines whether to append the category ids to an already set value or not [optional]
         #
         # * *Returns* :
         #   - an instance of AboutYou::SDK::Criteria::ProductSearchCriteria
         ###
-        def filterByFacetGroupSet(facetGroupSet, append = false)
-          self.filterByFacetIds(facetGroupSet.ids, append)
+        def filter_by_facet_group_set(facet_group_set, append = false)
+          filter_by_facet_ids(facet_group_set.ids, append)
         end
 
         ###
@@ -207,15 +204,15 @@ module AboutYou
         # * *Returns* :
         #   - an instance of AboutYou::SDK::Criteria::ProductSearchCriteria
         ###
-        def filterByPriceRange(from = 0, to = 0)
+        def filter_by_price_range(from = 0, to = 0)
           from = Integer(from)
           to = Integer(to)
           price = {}
-          
-          price["from"] = from if from > 0
-          price["to"] = to if to > 0
 
-          self.filterBy(FILTER_PRICE, price)
+          price['from'] = from if from > 0
+          price['to'] = to if to > 0
+
+          filter_by(FILTER_PRICE, price)
         end
 
         ###
@@ -228,11 +225,12 @@ module AboutYou
         # * *Returns* :
         #   - an instance of AboutYou::SDK::Criteria::ProductSearchCriteria
         ###
-        def sortBy(type, direction = SORT_ASC)
-          self.result['sort'] = {
+        def sort_by(type, direction = SORT_ASC)
+          result['sort'] = {
             'by'        => type,
-            'direction' => direction,
+            'direction' => direction
           }
+
           self
         end
 
@@ -246,12 +244,13 @@ module AboutYou
         # * *Returns* :
         #   - an instance of AboutYou::SDK::Criteria::ProductSearchCriteria
         ###
-        def setLimit(limit, offset = 0)
+        def set_limit(limit, offset = 0)
           limit = [[limit, 200].min, 0].max
-          self.result['limit'] = limit
+          result['limit'] = limit
 
           offset = [offset, 0].max
-          self.result['offset'] = offset
+          result['offset'] = offset
+
           self
         end
 
@@ -264,13 +263,13 @@ module AboutYou
         # * *Returns* :
         #   - an instance of AboutYou::SDK::Criteria::ProductSearchCriteria
         ###
-        def selectSales(enable = true)
+        def select_sales(enable = true)
           if enable
-            self.result['sale'] = true
-          else 
-            self.result['sale'] = nil
+            result['sale'] = true
+          else
+            result['sale'] = nil
           end
-          
+
           self
         end
 
@@ -283,40 +282,41 @@ module AboutYou
         # * *Returns* :
         #   - an instance of AboutYou::SDK::Criteria::ProductSearchCriteria
         ###
-        def selectPriceRanges(enable = true)
-          if enable 
-            self.result['price'] = true
-          else 
-            self.result['price'] = nil
+        def select_price_ranges(enable = true)
+          if enable
+            result['price'] = true
+          else
+            result['price'] = nil
           end
 
           self
         end
 
         ###
-        # this method adds a field to the api-request which enables you to 
+        # this method adds a field to the api-request which enables you to
         # receive Facets by a given group id
         #
         # * *Args*    :
-        #   - +groupId+ -> group id which facets you want to receive
+        #   - +group_id+ -> group id which facets you want to receive
         #   - +limit+ -> limits the received facets
         #
         # * *Returns* :
         #   - an instance of AboutYou::SDK::Criteria::ProductSearchCriteria
         ###
-        def selectFacetsByGroupId(groupId, limit)
-          self.checkFacetLimit(limit)
-          unless groupId.is_a?(Fixnum) && Integer(groupId)
-            raise "InvalidArgumentException! Group id must be an integer or a
-            string containing digits"
-          end
-          unless self.result['facets']
-            self.result['facets'] = {}
-          end
-          unless self.result['facets'][groupId]
-            self.result['facets'][String(groupId)] = {'limit' => limit}
-          end
-          
+        def select_facets_by_group_id(group_id, limit)
+          check_facet_limit(limit)
+
+          fail "InvalidArgumentException! Group id must be an integer or a
+          string containing digits" unless
+          group_id.is_a?(Fixnum) && Integer(group_id)
+
+          result['facets'] = {} unless result['facets']
+
+          result['facets'][String(group_id)] = {
+            'limit' => limit
+          } unless
+          result['facets'][group_id]
+
           self
         end
 
@@ -329,9 +329,13 @@ module AboutYou
         # * *Returns* :
         #   - an instance of AboutYou::SDK::Criteria::ProductSearchCriteria
         ###
-        def selectAllFacets(limit)
-          self.checkFacetLimit(limit)
-          self.result['facets'] = {self::FACETS_ALL => {'limit' => limit}}
+        def select_all_facets(limit)
+          check_facet_limit(limit)
+          result['facets'] = {
+            FACETS_ALL => {
+              'limit' => limit
+            }
+          }
 
           self
         end
@@ -345,13 +349,14 @@ module AboutYou
         # * *Returns* :
         #   - nil if there is no error
         ###
-        def checkFacetLimit(limit)
-          unless limit.is_a?(Fixnum) || limit.is_a?(Integer) 
-            raise 'InvalidArgumentException! limit must be an integer'
-          end
-          if limit < -1
-            raise 'InvalidArgumentException! limit must be positive or -1 for unlimited facets'
-          end
+        def check_facet_limit(limit)
+          fail 'InvalidArgumentException! limit must be an integer' unless
+          limit.is_a?(Fixnum) || limit.is_a?(Integer)
+
+          fail 'InvalidArgumentException! limit must be positive or -1
+            for unlimited facets' if limit < -1
+
+          nil
         end
 
         ###
@@ -363,11 +368,11 @@ module AboutYou
         # * *Returns* :
         #   - an instance of AboutYou::SDK::Criteria::ProductSearchCriteria
         ###
-        def selectCategories(enable = true)
+        def select_categories(enable = true)
           if enable
-            self.result['categories'] = true
-          else 
-            self.result['categories'] = nil
+            result['categories'] = true
+          else
+            result['categories'] = nil
           end
 
           self
@@ -383,17 +388,15 @@ module AboutYou
         # * *Returns* :
         #   - an instance of AboutYou::SDK::Criteria::ProductSearchCriteria
         ###
-        def boostProducts(ids)
-          mappedIds = []
-          ids = ids.map{
+        def boost_products(ids)
+          mapped_ids = []
+          ids = ids.map do
             |val|
-            if val.instance_of?(Product)
-              mappedIds.push(val.id)
-            end
-            mappedIds.push(val)
-          }
-          mappedIds = mappedIds.uniq
-          self.result['boosts'] = ids
+            mapped_ids.push(val.id) if val.instance_of?(Product)
+            mapped_ids.push(val)
+          end
+          mapped_ids = mapped_ids.uniq
+          result['boosts'] = ids
 
           self
         end
@@ -408,9 +411,9 @@ module AboutYou
         # * *Returns* :
         #   - an instance of AboutYou::SDK::Criteria::ProductSearchCriteria
         ###
-        def selectProductFields(fields)
-          self.result['fields'] = 
-            AboutYou::SDK::Criteria::ProductFields.filterFields(fields)
+        def select_product_fields(fields)
+          result['fields'] =
+            AboutYou::SDK::Criteria::ProductFields.filter_fields(fields)
 
           self
         end
@@ -421,12 +424,14 @@ module AboutYou
         # * *Returns* :
         #   - a boolean stating whether to fetch categories from api or not
         ###
-        def requiresCategories
-          (self.result.has_key?('fields') &&
-          AboutYou::SDK::Criteria::ProductFields.
-          requiresCategories(self.result['fields'])) ||
-          (self.result.has_key?('categories') &&
-          self.result['categories'])
+        def requires_categories
+          (result.key?('fields') &&
+            AboutYou::SDK::Criteria::ProductFields.requires_categories(
+            result['fields']
+          )) || (
+            result.key?('categories') &&
+              result['categories']
+            )
         end
 
         ###
@@ -435,35 +440,35 @@ module AboutYou
         # * *Returns* :
         #   - a boolean stating whether to fetch facets from api or not
         ###
-        def requiresFacets
-          (self.result.key?('fields') &&
-          AboutYou::SDK::Criteria::ProductFields.
-          requiresFacets(self.result['fields'])) ||
-          (self.result.key?('facets') &&
-          !self.result['facets'].empty?)
+        def requires_facets
+          (result.key?('fields') &&
+            AboutYou::SDK::Criteria::ProductFields.requires_facets(
+            result['fields']
+          )) || (
+            result.key?('facets') &&
+              !result['facets'].empty?
+            )
         end
 
         ###
         # this method turns an instance of
-        # AboutYou::SDK::Criteria::productSearchCriteria in an Array which can
-        # be passed directly to the api 
+        # AboutYou::SDK::Criteria::product_search_criteria in an Array which can
+        # be passed directly to the api
         #
         # * *Returns* :
         #   - an Array containing all the parameters set on the instance
         ###
-        def toArray
-          params = {'session_id' => self.sessionId}
+        def to_array
+          params = {
+            'session_id' => session_id
+          }
 
-          if self.result
-            params['result'] = self.result
-          end
-          unless @filter.empty?
-            params['filter'] = @filter
-          end
+          params['result'] = result if result
+          params['filter'] = @filter unless @filter.empty?
 
           params
         end
       end
     end
   end
-end
+end