Bug 359906

Summary: Applets lost "configurationRequired" functionality in Plasma5
Product: [Plasma] plasmashell Reporter: Friedrich W. H. Kossebau <kossebau>
Component: ContainmentAssignee: Sebastian Kügler <sebas>
Status: CONFIRMED ---    
Severity: minor CC: bshah, hein, justin.zobel, kde, plasma-bugs-null, simonandric5
Priority: NOR    
Version First Reported In: 5.5.4   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Friedrich W. H. Kossebau 2016-02-28 22:47:44 UTC 
While Plasma::Applet still has the API from KDE4 times (`Applet::configurationRequired()`, `Applet::setConfigurationRequired(bool needsConfiguring, const QString &reason)`), there is no `Plasmoid.configurationRequired` property on QML level.
And setting the property from C++ also has no effect in the UI, at least to what I can see.

It might be nice to have this property back in full swing, so there is a consistent way to mark applets from the code which have no sane default and first need some setting to be useful.

Examples: comic applet & weather applet

Reproducible: Always
Comment 1 Kai Uwe Broulik 2016-02-29 00:04:43 UTC 
The API is actually meant to be used imperatively, ie. you call plasmoid.setConfigurationRequired(true, i18n("reason")), I think there should be two distinct properties on the Plasmoid attached object

however the actual code that is supposed to do that looks like:

void AppletPrivate::showConfigurationRequiredMessage(bool show, const QString &reason)
{
    // reimplemented in the UI specific library
    Q_UNUSED(show)
    Q_UNUSED(reason)
}

Looks like we have one week to fix this for 5.6 if it's not already too late.
Comment 2 Justin Zobel 2020-11-19 04:01:19 UTC 
(In reply to Kai Uwe Broulik from comment #1)
> The API is actually meant to be used imperatively, ie. you call
> plasmoid.setConfigurationRequired(true, i18n("reason")), I think there
> should be two distinct properties on the Plasmoid attached object
> 
> however the actual code that is supposed to do that looks like:
> 
> void AppletPrivate::showConfigurationRequiredMessage(bool show, const
> QString &reason)
> {
>     // reimplemented in the UI specific library
>     Q_UNUSED(show)
>     Q_UNUSED(reason)
> }
> 
> Looks like we have one week to fix this for 5.6 if it's not already too late.

Kai was this merged into 5.6 or a later version?