Mailing List Archive: DBMail: users

sort in roundcubemail broken again

Index | Next | Previous | View Threaded

h.reindl at thelounge

May 14, 2012, 2:45 AM

Post #1 of 6 (342 views)
Permalink

sort in roundcubemail broken again

hi

http://git.dbmail.eu/paul/dbmail/commit/?h=dbmail_3_0&id=9e2f272541621d898d9485bfbf43818d7dce708c

in 2da7411789e03a9b51f2e286ea1ea6cc4f3e4303 this seems to be broken again
as long i reset user-preferences and do not touch sort all is fine
but from the moment you are sorting for anything it gets broken

Attachments:

signature.asc (0.26 KB)

pegasus at nerv

Jun 13, 2012, 1:21 AM

Post #2 of 6 (298 views)
Permalink

Re: sort in roundcubemail broken again [In reply to]

On Mon, 14 May 2012 11:45:42 +0200
Reindl Harald <h.reindl [at] thelounge> wrote:

> hi
>
> http://git.dbmail.eu/paul/dbmail/commit/?h=dbmail_3_0&id=9e2f272541621d898d9485bfbf43818d7dce708c
>
> in 2da7411789e03a9b51f2e286ea1ea6cc4f3e4303 this seems to be broken again
> as long i reset user-preferences and do not touch sort all is fine
> but from the moment you are sorting for anything it gets broken

What is the status of this now? Is it fixed yet?
Which patches do I need to apply on top of 3.0.2 to get it fixed? Anything else I need to do on the db?

--

Jure Pečar
http://jure.pecar.org
http://f5j.eu
_______________________________________________
DBmail mailing list
DBmail [at] dbmail
http://mailman.fastxs.nl/cgi-bin/mailman/listinfo/dbmail

h.reindl at thelounge

Jun 13, 2012, 2:00 AM

Post #3 of 6 (296 views)
Permalink

Re: sort in roundcubemail broken again [In reply to]

Am 13.06.2012 10:21, schrieb Jure Pečar:
> On Mon, 14 May 2012 11:45:42 +0200
> Reindl Harald <h.reindl [at] thelounge> wrote:
>
>> hi
>>
>> http://git.dbmail.eu/paul/dbmail/commit/?h=dbmail_3_0&id=9e2f272541621d898d9485bfbf43818d7dce708c
>>
>> in 2da7411789e03a9b51f2e286ea1ea6cc4f3e4303 this seems to be broken again
>> as long i reset user-preferences and do not touch sort all is fine
>> but from the moment you are sorting for anything it gets broken
>
> What is the status of this now? Is it fixed yet?
> Which patches do I need to apply on top of 3.0.2 to get it fixed? Anything else I need to do on the db?

you can use http://git.dbmail.eu/paul/dbmail/commit/?h=dbmail_3_0&id=22b964b653ba149b63b9b6787aeac7803c2121e5

this is the last snapshot where sort is working fine with
some additional improvements compared to 3.0.2

Attachments:

signature.asc (0.26 KB)

paul at nfg

Jun 13, 2012, 6:12 AM

Post #4 of 6 (297 views)
Permalink

Re: sort in roundcubemail broken again [In reply to]

On 06/13/2012 11:00 AM, Reindl Harald wrote:
>
>
> Am 13.06.2012 10:21, schrieb Jure Pečar:
>> On Mon, 14 May 2012 11:45:42 +0200
>> Reindl Harald <h.reindl [at] thelounge> wrote:
>>
>>> hi
>>>
>>> http://git.dbmail.eu/paul/dbmail/commit/?h=dbmail_3_0&id=9e2f272541621d898d9485bfbf43818d7dce708c

That fix (using DISTINCT) was broken - at least for non-MySQL users.

I've just pushed a fix on the master branch that should help for all
supported databases, but it requires a minor schema fix: one of the
views needs updating. This is a very fast schema change that can safely
be applied before restarting the IMAP daemons.

--
________________________________________________________________
Paul J Stevens pjstevns @ gmail, twitter, skype, linkedin

* Premium Hosting Services and Web Application Consultancy *

www.nfg.nl/info [at] nfg/+31.85.877.99.97
________________________________________________________________
_______________________________________________
DBmail mailing list
DBmail [at] dbmail
http://mailman.fastxs.nl/cgi-bin/mailman/listinfo/dbmail

pegasus at nerv

Jun 14, 2012, 7:07 AM

Post #5 of 6 (288 views)
Permalink

Re: sort in roundcubemail broken again [In reply to]

On Wed, 13 Jun 2012 15:12:19 +0200
Paul J Stevens <paul [at] nfg> wrote:

> That fix (using DISTINCT) was broken - at least for non-MySQL users.
>
> I've just pushed a fix on the master branch that should help for all
> supported databases, but it requires a minor schema fix: one of the
> views needs updating. This is a very fast schema change that can safely
> be applied before restarting the IMAP daemons.

FYI, imapd from master with this fix just sigaborts on startup without any message.

Taking just this fix back into 3.0.2 and recreating the db view results in still broken roundcube sort. Sort by size, subject or sender produce expected results, sort by date doesn't.

Also noticed that UID THREAD ORDEREDSUBJECT US-ASCII ALL imap command doesnt return any response from imapd and the client eventually times out.

--

Jure Pečar
http://jure.pecar.org
http://f5j.eu
_______________________________________________
DBmail mailing list
DBmail [at] dbmail
http://mailman.fastxs.nl/cgi-bin/mailman/listinfo/dbmail

paul at nfg

Jun 14, 2012, 7:33 AM

Post #6 of 6 (290 views)
Permalink

Re: sort in roundcubemail broken again [In reply to]

On 06/14/2012 04:07 PM, Jure Pečar wrote:

> FYI, imapd from master with this fix just sigaborts on startup without any message.

I need a backtrace from gdb. It works for me, so I need some more info.

>
> Taking just this fix back into 3.0.2 and recreating the db view results in still broken roundcube sort. Sort by size, subject or sender produce expected results, sort by date doesn't.

Unless someone provides me with the exact IMAP command used, there's
nothing I can do but grope in the dark. My own roundcube doesn't give
any problems, but that doesn't mean anything other than that I can't
reproduce this.

>
> Also noticed that UID THREAD ORDEREDSUBJECT US-ASCII ALL imap command doesnt return any response from imapd and the client eventually times out.

Confirmed. I'll look into it.

--
________________________________________________________________
Paul J Stevens pjstevns @ gmail, twitter, skype, linkedin

* Premium Hosting Services and Web Application Consultancy *

www.nfg.nl/info [at] nfg/+31.85.877.99.97
________________________________________________________________
_______________________________________________
DBmail mailing list
DBmail [at] dbmail
http://mailman.fastxs.nl/cgi-bin/mailman/listinfo/dbmail

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact Gossamer Threads