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

--- Comment #20 from Ismael Luceno <[email protected]> ---
(In reply to Julien Nabet from comment #19)
> 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?

+1

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

Reply via email to