import: Soft-deprecate WikiRevision::getUserObj and related methods

Reasoning:

* Not all code needs an actual User object. And even if, most code
  doesn't need it to be validated. That's expensive.
  Let the users decide.

* setUserObj() is almost never called anyway. And the few places that
  call it go the extra mile and create a User object just to please this
  setter. We can avoid or at least postpone this.

* All callers already fall back to getUser() when there is no
  getUserObj().

* For a while we try to replace the User class with much smaller
  interfaces, whenever possible. This is a chance to remove of one more.

* A "user IP" is a concept that doesn't even exist in this code. The
  setter writes to the same property as setUsername(). This makes it
  very easy to use these methods wrong, and this indeed already happened
  in some places.

Change-Id: I6687305a5be68e8a068f11482625a0a6bd9626f2

0 files changed, 0 insertions, 0 deletions