Pdfjsviewer template override master

.gitmodules

@@ -0,0 +1,3 @@
+[submodule "pdf.js"]
+	path = pdf.js
+	url = https://github.com/hypothesis/pdf.js-hypothes.is.git

HypothesisPlugin.inc.php

@@ -21,6 +21,7 @@ class HypothesisPlugin extends GenericPlugin {
 	function register($category, $path, $mainContextId = null) {
 		if (parent::register($category, $path, $mainContextId)) {
 			HookRegistry::register('ArticleHandler::download',array(&$this, 'callback'));
+			HookRegistry::register('TemplateManager::display', array(&$this, 'callbackTemplateDisplay'));
 			return true;
 		}
 		return false;
@@ -43,6 +44,36 @@ function callback($hookName, $args) {
 		return false;
 	}
 
+	/**
+	 * Hook callback function for TemplateManager::display
+	 * @param $hookName string
+	 * @param $args array
+	 * @return boolean
+	 */
+	function callbackTemplateDisplay($hookName, $args) {
+		if ($hookName != 'TemplateManager::display') return false;
+		$templateMgr = $args[0];
+		$template = $args[1];
+		$plugin = 'plugins-generic-pdfJsViewer';
+		$tpl = 'submissionGalley.tpl';
+		// template path contains the plugin path, and ends with the tpl file
+		if (strpos($template, $plugin) !== false && strpos($template, ':'.$tpl,  -1 - strlen($tpl)) !== false) {
+			$templateMgr->registerFilter("output", array($this, 'changePdfjsPath'));
+		}
+		return false;
+	}
+
+	/**
+	 * Output filter to create a new element in a registration form
+	 * @param $output string
+	 * @param $templateMgr TemplateManager
+	 * @return $string
+	 */
+	function changePdfjsPath($output, $templateMgr) {
+		$newOutput = str_replace('pdfJsViewer/pdf.js/web/viewer.html?file=', 'hypothesis/pdf.js/viewer/web/viewer.html?file=', $output);
+		return $newOutput;
+	}
+
 	/**
 	 * Get the display name of this plugin
 	 * @return string

README

@@ -18,7 +18,7 @@ COPYING for the complete terms of this license.
 
 System Requirements
 -------------------
-This plugin has been developed for OJS 3.0.x and may not work with other
+This plugin has been developed for OJS 3.2 and may not work with other
 versions.
 
 Installation
@@ -34,11 +34,8 @@ If you cannot use the Plugin Gallery, you can also install the plugin as follows
  - Read an HTML article in the public reader interface of OJS and verify that
    the Hypothes.is tool is embedded.
 
-In order to support Hypothes.is with OJS's PDF.js-based PDF reader, you'll need
-to make a change to the PDF.js Viewer plugin. To do this, replace the pdf.js
-implementation in plugins/generic/pdfJsViewer/pdf.js with the one from this
-repository: https://github.com/hypothesis/pdf.js-hypothes.is
-
+In order to support Hypothes.is with OJS's PDF.js-based PDF reader, this plugin
+hijacks the template display of the pdfJsViewer plugin.
 
 Contact/Support
 ---------------