stable
Clone or download
Read-only
request #14708: Disabling a plugin is not error prone
Following Bastien & Scapin's ergonomic criteria, disabling a plugin should display an advisory message requesting user confirmation because plugins will miss events and may be misconfigured or there can be data loss. In the list of all installed plugins, disable a plugin: a confirmation modal is displayed. The same if you disable a plugin from its properties page. Note: no need for confirmation modal when we enable a plugin since there is no data loss. Plus it would burden the installation process more than needed. Change-Id: I6608cf9e020b9d41e09c40a4a9964b0297da4a7b
Modified Files
Name | ||||
---|---|---|---|---|
M | plugins/pluginsadministration/include/PluginPropertiesPresenter.php | +10 | −4 | Go to diff View file |
M | plugins/pluginsadministration/include/PluginsAdministrationViews.class.php | +27 | −57 | Go to diff View file |
M | plugins/pluginsadministration/scripts/pluginsadministration.js | +22 | −3 | Go to diff View file |
M | plugins/pluginsadministration/site-content/fr_FR/LC_MESSAGES/tuleap-pluginsadministration.po | +29 | −0 | Go to diff View file |
A | plugins/pluginsadministration/templates/enable-disable-plugin-switch.mustache | +50 | −0 | Go to diff View file |
M | plugins/pluginsadministration/templates/installed-plugins.mustache | +4 | −4 | Go to diff View file |
M | plugins/pluginsadministration/templates/plugin-properties.mustache | +2 | −2 | Go to diff View file |
M | plugins/pluginsadministration/templates/uninstall-plugin-modal.mustache | +3 | −3 | Go to diff View file |