GitLab

Since these functions exit (or throw exception) after running, there
is no need to have a 'return' statement or similar afterwards.

clientError and serverError exit after they're done so no need for
break or return. Also, $this->format is default.

We also got rid of the incredibly verbose version of $this->isPost()
which was spread all over the place.

Not all of this cleaning up is done yet.

Fix XML API output for several profile update methods that returned a <user> entry but didn't set namespaces, causing XML parse failures.

* translator documentation updated.
* superfluous whitespace removed.

* use correct punctuation
* single quotes when replace was possible
* wording updated when needed

* use correct punctuation
* single quotes when replace was possible
* wording updated when needed

when there isn't an existing Design for the user. Plus a few other
fixups.

POST length > post_max_size in php.ini. I also added this check to
avatar upload, which was failing with huge files.