The response for SitesManager.updateSite is wrong. Calling it with an unknown idSite parameter results in
{"result":"success", "message":"ok"}
This makes no sense - and can't IMHO not be deserved, as delteSite returns
{"result":"error", "message":"website id = 124 not found"}
on error.
Thanks for your work!
Steffen
Attachment: Patch file
1585.diff
To may be speed it up a bit and make your life easier :) I've attached a patch, which adds the same check, which is also done in deleteSite()
Thanks, but getAllSitesId() requires superuser access. Some refactoring needs to be done.
(In [2928]) fixes #1585