Remove checks formerly for MySQL server version
authorKevin Israel <pleasestand@live.com>
Mon, 23 May 2016 11:30:52 +0000 (07:30 -0400)
committerAaron Schulz <aschulz@wikimedia.org>
Thu, 11 Aug 2016 04:55:06 +0000 (04:55 +0000)
commitc29dc2b65f4bfb1a928b7cc36c8d48c5524e87d3
tree530557ec1f5b86005c735d4c189aa545a34a4aaa
parent2442464754ab540163098e1f3c30b0996302bb7f
Remove checks formerly for MySQL server version

Removed these checks instead of fixing them, for the stated reasons:

* initEditCount.php: "instanceof DatabaseMysql" is not a correct way to
  check if the database server is MySQL/MariaDB, because DatabaseMysql is
  now a subclass of DatabaseMysqlBase, only used as a wrapper around the
  deprecated mysql PHP extension (not the newer mysqli extension). The
  check was intended for pre-4.1 versions of MySQL, which do not support
  subqueries.

  The effect of this change is to use single-query mode whenever there is
  only one configured database server, even when using the mysqli PHP
  extension, which would have been the original intent.

* fixBug20757.php: If the database server is not MySQL/MariaDB, the
  $lowerLeft variable would remain undefined. The script was only intended
  for use with MySQL, so the check was only necessary in order to support
  pre-4.1 versions of MySQL, which require different SQL.

Follows-up b74f88967bf090af.

Change-Id: I7f32aed4473e5ea39dc40449ddc0af5f9a10df16
maintenance/initEditCount.php
maintenance/storage/fixBug20757.php