From ddcd17a7f93fe2a94df62ab76bbff744d372fbe9 Mon Sep 17 00:00:00 2001
From: shinobu <niehoff.fabian@web.de>
Date: Fri, 19 Aug 2016 21:57:22 +0200
Subject: [PATCH] This fixes the require once error, removes the need of
 descriptions for queries and fixes a missing urlBase Parameter

---
 .../savedqueries/SavedqueriesController.php       | 15 +++++++++------
 extensions/savedqueries/SavedqueriesModule.php    |  9 ++-------
 .../templates/savedqueries/init.phtml             |  2 +-
 3 files changed, 12 insertions(+), 14 deletions(-)

diff --git a/extensions/savedqueries/SavedqueriesController.php b/extensions/savedqueries/SavedqueriesController.php
index 5f41351ca..a168b3352 100644
--- a/extensions/savedqueries/SavedqueriesController.php
+++ b/extensions/savedqueries/SavedqueriesController.php
@@ -6,8 +6,6 @@
  * @license   http://opensource.org/licenses/gpl-license.php GNU General Public License (GPL)
  */
 
-require_once 'OntoWiki/Controller/Component.php';
-
 /**
  * Mass Geocoding of Ressources via attributes (parameter r)
  *
@@ -32,14 +30,12 @@ public function init()
         parent::init();
         // m is automatically used and selected
         if ((!isset($this->_request->m)) && (!$this->_owApp->selectedModel)) {
-            require_once 'OntoWiki/Exception.php';
             throw new OntoWiki_Exception('No model pre-selected and missing parameter m (model)!');
         } else {
             $this->_model = $this->_owApp->selectedModel;
         }
 
         // disable tabs
-        require_once 'OntoWiki/Navigation.php';
         OntoWiki::getInstance()->getNavigation()->disableNavigation();
 
         // get translation object
@@ -61,8 +57,15 @@ public function init()
     public function initAction()
     {
         // create a new button on the toolbar
-        $queryResult = $this->_getQueryResult($this->queryString);
-
+        try {
+            $queryResult = $this->_getQueryResult($this->queryString);
+        } catch (Exception $e){
+            $queryResult = array(
+                array(
+                    "error" => "This Query contains errors and should be corrected in the Query Editor",
+                    ),
+                );
+        }
         $header = array();
         try {
             if (is_array($queryResult) && isset($queryResult[0]) && is_array($queryResult[0])) {
diff --git a/extensions/savedqueries/SavedqueriesModule.php b/extensions/savedqueries/SavedqueriesModule.php
index b530c434a..29bf543dd 100644
--- a/extensions/savedqueries/SavedqueriesModule.php
+++ b/extensions/savedqueries/SavedqueriesModule.php
@@ -6,11 +6,6 @@
  * @license   http://opensource.org/licenses/gpl-license.php GNU General Public License (GPL)
  */
 
-require_once 'Erfurt/Sparql/SimpleQuery.php';
-require_once 'OntoWiki/Module.php';
-require_once 'OntoWiki/Url.php';
-require_once 'OntoWiki/Utils.php';
-
 /**
  * @category   OntoWiki
  * @package    Extensions_Savedqueries
@@ -38,8 +33,8 @@ public function getContents()
                 ?query a <" . $this->_privateConfig->queryClass . "> .
                 ?query <" . $this->_privateConfig->queryLabel . "> ?label .
                 ?query <" . $this->_privateConfig->queryId . "> ?id .
-                ?query <" . $this->_privateConfig->queryDesc . "> ?description .
-                ?query <" . $this->_privateConfig->queryCode . "> ?code
+                ?query <" . $this->_privateConfig->queryCode . "> ?code.
+                OPTIONAL { ?query <" . $this->_privateConfig->queryDesc . "> ?description . }
         }";
 
         $elements = $storeGraph->sparqlQuery($query);
diff --git a/extensions/savedqueries/templates/savedqueries/init.phtml b/extensions/savedqueries/templates/savedqueries/init.phtml
index 9a95ef951..83a8d5d33 100644
--- a/extensions/savedqueries/templates/savedqueries/init.phtml
+++ b/extensions/savedqueries/templates/savedqueries/init.phtml
@@ -2,7 +2,7 @@
     <fieldset>
         <?php if (is_array($this->queryResult)): ?>
             <?php 
-               	echo $this->partial('partials/resultset.phtml', array('data' => $this->queryResult, 'header' => $this->header, 'caption'=>'')); 
+               	echo $this->partial('partials/resultset.phtml', array('data' => $this->queryResult, 'urlBase' => $this->urlBase, 'header' => $this->header, 'caption'=>'')); 
             ?>
         <?php else: ?>
             <pre><?php echo $this->escape($this->queryResult) ?></pre>