We are moving to Git Issues for bug tracking in future releases. During transition, content will be in both tools. If you'd like to file a new bug, please create an issue.

Bug 6802 - New submission notifications not sent to editors
New submission notifications not sent to editors
Status: RESOLVED FIXED
Product: OJS
Classification: Unclassified
Component: Editors
2.3.7
All All
: P3 normal
Assigned To: PKP Support
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-08-03 13:06 PDT by Matthew Crider
Modified: 2011-08-04 12:12 PDT (History)
1 user (show)

See Also:
Version Reported In:
Also Affects:


Attachments
Patch against OJS 2.3.6 (882 bytes, patch)
2011-08-03 13:11 PDT, Matthew Crider
Details | Diff
Patch against OJS 2.3.6 (749 bytes, patch)
2011-08-03 14:13 PDT, Matthew Crider
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Matthew Crider 2011-08-03 13:06:13 PDT
List of editors not properly constructed when sending new article notification.
Comment 1 Matthew Crider 2011-08-03 13:11:27 PDT
Created attachment 3606 [details]
Patch against OJS 2.3.6
Comment 2 Alec Smecher 2011-08-03 13:19:48 PDT
Needs porting to OCS. Also, beware of references -- if you pass the object off to another function it may behave unexpectedly. The best pattern to use is:

while ($object = $iterator->next()) {
   // Do something
}
Comment 3 Matthew Crider 2011-08-03 13:20:02 PDT
Fix editor list construction for new article notifications
https://github.com/pkp/ojs/commit/3d5e0ef8d00e31ae76b16da816e00cbcb9f6098e
Comment 4 Matthew Crider 2011-08-03 13:20:02 PDT
Fix editor list construction for new article notifications
https://github.com/pkp/ojs/commit/4ec1fb590dc5d9eacba6e0ca3890dd8eed68c348
Comment 5 Matthew Crider 2011-08-03 14:10:02 PDT
Cleaned up editor selection for new submission notification
https://github.com/pkp/ocs/commit/365a4c8f4136385984e33108c6ea80f103456852
Comment 6 Matthew Crider 2011-08-03 14:10:02 PDT
Cleaned up editor selection for new submission notification
https://github.com/pkp/ocs/commit/d10549d68ee04486e6c6941a42663a4b52fecb7e
Comment 7 Matthew Crider 2011-08-03 14:10:02 PDT
Cleaned up editor selection for new submission notification
https://github.com/pkp/ojs/commit/0aec01b4e9dd90e1b53d9d24b4dc22e05f0e5bcc
Comment 8 Matthew Crider 2011-08-03 14:13:38 PDT
Created attachment 3607 [details]
Patch against OJS 2.3.6
Comment 9 Alec Smecher 2011-08-03 14:18:17 PDT
Attachment #3607 [details] is OK, but in general, beware the reference -- best to use either:

while ($thing =& $iterator->next()) {
   // Do something, then...
   unset($thing);
}

...or don't use references...

while ($thing = $iterator->next()) {
   // Do something
}
Comment 10 Matthew Crider 2011-08-03 14:20:02 PDT
Cleaned up editor selection for new submission notification
https://github.com/pkp/ojs/commit/6e172849d367481b122d23cea01a96c267ce515c
Comment 11 Matthew Crider 2011-08-03 14:20:41 PDT
I am unsetting $editor -- its just not in the patch (was already doing it).
Comment 12 Alec Smecher 2011-08-03 14:36:52 PDT
Great! Didn't see that.
Comment 13 Matthew Crider 2011-08-04 12:12:31 PDT
Think we're all done here..