[LinkedIn] getusercontacts error #438

Closed
dcnl1980 opened this Issue Apr 7, 2015 · 4 comments

Projects

None yet

3 participants

@dcnl1980
dcnl1980 commented Apr 7, 2015

ErrorException (E_NOTICE)
HELP
Undefined variable: ch
Open: /var/www/clients/client1/web4/web/laravel/vendor/hybridauth/hybridauth/hybridauth/Hybrid/thirdparty/LinkedIn/LinkedIn.php

  // set the last url, headers
  $this->last_request_url = $url;
  $this->last_request_headers = $header;

  // gather the response
  $return_data['linkedin']        = curl_exec($handle);
  if( $return_data['linkedin'] === FALSE ) {
      Hybrid_Logger::error( "LinkedIn::fetch(). curl_exec error: ", curl_error($ch) );
  }
  $return_data['info']            = curl_getinfo($handle);
@StorytellerCZ StorytellerCZ added this to the 2.x milestone Apr 12, 2015
@StorytellerCZ StorytellerCZ changed the title from Linkedin getusercontacts error to [LinkedIn] getusercontacts error Apr 12, 2015
@VladyslavKurmaz
Contributor

Looks like a typo in
http://code.google.com/p/simple-linkedinphp/

      $return_data['linkedin']        = curl_exec($handle);
      if( $return_data['linkedin'] === FALSE ) {
          Hybrid_Logger::error( "LinkedIn::fetch(). curl_exec error: ", curl_error($ch) );
      }

$ch should be $handle
Latest simple-linkedinphp version (3.2.0) was published in Nov 2011.

@AdwinTrave Should we fix our instance of simple-linkedinphp library?
I'm afraid we will not receive new version of simple-linkedinphp soon.

@dcnl1980

Just change this in the code, but now we are unable to fetch the users. The last time it was working, since the update not.

@VladyslavKurmaz
Contributor

Additional error check was added at Feb 15, 2014
pull request #98 from mindon/master …
53c1ba6

if( $return_data['linkedin'] === FALSE ) {
          Hybrid_Logger::error( "LinkedIn::fetch(). curl_exec error: ", curl_error($ch) );
      }

It's clear now, I will fix it.

@dcnl1980

Super and thanks for your fix. I will wait for the pull request to get accepted. Found also some problems with Google+, but i will make some other thread.

@StorytellerCZ StorytellerCZ added a commit that referenced this issue Sep 7, 2015
@StorytellerCZ StorytellerCZ Release v2.5.0
2.5.0 - September 7 2015
      New:
        #444 + #445 - Slack provider
        $474 - WarGaming OpenID provider
        #454 - xuite's oauth provider
        #480 - Improved inline documentation for IDE support
        #370 + #495 - [github] getUserContacts() implementation
(followers & following)
        #372 + #495 - [instagram] getUserContacts() implementation
(follows & followed-by)
        #519 - [Twitter] Add email option to Twitter API
      Fixes:
        #438 + #445 - [LinkedIn] getusercontacts error
        #446 + #447 - [Steam] xml parsing error
        #340 + #476 - Ensure session is saved before sending response
        #472 - [Facebook] Obsolete 'read_friendlists' FB permission was
changed to 'read_custom_friendlists'
        #453 - [Vkontakte] fix Undefined property: stdClass::$email
        #499 + #500 - [Facebook] Bug: treating hometown as current
location
        #523 - [Draugiem] API does not always return the birthday object
        #525 - [Facebook] Fixes getUserProfile v2.4 compatibility
        #532 + #533 [soundcloud] Provider doesn't implement
getUserProfile() and getUserContacts()
        #535 - [Draugiem] API does not always return the birthday object
      Changes:
        #477 - [Vkontakte] Move Vkontakte fields to configuration
        71531f2 - [Facebook] Adding birthday field to be retrieved
      Removed:
        #455 - [Viadeo] Viadeo connect not existing anymore
972005a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment