Viewing the profile data, I get
"Notice: Undefined index: picture-url in include() (line 15 of /var/www/nkd7/TRUNK/drupal/sites/all/modules/contrib/linkedin/linkedin_profile/linkedin-user-page.tpl.php)."

I guess this is because this line
<?php if ($profile['picture-url']) : ?>
needs to use isset() or a similar check. It might be good to put the check in all such if statements in the templates.

CommentFileSizeAuthor
#6 1312628--linkedin-notice-missing-fields--6.patch3.76 KBalexiswatson

Comments

deltab’s picture

deltab commented
Priority: Normal » Major

Also in linkedin_profile.module, I get the following on trying to display Positions and Educations information

Notice: Undefined index: id in linkedin_profile_user_load() (line 117 of /var/www/html/sites/all/modules/linkedin/linkedin_profile/linkedin_profile.module).

alexiswatson’s picture

alexiswatson commented
Title: Undefined index: picture-url » PHP Notice when profile is missing photo
Priority: Major » Normal

deltab: This is probably better brought up as a separate issue (which I've also encountered).

As for the original issue, I'll roll a patch for it when I'm not headed to bed. :]

alexiswatson’s picture

alexiswatson commented
Title: PHP Notice when profile is missing photo » PHP Notice when fields are missing
Assigned: Unassigned » alexiswatson

Noticed the problem is a bit more generic than that. I'll whip up a patch that addresses as many as I can see.

paulhk’s picture

paulhk commented

I am also facing issues with the integration -
Notice: Undefined index: position in linkedin_profile_user_load() (line 114 of /home/website/public_html/website.com/sites/all/modules/linkedin/linkedin_profile/linkedin_profile.module).

alexiswatson’s picture

alexiswatson commented

Sorry for the delay. Some of these are fixed in #1357840: Position Template Always Interprets 'is-current' as TRUE and #1357846: Multivalue Structured Field Templates Not Receiving Variables. I'll be working on a few more of these in the near future.

alexiswatson’s picture

alexiswatson commented
Assigned: alexiswatson » Unassigned
Status: Active » Needs review
StatusFileSize
new 1312628--linkedin-notice-missing-fields--6.patch3.76 KB

And here's a patch for the rest. Try all three of them; if there's one that I missed, feel free to add to this one.

helmo’s picture

helmo commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Thanks the patch still applies and fixes the notice.

jaesperanza’s picture

jaesperanza commented

Patch still applies resolves issue. Just to add another error on line item not included in the patch, the same fix on "summary" item in case anyone else encounters this.

Notice: Undefined index: summary in include() (line 50 of /sites/all/modules/linkedin/linkedin_profile/linkedin-user-page-position-item.tpl.php).

in linkedin-user-page-position-item.tpl.php, changed:

<?php if ($position['summary']) : ?>

to

<?php if (isset($position['summary'])) : ?>