Change to Contact Item Field "mail" property?

Follow

Patrick Steil

• January 08, 2014 18:33

We have had this code running for months, made no changes today and today we get this warning:
Notice: Undefined index: mail in /var/www/helpdesk/libs/PTOrganization.class.php on line 645

Here is the code snippet and I marked line 645 in left margin...

foreach( $this->orgItem->fields as $afield ){

        $fieldclass = get_class($afield);

        //printTrace( "afield: ".dump($afield->__attributes) );

        switch( $fieldclass ){
            case 'PodioContactItemField':
                $fields[$afield->external_id."_name"] =  $afield->__attributes['values'][0]['value']['name'];

Line 645 ==> $fields[$afield->external_id."email"] = $afield->_attributes['values'][0]['value']['mail'][0];
break;
case 'PodioCategoryItemField':
$fields[$afield->external_id] = $afield->values[0]['value']['id'];

printTrace( "init: $afield->external_id:". $afield->values[0]['value']['id'] );

            break;
            default:
                $fields[$afield->external_id] =  $afield->humanized_value();
        }
    }

0

• 

  Patrick Steil

  • January 08, 2014 18:34

  Did something change in the data structure today?

  0

  Comment actions Permalink
• 

  Shaun Burns

  • January 09, 2014 12:00

  I noticed email was missing from my Zapier options. I'm not a developer, but I guess this is the same issue?

  0

  Comment actions Permalink
• 

  Patrick Steil

  • January 09, 2014 22:59

  Andreas???? Still waiting to hear on this... I have debugged this to see that the 'mail' property I was looking for previously has been removed... what's up with this?

  This is causing a production problem with our integration... thanks...

  Thanks!

  0

  Comment actions Permalink
• 

  Andreas Haugstrup Pedersen

  • January 14, 2014 19:37

  We've added the data back in, but it would be good if you could start keeping track of profile_ids on your end. The keep things performant we may have to start returning only profile_ids for contact fields

  0

  Comment actions Permalink
• 

  Patrick Steil

  • January 14, 2014 19:42

  Thank you Endress, I noticed this started working again yesterday. So your preference would be for us to get the profile ID and then do another API call to get the users profile in order to retrieve their email address?

  This would of course not mean better performance for us with another API call.

  Let me know if that's what you mean. Or you have some other idea.

  0

  Comment actions Permalink
• 

  Andreas Haugstrup Pedersen

  • January 14, 2014 19:46

  Preferably you would cache the profiles on your end since they will not change very often, but yet

  0

  Comment actions Permalink
• 

  Patrick Steil

  • January 14, 2014 19:49

  Sorry I didn't catch the way Siri spelled your name Andreas. Lol.

  Thanks for the help.

  0

  Comment actions Permalink

Please sign in to leave a comment.