Basic camera control API (just the mode)

[sfan5]

closes #12855

let's not bikeshed this. it's a simple and effective improvement.

To do

This PR is Ready for Review.

How to test

probably somewhat like this:

//lua =player:get_camera()
//lua player:set_camera({mode="first"})
//lua player:set_camera({mode="third"})
//lua player:set_camera({mode="asdsadasdasd"})
//lua player:set_camera({})

[Andrey2470T]

How about waiting for the SSCSM controller PR gets implemented not to do the camera control API server-oriented and not to continue "server to client" paradigm? Although this is purely a client-side feature and the server shouldn't load itself with the work which it hasn't actually to do.

[Andrey2470T]

Also FYI you could try to adapt already the pretty ready API, but for the secondary cameras which actually could be applied to the player camera: #14325

[sfan5]

I explicitly don't want to wait for SSCSM to happen nor adopt a high-complexity PR.

[lhofhansl]

I explicitly don't want to wait for SSCSM to happen nor adopt a high-complexity PR.

I agree. I like the simplicity of this (and the beginning of a Camera API that we can extend later - if we wanted to.)
Add this to core.features?

[lhofhansl]

Tried. Works.
Code looks good.

Right now if if you just want to set the mode in a mod, but allow the player to still change it if wanted you'd have to do something like:

      player:set_camera({mode = "third_front"})
      minetest.after(0.1, function(player) player:set_camera({mode = "any"}) end, player)

That might be something we'd want to make easier - but not important.

[appgurueu]

Add this to core.features?

I see no need to when users can just check if player.set_camera then ... end. Not to mention that they should ideally be doing both a serverside and a clientside availability check.