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

Julien Nabet <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |heiko.tietze@documentfounda
                   |                            |tion.org

--- Comment #19 from Julien Nabet <[email protected]> ---
Heiko/Xisco: in brief, I suppose we need an else case here:

162 if (officecfg::Office::Common::PackageKit::EnableBaseInstallation::get())
163 {
164     try
165     {
166         using namespace org::freedesktop::PackageKit;
167         using namespace svtools;
168         Reference< XSyncDbusSessionHelper >
xSyncDbusSessionHelper(SyncDbusSessionHelper::create(comphelper::getProcessComponentContext()));
169         Sequence< OUString > vPackages { "libreoffice-base" };
170         xSyncDbusSessionHelper->InstallPackageNames(vPackages, OUString());
171         // I'll be back (hopefully)!
172         SolarMutexGuard aGuard;
173         executeRestartDialog(comphelper::getProcessComponentContext(),
nullptr, RESTART_REASON_BIBLIOGRAPHY_INSTALL);
174     }
175     catch (const Exception &)
176     {
177         TOOLS_INFO_EXCEPTION("sfx.appl", "trying to install LibreOffice
Base");
178     }
179 }
180 return;

(See
https://opengrok.libreoffice.org/xref/core/sfx2/source/appl/appserv.cxx?r=e1138605#162)

The else case would contain a dialog indicating something like:
"You need to install Base part to use bibliography."
What do you think?

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to