Makes no sense by definition. In API.php should be only methods that are callable via the HTTP API.
Instead those methods should be moved in different classes. Such as a Model or something more meaningful.
Just to be sure. That would currently include the following API methods:
If any of those methods shouldn't be moved, let me know.
Otherwise won't do that in one big PR, but in one PR per plugin or so. That should make it easier and faster to review.
I'm now wondering how important it actually is. These methods can't be called through the API right? It doesn't seem all that important and I reckon we could close the issue if they can't be called directly through the API.
They can't be called directly. Nevertheless it might not make any sense to have public methods in an API class that should not be callable. But if it's not important right now, I'll stop working on it for now.
Feel free to close the issue or move it to a later milestone...
Agreed it doesn't make sense but it's also not any important. It really doesn't make any difference at the end of the day. Will close it.