Settings for journal managers

PlagiarismPlugin.inc.php

@@ -20,7 +20,7 @@ public function register($category, $path, $mainContextId = null) {
 		$success = parent::register($category, $path, $mainContextId);
 		$this->addLocaleData();
 
-		if ($success && Config::getVar('ithenticate', 'ithenticate') && $this->getEnabled()) {
+		if ($success && $this->getEnabled()) {
 			HookRegistry::register('submissionsubmitstep4form::execute', array($this, 'callback'));
 		}
 		return $success;
@@ -37,25 +37,67 @@ public function getDisplayName() {
 	 * @copydoc Plugin::getDescription()
 	 */
 	public function getDescription() {
-		return Config::getVar('ithenticate', 'ithenticate')?__('plugins.generic.plagiarism.description'):__('plugins.generic.plagiarism.description.seeReadme');
+		return __('plugins.generic.plagiarism.description');
 	}
 
 	/**
 	 * @copydoc LazyLoadPlugin::getCanEnable()
 	 */
-	function getCanEnable() {
-		if (!parent::getCanEnable()) return false;
-		return Config::getVar('ithenticate', 'ithenticate');
+	function getCanEnable($contextId = null) {
+		return !Config::getVar('ithenticate', 'ithenticate');
+	}
+
+	/**
+	 * @copydoc LazyLoadPlugin::getCanDisable()
+	 */
+	function getCanDisable($contextId = null) {
+		return !Config::getVar('ithenticate', 'ithenticate');
 	}
 
 	/**
 	 * @copydoc LazyLoadPlugin::getEnabled()
 	 */
 	function getEnabled($contextId = null) {
-		if (!parent::getEnabled($contextId)) return false;
-		return Config::getVar('ithenticate', 'ithenticate');
+		return parent::getEnabled($contextId) || Config::getVar('ithenticate', 'ithenticate');
+	}
+
+	/**
+	 * Fetch credentials from config.inc.php, if available
+	 * @return array username and password, or null(s)
+	**/
+	function getForcedCredentials() {
+		$request = Application::getRequest();
+		$context = $request->getContext();
+		$contextPath = $context->getPath();
+		$username = Config::getVar('ithenticate', 'username[' . $contextPath . ']',
+				Config::getVar('ithenticate', 'username'));
+		$password = Config::getVar('ithenticate', 'password[' . $contextPath . ']',
+				Config::getVar('ithenticate', 'password'));
+		return array($username, $password);
 	}
 
+	/**
+	 * Send the editor an error message
+	 * @param $submissionid int
+	 * @param $message string
+	 * @return void
+	**/
+	public function sendErrorMessage($submissionid, $message) {
+		$request = Application::getRequest();
+		$context = $request->getContext();
+		import('classes.notification.NotificationManager');
+		$notificationManager = new NotificationManager();
+		$roleDao = DAORegistry::getDAO('RoleDAO'); /* @var $roleDao RoleDAO */
+		// Get the managers.
+		$managers = $roleDao->getUsersByRoleId(ROLE_ID_MANAGER, $context->getId());
+		$managersArray = $managers->toAssociativeArray();
+		$allUserIds = array_keys($managersArray);
+		foreach ($allUserIds as $userId) {
+			$notificationManager->createTrivialNotification($userId, NOTIFICATION_TYPE_ERROR, array('contents' => __('plugins.generic.plagiarism.errorMessage', array('submissionId' => $submissionid, 'errorMessage' => $message))));
+		}
+		error_log('iThenticate submission '.$submissionid.' failed: '.$message);
+	}
+
 	/**
 	 * Send submission files to iThenticate.
 	 * @param $hookName string
@@ -65,26 +107,37 @@ public function callback($hookName, $args) {
 		$request = Application::getRequest();
 		$context = $request->getContext();
 		$contextPath = $context->getPath();
-		$submissionDao = Application::getSubmissionDAO();
+		$submissionDao = DAORegistry::getDAO('SubmissionDAO'); /* @var $submissionDao SubmissionDAO */
 		$submission = $submissionDao->getById($request->getUserVar('submissionId'));
 		$publication = $submission->getCurrentPublication();
 
 		require_once(dirname(__FILE__) . '/vendor/autoload.php');
 
 		// try to get credentials for current context otherwise use default config
-		$username = Config::getVar('ithenticate', 'username[' . $contextPath . ']',
-				Config::getVar('ithenticate', 'username'));
-		$password = Config::getVar('ithenticate', 'password[' . $contextPath . ']',
-				Config::getVar('ithenticate', 'password'));
-		$ithenticate = new \bsobbe\ithenticate\Ithenticate($username, $password);
+        	$contextId = $context->getId();
+		$credentials = $this->getForcedCredentials(); 
+		$username = $credentials[0];
+		$password = $credentials[1];
+		if (empty($username) || empty($password)) {
+			$username = $this->getSetting($contextId, 'ithenticate_user');
+			$password = $this->getSetting($contextId, 'ithenticate_pass');
+		}
+
+		$ithenticate = null;
+		try {
+			$ithenticate = new \bsobbe\ithenticate\Ithenticate($username, $password);
+		} catch (Exception $e) {
+			$this->sendErrorMessage($submission->getId(), $e->getMessage());
+			return false;
+		}
 		// Make sure there's a group list for this context, creating if necessary.
 		$groupList = $ithenticate->fetchGroupList();
 		$contextName = $context->getLocalizedName($context->getPrimaryLocale());
 		if (!($groupId = array_search($contextName, $groupList))) {
 			// No folder group found for the context; create one.
 			$groupId = $ithenticate->createGroup($contextName);
 			if (!$groupId) {
-				error_log('Could not create folder group for context ' . $contextName . ' on iThenticate.');
+				$this->sendErrorMessage($submission->getId(), 'Could not create folder group for context ' . $contextName . ' on iThenticate.');
 				return false;
 			}
 		}
@@ -97,7 +150,7 @@ public function callback($hookName, $args) {
 			true,
 			true
 		))) {
-			error_log('Could not create folder for submission ID ' . $submission->getId() . ' on iThenticate.');
+			$this->sendErrorMessage($submission->getId(), 'Could not create folder for submission ID ' . $submission->getId() . ' on iThenticate.');
 			return false;
 		}
 
@@ -117,12 +170,63 @@ public function callback($hookName, $args) {
 				Services::get('file')->fs->read($file->path),
 				$folderId
 			)) {
-				error_log('Could not submit "' . $submissionFile->getData('path') . '" to iThenticate.');
+				$this->sendErrorMessage($submission->getId(), 'Could not submit "' . $submissionFile->getData('path') . '" to iThenticate.');
 			}
 		}
 
 		return false;
 	}
+
+	/**
+     * @copydoc Plugin::getActions()
+     */
+    function getActions($request, $verb) {
+        $router = $request->getRouter();
+        import('lib.pkp.classes.linkAction.request.AjaxModal');
+        return array_merge(
+                $this->getEnabled() ? array(
+            new LinkAction(
+                    'settings',
+                    new AjaxModal(
+                            $router->url($request, null, null, 'manage', null, array('verb' => 'settings', 'plugin' => $this->getName(), 'category' => 'generic')),
+                            $this->getDisplayName()
+                    ),
+                    __('manager.plugins.settings'),
+                    null
+            ),
+                ) : array(),
+                parent::getActions($request, $verb)
+        );
+    }
+
+    /**
+     * @copydoc Plugin::manage()
+     */
+    function manage($args, $request) {
+        switch ($request->getUserVar('verb')) {
+            case 'settings':
+                $context = $request->getContext();
+
+                AppLocale::requireComponents(LOCALE_COMPONENT_APP_COMMON, LOCALE_COMPONENT_PKP_MANAGER);
+                $templateMgr = TemplateManager::getManager($request);
+                $templateMgr->registerPlugin('function', 'plugin_url', array($this, 'smartyPluginUrl'));
+
+                $this->import('PlagiarismSettingsForm');
+                $form = new PlagiarismSettingsForm($this, $context->getId());
+
+                if ($request->getUserVar('save')) {
+                    $form->readInputData();
+                    if ($form->validate()) {
+                        $form->execute();
+                        return new JSONMessage(true);
+                    }
+                } else {
+                    $form->initData();
+                }
+                return new JSONMessage(true, $form->fetch($request));
+        }
+        return parent::manage($args, $request);
+    }
 }
 
 /**

PlagiarismSettingsForm.inc.php

@@ -0,0 +1,70 @@
+<?php
+
+import('lib.pkp.classes.form.Form');
+
+class PlagiarismSettingsForm extends Form {
+
+	/** @var int */
+	var $_contextId;
+
+	/** @var object */
+	var $_plugin;
+
+	/**
+	 * Constructor
+	 * @param $plugin PlagiarismPlugin
+	 * @param $contextId int
+	 */
+	function __construct($plugin, $contextId) {
+		$this->_contextId = $contextId;
+		$this->_plugin = $plugin;
+
+		parent::__construct($plugin->getTemplateResource('settingsForm.tpl'));
+
+                $this->addCheck(new FormValidator($this, 'ithenticate_user', 'required', 'plugins.generic.plagiarism.manager.settings.usernameRequired'));
+		$this->addCheck(new FormValidator($this, 'ithenticate_pass', 'required', 'plugins.generic.plagiarism.manager.settings.passwordRequired'));
+
+		$this->addCheck(new FormValidatorPost($this));
+		$this->addCheck(new FormValidatorCSRF($this));
+	}
+
+	/**
+	 * Initialize form data.
+	 */
+	function initData() {
+		$credentials = $this->_plugin->getForcedCredentials();
+		$username = $credentials[0];
+		$password = $credentials[1];
+		$this->_data = array(
+                        'ithenticate_user' => $this->_plugin->getSetting($this->_contextId, 'ithenticate_user'),
+			'ithenticate_pass' => $this->_plugin->getSetting($this->_contextId, 'ithenticate_pass'),
+			'ithenticate_forced' => !empty($username) && !empty($password)
+		);
+	}
+
+	/**
+	 * Assign form data to user-submitted data.
+	 */
+	function readInputData() {
+                $this->readUserVars(array('ithenticate_user'));
+		$this->readUserVars(array('ithenticate_pass'));
+	}
+
+	/**
+	 * @copydoc Form::fetch()
+	 */
+	function fetch($request, $template = null, $display = false) {
+		$templateMgr = TemplateManager::getManager($request);
+		$templateMgr->assign('pluginName', $this->_plugin->getName());
+		return parent::fetch($request, $template, $display);
+	}
+
+	/**
+	 * @copydoc Form::execute()
+	 */
+	function execute(...$functionArgs) {
+                $this->_plugin->updateSetting($this->_contextId, 'ithenticate_user', trim($this->getData('ithenticate_user'), "\"\';"), 'string');
+		$this->_plugin->updateSetting($this->_contextId, 'ithenticate_pass', trim($this->getData('ithenticate_pass'), "\"\';"), 'string');
+		parent::execute(...$functionArgs);
+	}
+}

README.md

@@ -0,0 +1,54 @@
+# iThenticate Plagiarism Detector Plugin
+
+For OJS/OMP/OPS 3.x
+
+## Overview
+
+This plugin permits automatic submission of uploaded manuscripts to the [iThenticate service](http://www.ithenticate.com/) for plagiarism checking.
+1. You need an account of ithenticate.com (costs involved)
+   * paid via Crossref Similarity Check
+   * or, paid directly to iThenticate
+2. Install the plugin via the Plugin Gallery in the Dashboard
+3. Configure the plugin (see below)
+   * Enable the plugin via config.inc.php or in a specific journal/press/preprint context
+   * Configure the plugin with the username and password you get from ithenticate.com
+   * ![Example Settings configuration](ithenticate-settings.png)
+4. The author logs in and makes a submission
+   * The submission files will be sent to iThenticate in Step 4 of the submission process
+5. The Editor logs in to ithenticate.com to see the submission
+   * The submission will be found in a folder named by the Submission ID
+   * Click to see the report
+   * ![Example report review](ithenticate-report.png)
+
+Watch [the demo](https://www.ithenticate.com/demo) to know more about the features of iThenticate.
+
+## Configuration
+
+You may set the credentials in config.inc.php, or you may set the credentials per-journal in the plugin settings.  If credentials are present in config.inc.php, they will override those entered in the plugin settings form.
+
+The config.inc.php settings format is:
+
+```
+;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
+; iThenticate Plugin Settings ;
+;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
+
+[ithenticate]
+
+; Enable iThenticate to submit manuscripts after submit step 4
+;ithenticate = On
+
+; Credentials can be set by context : specify journal path
+; The username to access the API (usually an email address)
+;username[MyJournal_path] = "user@email.com"
+; The password to access the API
+;password[MyJournal_path] = "password"
+
+; default credentials
+; The username to access the API (usually an email address)
+;username = "user@email.com"
+
+; The password to access the API
+;password = "password"
+```
+

locale/en_US/locale.po

@@ -17,5 +17,23 @@ msgstr "iThenticate Plagiarism Detector Plugin"
 msgid "plugins.generic.plagiarism.description"
 msgstr "Send all submissions to iThenticate to be checked for possible plagiarism."
 
-msgid "plugins.generic.plagiarism.description.seeReadme"
-msgstr "Send all submissions to iThenticate to be checked for possible plagiarism. <strong>See the README document for this plugin for installation instructions.</strong>"
+msgid "plugins.generic.plagiarism.manager.settings.description"
+msgstr "Settings for the account used to upload submissions to iThenticate.  Contact your iThenticate administrator for details."
+
+msgid "plugins.generic.plagiarism.manager.settings.username"
+msgstr "iThenticate Usename"
+
+msgid "plugins.generic.plagiarism.manager.settings.password"
+msgstr "iThenticate Password"
+
+msgid "plugins.generic.plagiarism.manager.settings.usernameRequired"
+msgstr "iThenticate Usename is required"
+
+msgid "plugins.generic.plagiarism.manager.settings.passwordRequired"
+msgstr "iThenticate Password is required"
+
+msgid "plugins.generic.plagiarism.manager.settings.areForced"
+msgstr "iThenticate settings are in config.inc.php and changes here will not be used."
+
+msgid "plugins.generic.plagiarism.errorMessage"
+msgstr "Upload of submission {$submissionId} to iThenticate failed with error: {$errorMessage}"