Deprecate ORMTable::getFieldPrefix
authorThiemo Mättig <thiemo.maettig@wikimedia.de>
Mon, 6 Oct 2014 16:04:40 +0000 (18:04 +0200)
committerThiemo Mättig <thiemo.maettig@wikimedia.de>
Tue, 14 Oct 2014 14:14:17 +0000 (16:14 +0200)
commit9bd9c31c737522efd6f2530df883d8c144177380
treead673db6f9cb2ab6ca764dfd4f80e969f188b658
parent55c45e57b394b0ecf3e46948ad24c147cabb43a5
Deprecate ORMTable::getFieldPrefix

Internal calls to this method show up in my XDebug profiling as
the second most called method (after array_key_exists). Called
10000 times. Sure, it's very cheap and clearly not a bottleneck.
But just not necesarry to have a method for that. The benefit of
having a method is to have a dynamic prefix that can change
depending on other things in an object. But I think this is not
a good idea for a prefix.

Since this is protected it is part of the contract of this class.
I already cleaned known subclasses. Should be @deprecated for
some time and can be removed later.

Change-Id: I46a0d7e072d0a69e9aef5f77e92044b62e3d0ce7
includes/db/ORMTable.php
tests/phpunit/includes/db/ORMTableTest.php
tests/phpunit/includes/db/TestORMRowTest.php