ruby-trello 2.2.0 → 3.1.0

data/lib/trello/custom_field_item.rb

@@ -1,9 +1,30 @@
 module Trello
   # A custom field item contains the value for a custom field on a particular card.
   #
+  # @!attribute [r] id
+  #   @return [String]
+  # @!attribute [r] model_id
+  #   @return [String]
+  # @!attribute [r] model_type
+  #   @return [String]
+  # @!attribute [r] custom_field_id
+  #   @return [String]
+  # @!attribute [r] option_id
+  #   @return [String]
+  # @!attribute [rw] value
+  #   @return [Hash]
   class CustomFieldItem < BasicData
-    register_attributes :id, :model_id, :model_type, :custom_field_id, :value, :option_id,
-                        readonly: [ :id, :custom_field_id, :model_id, :model_type, :option_id ]
+
+    schema do
+      attribute :id, readonly: true, primary_key: true
+      attribute :model_id, readonly: true, remote_key: 'idModel'
+      attribute :model_type, readonly: true, remote_key: 'modelType'
+      attribute :custom_field_id, readonly: true, remote_key: 'idCustomField'
+      attribute :option_id, readonly: true, remote_key: 'idValue'
+
+      attribute :value
+    end
+
     validates_presence_of :id, :model_id, :custom_field_id
 
     # References the card with this custom field value
@@ -12,56 +33,33 @@ module Trello
     # References the parent custom field that this item is an instance of
     one :custom_field, path: 'customFields', using: :custom_field_id
 
-    # Update the fields of a custom field item.
-    #
-    # Supply a hash of string keyed data retrieved from the Trello API representing
-    # an item state.
-    def update_fields(fields)
-      attributes[:id]               = fields['id'] || fields[:id] || attributes[:id]
-      attributes[:model_id]         = fields['idModel'] || fields[:model_id] || attributes[:model_id]
-      attributes[:custom_field_id]  = fields['idCustomField'] || fields[:custom_field_id] || attributes[:custom_field_id]
-      attributes[:model_type]       = fields['modelType'] || fields[:model_type] || attributes[:model_type]
-      # Dropdown custom field items do not have a value, they have an ID that maps to
-      # a different endpoint to get the value
-      attributes[:option_id]       = fields['idValue'] if fields.has_key?('idValue')
-      # value format: { "text": "hello world" }
-      attributes[:value]            = fields['value'] if fields.has_key?('value')
-      self
-    end
+    def save
+      return update! if id
+
+      payload = {}
 
-    def update!
-      @previously_changed = changes
-      # extract only new values to build payload
-      payload = Hash[changes.map { |key, values| [key.to_sym, values[1]] }]
-      @changed_attributes.try(:clear)
-      changes_applied if respond_to?(:changes_applied)
+      schema.attrs.each do |_, attribute|
+        payload = attribute.build_payload_for_create(attributes, payload)
+      end
 
-      client.put("/card/#{model_id}/customField/#{custom_field_id}/item", payload)
+      put(element_path, payload)
     end
 
-    # Saves a record.
-    #
-    # @raise [Trello::Error] if the card could not be saved
-    #
-    # @return [String] The JSON representation of the saved custom field item returned by
-    # the Trello API.
-    def save
-      # If we have an id, just update our fields.
-      return update! if id
+    def collection_path
+      "/cards/#{model_id}/#{collection_name}"
+    end
 
-      from_response client.post("/card/#{model_id}/customField/#{custom_field_id}/item", {
-        value: value
-      })
+    def element_path
+      "/cards/#{model_id}/customField/#{custom_field_id}/item"
     end
 
     # You can't "delete" a custom field item, you can only clear the value
     def remove
       params = { value: {} }
-      client.put("/card/#{model_id}/customField/#{custom_field_id}/item", params)
+      client.put(element_path, params)
     end
 
     # Type is saved at the CustomField level
-    # Can equally be derived from :value, with a little parsing work: {"number": 42}
     def type
       custom_field.type
     end

data/lib/trello/custom_field_option.rb

@@ -1,22 +1,49 @@
 module Trello
   # A custom field option contains the individual items in a custom field dropdown menu.
   #
+  # @!attribute [r] id
+  #   @return [String]
+  # @!attribute [r] custom_field_id
+  #   @return [String]
+  # @!attribute [rw] value
+  #   @return [Hash]
+  # @!attribute [rw] color
+  #   @return [String]
+  # @!attribute [rw] position
+  #   @return [Float]
   class CustomFieldOption < BasicData
-    register_attributes :id, :value, :pos, :color,
-                        readonly: [:id]
+
+    schema do
+      # Readonly
+      attribute :id, remote_key: '_id', readonly: true, primary_key: true
+      attribute :custom_field_id, remote_key: 'idCustomField', readonly: true
+
+      # Writable but for create only
+      attribute :value, create_only: true
+      attribute :color, create_only: true
+      attribute :position, remote_key: 'pos', create_only: true
+    end
+
     validates_presence_of :id, :value
 
-    # Update the fields of a custom field option.
-    #
-    # Supply a hash of string keyed data retrieved from the Trello API representing
-    # an item state.
-    def update_fields(fields)
-      attributes[:id]               = fields['_id'] || fields[:id] || attributes[:id]
-      attributes[:color]            = fields['color'] || fields[:color] || attributes[:color]
-      attributes[:pos]              = fields['pos'] || fields[:pos] || attributes[:pos]
-      # value format: { "text": "hello world" }
-      attributes[:value]            = fields['value'] || fields[:value] || attributes[:value]
-      self
+    def self.find(id, params = {})
+      params = params.with_indifferent_access
+      custom_field_id = params.delete(:custom_field_id)
+      return if custom_field_id.nil?
+
+      from_response client.get("/customFields/#{custom_field_id}/options/#{id}")
+    end
+
+    def delete
+      client.delete(element_path)
+    end
+
+    def collection_path
+      "/customFields/#{custom_field_id}/options"
+    end
+
+    def element_path
+      "/customFields/#{custom_field_id}/options/#{id}"
     end
   end
 end

data/lib/trello/item.rb

@@ -5,31 +5,42 @@ module Trello
   #   @return [String]
   # @!attribute [r] name
   #   @return [String]
+  # @!attribute [r] name_data
+  #   @return [String]
   # @!attribute [r] type
   #   @return [Object]
   # @!attribute [r] state
   #   @return [Object]
-  # @!attribute [r] pos
+  # @!attribute [r] position
   #   @return [Object]
+  # @!attribute [r] due_date
+  #   @return [Datetime]
+  # @!attribute [r] member_id
+  #   @return [String]
+  # @!attribute [r] checklist_id
+  #   @return [String]
+  # @!attribute [w] checked
+  #   @return [Boolean]
   class Item < BasicData
-    register_attributes :id, :name, :type, :state, :pos, readonly: [ :id, :name, :type, :state, :pos ]
-    validates_presence_of :id, :type
 
-    # Updates the fields of an item.
-    #
-    # Supply a hash of string keyed data retrieved from the Trello API representing
-    # an item.
-    def update_fields(fields)
-      attributes[:id]           = fields['id'] || attributes[:id]
-      attributes[:card_id]      = fields['idCard'] || attributes[:card_id]
-      attributes[:checklist_id] = fields['idChecklist'] || attributes[:checklist_id]
-      attributes[:name]         = fields['name'] || attributes[:name]
-      attributes[:type]         = fields['type'] || attributes[:type]
-      attributes[:state]        = fields['state'] || attributes[:state]
-      attributes[:pos]          = fields['pos'] || attributes[:pos]
-      self
+    schema do
+      # Readonly
+      attribute :id, readonly: true, primary_key: true
+      attribute :type, readonly: true
+      attribute :name_data, readonly: true, remote_key: 'nameData'
+      attribute :due_date, readonly: true, remote_key: 'due', serializer: 'Time'
+      attribute :member_id, readonly: true, remote_key: 'idMember'
+      attribute :state, readonly: true
+      attribute :checklist_id, readonly: true, remote_key: 'idChecklist'
+
+      # Writable and write only
+      attribute :name, write_only: true
+      attribute :position, write_only: true, remote_key: 'pos'
+      attribute :checked, write_only: true
     end
 
+    validates_presence_of :id
+
     def complete?
       state == "complete"
     end

data/lib/trello/item_state.rb

@@ -5,26 +5,18 @@ module Trello
   #   @return [String]
   # @!attribute [r] state
   #   @return [Object]
-  # @!attribute [r] item_id
-  #   @return [String]
   class CheckItemState < BasicData
-    register_attributes :id, :state, :item_id, readonly: [ :id, :state, :item_id ]
-    validates_presence_of :id, :item_id
-
-    # Update the fields of an item state.
-    #
-    # Supply a hash of string keyed data retrieved from the Trello API representing
-    # an item state.
-    def update_fields(fields)
-      attributes[:id]      = fields['id'] || attributes[:id]
-      attributes[:state]   = fields['state'] || attributes[:state]
-      attributes[:item_id] = fields['idCheckItem'] || attributes[:item_id]
-      self
+    schema do
+      #Readonly
+      attribute :id, remote_key: 'idCheckItem', readonly: true, primary_key: true
+      attribute :state
     end
 
+    validates_presence_of :id
+
     # Return the item this state belongs to.
     def item
-      Item.find(item_id)
+      Item.find(id)
     end
   end
 end

data/lib/trello/json_utils.rb

@@ -7,7 +7,7 @@ module Trello
 
     module InstanceMethods
       def from_response(*args)
-        update_fields parse_json(*args)
+        initialize_fields parse_json(*args)
       end
 
       def parse_json(*args)

data/lib/trello/label.rb

@@ -6,104 +6,34 @@ module Trello
   #   @return [String]
   # @!attribute [rw] color
   #   @return [String]
+  # @!attribute [rw] name
+  #   @return [String]
+  # @!attribute [rw] board_id
+  #   @return [String]
   class Label < BasicData
-    register_attributes :id, :name, :board_id, :uses,
-      readonly: [ :id, :uses, :board_id ]
-    validates_presence_of :id, :board_id, :name
-    validates_length_of   :name,        in: 1..16384
-
-    SYMBOL_TO_STRING = {
-      id: 'id',
-      name: 'name',
-      board_id: 'idBoard',
-      color: 'color',
-      uses: 'uses'
-    }
-
-    class << self
-      VALID_LABEL_COLOURS = %w{green yellow orange red purple blue sky lime pink black} << ''
-
-      # Find a specific label by its id.
-      def find(id, params = {})
-        client.find(:label, id, params)
-      end
-
-      # Create a new label and save it on Trello.
-      def create(options)
-        client.create(:label,
-          'name' => options[:name],
-          'idBoard' => options[:board_id],
-          'color'   => options[:color],
-        )
-      end
-
-      # Label colours
-      def label_colours
-        VALID_LABEL_COLOURS
-      end
-    end
-
-    define_attribute_methods [:color]
 
-    def color
-      @__attributes[:color]
-    end
+    schema do
+      # Readonly
+      attribute :id, readonly: true, primary_key: true
 
-    def color= colour
-      unless Label.label_colours.include? colour
-        errors.add(:label, "color '#{colour}' does not exist")
-        return Trello.logger.warn "The label colour '#{colour}' does not exist."
-      end
+      # Writable
+      attribute :name
+      attribute :color
 
-      self.send(:"color_will_change!") unless colour == @__attributes[:color]
-      @__attributes[:color] = colour
+      # Writable but for create only
+      attribute :board_id, create_only: true, remote_key: 'idBoard'
     end
 
-    # Update the fields of a label.
-    #
-    # Supply a hash of stringkeyed data retrieved from the Trello API representing
-    # a label.
-    def update_fields(fields)
-      attributes[:id] = fields['id'] || attributes[:id]
-      attributes[:name]  = fields['name'] || fields[:name] || attributes[:name]
-      attributes[:color] = fields['color'] || fields[:color] || attributes[:color]
-      attributes[:board_id] = fields['idBoard'] || fields[:board_id] || attributes[:board_id]
-      attributes[:uses] = fields['uses'] if fields.has_key?('uses')
-      self
-    end
+    validates_presence_of :id, :board_id, :name
+    validates_length_of :name, in: 1..16384
 
     # Returns a reference to the board this label is currently connected.
     one :board, path: :boards, using: :board_id
 
-    # Saves a record.
-    def save
-      # If we have an id, just update our fields.
-      return update! if id
-
-      from_response client.post("/labels", {
-        name:   name,
-        color:   color,
-        idBoard: board_id,
-      })
-    end
-
-    # Update an existing record.
-    # Warning, this updates all fields using values already in memory. If
-    # an external resource has updated these fields, you should refresh!
-    # this object before making your changes, and before updating the record.
-    def update!
-      @previously_changed = changes
-      # extract only new values to build payload
-      payload = Hash[changes.map { |key, values| [SYMBOL_TO_STRING[key.to_sym].to_sym, values[1]] }]
-      @changed_attributes.try(:clear)
-      changes_applied if respond_to?(:changes_applied)
-
-      client.put("/labels/#{id}", payload)
-    end
-
     # Delete this label
     def delete
       client.delete("/labels/#{id}")
     end
+
   end
 end

data/lib/trello/label_name.rb

@@ -1,31 +1,28 @@
 module Trello
-
   # A colored Label attached to a card
+  #
+  # @!attribute [r] yellow
+  # @!attribute [r] red
+  # @!attribute [r] orange
+  # @!attribute [r] green
+  # @!attribute [r] purple
+  # @!attribute [r] blue
+  # @!attribute [r] sky
+  # @!attribute [r] pink
+  # @!attribute [r] lime
+  # @!attribute [r] black
   class LabelName < BasicData
-    register_attributes :yellow, :red, :orange, :green, :purple, :blue, :sky, :pink, :lime, :black
-
-    # Update the fields of a label.
-    #
-    # Supply a hash of stringkeyed data retrieved from the Trello API representing
-    # a label.
-    def update_fields(fields)
-      attributes[:yellow]  = fields['yellow'] || attributes[:yellow]
-      attributes[:red] = fields['red'] || attributes[:red]
-      attributes[:orange] = fields['orange'] || attributes[:orange]
-      attributes[:green] = fields['green'] || attributes[:green]
-      attributes[:purple] = fields['purple'] || attributes[:purple]
-      attributes[:blue] = fields['blue'] || attributes[:blue]
-      attributes[:sky] = fields['sky'] || attributes[:sky]
-      attributes[:pink] = fields['pink'] || attributes[:pink]
-      attributes[:lime] = fields['lime'] || attributes[:lime]
-      attributes[:black] = fields['black'] || attributes[:black]
-
-      self
+    schema do
+      attribute :yellow, readonly: true
+      attribute :red, readonly: true
+      attribute :orange, readonly: true
+      attribute :green, readonly: true
+      attribute :purple, readonly: true
+      attribute :blue, readonly: true
+      attribute :sky, readonly: true
+      attribute :pink, readonly: true
+      attribute :lime, readonly: true
+      attribute :black, readonly: true
     end
-
-    one :board, path: :boards, using: :board_id
-
-    many :cards, filter: :all
-
   end
 end

data/lib/trello/list.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Trello
   # A List is a container which holds cards. Lists are items on a board.
   #
@@ -11,10 +13,30 @@ module Trello
   #   @return [String] A 24-character hex string
   # @!attribute [rw] pos
   #   @return [Object]
+  # @!attribute [w] source_list_id
+  #   @return [String]
+  # @!attribute [w] subscribed
+  #   @return [Boolean]
   class List < BasicData
-    register_attributes :id, :name, :closed, :board_id, :pos, :source_list_id, readonly: [ :id, :board_id ]
+    schema do
+      # Readonly
+      attribute :id, readonly: true, primary_key: true
+
+      # Writable
+      attribute :name
+      attribute :pos
+      attribute :board_id, remote_key: 'idBoard'
+
+      # Writable but for create only
+      attribute :source_list_id, create_only: true, remote_key: 'idListSource'
+
+      # Writable but for update only
+      attribute :closed, update_only: true
+      attribute :subscribed, update_only: true
+    end
+
     validates_presence_of :id, :name, :board_id
-    validates_length_of   :name, in: 1..16384
+    validates_length_of   :name, in: 1..16_384
 
     include HasActions
 
@@ -26,48 +48,6 @@ module Trello
       def find(id, params = {})
         client.find(:list, id, params)
       end
-
-      def create(options)
-        client.create(:list,
-            'name'         => options[:name],
-            'idBoard'      => options[:board_id],
-            'pos'          => options[:pos],
-            'idListSource' => options[:source_list_id])
-      end
-    end
-
-    # Updates the fields of a list.
-    #
-    # Supply a hash of string keyed data retrieved from the Trello API representing
-    # a List.
-    def update_fields(fields)
-      attributes[:id]             = fields['id'] || attributes[:id]
-      attributes[:name]           = fields['name'] || fields[:name] || attributes[:name]
-      attributes[:closed]         = fields['closed'] if fields.has_key?('closed')
-      attributes[:board_id]       = fields['idBoard'] || fields[:board_id] || attributes[:board_id]
-      attributes[:pos]            = fields['pos'] || fields[:pos] || attributes[:pos]
-      attributes[:source_list_id] = fields['idListSource'] || fields[:source_list_id] || attributes[:source_list_id]
-      self
-    end
-
-    def save
-      return update! if id
-
-      from_response client.post("/lists", {
-        name: name,
-        closed: closed || false,
-        idBoard: board_id,
-        pos: pos,
-        idListSource: source_list_id
-      })
-    end
-
-    def update!
-      client.put("/lists/#{id}", {
-        name: name,
-        closed: closed,
-        pos: pos
-      })
     end
 
     # Check if the list is not active anymore.
@@ -101,6 +81,13 @@ module Trello
        })
     end
 
+    # Move list to another board. Accepts a `Trello::Board` or an id string.
+    def move_to_board(board)
+      board = board.id unless board.is_a?(String)
+
+      client.put("/lists/#{id}/idBoard", value: board)
+    end
+
     # Archives all the cards of the list
     def archive_all_cards
       client.post("/lists/#{id}/archiveAllCards")

data/lib/trello/member.rb

@@ -17,8 +17,84 @@ module Trello
   #   @return [String]
   # @!attribute [r] url
   #   @return [String]
+  # @!attribute [r] prefs
+  #   @return [Hash]
+  # @!attribute [r] activity_blocked
+  #   @return [Boolean]
+  # @!attribute [r] bio_data
+  #   @return [Hash]
+  # @!attribute [r] confirmed
+  #   @return [Boolean]
+  # @!attribute [r] enterprise_id
+  #   @return [String]
+  # @!attribute [r] deactivated_enterprise_ids
+  #   @return [String]
+  # @!attribute [r] referrer_member_id
+  #   @return [String]
+  # @!attribute [r] admin_orgs_perm_id
+  #   @return [String]
+  # @!attribute [r] member_type
+  #   @return [String]
+  # @!attribute [r] non_public
+  #   @return [Hash]
+  # @!attribute [r] non_public_available
+  #   @return [Hash]
+  # @!attribute [r] products
+  #   @return [Array]
+  # @!attribute [r] status
+  #   @return [String]
+  # @!attribute [r] board_ids
+  #   @return [Array<String>]
+  # @!attribute [r] organization_ids
+  #   @return [Array<String>]
+  # @!attribute [r] admin_enterprise_ids
+  #   @return [Array<String>]
+  # @!attribute [r] limits
+  #   @return [Hash]
+  # @!attribute [r] login_types
+  #   @return [Array<String>]
+  # @!attribute [r] marketing_opt_in
+  #   @return [Hash]
+  # @!attribute [r] pinned_board_ids
+  #   @return [String]
+  # @!attribute [w] avatar_source
+  #   @return [String]
   class Member < BasicData
-    register_attributes :id, :username, :email, :full_name, :initials, :avatar_id, :bio, :url, readonly: [ :id, :username, :avatar_id, :url ]
+    schema do
+      # readonly
+      attribute :id, readonly: true, primary_key: true
+      attribute :email, readonly: true
+      attribute :avatar_id, readonly: true, remote_key: 'avatarHash'
+      attribute :url, readonly: true
+      attribute :prefs, readonly: true
+      attribute :activity_blocked, readonly: true, remote_key: 'activityBlocked'
+      attribute :bio_data, readonly: true, remote_key: 'bioData'
+      attribute :confirmed, readonly: true
+      attribute :enterprise_id, readonly: true, remote_key: 'idEnterprise'
+      attribute :deactivated_enterprise_ids, readonly: true, remote_key: 'idEnterprisesDeactivated'
+      attribute :referrer_member_id, readonly: true, remote_key: 'idMemberReferrer'
+      attribute :admin_orgs_perm_id, readonly: true, remote_key: 'idPremOrgsAdmin'
+      attribute :member_type, readonly: true, remote_key: 'memberType'
+      attribute :non_public, readonly: true, remote_key: 'nonPublic'
+      attribute :non_public_available, readonly: true, remote_key: 'nonPublicAvailable'
+      attribute :products, readonly: true
+      attribute :status, readonly: true
+      attribute :board_ids, readonly: true, remote_key: 'idBoards'
+      attribute :organization_ids, readonly: true, remote_key: 'idOrganizations'
+      attribute :admin_enterprise_ids, readonly: true, remote_key: 'idEnterprisesAdmin'
+      attribute :limits, readonly: true
+      attribute :login_types, readonly: true, remote_key: 'loginTypes'
+      attribute :marketing_opt_in, readonly: true, remote_key: 'marketingOptIn'
+      attribute :pinned_board_ids, readonly: true, remote_key: 'idBoardsPinned'
+
+      # writable
+      attribute :username
+      attribute :full_name, remote_key: 'fullName'
+      attribute :initials
+      attribute :bio
+      attribute :avatar_source, remote_key: 'avatarSource'
+    end
+
     validates_presence_of :id, :username
     validates_length_of   :full_name, minimum: 4
     validates_length_of   :bio,       maximum: 16384
@@ -34,22 +110,6 @@ module Trello
       end
     end
 
-    # Update the fields of a member.
-    #
-    # Supply a hash of string keyed data retrieved from the Trello API representing
-    # an Member.
-    def update_fields(fields)
-      attributes[:id]        = fields['id'] || attributes[:id]
-      attributes[:full_name] = fields['fullName'] || attributes[:full_name]
-      attributes[:email]     = fields['email'] || attributes[:email]
-      attributes[:username]  = fields['username'] || attributes[:username]
-      attributes[:initials]  = fields['initials'] || attributes[:initials]
-      attributes[:avatar_id] = fields['avatarHash'] || attributes[:avatar_id]
-      attributes[:bio]       = fields['bio'] || attributes[:bio]
-      attributes[:url]       = fields['url'] || attributes[:url]
-      self
-    end
-
     # Retrieve a URL to the avatar.
     #
     # Valid values for options are:
@@ -90,21 +150,6 @@ module Trello
     # Returns a list of notifications for the user
     many :notifications
 
-    def save
-      @previously_changed = changes
-      @changed_attributes.try(:clear)
-      changes_applied if respond_to?(:changes_applied)
-
-      return update! if id
-    end
-
-    def update!
-      from_response client.put(request_prefix, {
-        fullName: full_name,
-        bio: bio
-      })
-    end
-
     # :nodoc:
     def request_prefix
       "/members/#{id}"