ruby-trello 2.0.0 → 2.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: aef36c181b18bcb01b8a45c21aad954536f40188
-  data.tar.gz: 1271b238eae444a32c6c84ea9c3cf8c26fe50a98
+SHA256:
+  metadata.gz: 77b35b3b765d1056e69ec92a4d403439849c66edfdb01747d590d79026b7ac67
+  data.tar.gz: 35173e15c5deb8a4c8404831c2cb524e3b8283cc60ac69d0d9519b9b9b22d66c
 SHA512:
-  metadata.gz: 86b932d63a9e11a9d63ab20233e67aeb241cc5203bfd689d56f5323c8a31dfbb5dfd1d6b5873571d50fad68487033ead6b70282b7e52557ffd72b7bb81cd70fa
-  data.tar.gz: b03c3113250ff01a490bb39304e7d7ed4dc7a64f61a6c7feb923138a7621e0034b73bfeee58c796e017485a0d2b0d5f46c80ab907c68b19c0f2f19dc503d91bc
+  metadata.gz: 155bdc62579174c62bf3ade75dd2921cdfb29aa5ced74176d90ea62a94fa279f0e164a54cd0970392296fa04dec7d53a293d45bb188f9eeac3f662d23109f309
+  data.tar.gz: d4e75692b37cd6d1befa4d14dff5bce2acdca231a90287e898af1cadcaec4d5f006e8fb6ed05aeeb6982ab741a1f93e003bda3b8f3aa1eb9974fcdff9a6fd920

data/README.md

@@ -4,7 +4,7 @@
 [![Build Status](https://secure.travis-ci.org/jeremytregunna/ruby-trello.png)](http://travis-ci.org/jeremytregunna/ruby-trello) [![Dependency Status](https://gemnasium.com/jeremytregunna/ruby-trello.png)](https://gemnasium.com/jeremytregunna/ruby-trello)
 [![Code Climate](https://codeclimate.com/github/jeremytregunna/ruby-trello/badges/gpa.svg)](https://codeclimate.com/github/jeremytregunna/ruby-trello)
 
-This library implements the [Trello](http://www.trello.com/) [API](http://trello.com/api).
+This library implements the [Trello](http://www.trello.com/) [API](https://developers.trello.com/).
 
 Trello is an awesome tool for organization. Not just aimed at developers, but everybody.
 Seriously, [check it out](http://www.trello.com/).
@@ -22,12 +22,12 @@ to, please [let us know](https://trello.com/card/spot-a-bug-report-it/4f092b2ee2
 
 Supports Ruby 2.1.0 or newer.
 
-Use version 1.3.0 or earlier for ruby 1.9.3 support.
-Use version 1.4.x or earlier for ruby 2.0.0 support.
+Use version 1.3.0 or earlier for Ruby 1.9.3 support.
+Use version 1.4.x or earlier for Ruby 2.0.0 support.
 
 ## Configuration
 
-####Basic authorization:
+#### Basic authorization:
 
 1. Get your API public key from Trello via the irb console:
 
@@ -50,7 +50,7 @@ Trello.configure do |config|
 end
 ```
 
-####2-legged OAuth authorization
+#### 2-legged OAuth authorization
 
 ```ruby
 Trello.configure do |config|
@@ -61,7 +61,7 @@ Trello.configure do |config|
 end
 ```
 
-####3-legged OAuth authorization
+#### 3-legged OAuth authorization
 
 ```ruby
 Trello.configure do |config|
@@ -72,18 +72,67 @@ Trello.configure do |config|
 end
 ```
 
-All the calls this library make to Trello require authentication using these keys. Be sure to protect them.
+All the calls this library makes to Trello require authentication using these keys. Be sure to protect them.
 
-So lets say you want to get information about the user *bobtester*. We can do something like this:
+#### Usage
+
+So let's say you want to get information about the user *bobtester*. We can do something like this:
 
 ```ruby
 bob = Trello::Member.find("bobtester")
+
 # Print out his name
 puts bob.full_name # "Bob Tester"
+
 # Print his bio
 puts bob.bio # A wonderfully delightful test user
+
 # How about a list of his boards?
 bob.boards
+
+# And then to read the lists of the first board do : 
+bob.boards.first.lists
+```
+
+##### Accessing specific items
+
+There is no find by name method in the trello API, to access a specific item, you have to know it's ID.
+The best way is to pretty print the elements and then find the id of the element you are looking for.
+
+```ruby
+# With bob
+pp bob.boards # Will pretty print all boards, allowing us to find our board id
+
+# We can now access it's lists
+pp Trello::Board.find( board_id ).lists # will pretty print all lists. Let's get the list id
+
+# We can now access the cards of the list
+pp Trello::List.find( list_id ).cards
+
+# We can now access the checklists of the card
+pp Trello::Card.find( card_id ).checklists
+
+# and so on ...
+```
+
+##### Changing a checkbox state
+```ruby
+# First get your checklist id 
+checklist = Trello::Checklist.find( checklist_id )
+
+# At this point, there is no more ids. To get your checklist item, 
+# you have to know it's position (same as in the trello interface).
+# Let's take the first
+checklist_item = checklist.items.first
+
+# Then we can read the status
+checklist_item.state # return 'complete' or 'incomplete'
+
+# We can update it (note we call update_item_state from checklist, not from checklist_item)
+checklist.update_item_state( checklist_item.id, 'complete' ) # or 'incomplete'
+
+# You can also use true or false instead of 'complete' or 'incomplete'
+checklist.update_item_state( checklist_item.id, true ) # or false
 ```
 
 #### Multiple Users

data/lib/trello.rb

@@ -37,6 +37,7 @@ require 'addressable/uri'
 #
 # Feel free to {peruse and participate in our Trello board}[https://trello.com/board/ruby-trello/4f092b2ee23cb6fe6d1aaabd]. It's completely open to the public.
 module Trello
+  autoload :Error,             'trello/error'
   autoload :Action,            'trello/action'
   autoload :Comment,           'trello/comment'
   autoload :Association,       'trello/association'
@@ -75,9 +76,6 @@ module Trello
   # Version of the Trello API that we use by default.
   API_VERSION = 1
 
-  # Raise this when we hit a Trello error.
-  Error = Class.new(StandardError)
-
   # This specific error is thrown when your access token is invalid. You should get a new one.
   InvalidAccessToken = Class.new(Error)
 

data/lib/trello/action.rb

@@ -38,12 +38,12 @@ module Trello
     # Supply a hash of string keyed data retrieved from the Trello API representing
     # an Action.
     def update_fields(fields)
-      attributes[:id]                 = fields['id']
-      attributes[:type]               = fields['type']
-      attributes[:data]               = fields['data']
-      attributes[:date]               = Time.iso8601(fields['date'])
-      attributes[:member_creator_id]  = fields['idMemberCreator']
-      attributes[:member_participant] = fields['member']
+      attributes[:id]                 = fields['id'] || attributes[:id]
+      attributes[:type]               = fields['type'] || attributes[:type]
+      attributes[:data]               = fields['data'] || attributes[:data]
+      attributes[:date]               = Time.iso8601(fields['date']) rescue nil if fields.has_key?('date')
+      attributes[:member_creator_id]  = fields['idMemberCreator'] || attributes[:member_creator_id]
+      attributes[:member_participant] = fields['member'] || attributes[:member_participant]
       self
     end
 

data/lib/trello/attachment.rb

@@ -7,6 +7,8 @@ module Trello
   #   @return [String]
   # @!attribute url
   #   @return [String]
+  # @!attribute pos
+  #   @return [Float]
   # @!attribute bytes
   #   @return [Fixnum]
   # @!attribute date
@@ -16,21 +18,22 @@ module Trello
   # @!attribute mime_type
   #   @return [String]
   class Attachment < BasicData
-    register_attributes :name, :id, :url, :bytes, :member_id, :date, :is_upload, :mime_type, :previews
+    register_attributes :name, :id, :pos, :url, :bytes, :member_id, :date, :is_upload, :mime_type, :previews
     # Update the fields of an attachment.
     #
     # Supply a hash of stringkeyed data retrieved from the Trello API representing
     # an attachment.
     def update_fields(fields)
-      attributes[:name]      = fields['name']
-      attributes[:id]        = fields['id']
-      attributes[:url]       = fields['url']
-      attributes[:bytes]     = fields['bytes'].to_i
-      attributes[:member_id] = fields['idMember']
-      attributes[:date]      = Time.parse(fields['date'])
-      attributes[:is_upload] = fields['isUpload']
-      attributes[:mime_type] = fields['mimeType']
-      attributes[:previews]  = fields['previews']
+      attributes[:name]      = fields['name'] || attributes[:name]
+      attributes[:id]        = fields['id'] || attributes[:id]
+      attributes[:pos]       = fields['pos'] || attributes[:pos]
+      attributes[:url]       = fields['url'] || attributes[:url]
+      attributes[:bytes]     = fields['bytes'].to_i || attributes[:bytes]
+      attributes[:member_id] = fields['idMember'] || attributes[:member_id]
+      attributes[:date]      = Time.parse(fields['date']).presence || attributes[:date]
+      attributes[:is_upload] = fields['isUpload'] if fields.has_key?('isUpload')
+      attributes[:mime_type] = fields['mimeType'] || attributes[:mime_type]
+      attributes[:previews]  = fields['previews'] if fields.has_key?('previews')
       self
     end
   end

data/lib/trello/board.rb

@@ -151,12 +151,22 @@ module Trello
     #
     # This method, when called, can take a hash table with a filter key containing any
     # of the following values:
-    #    :filter => [ :none, :normal, :owners, :all ] # default :all
+    #    :filter => [ :none, :normal, :owners, :admins, :all ] # default :all
     many :members, filter: :all
 
+    # Returns a list of checklists associated with the board.
+    #
+    # The options hash may have a filter key which can have its value set as any
+    # of the following values:
+    #    :filter => [ :none, :all ] # default :all
+    many :checklists, filter: :all
+
     # Returns a reference to the organization this board belongs to.
     one :organization, path: :organizations, using: :organization_id
 
+    # Returns a list of plugins associated with the board
+    many :plugin_data, path: "pluginData"
+
     def labels(params = {})
       # Set the limit to as high as possible given there is no pagination in this API.
       params[:limit] = 1000 unless params[:limit]

data/lib/trello/card.rb

@@ -169,28 +169,29 @@ module Trello
     #
     # @return [Trello::Card] self
     def update_fields(fields)
-      attributes[:id]                     = fields[SYMBOL_TO_STRING[:id]]
-      attributes[:short_id]               = fields[SYMBOL_TO_STRING[:short_id]]
-      attributes[:name]                   = fields[SYMBOL_TO_STRING[:name]] || fields[:name]
-      attributes[:desc]                   = fields[SYMBOL_TO_STRING[:desc]] || fields[:desc]
-      attributes[:due]                    = Time.iso8601(fields[SYMBOL_TO_STRING[:due]]) rescue nil
-      attributes[:due]                    ||= fields[:due]
-      attributes[:due_complete]           = fields[SYMBOL_TO_STRING[:due_complete]] || false
-      attributes[:closed]                 = fields[SYMBOL_TO_STRING[:closed]]
-      attributes[:url]                    = fields[SYMBOL_TO_STRING[:url]]
-      attributes[:short_url]              = fields[SYMBOL_TO_STRING[:short_url]]
-      attributes[:board_id]               = fields[SYMBOL_TO_STRING[:board_id]]
-      attributes[:member_ids]             = fields[SYMBOL_TO_STRING[:member_ids]] || fields[:member_ids]
-      attributes[:list_id]                = fields[SYMBOL_TO_STRING[:list_id]] || fields[:list_id]
-      attributes[:pos]                    = fields[SYMBOL_TO_STRING[:pos]] || fields[:pos]
-      attributes[:labels]                 = (fields[SYMBOL_TO_STRING[:labels]] || []).map { |lbl| Trello::Label.new(lbl) }
-      attributes[:card_labels]            = fields[SYMBOL_TO_STRING[:card_labels]] || fields[:card_labels]
-      attributes[:last_activity_date]     = Time.iso8601(fields[SYMBOL_TO_STRING[:last_activity_date]]) rescue nil
-      attributes[:cover_image_id]         = fields[SYMBOL_TO_STRING[:cover_image_id]]
-      attributes[:badges]                 = fields[SYMBOL_TO_STRING[:badges]]
-      attributes[:card_members]           = fields[SYMBOL_TO_STRING[:card_members]]
-      attributes[:source_card_id]         = fields[SYMBOL_TO_STRING[:source_card_id]] || fields[:source_card_id]
-      attributes[:source_card_properties] = fields[SYMBOL_TO_STRING[:source_card_properties]] || fields[:source_card_properties]
+      attributes[:id]                     = fields[SYMBOL_TO_STRING[:id]] || attributes[:id]
+      attributes[:short_id]               = fields[SYMBOL_TO_STRING[:short_id]] || attributes[:short_id]
+      attributes[:name]                   = fields[SYMBOL_TO_STRING[:name]] || fields[:name] || attributes[:name]
+      attributes[:desc]                   = fields[SYMBOL_TO_STRING[:desc]] || fields[:desc] || attributes[:desc]
+      attributes[:due]                    = Time.iso8601(fields[SYMBOL_TO_STRING[:due]]) rescue nil if fields.has_key?(SYMBOL_TO_STRING[:due])
+      attributes[:due]                    = fields[:due] if fields.has_key?(:due)
+      attributes[:due_complete]           = fields[SYMBOL_TO_STRING[:due_complete]] if fields.has_key?(SYMBOL_TO_STRING[:due_complete])
+      attributes[:due_complete]           ||= false
+      attributes[:closed]                 = fields[SYMBOL_TO_STRING[:closed]] if fields.has_key?(SYMBOL_TO_STRING[:closed])
+      attributes[:url]                    = fields[SYMBOL_TO_STRING[:url]] || attributes[:url]
+      attributes[:short_url]              = fields[SYMBOL_TO_STRING[:short_url]] || attributes[:short_url]
+      attributes[:board_id]               = fields[SYMBOL_TO_STRING[:board_id]] || attributes[:board_id]
+      attributes[:member_ids]             = fields[SYMBOL_TO_STRING[:member_ids]] || fields[:member_ids] || attributes[:member_ids]
+      attributes[:list_id]                = fields[SYMBOL_TO_STRING[:list_id]] || fields[:list_id] || attributes[:list_id]
+      attributes[:pos]                    = fields[SYMBOL_TO_STRING[:pos]] || fields[:pos] || attributes[:pos]
+      attributes[:labels]                 = (fields[SYMBOL_TO_STRING[:labels]] || []).map { |lbl| Trello::Label.new(lbl) }.presence || attributes[:labels].presence || []
+      attributes[:card_labels]            = fields[SYMBOL_TO_STRING[:card_labels]] || fields[:card_labels] || attributes[:card_labels]
+      attributes[:last_activity_date]     = Time.iso8601(fields[SYMBOL_TO_STRING[:last_activity_date]]) rescue nil if fields.has_key?(SYMBOL_TO_STRING[:last_activity_date])
+      attributes[:cover_image_id]         = fields[SYMBOL_TO_STRING[:cover_image_id]] || attributes[:cover_image_id]
+      attributes[:badges]                 = fields[SYMBOL_TO_STRING[:badges]] || attributes[:badges]
+      attributes[:card_members]           = fields[SYMBOL_TO_STRING[:card_members]] || attributes[:card_members]
+      attributes[:source_card_id]         = fields[SYMBOL_TO_STRING[:source_card_id]] || fields[:source_card_id] || attributes[:source_card_id]
+      attributes[:source_card_properties] = fields[SYMBOL_TO_STRING[:source_card_properties]] || fields[:source_card_properties] || attributes[:source_card_properties]
       self
     end
 
@@ -342,6 +343,16 @@ module Trello
       end
     end
 
+    # Moves this card to the given list no matter which board it is on
+    def move_to_list_on_any_board(list_id)
+      list = List.find(list_id)
+      if board.id == list.board_id
+        move_to_list(list_id)
+      else
+        move_to_board(Board.find(list.board_id), list)
+      end
+    end
+
     # Move this card to the given board (and optional list on this board)
     def move_to_board(new_board, new_list = nil)
       unless board_id == new_board.id

data/lib/trello/checklist.rb

@@ -45,17 +45,17 @@ module Trello
     # Supply a hash of string keyed data retrieved from the Trello API representing
     # a checklist.
     def update_fields(fields)
-      attributes[:id] = fields['id']
-      attributes[:name] = fields['name'] || fields[:name]
-      attributes[:description] = fields['desc']
-      attributes[:closed] = fields['closed']
-      attributes[:url] = fields['url']
-      attributes[:check_items] = fields['checkItems']
-      attributes[:position] = fields['pos']
-      attributes[:board_id] = fields['idBoard']
-      attributes[:card_id] = fields['idCard'] || fields[:card_id]
-      attributes[:list_id] = fields['idList']
-      attributes[:member_ids] = fields['idMembers']
+      attributes[:id] = fields['id'] || attributes[:id]
+      attributes[:name] = fields['name'] || fields[:name] || attributes[:name]
+      attributes[:description] = fields['desc'] || attributes[:description]
+      attributes[:closed] = fields['closed'] if fields.has_key?('closed')
+      attributes[:url] = fields['url'] || attributes[:url]
+      attributes[:check_items] = fields['checkItems'] if fields.has_key?('checkItems')
+      attributes[:position] = fields['pos'] || attributes[:position]
+      attributes[:board_id] = fields['idBoard'] || attributes[:board_id]
+      attributes[:card_id] = fields['idCard'] || fields[:card_id] || attributes[:card_id]
+      attributes[:list_id] = fields['idList'] || attributes[:list_id]
+      attributes[:member_ids] = fields['idMembers'] || attributes[:member_ids]
       self
     end
 
@@ -109,9 +109,10 @@ module Trello
 
     # Update a checklist item's state, e.g.: "complete" or "incomplete"
     def update_item_state(item_id, state)
+      state = ( state ? 'complete' : 'incomplete' ) unless state.is_a?(String)
       client.put(
-        "/cards/#{card_id}/checklist/#{id}/checkItem/#{item_id}/state",
-        value: state,
+          "/cards/#{card_id}/checkItem/#{item_id}",
+          state: state
       )
     end
 

data/lib/trello/client.rb

@@ -96,7 +96,7 @@ module Trello
 
       unless [200, 201].include? response.code
         Trello.logger.error("[#{response.code} #{name.to_s.upcase} #{uri}]: #{response.body}")
-        raise Error, response.body
+        raise Error.new(response.body, response.code)
       end
 
       response.body

data/lib/trello/comment.rb

@@ -27,10 +27,10 @@ module Trello
     # Supply a hash of string keyed data retrieved from the Trello API representing
     # a Comment.
     def update_fields(fields)
-      attributes[:action_id]          = fields['id']
-      attributes[:text]               = fields['data']['text']
-      attributes[:date]               = Time.iso8601(fields['date'])
-      attributes[:member_creator_id]  = fields['idMemberCreator']
+      attributes[:action_id]          = fields['id'] || attributes[:action_id]
+      attributes[:text]               = fields['data']['text'] || attributes[:text]
+      attributes[:date]               = Time.iso8601(fields['date']) if fields.has_key?('date')
+      attributes[:member_creator_id]  = fields['idMemberCreator'] || attributes[:member_creator_id]
       self
     end
 

data/lib/trello/error.rb

@@ -0,0 +1,12 @@
+module Trello
+  class Error < StandardError
+
+    attr_reader :status_code
+
+    def initialize(message, status_code=nil)
+      @status_code = status_code
+      super(message)
+    end
+
+  end
+end

data/lib/trello/item.rb

@@ -20,13 +20,13 @@ module Trello
     # Supply a hash of string keyed data retrieved from the Trello API representing
     # an item.
     def update_fields(fields)
-      attributes[:id]           = fields['id']
-      attributes[:card_id]      = fields['idCard']
-      attributes[:checklist_id] = fields['idChecklist']
-      attributes[:name]         = fields['name']
-      attributes[:type]         = fields['type']
-      attributes[:state]        = fields['state']
-      attributes[:pos]          = fields['pos']
+      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
     end
 

data/lib/trello/item_state.rb

@@ -16,9 +16,9 @@ module Trello
     # 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[:state]   = fields['state']
-      attributes[:item_id] = fields['idCheckItem']
+      attributes[:id]      = fields['id'] || attributes[:id]
+      attributes[:state]   = fields['state'] || attributes[:state]
+      attributes[:item_id] = fields['idCheckItem'] || attributes[:item_id]
       self
     end
 

data/lib/trello/label.rb

@@ -64,11 +64,11 @@ module Trello
     # Supply a hash of stringkeyed data retrieved from the Trello API representing
     # a label.
     def update_fields(fields)
-      attributes[:id] = fields['id']
-      attributes[:name]  = fields['name'] || fields[:name]
-      attributes[:color] = fields['color'] || fields[:color]
-      attributes[:board_id] = fields['idBoard'] || fields[:board_id]
-      attributes[:uses] = fields['uses']
+      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
 

data/lib/trello/label_name.rb

@@ -9,16 +9,16 @@ module Trello
     # Supply a hash of stringkeyed data retrieved from the Trello API representing
     # a label.
     def update_fields(fields)
-      attributes[:yellow]  = fields['yellow']
-      attributes[:red] = fields['red']
-      attributes[:orange] = fields['orange']
-      attributes[:green] = fields['green']
-      attributes[:purple] = fields['purple']
-      attributes[:blue] = fields['blue']
-      attributes[:sky] = fields['sky']
-      attributes[:pink] = fields['pink']
-      attributes[:lime] = fields['lime']
-      attributes[:black] = fields['black']
+      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
     end

data/lib/trello/list.rb

@@ -41,12 +41,12 @@ module Trello
     # Supply a hash of string keyed data retrieved from the Trello API representing
     # a List.
     def update_fields(fields)
-      attributes[:id]             = fields['id']
-      attributes[:name]           = fields['name'] || fields[:name]
-      attributes[:closed]         = fields['closed']
-      attributes[:board_id]       = fields['idBoard'] || fields[:board_id]
-      attributes[:pos]            = fields['pos'] || fields[:pos]
-      attributes[:source_list_id] = fields['idListSource'] || fields[:source_list_id]
+      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
 

data/lib/trello/member.rb

@@ -39,14 +39,14 @@ module Trello
     # Supply a hash of string keyed data retrieved from the Trello API representing
     # an Member.
     def update_fields(fields)
-      attributes[:id]        = fields['id']
-      attributes[:full_name] = fields['fullName']
-      attributes[:email]     = fields['email']
-      attributes[:username]  = fields['username']
-      attributes[:initials]  = fields['initials']
-      attributes[:avatar_id] = fields['avatarHash']
-      attributes[:bio]       = fields['bio']
-      attributes[:url]       = fields['url']
+      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
 

data/lib/trello/notification.rb

@@ -25,12 +25,12 @@ module Trello
     end
 
     def update_fields(fields)
-      attributes[:id]                = fields['id']
-      attributes[:unread]            = fields['unread']
-      attributes[:type]              = fields['type']
-      attributes[:date]              = fields['date']
-      attributes[:data]              = fields['data']
-      attributes[:member_creator_id] = fields['idMemberCreator']
+      attributes[:id]                = fields['id'] || attributes[:id]
+      attributes[:unread]            = fields['unread'] if fields.has_key?('unread')
+      attributes[:type]              = fields['type'] || attributes[:type]
+      attributes[:date]              = fields['date'] || attributes[:date]
+      attributes[:data]              = fields['data'] || attributes[:data]
+      attributes[:member_creator_id] = fields['idMemberCreator'] || attributes[:member_creator_id]
       self
     end
 

data/lib/trello/organization.rb

@@ -34,17 +34,17 @@ module Trello
     # Supply a hash of string keyed data retrieved from the Trello API representing
     # an Organization.
     def update_fields(fields)
-      attributes[:id]                           = fields['id']
-      attributes[:name]                         = fields['name']
-      attributes[:display_name]                 = fields['displayName']
-      attributes[:description]                  = fields['desc']
-      attributes[:url]                          = fields['url']
-      attributes[:invited]                      = fields['invited']
-      attributes[:website]                      = fields['website']
-      attributes[:logo_hash]                    = fields['logoHash']
-      attributes[:billable_member_count]        = fields['billableMemberCount']
-      attributes[:active_billable_member_count] = fields['activeBillableMemberCount']
-      attributes[:memberships]                  = fields['memberships']
+      attributes[:id]                           = fields['id'] || attributes[:id]
+      attributes[:name]                         = fields['name'] || attributes[:name]
+      attributes[:display_name]                 = fields['displayName'] || attributes[:display_name]
+      attributes[:description]                  = fields['desc'] || attributes[:description]
+      attributes[:url]                          = fields['url'] || attributes[:url]
+      attributes[:invited]                      = fields['invited'] if fields.has_key?('invited')
+      attributes[:website]                      = fields['website'] || attributes[:website]
+      attributes[:logo_hash]                    = fields['logoHash'] || attributes[:logo_hash]
+      attributes[:billable_member_count]        = fields['billableMemberCount'] || attributes[:billable_member_count]
+      attributes[:active_billable_member_count] = fields['activeBillableMemberCount'] || attributes[:active_billable_member_count]
+      attributes[:memberships]                  = fields['memberships'] || attributes[:memberships]
       self
     end
 

data/lib/trello/plugin_datum.rb

@@ -21,12 +21,12 @@ module Trello
     # Supply a hash of stringkeyed data retrieved from the Trello API representing
     # an attachment.
     def update_fields(fields)
-      attributes[:id]        = fields['id']
-      attributes[:idPlugin]  = fields['idPlugin']
-      attributes[:scope]     = fields['scope']
-      attributes[:value]     = JSON.parse fields['value']
-      attributes[:idModel]   = fields['idModel']
-      attributes[:access]    = fields['access']
+      attributes[:id]        = fields['id'] || attributes[:id]
+      attributes[:idPlugin]  = fields['idPlugin'] || attributes[:idPlugin]
+      attributes[:scope]     = fields['scope'] || attributes[:scope]
+      attributes[:value]     = JSON.parse(fields['value']).presence if fields.has_key?('value')
+      attributes[:idModel]   = fields['idModel'] || attributes[:idModel]
+      attributes[:access]    = fields['access'] || attributes[:access]
       self
     end
   end

data/lib/trello/token.rb

@@ -23,11 +23,12 @@ module Trello
 
     # :nodoc:
     def update_fields(fields)
-      attributes[:id]          = fields['id']
-      attributes[:member_id]   = fields['idMember']
-      attributes[:created_at]  = Time.iso8601(fields['dateCreated'])
-      attributes[:permissions] = fields['permissions'] || {}
-      attributes[:webhooks]    = fields['webhooks']
+      attributes[:id]          = fields['id'] || attributes[:id]
+      attributes[:member_id]   = fields['idMember'] || attributes[:member_id]
+      attributes[:created_at]  = Time.iso8601(fields['dateCreated']) rescue nil if fields.has_key?('dateCreated')
+      attributes[:permissions] = fields['permissions'] || attributes[:permissions]
+      attributes[:permissions] ||= {}
+      attributes[:webhooks]    = fields['webhooks'] || attributes[:webhooks]
     end
 
     # Returns a reference to the user who authorized the token.

data/lib/trello/webhook.rb

@@ -48,11 +48,11 @@ module Trello
 
     # @return [Trello::Webhook] self
     def update_fields(fields)
-      attributes[:id]              = fields['id']
-      attributes[:description]     = fields['description'] || fields[:description]
-      attributes[:id_model]        = fields['idModel'] || fields[:id_model]
-      attributes[:callback_url]    = fields['callbackURL'] || fields[:callback_url]
-      attributes[:active]          = fields['active']
+      attributes[:id]              = fields['id'] || attributes[:id]
+      attributes[:description]     = fields['description'] || fields[:description] || attributes[:description]
+      attributes[:id_model]        = fields['idModel'] || fields[:id_model] || attributes[:id_model]
+      attributes[:callback_url]    = fields['callbackURL'] || fields[:callback_url] || attributes[:callback_url]
+      attributes[:active]          = fields['active'] if fields.has_key?('active')
       self
     end
 

data/spec/action_spec.rb

@@ -124,5 +124,26 @@ module Trello
         expect(action.member_creator).to_not be_nil
       end
     end
+
+    describe "#update_fields" do
+      it "does not set any fields when the fields argument is empty" do
+        expected = {
+          'id' => 'id',
+          'type' => 'type',
+          'data' => 'data',
+          'idMemberCreator' => 'member_creator_id',
+          'member' => 'member_participant'
+        }
+
+        action = Action.new(expected)
+        action.client = client
+
+        action.update_fields({})
+
+        expected.each do |key, value|
+          expect(action.send(value)).to eq expected[key]
+        end
+      end
+    end
   end
 end

data/spec/board_spec.rb

@@ -225,6 +225,19 @@ module Trello
       end
     end
 
+    context "checklists" do
+      before do
+        allow(client)
+          .to receive(:get)
+          .with("/boards/abcdef123456789123456789/checklists", {filter: :all})
+          .and_return checklists_payload
+      end
+
+      it "has a list of checklists" do
+        expect(board.checklists.count).to be > 0
+      end
+    end
+
     context "organization" do
       before do
         allow(client)

data/spec/card_spec.rb

@@ -399,6 +399,45 @@ module Trello
         card.move_to_board(other_board, other_list)
       end
 
+      it 'can be moved to a list on the same board' do
+        current_board = double(id: 'abcdef123456789123456789')
+        other_list = double(
+          id: '987654321987654321fedcba',
+          board_id: 'abcdef123456789123456789'
+        )
+        allow(List).to receive(:find).with('987654321987654321fedcba').
+          and_return(other_list)
+        allow(card).to receive(:board).and_return(current_board)
+        payload = {value: other_list.id}
+
+        expect(client)
+          .to receive(:put)
+          .with('/cards/abcdef123456789123456789/idList', payload)
+
+        card.move_to_list_on_any_board(other_list.id)
+      end
+
+      it 'can be moved to a list on another board' do
+        current_board = double(id: 'abcdef123456789123456789')
+        other_board = double(id: '987654321987654321fedcba')
+        other_list = double(
+          id: '987654321987654321aalist',
+          board_id: '987654321987654321fedcba'
+        )
+        allow(List).to receive(:find).with('987654321987654321aalist').
+          and_return(other_list)
+        allow(card).to receive(:board).and_return(current_board)
+        allow(Board).to receive(:find).with('987654321987654321fedcba').
+          and_return(other_board)
+        payload = { value: other_board.id, idList: other_list.id }
+
+        expect(client)
+          .to receive(:put)
+          .with('/cards/abcdef123456789123456789/idBoard', payload)
+
+        card.move_to_list_on_any_board(other_list.id)
+      end
+
       it 'should not be moved if new board is identical with old board', focus: true do
         other_board = double(id: 'abcdef123456789123456789')
         expect(client).to_not receive(:put)
@@ -753,5 +792,31 @@ module Trello
         expect(card.due_complete).to be true
       end
     end
+
+    describe "#update_fields" do
+      it "does not set any fields when the fields argument is empty" do
+        expected = cards_details.first
+
+        card = Card.new(expected)
+        card.client = client
+
+        card.update_fields({})
+
+        expected.each do |key, value|
+          if card.respond_to?(key) && key != 'labels'
+            expect(card.send(key)).to eq value
+          end
+
+          expect(card.labels).to eq expected['labels'].map { |lbl| Trello::Label.new(lbl) }
+          expect(card.short_id).to eq expected['idShort']
+          expect(card.short_url).to eq expected['shortUrl']
+          expect(card.board_id).to eq expected['idBoard']
+          expect(card.member_ids).to eq expected['idMembers']
+          expect(card.cover_image_id).to eq expected['idAttachmentCover']
+          expect(card.list_id).to eq expected['idList']
+          expect(card.card_labels).to eq expected['idLabels']
+        end
+      end
+    end
   end
 end

data/spec/checklist_spec.rb

@@ -167,12 +167,21 @@ module Trello
         expected_item_id = "1234"
         expected_state = "incomplete"
         expected_resource =
-          "/cards/abccardid/checklist/abcdef123456789123456789" \
-          "/checkItem/#{expected_item_id}/state"
-        payload = { value: expected_state }
+          "/cards/abccardid/checkItem/#{expected_item_id}"
+        payload = { state: expected_state }
         expect(client).to receive(:put).once.with(expected_resource, payload)
         checklist.update_item_state(expected_item_id, expected_state)
       end
+
+      it "updates an item's with boolean input" do
+        expected_item_id = "1234"
+        expected_state = "complete"
+        expected_resource =
+            "/cards/abccardid/checkItem/#{expected_item_id}"
+        payload = { state: expected_state }
+        expect(client).to receive(:put).once.with(expected_resource, payload)
+        checklist.update_item_state(expected_item_id, true)
+      end
     end
 
     context "board" do
@@ -256,5 +265,32 @@ module Trello
         checklist.copy
       end
     end
+
+    describe "#update_fields" do
+      it "does not set any fields when the fields argument is empty" do
+        expected = {
+          'id' => 'id',
+          'name' => 'name',
+          'desc' => 'description',
+          'closed' => 'closed',
+          'url' => 'url',
+          'checkItems' => 'check_items',
+          'pos' => 'position',
+          'idBoard' => 'board_id',
+          'idCard' => 'card_id',
+          'idList' => 'list_id',
+          'idMembers' => 'member_ids'
+        }
+
+        checklist = Checklist.new(expected)
+        checklist.client = client
+
+        checklist.update_fields({})
+
+        expected.each do |key, value|
+          expect(checklist.send(value)).to eq expected[key]
+        end
+      end
+    end
   end
 end

data/spec/client_spec.rb

@@ -50,7 +50,11 @@ describe Client do
           .to receive(:try_execute)
           .and_return(response_with_non_200_status)
 
-        expect { client.get "/xxx" }.to raise_error(Error, "404 error response")
+        expect { client.get "/xxx" }.to raise_error do |error|
+          expect(error).to be_a(Error)
+          expect(error.message).to eq("404 error response")
+          expect(error.status_code).to eq(404)
+        end
       end
     end
   end

data/spec/item_spec.rb

@@ -51,5 +51,25 @@ module Trello
         it { expect(item).not_to be_complete }
       end
     end
+
+    describe "#update_fields" do
+      it "does not set any fields when the fields argument is empty" do
+        expected = {
+          'id' => 'id',
+          'name' => 'name',
+          'type' => 'type',
+          'state' => 'state',
+          'pos' => 'pos'
+        }
+
+        item = Item.new(expected)
+
+        item.update_fields({})
+
+        expected.each do |key, value|
+          expect(item.send(value)).to eq expected[key]
+        end
+      end
+    end
   end
 end

data/spec/label_spec.rb

@@ -181,5 +181,25 @@ module Trello
         expect(label.board).to_not be_nil
       end
     end
+
+    describe "#update_fields" do
+      it "does not set any fields when the fields argument is empty" do
+        expected = {
+          'id' => 'id',
+          'name' => 'name',
+          'color' => 'color',
+          'idBoard' => 'board_id',
+          'uses' => 'uses'
+        }
+
+        label = Label.new(expected)
+
+        label.update_fields({})
+
+        expected.each do |key, value|
+          expect(label.send(value)).to eq expected[key]
+        end
+      end
+    end
   end
 end

data/spec/list_spec.rb

@@ -228,5 +228,26 @@ module Trello
         list.close!
       end
     end
+
+    describe "#update_fields" do
+      it "does not set any fields when the fields argument is empty" do
+        expected = {
+          'id' => 'id',
+          'name' => 'name',
+          'closed' => 'closed',
+          'idBoard' => 'board_id',
+          'pos' => 'pos',
+          'idListSource' => 'source_list_id'
+        }
+
+        label = List.new(expected)
+
+        label.update_fields({})
+
+        expected.each do |key, value|
+          expect(label.send(value)).to eq expected[key]
+        end
+      end
+    end
   end
 end

data/spec/member_spec.rb

@@ -132,5 +132,28 @@ module Trello
         expect { member.id = '42' }.to raise_error NoMethodError
       end
     end
+
+    describe "#update_fields" do
+      it "does not set any fields when the fields argument is empty" do
+        expected = {
+          'id' => 'id',
+          'fullName' => 'full_name',
+          'email' => 'email',
+          'username' => 'username',
+          'initials' => 'initials',
+          'avatarHash' => 'avatar_id',
+          'bio' => 'bio',
+          'url' => 'url'
+        }
+
+        member = Member.new(expected)
+
+        member.update_fields({})
+
+        expected.each do |key, value|
+          expect(member.send(value)).to eq expected[key]
+        end
+      end
+    end
   end
 end

data/spec/notification_spec.rb

@@ -118,5 +118,26 @@ module Trello
         expect(notification.member_creator_id).to eq notification_details['idMemberCreator']
       end
     end
+
+    describe "#update_fields" do
+      it "does not set any fields when the fields argument is empty" do
+        expected = {
+          'id' => 'id',
+          'unread' => 'unread',
+          'type' => 'type',
+          'date' => 'date',
+          'data' => 'data',
+          'idMemberCreator' => 'member_creator_id'
+        }
+
+        notification = Notification.new(expected)
+
+        notification.update_fields({})
+
+        expected.each do |key, value|
+          expect(notification.send(value)).to eq expected[key]
+        end
+      end
+    end
   end
 end

data/spec/organization_spec.rb

@@ -40,6 +40,32 @@ module Trello
         expect(organization.actions.count).to be > 0
       end
     end
+
+    describe "#update_fields" do
+      it "does not set any fields when the fields argument is empty" do
+        expected = {
+          'id' => 'id',
+          'name' => 'name',
+          'displayName' => 'display_name',
+          'desc' => 'description',
+          'url' => 'url',
+          'invited' => 'invited',
+          'website' => 'website',
+          'logoHash' => 'logo_hash',
+          'billableMemberCount' => 'billable_member_count',
+          'activeBillableMemberCount' => 'active_billable_member_count',
+          'memberships' => 'memberships'
+        }
+
+        organization = Organization.new(expected)
+
+        organization.update_fields({})
+
+        expected.each do |key, value|
+          expect(organization.send(value)).to eq expected[key]
+        end
+      end
+    end
   end
 end
 

data/spec/token_spec.rb

@@ -66,5 +66,24 @@ module Trello
         expect(token.member).to eq Member.find('abcdef123456789123456789')
       end
     end
+
+    describe "#update_fields" do
+      it "does not set any fields when the fields argument is empty" do
+        expected = {
+          'id' => 'id',
+          'idMember' => 'member_id',
+          'permissions' => 'permissions',
+          'webhooks' => 'webhooks'
+        }
+
+        token = Token.new(expected)
+
+        token.update_fields({})
+
+        expected.each do |key, value|
+          expect(token.send(value)).to eq expected[key]
+        end
+      end
+    end
   end
 end

data/spec/webhook_spec.rb

@@ -106,5 +106,25 @@ module Trello
         expect(webhook).to be_activated
       end
     end
+
+    describe "#update_fields" do
+      it "does not set any fields when the fields argument is empty" do
+        expected = {
+          'id' => 'id',
+          'description' => 'description',
+          'idModel' => 'id_model',
+          'callbackURL' => 'callback_url',
+          'active' => 'active'
+        }
+
+        webhook = Webhook.new(expected)
+
+        webhook.update_fields({})
+
+        expected.each do |key, value|
+          expect(webhook.send(value)).to eq expected[key]
+        end
+      end
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ruby-trello
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.0.1
 platform: ruby
 authors:
 - Jeremy Tregunna
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-02-05 00:00:00.000000000 Z
+date: 2018-02-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
@@ -105,6 +105,7 @@ files:
 - lib/trello/core_ext/hash.rb
 - lib/trello/core_ext/string.rb
 - lib/trello/cover_image.rb
+- lib/trello/error.rb
 - lib/trello/has_actions.rb
 - lib/trello/item.rb
 - lib/trello/item_state.rb
@@ -167,34 +168,34 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: ruby-trello
-rubygems_version: 2.5.1
+rubygems_version: 2.7.3
 signing_key: 
 specification_version: 4
 summary: A wrapper around the trello.com API.
 test_files:
-- spec/action_spec.rb
-- spec/array_spec.rb
 - spec/association_spec.rb
-- spec/basic_auth_policy_spec.rb
-- spec/board_spec.rb
-- spec/card_spec.rb
-- spec/checklist_spec.rb
-- spec/client_spec.rb
-- spec/configuration_spec.rb
-- spec/hash_spec.rb
-- spec/integration/how_to_authorize_spec.rb
+- spec/notification_spec.rb
+- spec/spec_helper.rb
 - spec/integration/how_to_use_boards_spec.rb
+- spec/integration/how_to_authorize_spec.rb
 - spec/integration/integration_test.rb
-- spec/item_spec.rb
-- spec/json_utils_spec.rb
-- spec/label_spec.rb
-- spec/list_spec.rb
+- spec/card_spec.rb
+- spec/hash_spec.rb
+- spec/trello_spec.rb
+- spec/board_spec.rb
 - spec/member_spec.rb
-- spec/notification_spec.rb
 - spec/oauth_policy_spec.rb
+- spec/configuration_spec.rb
+- spec/json_utils_spec.rb
+- spec/checklist_spec.rb
+- spec/list_spec.rb
+- spec/webhook_spec.rb
 - spec/organization_spec.rb
-- spec/spec_helper.rb
-- spec/string_spec.rb
+- spec/basic_auth_policy_spec.rb
 - spec/token_spec.rb
-- spec/trello_spec.rb
-- spec/webhook_spec.rb
+- spec/string_spec.rb
+- spec/action_spec.rb
+- spec/array_spec.rb
+- spec/client_spec.rb
+- spec/item_spec.rb
+- spec/label_spec.rb