diff options
| author | niv-openerp <nicolas.vanhoren@openerp.com> | 2012-08-22 11:17:59 +0200 |
|---|---|---|
| committer | niv-openerp <nicolas.vanhoren@openerp.com> | 2012-08-22 11:17:59 +0200 |
| commit | a540de6f75cf30c8dc44fcccf3484b74eb1974a5 (patch) | |
| tree | 3e1fdd36aa42693f49e94bab7fcc53b4b692dada /addons/web_linkedin/static/src/js/linkedin.js | |
| parent | e9f1287d7eeff3ef7f39511aa0ad5b6bf3ef70c6 (diff) | |
[FIX] web_linkedin: rare case when linkedin returns an useless people information
bzr revid: nicolas.vanhoren@openerp.com-20120822091759-mnvq11xwvxvmv6d3
Diffstat (limited to 'addons/web_linkedin/static/src/js/linkedin.js')
| -rw-r--r-- | addons/web_linkedin/static/src/js/linkedin.js | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/addons/web_linkedin/static/src/js/linkedin.js b/addons/web_linkedin/static/src/js/linkedin.js index 483452d4278..575a4cf9a85 100644 --- a/addons/web_linkedin/static/src/js/linkedin.js +++ b/addons/web_linkedin/static/src/js/linkedin.js @@ -123,7 +123,10 @@ openerp.web_linkedin = function(instance) { children_def.resolve(result); }); defs.push(children_def.pipe(function(result) { - var defs = _.map(result.people.values || [], function(el) { + result = _.reject(result.people.values || [], function(el) { + return ! el.formattedName; + }); + var defs = _.map(result, function(el) { el.__type = "people"; return self.create_on_change(el); }); @@ -147,14 +150,14 @@ openerp.web_linkedin = function(instance) { } to_change.mobile = false; to_change.phone = false; - _.each(entity.phoneNumbers.values || [], function(el) { + _.each((entity.phoneNumbers || {}).values || [], function(el) { if (el.phoneType === "mobile") { to_change.mobile = el.phoneNumber; } else { to_change.phone = el.phoneNumber; } }); - var positions = entity.positions.values || []; + var positions = (entity.positions || {}).values || []; to_change.function = positions.length > 0 ? positions[0].title : false; /* TODO to_change.linkedinUrl = entity.publicProfileUrl; |