gianlucagiacometti / Roundcube-Forward

RoundCube plugin to set forward addresses in PostfixAdmin database
GNU General Public License v2.0
7 stars 10 forks source link

forward plugin not compatible with roundcube 0.9 beta #1

Closed sblaisot closed 9 years ago

sblaisot commented 11 years ago

Hi,

Due to changes in the way roundcube access database in the 0.9 beta version, the forward plugin doesn't work anymore starting with this version.

Error log says : PHP Fatal error: Class 'rcube_mdb2' not found in /plugins/forward/lib/drivers/sql.php on line 39

this is related with the following changelog line of roundcube 0.9 :

A workaround is to replace $db = new rcube_mdb2($dsn, '', FALSE); by $db = rcube_db::factory($dsn, '', FALSE);

regards,

S.B.

sblaisot commented 11 years ago

for better compatibility with pre-è0.9 versions, it is in fact better to replace with if (!class_exists('rcube_db')) { $db = new rcube_mdb2($dsn, '', FALSE); } else { $db = rcube_db::factory($dsn, '', FALSE); }

which can handle both situations.

regards,

S.B.

gianlucagiacometti commented 11 years ago

Thank you very much! as soon as I can I will update the official repository thanking you.

Spedito da Samsung Galaxy S2

-----Original Message----- From: Sebastien BLAISOT notifications@github.com To: gianlucagiacometti/roundcube-forward roundcube-forward@noreply.github.com Sent: mar, 08 gen 2013 23:57 Subject: Re: [roundcube-forward] forward plugin not compatible with roundcube 0.9 beta (#1)

for better compatibility with pre-è0.9 versions, it is in fact better to replace with if (!class_exists('rcube_db')) { $db = new rcube_mdb2($dsn, '', FALSE); } else { $db = rcube_db::factory($dsn, '', FALSE); }

which can handle both situations.

regards,

S.B.


Reply to this email directly or view it on GitHub: https://github.com/gianlucagiacometti/roundcube-forward/issues/1#issuecomment-12022103

IdahoPL commented 11 years ago

When can we expect updated version? :)

zuloo commented 10 years ago

Hello, i guess i will fork it and offer you a pull request...

gianlucagiacometti commented 10 years ago

Great, I printed my phd thesis today and I'll go for it within the end of the month. Two minds work better!

Il 15 febbraio 2014 21:59:44 zuloo notifications@github.com ha scritto:

Hello, i guess i will fork it and offer you a pull request...


Reply to this email directly or view it on GitHub: https://github.com/gianlucagiacometti/roundcube-forward/issues/1#issuecomment-35167595

IdahoPL commented 10 years ago

Thank you, I'm eagerly waiting :)

IdahoPL commented 9 years ago

Hi,

Is this project alive?

gianlucagiacometti commented 9 years ago

I hope, I'm starting working on a new version right now

IdahoPL commented 9 years ago

It's a great news :) If I may suggest one feature - it would be great to support postfixadmin's vacation alias format. postfixadmin use special alias format when vacation is active. It looks like this:

login#domain.tld@vacation.domain2.tld

I think forward should hide this type of alias from user, as it can be scary for regular user ;) If this type of alias is set then hide it and leave it alone, other plugin should manage vacation settings.

gianlucagiacometti commented 9 years ago

I'm back... at last! :-)