https://bugs.documentfoundation.org/show_bug.cgi?id=131850

Maxim Monastirsky <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #6 from Maxim Monastirsky <[email protected]> ---
(In reply to Mike Kaganski from comment #5)
> It might of course had worsened with the fix for tdf#125756. However, that
> partial regression is undoubtedly not to be fixed by its own
I disagree. There's absolutely no reason to check for encryption status while
just getting the default image for the given module to be used as an overlay.
But that exactly what the overlay code does, by calling
RecentDocsView::getDefaultThumbnail which has the encryption check. Before
introducing the overlay feature, the encryption check was triggered only for
files which didn't have thumbnails stored in the user profile, which usually
not the case.

> and the core problem *is* the one introduced in the commit mentioned in 
> comment 1.
Note that for the case there is no thumbnail in the user profile, we were
trying to read the file anyway to get the thumbnail stored inside it, way
before the mentioned commit.

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to