6.0.0-beta1
7/5/25

[#12171] ActiveSync and Nokia N900: New contact is ignored
Summary ActiveSync and Nokia N900: New contact is ignored
Queue Synchronization
Queue Version Git master
Type Bug
State Not A Bug
Priority 1. Low
Owners mrubinsk (at) horde (dot) org
Requester thomas.jarosch (at) intra2net (dot) com
Created 04/10/2013 (4469 days ago)
Due
Updated 04/16/2013 (4463 days ago)
Assigned 04/11/2013 (4468 days ago)
Resolved 04/15/2013 (4464 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
04/16/2013 09:32:41 PM Git Commit Comment #11 Reply to this comment
Changes have been made in Git (master):

commit c363c49c2d2a553fa1005fe4de0cddbdef8dba59
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Tue Apr 16 17:00:35 2013 -0400

     Bug: 12171 Set logger on importer.

  .../lib/Horde/ActiveSync/Request/Base.php          |    1 +
  1 files changed, 1 insertions(+), 0 deletions(-)

http://git.horde.org/horde-git/-/commit/c363c49c2d2a553fa1005fe4de0cddbdef8dba59
04/16/2013 09:01:14 PM Git Commit Comment #10 Reply to this comment
Changes have been made in Git (FRAMEWORK_5_0):

commit d68a900b79c6c7d3d3d2a28d5179f75fba58d1b9
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Tue Apr 16 17:00:35 2013 -0400

     Bug: 12171 Set logger on importer.

  .../lib/Horde/ActiveSync/Request/Base.php          |    1 +
  1 files changed, 1 insertions(+), 0 deletions(-)

http://git.horde.org/horde-git/-/commit/d68a900b79c6c7d3d3d2a28d5179f75fba58d1b9
04/16/2013 02:50:09 PM Thomas Jarosch Comment #9 Reply to this comment
The new logger introduced a new problem:

[Tue Apr 16 16:36:21 2013] [error] [client 172.17.2.102] PHP Fatal 
error:  Call to a member function err() on a non-object in 
/datastore/DEVEL/horde/framework/ActiveSync/lib/Horde/ActiveSync/Connector/Importer.php on line 
153

This is on FRAMEWORK_5 code. Looks like the _logger object is not set yet.
I've added a Horde::debug() statement so I can show you a backtrace:

Backtrace:
1. Horde_Rpc_ActiveSync->getResponse() 
/datastore/DEVEL/horde/horde/rpc.php:156
2. Horde_ActiveSync->handleRequest() 
/datastore/DEVEL/horde/framework/Rpc/lib/Horde/Rpc/ActiveSync.php:141
3. Horde_ActiveSync_Request_Base->handle() 
/datastore/DEVEL/horde/framework/ActiveSync/lib/Horde/ActiveSync.php:647
4. Horde_ActiveSync_Request_Sync->_handle() 
/datastore/DEVEL/horde/framework/ActiveSync/lib/Horde/ActiveSync/Request/Base.php:218
5. Horde_ActiveSync_Request_Sync->_parseSyncFolders() 
/datastore/DEVEL/horde/framework/ActiveSync/lib/Horde/ActiveSync/Request/Sync.php:217
6. Horde_ActiveSync_Request_Sync->_parseSyncCommands() 
/datastore/DEVEL/horde/framework/ActiveSync/lib/Horde/ActiveSync/Request/Sync.php:973
7. Horde_ActiveSync_Connector_Importer->importMessageChange() 
/datastore/DEVEL/horde/framework/ActiveSync/lib/Horde/ActiveSync/Request/Sync.php:1144
8. Horde::debug() 
/datastore/DEVEL/horde/framework/ActiveSync/lib/Horde/ActiveSync/Connector/Importer.php:155

04/15/2013 02:32:25 PM Michael Rubinsky State ⇒ Not A Bug
 
04/15/2013 10:39:02 AM Thomas Jarosch Comment #8 Reply to this comment
I've added some logging to that method to better determine why it 
exists. See if you can it to happen again.
I've tried to clear all contacts -> sync -> create "first" contact
and I've also re-configured the ActiveSync account. Couldn't trigger it again.

Let's close this issue for now. We have the new logging in place 
should I happen again.

04/12/2013 02:24:47 PM Git Commit Comment #7 Reply to this comment
Changes have been made in Git (FRAMEWORK_5_0):

commit d2216974b4a39d3f1a6aa8b5dfb4456ad66d2efd
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Fri Apr 12 09:37:02 2013 -0400

     Add some logging to track down import failures.

     See Bug: 12171

  .../lib/Horde/ActiveSync/Connector/Importer.php    |   15 ++++++++++++++-
  1 files changed, 14 insertions(+), 1 deletions(-)

http://git.horde.org/horde-git/-/commit/d2216974b4a39d3f1a6aa8b5dfb4456ad66d2efd
04/12/2013 01:38:59 PM Michael Rubinsky Comment #6 Reply to this comment
I've added some logging to that method to better determine why it 
exists. See if you can it to happen again.
04/12/2013 01:38:11 PM Git Commit Comment #5 Reply to this comment
Changes have been made in Git (master):

commit d7ef6c9762ad2cd6c829051b71f727f904d44fd3
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date:   Fri Apr 12 09:37:02 2013 -0400

     Add some logging to track down import failures.

     See Bug: 12171

  .../lib/Horde/ActiveSync/Connector/Importer.php    |   15 ++++++++++++++-
  1 files changed, 14 insertions(+), 1 deletions(-)

http://git.horde.org/horde-git/-/commit/d7ef6c9762ad2cd6c829051b71f727f904d44fd3
04/11/2013 04:24:35 PM Thomas Jarosch Comment #4 Reply to this comment
I've analyzed the log file again. What I'm missing for the ignored 
contact is something like this:

2013-04-11T17:38:23+02:00 DEBUG: [24359] I       </Data>
2013-04-11T17:38:23+02:00 DEBUG: [24359] 
Horde_Core_ActiveSync_Driver::changeMessage(@Contacts@,  ...)
2013-04-11T17:38:24+02:00 DEBUG: [24359] 
Horde_Core_ActiveSync_Driver::_smartStatMessage(@Contacts@, 
5166d8f0-2aac-4a78-9c2e-5f27ac10017b)

Looking at the importMessageChange() code, there are a few places 
where it returns from the function without logging it. Not sure if 
it's worth the effort to add logging there, it's basically the dummy 
folder and duplicate detection.

04/11/2013 02:51:53 PM Thomas Jarosch Comment #3 Reply to this comment
The only thing remotely odd that I see is the contact that didn't 
sync does not contain any data, other than the name, where the one 
that did sync contains an email address.

That being said, creating a new contact containing only a name works 
fine for me here.
Yes, I've noticed that too and created a third contact later on.
This third contact only contained a name and worked too.

I'll try to reset the device and see if I can reproduce it.

Thanks for checking the log though!

04/11/2013 02:46:15 PM Michael Rubinsky Comment #2
State ⇒ Feedback
Assigned to Michael Rubinsky
Reply to this comment
The only thing remotely odd that I see is the contact that didn't sync 
does not contain any data, other than the name, where the one that did 
sync contains an email address.

That being said, creating a new contact containing only a name works 
fine for me here.
04/11/2013 02:43:13 PM Michael Rubinsky Version ⇒ Git master
Queue ⇒ Synchronization
 
04/10/2013 03:14:17 PM Thomas Jarosch Comment #1
Priority ⇒ 1. Low
State ⇒ Unconfirmed
New Attachment: n900_contact_log.txt.gz Download
Patch ⇒ No
Milestone ⇒
Queue ⇒ Horde Framework Packages
Summary ⇒ ActiveSync and Nokia N900: New contact is ignored
Type ⇒ Bug
Reply to this comment
Hi,

I still have this dinosaur of a Nokia N900 for testing with ActiveSync.

I've added a contact on the phone and it was ignored by ActiveSync.
Before submitting this ticket I've added another contact and it synced fine.

Attached is the log file, the missing contact is "N900 test Testme",
the other one is "N900 second test".

Bug on the device?

Cheers,
Thomas

Saved Queries