KMessageBox runtime dependency on FrameworkIntegrationPlugin makes it useless

[Albert Astals Cid]

El diumenge, 12 de juny de 2016, a les 20:38:28 CEST, Hugo Pereira Da Costa va 
escriure:
> On 06/12/2016 07:51 PM, Albert Astals Cid wrote:
> > Having KMessageBox "dontShowMeAgain" feature depend on an integration
> > plugin is a very bad idea.
> > 
> > Basically it means programs that use KMessageBox can never asusme it will
> > work so basically they have to use alternative methods to have the
> > "dontShowMeAgain" feature or not have it at all.
> > 
> > I understand someone thought that it was a better idea having a feature
> > that may work or not randomly that increasing the dependency chain of
> > KMessageBox, but I disagree.
> > 
> > I don't think the status quo is good at all, my program basically gets a
> > runtime dependency that is not specified anywhere and that makes some
> > features work or not randomly.
> > 
> > The options I can see are:
> >   * Remove the "dontShowMeAgain" feature from KMessageBox
> >   * Make the "dontShowMeAgain" feature use QSettings (always or if
> > 
> > FrameworkIntegrationPlugin is not available)
> > 
> >   * Show a KMessageBox warning when trying to use the "dontShowMeAgain"
> >   feature> 
> > and the FrameworkIntegrationPlugin is not available saying the user to
> > install that package if he wants to get the functionality.
> > 
> > Suggestions?
> > 
> > For those that say this is an hipothetical situation that makes no sense
> > to
> > fix, please go and read https://bugs.kde.org/show_bug.cgi?id=362082
> > 
> > I lost hours of my life trying to figure out what was wrong.
> 
> Hello Albert,
> 
> Just a naive question (sorry if I miss the point). If the use of the
> don't-show-again pluggin is critical to your app, can't you also add a
> strong dependency on frameworkintegration to your app ?

Yes, You miss the point.

This is not about me, it is about a class providing a feature that doesn't 
work at all unless you use have another plugin installed, a plugin whose name 
is not "makeKMessageBoxWorkCorrectly" so it's no immediately obvious you're 
missing it, and also about the class not saying anywhere in it's documentation 
that one of its core features don't work without the plugin.

Cheers,
  Albert

> 
> Best,
> 
> Hugo
> 
> > Cheers,
> > 
> >    Albert
> > 
> > _______________________________________________
> > Kde-frameworks-devel mailing list
> > Kde-frameworks-devel at kde.org
> > https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
> 
> _______________________________________________
> Kde-frameworks-devel mailing list
> Kde-frameworks-devel at kde.org
> https://mail.kde.org/mailman/listinfo/kde-frameworks-devel