OJS v.2 - Copyediting - is this a bug?

Are you an Editor, Author, or Journal Manager in need of help? Want to talk to us about workflow issues? This is your forum.

Moderators: jmacgreg, michael, vgabler, John

Forum rules
The Public Knowledge Project Support Forum is moving to http://forum.pkp.sfu.ca

This forum will be maintained permanently as an archived historical resource, but all new questions should be added to the new forum. Questions will no longer be monitored on this old forum after March 30, 2015.
Posts: 14
Joined: Mon Aug 15, 2005 8:28 am
Location: Brasil

OJS v.2 - Copyediting - is this a bug?

Postby roberta_takenaka » Mon Aug 15, 2005 8:49 am

I am trying to understand the code in classes/submission/copyeditor/CopyeditorAction.inc.php at function uploadCopyeditVersion($copyeditorSubmission, $copyeditStage), where there are the lines:

Code: Select all

// Only allow an upload if they're in the initial or final copyediting
      // stages.
      if ($copyeditStage == 'initial' && ($copyeditorSubmission->getDateNotified() == null || $copyeditorSubmission->getDateCompleted() != null)) return;
      else if ($copyeditorSubmission->getDateFinalNotified() == null || $copyeditorSubmission->getDateFinalCompleted() != null) return;

Is anything missing in else if condition, like
$copyeditStage == 'final' &&

I think there is a bug here, because the copyeditor is allowed to upload a file, but the file can not be saved because the else if condition is true, so the return statement is executed.

Posts: 338
Joined: Tue Oct 14, 2003 8:23 pm

Postby kevin » Mon Aug 15, 2005 8:22 pm

Yes, that does seem to be a bug. Your suggested fix should work.


Posts: 9
Joined: Thu Oct 20, 2005 9:01 am

Postby nfeasey » Mon Oct 31, 2005 1:19 pm

Did this issue get fixed in a subsequent release. We seem to be having the same issue in which everything seems to go ok with the upload but the file is not, in fact updated.

If this is the fix, how should it be coded??

Thanks in advance.

Posts: 10015
Joined: Wed Aug 10, 2005 12:56 pm

Postby asmecher » Mon Oct 31, 2005 1:53 pm


This issue is corrected in the current version, OJS 2.0.2-1. Please check that you're using the most up-to-date version.

Alec Smecher
Open Journal Systems


Postby Guest » Mon Oct 31, 2005 1:58 pm

Thanks for you reply.

Unfortunately our version is being customized for version 2.0.1 so I hesitate to upgrade at this point as it will affect an ongoing project.

Anyway you can give me the code in question as per this post so that I can just stick the fix in for now?

It would be really appreciated if possible.

Thanks in advance.


Postby Guest » Mon Oct 31, 2005 2:13 pm


I thinking about it, I can just download the latest version and file the file in this post from the latest version and make the necessary changes.

I assume this is a safe thing to do.

Posts: 10015
Joined: Wed Aug 10, 2005 12:56 pm

Postby asmecher » Tue Nov 01, 2005 12:42 pm

The above change in the snippet of code should be enough to correct the issue.

You might consider downloading a patch from 2.0.1 to 2.0.2-1 (see http://pkp.sfu.ca/ojs/download/patch/) and doing a dry run (e.g. patch --dry-run -p1 < the.patch.file). It's possible that you could still perform an upgrade with minimal conflicts.

Alec Smecher
Open Journal Systems Team

Return to “OJS Editorial Support and Discussion”

Who is online

Users browsing this forum: No registered users and 1 guest