* Vincent Bernat <[email protected]> [2009-03-10 19:52]:
> Googling a bit, I see that the option "disable_smart_seqname" is here to
> get better performance. Therefore, I  propose to just patch roundcube to

Yeas, seems so.

> not  use this  option. Having  a small  performance impact  seems  to be
> better than to have a non working program.
> 
> What do you think?

Ok, I patched roudcube (oneline comment patch in attachement ;) )
and it now works. Well, sort of.

I hit another bug: http://trac.roundcube.net/ticket/1484954
So now users can't edit their identieties (saving them to db fails).
Solution is the same as before - update php-mdb2-driver-pgsql to 1.5.0.

What is more - from the bug above it seems this affects sqlite backend, too.

Should I fill a wishlist bug against php-mdb2-driver-pgsql for new
version?
I know it is called 'alpha', but considering date when it was released
and that probably a lot of people is using it...

greetings
silk
Only in roundcube-0.2~stable/debian/patches: 519104-workaround.patch
diff -ur roundcube-0.2~stable/program/include/rcube_mdb2.php roundcube-0.2~stable-patched/program/include/rcube_mdb2.php
--- roundcube-0.2~stable/program/include/rcube_mdb2.php	2008-10-14 14:49:44.000000000 +0200
+++ roundcube-0.2~stable-patched/program/include/rcube_mdb2.php	2009-03-11 11:16:43.000000000 +0100
@@ -86,7 +86,8 @@
         'portability' => MDB2_PORTABILITY_ALL ^ MDB2_PORTABILITY_EMPTY_TO_NULL);
 
     if ($this->db_provider == 'pgsql') {
-      $db_options['disable_smart_seqname'] = true;
+    // commented out due toa php-mdb2-driver-pgsql < 1.5.0a in debian 
+    //  $db_options['disable_smart_seqname'] = true;
       $db_options['seqname_format'] = '%s';
     }
 

Reply via email to