Uploaded image for project: 'phpBB3'
  1. phpBB3
  2. PHPBB3-11994

Admin options for extensions are bad/misleading

    Details

    • Type: Bug
    • Status: Unverified Fix
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 3.1.0-a1
    • Fix Version/s: 3.1.0-a2
    • Component/s: ACP, Extensions
    • Labels:
      None

      Description

      I guess, we all know what enable and disable mean. However there is no install option or anything similar. So does this mean that enable == install and disable == uninstall?
      Also what exactly does purge do?

      I know, that purge is uninstall. But this needs clearification.
      basically the options in the UI should be:

      • install
      • disable (db changes is retained)
      • uninstall (db changes are removed)

        Issue Links

          Activity

          Hide
          imkingdavid David King added a comment -

          I'm positive that we've had discussions about terminology for this before, but I don't currently have time to look them up.

          Adding the "db changes..." bits just make the links look ugly. We could probably put that in a tooltip, but not the actual link.

          Install should toggle between "install" and "enable" based on whether or not the extension is already installed (i.e. if an extension is disabled, it is installed but is not enabled, so the correct term would be "enable"; if an extension has not been installed or was purged, it should show "install").

          I can agree to changing "purge" to "uninstall".

          Show
          imkingdavid David King added a comment - I'm positive that we've had discussions about terminology for this before, but I don't currently have time to look them up. Adding the "db changes..." bits just make the links look ugly. We could probably put that in a tooltip, but not the actual link. Install should toggle between "install" and "enable" based on whether or not the extension is already installed (i.e. if an extension is disabled, it is installed but is not enabled, so the correct term would be "enable"; if an extension has not been installed or was purged, it should show "install"). I can agree to changing "purge" to "uninstall".
          Hide
          nickvergessen Joas Schilling added a comment -

          > Adding the "db changes..." bits just make the links look ugly. We could probably put that in a tooltip, but not the actual link.

          That part was only ment for explanation of what the option should do

          Show
          nickvergessen Joas Schilling added a comment - > Adding the "db changes..." bits just make the links look ugly. We could probably put that in a tooltip, but not the actual link. That part was only ment for explanation of what the option should do

            People

            • Assignee:
              nickvergessen Joas Schilling
              Reporter:
              nickvergessen Joas Schilling
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development