[Pkg-roundcube-maintainers] Bug#994128: roundcube: search preference configuration setting for folder scope gets ignored

[Steve Dondley]

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