Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature request: Order mv list output alphabetically #1960

Open
SlimeDog opened this issue Jan 26, 2019 · 7 comments · May be fixed by #2824
Open

Feature request: Order mv list output alphabetically #1960

SlimeDog opened this issue Jan 26, 2019 · 7 comments · May be fixed by #2824
Labels
Resolution: Accepted MV-Team is aware of the issue/PR and will be looking into it. Type: Idea Suggestion for a new feature or enhancement.

Comments

@SlimeDog
Copy link

Spigot 1.13.2
MV-Core 2.7.0-SNAPSHOT-b747

>mv list
[22:58:17 INFO]: ====[ Multiverse World List ]====
[22:58:17 INFO]: minigames - NORMAL
[22:58:17 INFO]: yellowdog - NORMAL
[22:58:17 INFO]: skyblock-the-end - THE_END
[22:58:17 INFO]: void - NORMAL
[22:58:17 INFO]: skyblock-nether - NETHER
[22:58:17 INFO]: luckyblock - NORMAL
[22:58:17 INFO]: parkour - NORMAL
[22:58:17 INFO]: skyblock - NORMAL
[22:58:17 INFO]: playground-the-end - THE_END
[22:58:17 INFO]: playground - NORMAL
[22:58:17 INFO]: playground-nether - NETHER
[22:58:17 INFO]: yellowdog-nether - NETHER
[22:58:17 INFO]: lobby - NORMAL
[22:58:17 INFO]: city - NORMAL
[22:58:17 INFO]: city-nether - NETHER

would be more user-friendly if it were

>mv list
[22:58:17 INFO]: ====[ Multiverse World List ]====
[22:58:17 INFO]: city - NORMAL
[22:58:17 INFO]: city-nether - NETHER
[22:58:17 INFO]: lobby - NORMAL
[22:58:17 INFO]: luckyblock - NORMAL
[22:58:17 INFO]: minigames - NORMAL 
[22:58:17 INFO]: parkour - NORMAL
[22:58:17 INFO]: playground - NORMAL
[22:58:17 INFO]: playground-nether - NETHER
[22:58:17 INFO]: playground-the-end - THE_END
[22:58:17 INFO]: skyblock - NORMAL
[22:58:17 INFO]: skyblock-nether - NETHER
[22:58:17 INFO]: skyblock-the-end - THE_END
[22:58:17 INFO]: void - NORMAL
[22:58:17 INFO]: yellowdog - NORMAL
[22:58:17 INFO]: yellowdog-nether - NETHER
@dumptruckman
Copy link
Member

This is a neat idea. However, what if someone would prefer some kind of logical grouping? For instance, if they had a group of survival worlds and a group of creative worlds. I imagine they would prefer those to show up together. As such, I have some ideas for tackling this and providing a way to satisfy both desires.

@dumptruckman dumptruckman added the PR: Enhancement Pull requests to implement a feature or improvement in code. label Jan 30, 2019
@SlimeDog
Copy link
Author

SlimeDog commented Jan 31, 2019

A variety of useful sorting schemes would be great. Alphabetical just makes them easier to find in the list (I have 23 worlds), and I would think the easiest to implement. Grouping is a whole other kettle of fish.

@VL4DST3R
Copy link

What is the current reasoning behind how worlds get sorted? It is so annoying having them mixed at random, even if you import them in a specific order. Even more so, if you try and edit the worlds.yml file to manually sort them, they reset back to the "random" order.

@SlimeDog
Copy link
Author

SlimeDog commented Aug 28, 2019

Since it's been raised again, count again my voice for alphanumeric sorting.

@timriker
Copy link

timriker commented Sep 4, 2019

Another vote for alpha sort. This should include /mv list as well as the api for plugins like dynmap so that worlds are sorted there as well.

@VL4DST3R
Copy link

VL4DST3R commented Sep 4, 2019

Ah yes, dynmap is also affected by this. Iirc you can fix it on the dynmap side by manually rearranging the generated web files, but again, still a pain. Please add alphabetical sort!

@timriker
Copy link

timriker commented Sep 4, 2019

It might be as easy as adding a sort(compareByAlias) to getMVWorlds() with something like:

Collection<MultiverseWorld> compareByAlias = new Comparator<MultiverseWorld>() {
    @Override
    public int compare(MultiverseWorlds a, MultiverseWorld b) {
        return a.getAlias().compareTo(b.getAlias());
    }
};

@benwoo1110 benwoo1110 added this to the Multiverse 5.0.0 milestone Oct 7, 2020
@benwoo1110 benwoo1110 added Type: Idea Suggestion for a new feature or enhancement. Resolution: Accepted MV-Team is aware of the issue/PR and will be looking into it. and removed PR: Enhancement Pull requests to implement a feature or improvement in code. labels Oct 12, 2020
@benwoo1110 benwoo1110 linked a pull request Feb 10, 2023 that will close this issue
75 tasks
@zax71 zax71 linked a pull request Jan 12, 2025 that will close this issue
75 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Accepted MV-Team is aware of the issue/PR and will be looking into it. Type: Idea Suggestion for a new feature or enhancement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants