Merge LocalFile::decodeRow() with its only caller loadFromRow(), and replace dynamic assignment $this->$field with plain assignment. This makes the data flow more obvious. The point of dynamic assignment was defeated by the fact that almost every field required some kind of normalization after loading it from the database. There's no point looping over all fields when you need to do something different for each field. Dynamic assignment is retained in loadFromCache() and setProps(), where there is less normalization. Change-Id: Ie103884c4436ab63c29652edf3bab331a34081c9 |
||
|---|---|---|
| .. | ||
| api-testing | ||
| common | ||
| integration/includes/http | ||
| parser | ||
| phpunit | ||
| qunit | ||
| selenium | ||
| .eslintrc.json | ||
| .htaccess | ||