Created attachment 3053[details]
Patch to Streamline EditorSubmission SELECT statement
Enclosed is a patch for the OJS EditorSubmission SELECT statement that calls the DISTINCT switch only when necessary, and also omits many of the joins unless they are specifically needed.

Jacob, a few small things:
- It looks like you're using spaces rather than tabs to indent
- We generally use unified diffs (diff -u); I find them more readable
- Now this is getting really pedantic: For consistency, stick with $lowerCamelCase for variable names rather than $justlowercase.
Otherwise, looking good, thanks!
Alec

Hi Alec,
I would usually use tabs over spaces in my code - It must be a transition between Eclipse (Win)/Linux that caused an issue there. Will make sure that is not lost in the next patch.
Will definitely adhere to -u / Camel-Case to keep to standards in the future.
Cheers,
Jake

Created attachment 3147[details]
Patch against OJS pre-2.3.2 CVS
Sorry I dropped this for so long, Jake; my penance is to update it to work with current CVS (see attachment). Before this can be committed, it'll also need to be ported to the SectionEditorSubmissionDAO, which should be pretty similar. Any chance you could have a crack at it?