Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

add account_id to parse_contact #3

Merged
merged 1 commit into from

2 participants

@afittro

parse_opportunity and parse_lead already have an account_id element. This commit brings parse_contact in-line with those others by storing the parent-account's id. Without this the coder is left to monkey patch every call to store parse_contact results with the containing account's id.

@j4y j4y merged commit c33aed5 into from
@j4y
Owner

I merged the pull request in and released a new version. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jun 28, 2011
  1. @afittro
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 0 deletions.
  1. +1 −0  lib/landslider.rb
View
1  lib/landslider.rb
@@ -605,6 +605,7 @@ def parse_contact(node)
:reports_to => xml_to_str(node, './reportsTo/text()'),
:owner_id => xml_to_int(node, './ownerId/text()'),
:contact_id => xml_to_int(node, './contactId/text()'),
+ :account_id => xml_to_int(node, './accountId/text()'),
:custom_fields => node.xpath('./customFields', ns).map { |child| parse_custom_field(child) }
}
end
Something went wrong with that request. Please try again.