Slow me missed that one :D
Screenshots look as expected I think. There is potentially a third case one if you enter a site with a value being int-casted
0 (at least I think that is what is going on there), like a literal zero or something non-numeric. Would that be relevant?
hm... shouldn't hurt to add a test for that as well
Actually setting a invalid site currently results in:
Imho we maybe shouldn't show such an error message to non super users. Most users might not even understand the error, so maybe we should simply redirect to index or similar. Will create a new issue for this, and won't add a test for it now