stbaldricks 12.14.0.alpha.1 → 12.14.0.alpha.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2420525ef5b22e14f0371c1e9c1f8a90393c0c95f0cf9b061ef3e92f7a859d4e
-  data.tar.gz: 4b9b0881b050a9ea26a8f5f3279e390fa7f3cbf8cdaebf5a932d9c6ebd1e434f
+  metadata.gz: 1cac242569a8fe46b775cd30555fd887676b601727e1eddd280fbb0f9b9a6feb
+  data.tar.gz: febdac347c36fdba7434e9c784b2934fa831cd28bf29806c5ffa6395a1491cb0
 SHA512:
-  metadata.gz: 9fcefefb1881d41e6a4480c077a5c0fa5073d3a8b6b19c70325586d5282409eacb8b756ba591482e23540b80a4ead06d5796c4a5ed4803d02ffccc30d03c9eaa
-  data.tar.gz: c2a13304a82acb38b072ec5f7a72a5e23e90c361afe5e9dc093309344cd1637464ca574a36680af2380f92af7f19f29dc8c4edc5b7f39af1e284ef9a4da56031
+  metadata.gz: d144d4f5416ee4e828c7471cfd5b250d1a1ff36f6c2c17f9b04eb52d61afce9665e207acb76e1523307988ff277bd2a94a28b842a319b8770053533e4d3ad894
+  data.tar.gz: c7c4a9e7a8cf846a4b271892831a38de3f1b2d14dfc14d5e877c7acaf5b71d61066152082af371500284fa49e752a303d035b0679cef1142f8e6da9143380e14

data/lib/stbaldricks/configuration.rb

@@ -16,11 +16,11 @@ module SBF
       end
 
       def self.base_uri
-        @base_uri ||= ENV.fetch('API_ENDPOINT', DEFAULT_API_ENDPOINT)
+        @base_uri ||= ENV['API_ENDPOINT'] || DEFAULT_API_ENDPOINT
       end
 
       def self.general_token
-        @general_token ||= ENV.fetch('API_KEY', nil) || raise(SBF::Client::Configuration::Error.new('general_token', 'must be set'))
+        @general_token ||= ENV['API_KEY'] || raise(SBF::Client::Configuration::Error.new('general_token', 'must be set'))
       end
 
       def self.request_id

data/lib/stbaldricks/default_logger.rb

@@ -3,7 +3,7 @@ require 'logger'
 module SBF
   module Client
     module DefaultLogger
-      def self.instance(default_stream = $stdout)
+      def self.instance(default_stream = STDOUT)
         formatter = proc { |severity, datetime, _, msg|
           # ruby logger docs suggested this method for escaping messages since messages may contain user input
           # and are not escaped by default.
@@ -16,7 +16,7 @@ module SBF
 
         logger.formatter = formatter
         logger.level = ::Logger::DEBUG
-        logger.level = ENV.fetch('SBF_CLIENT_LOG_LEVEL', ::Logger::DEBUG)
+        logger.level = ENV['SBF_CLIENT_LOG_LEVEL'] || ::Logger::DEBUG
         logger
       end
     end

data/lib/stbaldricks/endpoints/blog_post.rb

@@ -27,17 +27,17 @@ module SBF
         end
       end
 
-      def find_by_term(term, order = {}, limit = 10, offset = 0, show_hidden: true)
+      def find_by_term(term, order = {}, limit = 10, offset = 0, show_hidden = true)
         # Build the parameter list for search; an empty find is supported
         filter = {}
         filter[:type] = FILTER_BY_TERM
         filter[:term] = term unless term.empty?
         filter[:show_hidden] = show_hidden
 
-        find(filter: filter, order: order, limit: limit, offset: offset)
+        find(filter, order, limit, offset)
       end
 
-      def find_by_author(author, order = {}, limit = 10, offset = 0, show_hidden: true)
+      def find_by_author(author, order = {}, limit = 10, offset = 0, show_hidden = true)
         raise SBF::Client::Error, 'Must provide author for the query' if author.empty?
 
         filter = {}
@@ -45,10 +45,10 @@ module SBF
         filter[:term] = author
         filter[:show_hidden] = show_hidden
 
-        find(filter: filter, order: order, limit: limit, offset: offset)
+        find(filter, order, limit, offset)
       end
 
-      def find_by_category(category, order = {}, limit = 10, offset = 0, show_hidden: true)
+      def find_by_category(category, order = {}, limit = 10, offset = 0, show_hidden = true)
         raise SBF::Client::Error, 'Must provide category for the query' if category.empty?
 
         filter = {}
@@ -56,10 +56,10 @@ module SBF
         filter[:term] = category
         filter[:show_hidden] = show_hidden
 
-        find(filter: filter, order: order, limit: limit, offset: offset)
+        find(filter, order, limit, offset)
       end
 
-      def find_by_tag(tag, order = {}, limit = 10, offset = 0, show_hidden: true)
+      def find_by_tag(tag, order = {}, limit = 10, offset = 0, show_hidden = true)
         raise SBF::Client::Error, 'Must provide tag for the query' if tag.empty?
 
         filter = {}
@@ -67,7 +67,7 @@ module SBF
         filter[:term] = tag
         filter[:show_hidden] = show_hidden
 
-        find(filter: filter, order: order, limit: limit, offset: offset)
+        find(filter, order, limit, offset)
       end
     end
   end

data/lib/stbaldricks/endpoints/communicate.rb

@@ -48,7 +48,7 @@ module SBF
           group_id: group_id
         )
         parsed_response_body = JSON.parse(response.body).symbolize!
-        parsed_response_body.map! { |c| SBF::Client::Contact.new(c, clear_changes: false) } if ok?(response)
+        parsed_response_body.map! { |c| SBF::Client::Contact.new(c) } if ok?(response)
 
         handle_parsed_response(parsed_response_body, response)
       end

data/lib/stbaldricks/endpoints/config.rb

@@ -11,7 +11,7 @@ module SBF
         raise SBF::Client::StandardError, 'Unable to retrieve settings from the api' unless ok?(response)
 
         # re-call initalize with the new api data. This should re-set all of the instance variables on the instance
-        entity.send(:initialize, parsed_response_body, clear_changes: false)
+        entity.send(:initialize, parsed_response_body)
       end
     end
   end

data/lib/stbaldricks/endpoints/deduplicator_match.rb

@@ -3,7 +3,7 @@ require 'stbaldricks/endpoints/lib/entity'
 module SBF
   module Client
     class DeduplicatorMatchEndpoint < SBF::Client::EntityEndpoint
-      def checkout(filter: [], order: {}, limit: 20, offset: 0, with: {})
+      def checkout(filter = [], order = {}, limit = 20, offset = 0, with = {})
         filter = filter.to_json unless filter.is_a? String
         order = order.to_json unless order.is_a? String
         with = normalize_with(with)
@@ -15,7 +15,7 @@ module SBF
         parsed_response_body = JSON.parse(response.body).symbolize!
 
         if ok?(response)
-          parsed_response_body[:results].map! { |entity_data| target_class.new(entity_data, clear_changes: true) }
+          parsed_response_body[:results].map! { |entity_data| target_class.new(entity_data, true) }
           SBF::Client::EntityCollection.new(parsed_response_body[:results], parsed_response_body[:total_count])
         else
           parsed_response_body = JSON.parse(response.body).symbolize!

data/lib/stbaldricks/endpoints/kid.rb

@@ -12,7 +12,7 @@ module SBF
         parsed_response_body = JSON.parse(response.body).symbolize!
 
         if ok?(response)
-          parsed_response_body[:results].map! { |entity_data| target_class.new(entity_data, clear_changes: false) }
+          parsed_response_body[:results].map! { |entity_data| target_class.new(entity_data) }
           SBF::Client::EntityCollection.new(parsed_response_body[:results], parsed_response_body[:total_count])
         else
           parsed_response_body = JSON.parse(response.body).symbolize!

data/lib/stbaldricks/endpoints/lib/entity.rb

@@ -10,7 +10,8 @@ require 'json'
 module SBF
   module Client
     class EntityEndpoint
-      attr_reader :orig_target_class, :base_uri
+      attr_reader :orig_target_class
+      attr_reader :base_uri
 
       def initialize(target_class)
         @orig_target_class = target_class
@@ -33,7 +34,7 @@ module SBF
       # @param with [Hash] The optional entity fields to include in the response
       # @return entity [SBF::Client::BaseEntity]
       def create(entity_or_hash, with = {})
-        entity = entity_or_hash.is_a?(Hash) ? target_class.new(entity_or_hash, clear_changes: false) : entity_or_hash
+        entity = entity_or_hash.is_a?(Hash) ? target_class.new(entity_or_hash) : entity_or_hash
         raise SBF::Client::Error, 'Invalid Entity' unless entity.is_a?(SBF::Client::BaseEntity)
 
         with = normalize_with(with, entity)
@@ -49,11 +50,10 @@ module SBF
       # @param with [Hash] The optional entity fields to include in the response
       # @return entity [SBF::Client::BaseEntity]
       def update(id = nil, entity_or_hash = nil, with = {})
-        case entity_or_hash
-        when SBF::Client::BaseEntity
+        if entity_or_hash.is_a?(SBF::Client::BaseEntity)
           # If someone has passed in an entity, just convert it to a hash
           data = entity_or_hash.dirty_data
-        when Hash
+        elsif entity_or_hash.is_a?(Hash)
           # If someone has passed in a hash, make sure all of it's values match fields in the entity class
           data = sanitize(entity_or_hash)
         else
@@ -89,7 +89,7 @@ module SBF
       end
 
       # Calls the find route for the entity.
-      def find(filter: [], order: {}, limit: 20, offset: 0, with: {})
+      def find(filter = [], order = {}, limit = 20, offset = 0, with = {})
         filter = filter.to_json unless filter.is_a? String
         order = order.to_json unless order.is_a? String
         with = normalize_with(with)
@@ -98,9 +98,7 @@ module SBF
         parsed_response_body = JSON.parse(response.body).symbolize!
 
         if ok?(response)
-          parsed_response_body[:results].map! do |entity_data|
-            target_class.new(entity_data, clear_changes: true)
-          end
+          parsed_response_body[:results].map! { |entity_data| target_class.new(entity_data, true) }
           SBF::Client::EntityCollection.new(parsed_response_body[:results], parsed_response_body[:total_count])
         else
           parsed_response_body = JSON.parse(response.body).symbolize!
@@ -115,7 +113,7 @@ module SBF
 
       # Calls the find route for the entity.
       def find_first(filter = [], order = {}, with = {})
-        response = find(filter: filter, order: order, limit: 1, with: with)
+        response = find(filter, order, 1, 0, with)
         if response.is_a?(SBF::Client::EntityCollection)
           response.empty? ? nil : response.first
         else
@@ -150,7 +148,7 @@ module SBF
 
       def target_class
         class_name_pieces = @orig_target_class.to_s.split('::')
-        full_name = (class_name_pieces[0..-2] + ["Full#{class_name_pieces.last}"]).join('::')
+        full_name = (class_name_pieces[0..-2] + ['Full' + class_name_pieces.last]).join('::')
         Object.const_defined?(full_name) ? Object.const_get(full_name) : @orig_target_class
       end
       private :target_class
@@ -249,7 +247,7 @@ module SBF
             entity_or_hash.send(:initialize_attributes, new_data)
             entity = entity_or_hash
           else
-            entity = target_class.new(new_data, clear_changes: false)
+            entity = target_class.new(new_data)
           end
 
           # Reload entity and its sub-entities so that no fields have a 'changed' state
@@ -258,7 +256,7 @@ module SBF
           entity
         else
           error = SBF::Client::ApiErrorEntity.new(parsed_response_body.merge(http_code: response.code))
-          entity = entity_or_hash.is_a?(SBF::Client::BaseEntity) ? entity_or_hash : target_class.new(entity_or_hash, clear_changes: false)
+          entity = entity_or_hash.is_a?(SBF::Client::BaseEntity) ? entity_or_hash : target_class.new(entity_or_hash)
           entity.add_errors(error)
           entity.errors_http_code = response.code
 

data/lib/stbaldricks/endpoints/newsletter_recipient.rb

@@ -3,14 +3,9 @@ require 'stbaldricks/endpoints/lib/entity'
 module SBF
   module Client
     class NewsletterRecipientEndpoint < EntityEndpoint
-      def subscribe(entity_or_email, first_name = nil, last_name = nil, welcome_email: true)
+      def subscribe(entity_or_email, first_name = nil, last_name = nil, welcome_email = true)
         if entity_or_email.is_a?(String) && entity_or_email && first_name && last_name
-          entity = target_class.new({
-                                      email_address: entity_or_email,
-                                      first_name: first_name,
-                                      last_name: last_name,
-                                      welcome_email: welcome_email
-                                    }, clear_changes: false)
+          entity = target_class.new(email_address: entity_or_email, first_name: first_name, last_name: last_name, welcome_email: welcome_email)
         elsif entity_or_email.is_a?(SBF::Client::BaseEntity)
           entity = entity_or_email
         else
@@ -26,10 +21,9 @@ module SBF
       end
 
       def unsubscribe(entity_or_email)
-        case entity_or_email
-        when String
-          entity = target_class.new({email_address: entity_or_email}, clear_changes: false)
-        when SBF::Client::BaseEntity
+        if entity_or_email.is_a?(String)
+          entity = target_class.new(email_address: entity_or_email)
+        elsif entity_or_email.is_a?(SBF::Client::BaseEntity)
           entity = entity_or_email
         else
           raise SBF::Client::Error, 'Invalid Entity'

data/lib/stbaldricks/endpoints/participant.rb

@@ -7,7 +7,7 @@ module SBF
         response = SBF::Client::Api::Request.post_request("#{base_uri}/move_to_different_team", id: participant_id, new_team_id: new_team_id)
 
         if ok?(response)
-          data = SBF::Client::FullParticipant.new(JSON.parse(response.body).symbolize!, clear_changes: false)
+          data = SBF::Client::FullParticipant.new(JSON.parse(response.body).symbolize!)
         else
           error = SBF::Client::ErrorEntity.new(JSON.parse(response.body).symbolize!)
         end
@@ -26,7 +26,7 @@ module SBF
         )
 
         if ok?(response)
-          data =  SBF::Client::FullParticipant.new(JSON.parse(response.body).symbolize!, clear_changes: false)
+          data =  SBF::Client::FullParticipant.new(JSON.parse(response.body).symbolize!)
         else
           error = SBF::Client::ErrorEntity.new(JSON.parse(response.body).symbolize!)
         end
@@ -38,7 +38,7 @@ module SBF
         response = SBF::Client::Api::Request.post_request("#{base_uri}/move_to_different_team", id: participant_id, new_team_id: 0)
 
         if ok?(response)
-          data = SBF::Client::FullParticipant.new(JSON.parse(response.body).symbolize!, clear_changes: false)
+          data = SBF::Client::FullParticipant.new(JSON.parse(response.body).symbolize!)
         else
           error = SBF::Client::ErrorEntity.new(JSON.parse(response.body).symbolize!)
         end

data/lib/stbaldricks/endpoints/photo.rb

@@ -4,7 +4,7 @@ module SBF
   module Client
     class PhotoEndpoint < SBF::Client::EntityEndpoint
       def create(entity_or_hash, with = {})
-        entity = entity_or_hash.is_a?(Hash) ? target_class.new(entity_or_hash, clear_changes: false) : entity_or_hash
+        entity = entity_or_hash.is_a?(Hash) ? target_class.new(entity_or_hash) : entity_or_hash
         raise SBF::Client::Error, 'Invalid Entity' unless entity.is_a?(SBF::Client::BaseEntity)
 
         with = normalize_with(with)
@@ -20,13 +20,12 @@ module SBF
       end
 
       def update(id = nil, entity_or_hash = nil, with = {})
-        case entity_or_hash
-        when SBF::Client::BaseEntity
+        if entity_or_hash.is_a?(SBF::Client::BaseEntity)
           # If someone has passed in an entity, just convert it to a hash
           data = entity_or_hash.dirty_data
           data[:id] = id unless id.nil?
           data[:id] = entity_or_hash.id if id.nil? && !entity_or_hash.id.nil?
-        when Hash
+        elsif entity_or_hash.is_a?(Hash)
           # If someone has passed in a hash, make sure all of it's values match fields in the entity class
           data = sanitize(entity_or_hash)
         else
@@ -67,7 +66,7 @@ module SBF
           data.file = file
           data
         else
-          photo = SBF::Client::Photo.new((data.is_a?(Hash) && data.any?) ? data : {}, clear_changes: false)
+          photo = SBF::Client::Photo.new(data.is_a?(Hash) && data.any? ? data : {})
           photo.file = file
           photo
         end

data/lib/stbaldricks/endpoints/search.rb

@@ -18,7 +18,7 @@ module SBF
       # @option [Integer] :offset The query offset.
       # @return [Client::Api::Response] The data attribute on a successful response will be
       #                                 an array of search entities matching the model type(s) specified
-      def find(model_type, search_text: nil, filters: [], geo_location: {}, options: {})
+      def find(model_type, search_text = nil, filters = [], geo_location = {}, options = {})
         if [search_text, filters, geo_location].all?(&:empty?)
           raise SBF::Client::Error, 'Must provide either a search_text, filter or geo_location for the query'
         end
@@ -45,7 +45,7 @@ module SBF
           response_data[:total_count] = parsed_response_body[:total]
           response_data[:results] = [].tap do |arr|
             parsed_response_body[:hits].each do |entity_data|
-              arr << target_class.const_get(entity_data[:type].capitalize).new(entity_data[:source], clear_changes: false)
+              arr << target_class.const_get(entity_data[:type].capitalize).new(entity_data[:source])
             end
           end
 

data/lib/stbaldricks/endpoints/user.rb

@@ -20,7 +20,7 @@ module SBF
 
         if ok?(response)
           SBF::Client::Configuration.user_token = parsed_response[:token]
-          data = parsed_response.merge(user: target_class.new(parsed_response[:user], clear_changes: false))
+          data = parsed_response.merge(user: target_class.new(parsed_response[:user]))
         else
           SBF::Client::Configuration.user_token = nil
           error = SBF::Client::ErrorEntity.new(parsed_response)

data/lib/stbaldricks/entities/batch.rb

@@ -12,15 +12,17 @@ module SBF
         PROCESSED = 'processed'.freeze
       end
 
-      attr_accessor :id, :type, :amount, :filename, :created_at, :modified_at, :processed_at, :submitted_by
-
+      attr_accessor :id
+      attr_accessor :type
       multitype_attr_accessor(
         :payment_details, [
           [->(v) { v[:type] == SBF::Client::Payment::Type::CREDIT_CARD }, 'SBF::Client::Payment::CreditCardDetails'],
           [->(v) { v[:type] == SBF::Client::Payment::Type::NONCE }, 'SBF::Client::Payment::NonceDetails']
         ]
       )
-
+      attr_accessor :amount
+      attr_accessor :filename
+      attr_accessor :created_at, :modified_at, :processed_at, :submitted_by
       entity_collection_attr_accessor :donations, 'SBF::Client::FullDonation', 'SBF::Client::PartialDonation'
     end
   end

data/lib/stbaldricks/entities/blog_post.rb

@@ -12,7 +12,13 @@ module SBF
         action :find_by_category
         action :find_by_tag
 
-        attr_reader :id, :slug, :url, :title, :content, :published_at, :modified_at
+        attr_reader :id
+        attr_reader :slug
+        attr_reader :url
+        attr_reader :title
+        attr_reader :content
+        attr_reader :published_at
+        attr_reader :modified_at
       end
     end
   end

data/lib/stbaldricks/entities/campaign.rb

@@ -12,8 +12,12 @@ module SBF
       disallow_instantiation
 
       class Totals < SBF::Client::BaseEntity
-        attr_reader :current_year_amount_raised, :previous_year_amount_raised, :all_years_amount_raised, :participants,
-                    :shavees, :number_of_events
+        attr_reader :current_year_amount_raised
+        attr_reader :previous_year_amount_raised
+        attr_reader :all_years_amount_raised
+        attr_reader :participants
+        attr_reader :shavees
+        attr_reader :number_of_events
       end
     end
 
@@ -22,12 +26,15 @@ module SBF
     end
 
     class FullCampaign < Campaign
-      attr_accessor :id, :name, :website_url, :is_enabled
+      attr_accessor :id
+      attr_accessor :name
+      attr_accessor :website_url
+      attr_accessor :is_enabled
 
-      entity_attr_reader :totals, 'SBF::Client::Campaign::Totals', nil, optional: true
-      entity_attr_accessor :web_page, 'SBF::Client::WebPage', nil, optional: true
+      entity_attr_reader :totals, 'SBF::Client::Campaign::Totals', nil, true
+      entity_attr_accessor :web_page, 'SBF::Client::WebPage', nil, true
 
-      entity_attr_accessor :photo, 'SBF::Client::Photo', nil, optional: true
+      entity_attr_accessor :photo, 'SBF::Client::Photo', nil, true
     end
   end
 end

data/lib/stbaldricks/entities/challenge.rb

@@ -16,11 +16,13 @@ module SBF
         DECLINED = 'declined'.freeze
       end
 
-      attr_accessor :id, :status, :end_date, :modified_by
-
-      entity_attr_accessor :challenger, 'SBF::Client::Challenger', nil, optional: true
-      entity_attr_accessor :challengee, 'SBF::Client::Challenger', nil, optional: true
+      attr_accessor :id
+      attr_accessor :status
+      attr_accessor :end_date
+      entity_attr_accessor :challenger, 'SBF::Client::Challenger', nil, true
+      entity_attr_accessor :challengee, 'SBF::Client::Challenger', nil, true
       attr_reader :created_at, :modified_at
+      attr_accessor :modified_by
     end
   end
 end

data/lib/stbaldricks/entities/challenger.rb

@@ -97,9 +97,11 @@ module SBF
         end
       end
 
-      attr_accessor :id, :challenge_id, :type, :is_accepted
-
-      entity_attr_reader :totals, 'SBF::Client::Challenger::Totals', nil, optional: true
+      attr_accessor :id
+      attr_accessor :challenge_id
+      attr_accessor :type
+      attr_accessor :is_accepted
+      entity_attr_reader :totals, 'SBF::Client::Challenger::Totals', nil, true
 
       multitype_attr_accessor(
         :entity,
@@ -125,7 +127,7 @@ module SBF
             'SBF::Client::Challenger::PartialEvent'
           ]
         ],
-        optional: true
+        true
       )
     end
   end

data/lib/stbaldricks/entities/concerns/entity_response_concern.rb

@@ -59,7 +59,7 @@ module EntityResponseConcern
       caller_message = method_caller.nil? ? '' : " Called from #{method_caller.first}"
 
       "[DEPRECATION] Use of the SBF::Client::Api::Response Interface (including the `#{method_name}` method) is deprecated"\
-        " for #{self.class}. Please update your code to use the #{self.class} interface accordingly. #{caller_message}"
+          " for #{self.class}. Please update your code to use the #{self.class} interface accordingly. #{caller_message}"
     end
   end
 end

data/lib/stbaldricks/entities/config.rb

@@ -11,9 +11,14 @@ module SBF
       action :get
       instance_action :reload
 
-      attr_reader :default_year, :open_registration_years, :website_url, :years_for_squire, :years_for_knight, :years_for_knight_commander,
-                  :years_for_crusader, :years_for_baron
-
+      attr_reader :default_year
+      attr_reader :open_registration_years
+      attr_reader :website_url
+      attr_reader :years_for_squire
+      attr_reader :years_for_knight
+      attr_reader :years_for_knight_commander
+      attr_reader :years_for_crusader
+      attr_reader :years_for_baron
       # NOTE: If you add another attribute, be sure to mock it out in `configure :test` of app.rb
 
       # Attempts to call the method on the cached configuration object

data/lib/stbaldricks/entities/contact.rb

@@ -14,19 +14,22 @@ module SBF
         WEBSITE = 'website'.freeze
       end
 
-      attr_reader :id, :display_name, :email, :created_at, :modified_at
-
+      attr_reader :id
       entity_attr_accessor :owner, 'SBF::Client::FullPerson', 'SBF::Client::PartialPerson'
       entity_attr_accessor :person, 'SBF::Client::FullPerson', 'SBF::Client::PartialPerson'
       attr_accessor :type
-
+      attr_reader :display_name
+      attr_reader :email
+      attr_reader :created_at, :modified_at
       entity_collection_attr_reader :contact_groups, 'SBF::Client::ContactGroup'
     end
 
     class CustomContact < SBF::Client::Contact
       endpoint SBF::Client::ContactEndpoint
 
-      attr_accessor :id, :display_name, :email
+      attr_accessor :id
+      attr_accessor :display_name
+      attr_accessor :email
     end
   end
 end

data/lib/stbaldricks/entities/contact_group.rb

@@ -8,8 +8,8 @@ module SBF
     class ContactGroup < SBF::Client::TopLevelEntity
       actions DEFAULT_CRUD_ACTIONS
 
-      attr_accessor :id, :display_name
-
+      attr_accessor :id
+      attr_accessor :display_name
       entity_attr_accessor :owner, 'SBF::Client::FullPerson'
       multitype_collection_attr_accessor(
         :contacts,

data/lib/stbaldricks/entities/deduplicator_history.rb

@@ -6,7 +6,12 @@ module SBF
       action :get
       action :find
 
-      attr_accessor :id, :from_profile_id, :to_profile_id, :checkout_id, :change_history, :date_merged
+      attr_accessor :id
+      attr_accessor :from_profile_id
+      attr_accessor :to_profile_id
+      attr_accessor :checkout_id
+      attr_accessor :change_history
+      attr_accessor :date_merged
     end
   end
 end

data/lib/stbaldricks/entities/deduplicator_match.rb

@@ -27,10 +27,13 @@ module SBF
         ERROR_OUT_OF_SYNC_UPDATE = 'error_out_of_sync_update'.freeze
       end
 
-      attr_accessor :id, :checkout_id, :status, :date_created, :date_modified
-
-      entity_collection_attr_accessor :persons, 'SBF::Client::FullPerson', nil, optional: true
-      entity_collection_attr_accessor :organizations, 'SBF::Client::FullOrganization', nil, optional: true
+      attr_accessor :id
+      attr_accessor :checkout_id
+      attr_accessor :status
+      attr_accessor :date_created
+      attr_accessor :date_modified
+      entity_collection_attr_accessor :persons, 'SBF::Client::FullPerson', nil, true
+      entity_collection_attr_accessor :organizations, 'SBF::Client::FullOrganization', nil, true
     end
   end
 end

data/lib/stbaldricks/entities/disease.rb

@@ -13,8 +13,11 @@ module SBF
 
       disallow_instantiation
 
-      attr_accessor :id, :name, :description
-      attr_reader :created_at, :modified_at
+      attr_accessor :id
+      attr_accessor :name
+      attr_accessor :description
+      attr_reader :created_at
+      attr_reader :modified_at
     end
   end
 end

data/lib/stbaldricks/entities/document_library.rb

@@ -6,9 +6,14 @@ module SBF
     class DocumentLibrary < SBF::Client::TopLevelEntity
       actions DEFAULT_CRUD_ACTIONS
 
-      attr_accessor :id, :type, :title, :description, :url, :media_storage_id, :sort_order
-
-      entity_attr_reader :category, 'SBF::Client::DocumentLibraryCategory', nil, optional: true
+      attr_accessor :id
+      attr_accessor :type
+      attr_accessor :title
+      attr_accessor :description
+      attr_accessor :url
+      attr_accessor :media_storage_id
+      attr_accessor :sort_order
+      entity_attr_reader :category, 'SBF::Client::DocumentLibraryCategory', nil, true
     end
   end
 end

data/lib/stbaldricks/entities/document_library_category.rb

@@ -5,7 +5,10 @@ module SBF
     class DocumentLibraryCategory < SBF::Client::TopLevelEntity
       actions DEFAULT_CRUD_ACTIONS
 
-      attr_accessor :id, :top_level_category, :name, :sort_order
+      attr_accessor :id
+      attr_accessor :top_level_category
+      attr_accessor :name
+      attr_accessor :sort_order
     end
   end
 end