[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         | skhorde at smail.inf.fh-bonn-rhein-sieg.de
  Summary            | problem in _getAttribute() of
                     | /usr/share/php/Horde/ActiveSync/Message/Base.php
  Queue              | Synchronization
  Version            | Git master
  Type               | Bug
  State              | Feedback
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


skhorde at smail.inf.fh-bonn-rhein-sieg.de (2013-05-24 07:40) wrote:

> 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.

Thanks, I changed to array_key_exists().

> 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.

My report was not necessarily about that the return value is wrong,  
but to get rid of the PHP error message, and _maybe_ telling the  
developer, that something is wrong here.

I have added var_export(debug_backtrace(), true), let's see if I get  
something.