6.0.0-beta1
10/19/25

[#6044] php error on first login of kolab user
Summary php error on first login of kolab user
Queue Turba
Queue Version HEAD
Type Bug
State Duplicate
Priority 1. Low
Owners mrubinsk (at) horde (dot) org
Requester m.gabriel (at) das-netzwerkteam (dot) de
Created 12/25/2007 (6508 days ago)
Due
Updated 12/25/2007 (6508 days ago)
Assigned 12/25/2007 (6508 days ago)
Resolved 12/25/2007 (6508 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
12/25/2007 07:59:05 PM Michael Rubinsky Assigned to Michael Rubinsky
Taken from Gunnar Wrobel
State ⇒ Duplicate
 
12/25/2007 07:58:33 PM Michael Rubinsky Comment #3
Version ⇒ HEAD
Queue ⇒ Turba
Reply to this comment
This is also already fixed in CVS,  by the same commit as mentioned in 
Bug: 6037

...also moving to Turba since it's not really Kolab specific.
12/25/2007 03:27:45 AM Chuck Hagenbuch Comment #2
State ⇒ Assigned
Assigned to Gunnar Wrobel
Reply to this comment
It'd be helpful if you could attach the patches to the tickets instead 
of putting them inline - easier to avoid line wrapping confusion that 
way. Also please take a quick read through 
http://www.horde.org/source/contribute.php - unified diffs are usually 
easier to read for patches, etc. Thanks!
12/25/2007 12:21:15 AM m (dot) gabriel (at) das-netzwerkteam (dot) de Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ php error on first login of kolab user
Queue ⇒ Kolab
State ⇒ Unconfirmed
Reply to this comment
i am using kolab-2.1 (debian unstable) and horde-webmail-1.1rc1



the following patch sedates some php errors for kolab users on first login:



diff -r 
horde-webmail-1.1-rc1/turba/lib/Maintenance/Task/upgradelists.php 
horde-webmail-1.1-rc1-netzwerkteam/turba/lib/Maintenance/Task/upgradelists.php

34,55c34,57

<                 $cnt = $lists->count();

<                 for ($j = 0; $j < $cnt; ++$j) {

<                     $list = $lists->next();

<                     $attributes = $list->getAttributes();

<                     $members = @unserialize($attributes['__members']);

<                     $c = count($members);

<                     for ($i = 0; $i < $c; ++$i) {

<                         if (substr_count($members[$i], ':') == 2) {

<                             
preg_match('/^([a-zA-Z0-9]+:[a-zA-Z0-9]+)(:[a-zA-Z0-9]+)$/', 
$members[$i], $matches);

<                             $source = $matches[1];

<                             $contact_key = substr($matches[2], 1);

<                         } elseif (substr_count($members[$i], ':') == 1) {

<                             list($source, $contact_key) = 
explode(':', $members[$i]);

<                         } else {

<                             break;

<                         }

<                         $source = $this->_updateShareName($source);

<                         $members[$i] = $source . ':' . $contact_key;

<                     }

<                     $list->setValue('__members', serialize($members));

<                     $list->store();

<                 }

---

[Show Quoted Text - 27 lines]
best,

mike

Saved Queries