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

Sort Extensions by Name in ACP Ext Mgr

    Details

    • Type: Bug
    • Status: Unverified Fix
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.1.0-a2
    • Fix Version/s: 3.1.0-b3
    • Component/s: Extensions
    • Labels:
      None

      Description

      Currently, installed / uninstalled Extensions in the Ext Manager in the ACP seems order arbitrarily.

      I think it should be alphabetically ordered by Ext Display Name.

      Just makes it a lot easier to find them if you need to manage them.

        Activity

        Hide
        nickvergessen Joas Schilling added a comment -

        Should be sorted by vendor/ext-name in my opinion?

        Show
        nickvergessen Joas Schilling added a comment - Should be sorted by vendor/ext-name in my opinion?
        Hide
        VSE Matt Friedman added a comment - - edited

        In the Ext Mgr, you do not see vendor names... Just the names of the extensions. Which is fine. The Admin won't care where it is from, they may just know they want to remove their "Board Rules" or "Gallery" extension for some reason, so its easiest to find just by looking at the list of installed extensions if sorted by name.

        Trying to do it by vendor name will not make sense, since the vendor name is not shown, so it won't appear to be sorted by any logical means if the vendor name is part of the sort. And again, admin's won't care who the vendor is, they'll just know their extensions by name for the most part.

        Show
        VSE Matt Friedman added a comment - - edited In the Ext Mgr, you do not see vendor names... Just the names of the extensions. Which is fine. The Admin won't care where it is from, they may just know they want to remove their "Board Rules" or "Gallery" extension for some reason, so its easiest to find just by looking at the list of installed extensions if sorted by name. Trying to do it by vendor name will not make sense, since the vendor name is not shown, so it won't appear to be sorted by any logical means if the vendor name is part of the sort. And again, admin's won't care who the vendor is, they'll just know their extensions by name for the most part.
        Hide
        imkingdavid David King added a comment -

        I agree with Matt - whatever is visible to the user should be used for sorting, not a hidden vendor name. Perhaps in later versions we can include sort/filter options as necessary/appropriate.

        Show
        imkingdavid David King added a comment - I agree with Matt - whatever is visible to the user should be used for sorting, not a hidden vendor name. Perhaps in later versions we can include sort/filter options as necessary/appropriate.
        Hide
        nickvergessen Joas Schilling added a comment -

        Yeah when its not displayed, it does not make much sense, thats right.

        Show
        nickvergessen Joas Schilling added a comment - Yeah when its not displayed, it does not make much sense, thats right.
        Hide
        geetakshi geetakshi [X] (Inactive) added a comment -

        The list seems to be sorted in alphabetical order. The only thing I could notice was that in case of disabled extensions: The extensions with deleted data occur after the ones which are only disabled, with their data still present, irrespective of their alphabetical order. Also these two are sorted within themselves. But I think it is intentionally kept this way, to separate them.

        Show
        geetakshi geetakshi [X] (Inactive) added a comment - The list seems to be sorted in alphabetical order. The only thing I could notice was that in case of disabled extensions: The extensions with deleted data occur after the ones which are only disabled, with their data still present, irrespective of their alphabetical order. Also these two are sorted within themselves. But I think it is intentionally kept this way, to separate them.
        Hide
        VSE Matt Friedman added a comment - - edited

        They are alphabetical... But not in the right way.

        They are alphabetized by
        vendor name first, then extension name

        We want them listed alphabetically by their display name.

        Right now you could have this:
        phpbb/boardrules (display name Board Rules)
        acme/demo (display name Foobar Demo)

        They would appear like this:
        "Foobar Demo"
        "Board Rules"

        As you can image, with somebody having dozens of extensions installed, it will get very hard to find them in the list if they are not alphabetized by their display name.

        Show
        VSE Matt Friedman added a comment - - edited They are alphabetical... But not in the right way. They are alphabetized by vendor name first, then extension name We want them listed alphabetically by their display name. Right now you could have this: phpbb/boardrules (display name Board Rules) acme/demo (display name Foobar Demo) They would appear like this: "Foobar Demo" "Board Rules" As you can image, with somebody having dozens of extensions installed, it will get very hard to find them in the list if they are not alphabetized by their display name.
        Hide
        VSE Matt Friedman added a comment - - edited

        See screenshot attached above

        Any thoughts on breaking up the extensions list further by all the possible groupings available?

        Enabled
        Disabled
        Uninstalled (or Available ?)
        Invalid

        Show
        VSE Matt Friedman added a comment - - edited See screenshot attached above Any thoughts on breaking up the extensions list further by all the possible groupings available? Enabled Disabled Uninstalled (or Available ?) Invalid
        Hide
        imkingdavid David King added a comment - - edited

        Well, "Uninstalled" isn't going to work because extensions aren't "installed" as it is (they are simply "enabled"). I don't think we should make so much of a differentiation between disabled and available (except by the addition of the "Delete data" link). Perhaps just group them all under Available? "Disabled" implies that the extension was previously enabled and an action having been taken to disable it, which is not true in all cases (i.e. extensions that were uploaded but not yet enabled).

        So I say we go with "Enabled", "Available", and "Invalid".

        Show
        imkingdavid David King added a comment - - edited Well, "Uninstalled" isn't going to work because extensions aren't "installed" as it is (they are simply "enabled"). I don't think we should make so much of a differentiation between disabled and available (except by the addition of the "Delete data" link). Perhaps just group them all under Available? "Disabled" implies that the extension was previously enabled and an action having been taken to disable it, which is not true in all cases (i.e. extensions that were uploaded but not yet enabled). So I say we go with "Enabled", "Available", and "Invalid".

          People

          • Assignee:
            nickvergessen Joas Schilling
            Reporter:
            VSE Matt Friedman
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development