[Tickets #12605] Re: AS: Fix loss of PINGABLE flag

[noreply at bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/12605
------------------------------------------------------------------------------
  Ticket             | 12605
  Updated By         | Thomas Jarosch <thomas.jarosch at intra2net.com>
  Summary            | AS: Fix loss of PINGABLE flag
  Queue              | Synchronization
  Version            | Git master
  Type               | Bug
  State              | Feedback
  Priority           | 2. Medium
  Milestone          |
  Patch              | 1
  Owners             | Michael Rubinsky
+New Attachment     | activesync.log
------------------------------------------------------------------------------


Thomas Jarosch <thomas.jarosch at intra2net.com> (2013-08-27 22:33) wrote:

> Can you check if these changes fix the pingable flag from getting  
> lost in your case?

I've tested it now. It does work, but not as anticipated :o)

When the ping is processed we don't have the folder in the sync cache yet.
We added the collection in memory to "this->_collections" but it's not
part of the collection we get from the syncCache.

Please see the attached log, I've added a logger to dump
$this->_collections and $collections in updatePingableFlag().


Later on everything is fine, the device sends a Sync
for the folder and another Ping request.  Wild guess:
Through the modified behavior (we don't tell the client there are changes),
the device is behaving "correctly" now.




Thomas Jarosch <thomas.jarosch at intra2net.com> (2013-08-27 22:33)  
uploaded: activesync.log

http://bugs.horde.org/h/services/download/?app=whups&actionID=download_file&file=activesync.log&ticket=12605&fn=%2Factivesync.log