rdbms: Remove weird use of serialize() in MssqlBlob and DatabaseMssql
authorBrad Jorsch <bjorsch@wikimedia.org>
Mon, 8 Apr 2019 15:15:54 +0000 (11:15 -0400)
committerBrad Jorsch <bjorsch@wikimedia.org>
Mon, 8 Apr 2019 15:15:54 +0000 (11:15 -0400)
commit209683eff02354637761988e499b946525d7aa8f
treef4ddde5e984491fd3ff955d905c18adf870aec82
parentcf8f1fe9b0b5647d1ed3955f6ea6bce7d76fed44
rdbms: Remove weird use of serialize() in MssqlBlob and DatabaseMssql

The use in MssqlBlob could never be reached, since the test used `&&`
rather than `||`.

The use in DatabaseMssql should never be reached, and all our other
Database subclasses will give PHP warnings or errors when it tries to
stringify an array or object rather than silently serializing. MSSQL
should be consistent there. Further, MSSQL never *un*serializes, so it
seems rather useless anyway.

Change-Id: I7b193c6be28662edb38d416ac02371f7a293ab56
includes/libs/rdbms/database/DatabaseMssql.php
includes/libs/rdbms/encasing/MssqlBlob.php