Bug 7776 - Books for review assignments not transferred when merging users
Books for review assignments not transferred when merging users
Status: RESOLVED FIXED
Product: OJS
Classification: Unclassified
Component: Plug-ins
2.3.8
All All
: P3 normal
Assigned To: Michael Felczak
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-08-17 06:16 PDT by Michael Felczak
Modified: 2012-08-17 17:55 PDT (History)
2 users (show)

See Also:
Version Reported In:
Also Affects:


Attachments
Patch against OJS 2.3.7 (should also work with 2.3.6, 2.3.5, etc.) (3.11 KB, patch)
2012-08-17 17:54 PDT, Michael Felczak
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Felczak 2012-08-17 06:16:45 PDT
Add hook to UserAction::mergeUsers and transfer book for review records across merged users.
Comment 1 Alec Smecher 2012-08-17 08:43:02 PDT
Moving to 2.4.1 -- the next release will be 2.4.0, but practically speaking I won't have time to implement this before that goes out.
Comment 2 Michael Felczak 2012-08-17 15:21:01 PDT
Thanks Alec.

I should be able to get to this in the next couple of days if you'd still like to include it for 2.4.0
Comment 3 Alec Smecher 2012-08-17 15:27:54 PDT
That would be great, Michael -- didn't know you were looking to do some coding. Please do.
Comment 4 Michael Felczak 2012-08-17 16:42:28 PDT
Sounds good -- I'd just like to have it fixed really.

(I haven't seen any reports of this affecting any installations yet)
Comment 5 Michael Felczak 2012-08-17 17:45:02 PDT
Books for review assignments not transferred when merging users
https://github.com/pkp/ojs/commit/15e157e7805b41567e56032a130d59ad7c6e1b78
Comment 6 Michael Felczak 2012-08-17 17:50:02 PDT
Books for review assignments not transferred when merging users
https://github.com/pkp/ojs/commit/397ddc4cd799413f3a85e44ba5a2a8cbf260e431
Comment 7 Michael Felczak 2012-08-17 17:54:10 PDT
Created attachment 3846 [details]
Patch against OJS 2.3.7 (should also work with 2.3.6, 2.3.5, etc.)
Comment 8 Michael Felczak 2012-08-17 17:55:23 PDT
Fixed.