PKP Bugzilla – Bug 7275
MergeUsers errors when updating comments
Last modified: 2012-06-07 15:46:14 PDT
The MergeUsers utility currently generates a fatal error when it attempts to migrate comments attached to articles. Line 52 of UserAction.inc.php attempts to call $comment->setUserId(int) and no such method exists. (there is a setUser(int) method, though).
There probably won't be a 2.3.8, but just in case there is, rescheduling for it.
There is also a problem on line 276 in CommentDAO. $comment->getUser() on line 255 returns the user id, not the object, so the call to $user->getId() fails.
Update docs for Comment::setUser https://github.com/pkp/pkp-lib/commit/8f6235cf7595f17b6d58eec1f8429242f507987f
Update docs for Comment::setUser https://github.com/pkp/pkp-lib/commit/83590f04988bede7bb0d30cdc422ebc80d8f6fb4
Fix MergeUsers error when updating comments https://github.com/pkp/ocs/commit/5d9b3a5479ef2d605b5fbb5be321abc56576992b
Fix MergeUsers error when updating comments https://github.com/pkp/ojs/commit/673b7cc54c7c7c850a36a50928f85cca0515fca8
Fix MergeUsers error when updating comments https://github.com/pkp/ojs/commit/c764ff631c2b71c44e0ef0043553d488b5745fa6
Created attachment 3791 [details] Patch against OJS 2.3.7
Created attachment 3792 [details] Patch against OCS 2.3.5
Fix MergeUsers error when updating comments https://github.com/pkp/ocs/commit/43fb4e17fed4b863df8a4a1f466a011e523a9ce2
Fixed and added to recommended patches list; closing.