stbaldricks 12.10.0.alpha.2 → 12.10.0.alpha.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6329ad2d96a64c2eeba118360b5884851a82f632472e57e7ff4418a605da882d
-  data.tar.gz: 48c5b9afa7e1aed7a0e6de385b07081fa97f293e15c3de50c66e97a793727458
+  metadata.gz: 34a3ee4a9b2cd1f2e19c0a412daa09cb3d8045497844d1dbdf28b90060f8e9f3
+  data.tar.gz: 705928ce7a4bc56878954da837b2de2e52f2ef700e66b0eae427e1d221f5ff42
 SHA512:
-  metadata.gz: 64ec0cc8de11735db6f4996bb7c9c4bc49c9abca80fc6d44c7a3f624697f4cdfc30d67f1b773ca379f1ea566be5fe3989dd2e4bf24e495cf65234b2351f80d6e
-  data.tar.gz: d9d4c2cf4a3c048e7b9478d872ead7c6ad3e3ecbc533b8cf38b27afdab10342d9ecd7e9bed5b33d9676b36b0ca5be92ec908058769de1bdebdb3ae56c7dcd8b8
+  metadata.gz: 17065359c81d0ab33acf0937cde7260182cc3bcfd7860d44691d8ae6892c7a1d0a13278197a14aed7b29ba541e65c9bb1113eee2cf6d338e45449c6471f312c9
+  data.tar.gz: 3e5c458c635791f059424ac420862e91e75d55c2f147dadab7741b9ade1709f6817b55d07dc04a9f0d35a0166b09ed318f95ff8b9348ee596211b1db8428e70b

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, **(;{})) } 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, **(;{}))
+        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, **(;{})) }
+          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, **(;{})) : 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, **(;{}))
+            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, **(;{}))
+          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
-                                    }, **(;{}))
+          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}, **(;{}))
-        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!, **(;{}))
+          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!, **(;{}))
+          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!, **(;{}))
+          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, **(;{})) : 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 : {}, **(;{}))
+          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], **(;{}))
+              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], **(;{})))
+          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