On 2021-09-12 02:28 PM, Steve Dondley wrote:
On 2021-09-12 02:16 PM, Steve Dondley wrote:
On 2021-09-12 02:07 PM, Guilhem Moulin wrote:
On Sun, 12 Sep 2021 at 13:02:41 -0400, Steve Dondley via
Pkg-roundcube-maintainers wrote:
That said, there must still be a bug because roundcube is supposed to remember the search scope feature from the $_SESSION variable and it's not doing that. Looking at the PHP code, roundcube is most certainly intended to
has this ability. If you manually refresh the page after selecting a
different folder the setting does, in fact, kick in.

The $_SESSION variable for search_scope gets ignored and overridden by some javascript. And replacing app.min.js with app.js fixes the problem. Why that
fixes the problem, I have no earthly idea.

I can't reproduce this in a clean Bullseye chroot. Removed app.min.js so the non-minified version is used instead, but I still have to reload the page to prefill the search dialog with Scope=all. Does your app.js
matches what we shipped in roundcube-core?  DEBIAN/md5sums snippet:

b4865d6536b9188b7f5c3ae0361a1cb5 usr/share/roundcube/program/js/app.js 7a0c881ca3e9cbff512853fcdb8e6650 usr/share/roundcube/program/js/app.min.js
    4f3ace3cd9376165005b754226ca1bd8
usr/share/roundcube/program/js/app.min.js.gz
    5b9f0a46d6913f9b3481fece00930cd9
usr/share/roundcube/program/js/app.min.js.map

They all match except for app.js. I did modify it when troubleshooting
the problem but I'm pretty sure i reverted everything back.

What's the easiest way to get my hands on the original file from the package?

OK, I figured it out. So this is really bizarre.

The freshly downlaoded copy of app.js still does not match your md5. I get:

53dc7e81f7db265b0e3860ec718a3e1e

I get this for both the installed copy of app.js and the one I
downloaded from the .deb package.

Here is my version of app.js: https://gist.github.com/sdondley/9db6dbffb8fb751c4afcd1092ab24fd0

Reply via email to