DumpStringOutput: Rename getOutput() to __toString()
authorKevin Israel <pleasestand@live.com>
Thu, 14 Jul 2016 10:28:16 +0000 (06:28 -0400)
committerKevin Israel <pleasestand@live.com>
Thu, 14 Jul 2016 10:28:16 +0000 (06:28 -0400)
commitbe46ffa771e9bc6b09526b8acfd09d468987f91e
tree0f3a7ae8b0d5736790cb4f03f08431d1b1c61fe7
parent1a7bde454ea4e0825e48037fba66638b4e967333
DumpStringOutput: Rename getOutput() to __toString()

Though getOutput() is what first came to mind, I do not particularly
like the name, partly because it is used in many, many places as a
method that returns an OutputPage object.

* Renamed the method to __toString(). This is appropriate because
  each instance, at any given time, corresponds to a single string
  value (and exceptions cannot occur during this conversion).
* Removed unnecessary variables from ApiQuery and ExportTest. In
  these and most other cases, it should no longer be necessary to
  call getOutput() explicitly.

Change-Id: Icf202743d1f332f8981338f42eb6e3e5a04abdf1
includes/api/ApiQuery.php
includes/export/DumpStringOutput.php
tests/phpunit/includes/ExportTest.php