Proposal: Default extendables' appliesTo to extendables/subfolder about klasa HOT 12 CLOSED

Every piece could have a subfolder property (similar to category/subcategory in commands) in which would contain the location of the piece. Every piece would have it assigned to variable its suppose to be or null, and then can be used in the piece that it might be needed in, i.e. for appliesTo.

Issues I could see with this:
- Performance drops for having to read extra dirs that may or may not be empty. (All other pieces beside commands to my knowledge only read the top dir). Klasa does this already.

• Pieces that require specific subfolder names would error.

from klasa.

+1 to this idea, makes it way more organizied and this should be easy to implement since klasa kinda does something simular to command folder

from klasa.

If this idea is implemented, it should be moved to the base class, and all of the pieces should inherit it, regardless of whether they use it or not to keep it consistent.

from klasa.

Would subfolder just be ignored if they don't use it?

from klasa.

Edit: I thought of another caveat

from klasa.

Fair point, but what do you mean by "Pieces that require specific subfolder names would error"?

from klasa.

So like if you want to have a User extendables folder, and you incorrectly spelled User or anything that would make it not "User" when you try to apply it, it would fail.

from klasa.

Your proposal lacks of a little detail: the option klasa from ExtendableOptions.

from klasa.

I like this. IMO the klasa ones should go into a subcategory called klasa and have an appliesTo arg regardless.

from klasa.

Jacz suggested that, too. ./extendables/klasa/KlasaUser/profile.js

from klasa.

Oh, I misunderstood you. That would work, too.

from klasa.

No, while you can already organize pieces into subfolders, it's not really fitting to make that have meaning here.

from klasa.