[Tickets #12262] Re: problem in _getAttribute() of /usr/share/php/Horde/ActiveSync/Message/Base.php

[noreply at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/12262
------------------------------------------------------------------------------
  Ticket             | 12262
  Updated By         | Michael Rubinsky <mrubinsk at horde.org>
  Summary            | problem in _getAttribute() of
                     | /usr/share/php/Horde/ActiveSync/Message/Base.php
  Queue              | Kronolith
  Version            | 4.1.0beta2
  Type               | Bug
-State              | Unconfirmed
+State              | Feedback
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


Michael Rubinsky <mrubinsk at horde.org> (2013-05-17 14:24) wrote:

> I have problems with GUI - ActiveSync - Lightning,
>
> in the log I have:
>
> [kronolith] PHP ERROR: Undefined index: body [pid 30676 on line 497  
> of "/usr/share/php/Horde/ActiveSync/Message/Base.php"]
>
> I have added a log message there with:
>
> if(empty($this->_properties[$name])) {	// SKA
> 		Horde::logMessage('SKA: property "'.$name.'" not set', 'WARN');
> 		return $default;
> 	}

An empty value for $this->_properties[$name] is perfectly valid here -  
the default value for most attributes is false, so empty() will return  
true. Instead, what you want to do is check if the array_key_exists()  
there.

Regardless, I would need a backtrace to see where _getAttributes() is  
being called from, as it should only be asking for properties that are  
known to exist in a given EAS version.