Don't attempt to copy to ip_changes in PageArchive class.
authorMusikAnimal <musikanimal@gmail.com>
Tue, 19 Sep 2017 22:11:28 +0000 (18:11 -0400)
committerMusikAnimal <musikanimal@gmail.com>
Tue, 19 Sep 2017 22:13:56 +0000 (18:13 -0400)
commit5e7fd24297cb886f2b59a1b9c83db7e2581f31fe
tree64c58c887b03028abe128b8eebcb76cc83aad8c1
parent617d05ae4c195d5edfb14ab21286234e8e79d888
Don't attempt to copy to ip_changes in PageArchive class.

This actually happens in Revision::insertOn, so the code is redundant,
but also shouldn't ever execute because it erroneously was checking
for ar_rev_id=0 and not ar_user=0.

This is sometimes causing errors in production, in the rare cases
where ar_rev_id is NULL, and hence (int)ar_rev_id evaluates to 0.

Change-Id: I35a7310f600a9db683d7dc0e730da8ff31e337a2
includes/page/PageArchive.php