Add multiple prefix support and adds ; as a prefix.

[SuperCater]

This simply adds the ability for commands to take multiple prefixes, I added ; and : as the default prefixes. This removes the need for #1829

Known "issues":

1. The first prefix is the one that shows up in autocomplete, the others are ignored and won't show up.
2. You can no longer change prefixes in the user panel

POF

[WalkerOfBacon]

Can this still allow you to add a single prefix if needed?

[SuperCater]

dont merge yet found an issue i need to quickly fix

[WalkerOfBacon]

Can this still allow you to add a single prefix if needed?

As a string*

[SuperCater]

Can this still allow you to add a single prefix if needed?

Yep still supports taking a string as the Prefix for backwards compatibility

[SuperCater]

dont merge yet found an issue i need to quickly fix

"foxed" it

[SuperCater]

okay yeah dont merge this im stupid and found a couple of issues i need to fix 👍

[WalkerOfBacon]

draft!!

[SuperCater]

Okay made it so server.Settings.Prefix will return the first prefix if using the concat operator, this should fix a couple issues with it.

[SuperCater]

Alright all commands have been updated to work with the new system, will check out the plugins and cores now

[WalkerOfBacon]

there should also be an urgent message regarding these changes also

[SuperCater]

there should also be an urgent message regarding these changes also

Who should it notify? Moderators+ ?

[SuperCater]

btw i used some metatable magic so that it shouldn't break old plugins

I made it so __tostring and __concat return the first prefix in the table, only thing that will break is if you use the length operator or directly reference it

[WalkerOfBacon]

Who should it notify? Moderators+ ?

Creator+

[SuperCater]

Who should it notify? Moderators+ ?

Creator+

@Dimenpsyonal do you think we should do this?

[Dimenpsyonal]

btw i used some metatable magic so that it shouldn't break old plugins

I made it so __tostring and __concat return the first prefix in the table, only thing that will break is if you use the length operator or directly reference it

How do you people even come up with stuff like this

[SuperCater]

Alright I'm pretty sure i've fixed everything, only thing i haven't checked are all the UI modules and frankly im far too lazy to, i tested it most of the important things matter and any remaining bugs shouldn't be too hard to fix if they do pop up

[Dimenpsyonal]

Lint failed

[SuperCater]

btw i used some metatable magic so that it shouldn't break old plugins
I made it so __tostring and __concat return the first prefix in the table, only thing that will break is if you use the length operator or directly reference it

How do you people even come up with stuff like this

wasn't too difficult, added this line of code in the server load process

		if type(server.Settings.Prefix) == "table" then
			setmetatable(server.Settings.Prefix, {
				__concat = function(self, value)
					return self[1]
				end,
				__tostring = function(self)
					return self[1]
				end,

			})
		end

[Dimenpsyonal]

Fascinating

[PurpleCreativity]

Alright I'm pretty sure i've fixed everything, only thing i haven't checked are all the UI modules and frankly im far too lazy to, i tested it most of the important things matter and any remaining bugs shouldn't be too hard to fix if they do pop up

famous last words

[SuperCater]

Alright I'm pretty sure i've fixed everything, only thing i haven't checked are all the UI modules and frankly im far too lazy to, i tested it most of the important things matter and any remaining bugs shouldn't be too hard to fix if they do pop up

famous last words

no doubt there are more issues present since this changes the type of a core setting but they should be obvious and pop up as people use commands.

[SuperCater]

I'm going to look into making it so you can change the prefix ingame

[SuperCater]

okay i dont think you could ever change prefix in game for some reason

[WalkerOfBacon]

okay i dont think you could ever change prefix in game for some reason

Apparently you were never able to individually

[PurpleCreativity]

you shouldn't be able to unless we're talking per-player

[SuperCater]

just realized im stupid and pasted something wrong one sec 😭

[SuperCater]

balls

[SuperCater]

ill open another PR real quick to fix the stupid issue

[Dimenpsyonal]

GOTTEM