Query string related changes - use addQS and friends.

git-svn-id: https://kt-dms.svn.sourceforge.net/svnroot/kt-dms/trunk@4777 c91229c3-7414-0410-bfa2-8a42b809f60b

Query string related changes - use addQS and friends.
git-svn-id: https://kt-dms.svn.sourceforge.net/svnroot/kt-dms/trunk@4777 c91229c3-7414-0410-bfa2-8a42b809f60b
nbm
1 parent 7e2d2c1a
Showing 10 changed files with 25 additions and 20 deletions
lib/session/control.inc
lib/templating/kt3template.inc.php
plugins/ktcore/admin/documentFields.php
plugins/ktcore/admin/userManagement.php
plugins/ktcore/admin/workflows.php
plugins/ktcore/authentication/authenticationadminpage.inc.php
plugins/ktstandard/KTDiscussion.php
plugins/ktstandard/ldap/ldapauthenticationprovider.inc.php
templates/ktcore/authentication/manage.smarty
templates/ktcore/workflow/admin_portlet.smarty
@@ -139,7 +139,7 @@ function checkSessionAndRedirect($bRedirect, $bDownload = false) {
             } else {
             	$url = generateControllerUrl("login");
             }
-            $redirect = urlencode($_SERVER["PHP_SELF"] . "?" . $_SERVER["QUERY_STRING"]);
+            $redirect = urlencode(KTUtil::addQueryStringSelf($_SERVER["QUERY_STRING"]));
             if ((strlen($redirect) > 1)) {
                 $default->log->debug("checkSession:: redirect url=$redirect");
                 // this session verification failure represents either the first visit to
@@ -282,6 +282,8 @@ class KTPage {
                $sUrl = KTUtil::addQueryString($sUrl, $sQuery);
            }
 		   $aTuple["url"] = $sUrl;
+        } else if ($aActionTuple["query"]) {
+           $aTuple['url'] = KTUtil::addQueryStringSelf($aActionTuple["query"]);
 		} else {
 		   $aTuple["url"] = false;
 		}
@@ -721,7 +721,7 @@ class KTDocumentFieldDispatcher extends KTAdminDispatcher {
         // since the root is virtual, we need to fake it here.
         // the inner section is generised.
         $treeStr .= '<ul class="kt_treenodes"><li class="treenode active"><a class="pathnode"  onclick="toggleElementClass(\'active\', this.parentNode);">Root</a>';
-        $treeStr .= ' (<a href="?action=editTree&field_id='.$treeToRender->field_id.'&current_node=0">edit</a>)';
+        $treeStr .= ' (<a href="' . KTUtil::addQueryStringSelf('action=editTree&field_id='.$treeToRender->field_id.'&current_node=0') . '">edit</a>)';
         $treeStr .= '<ul>';
         //$default->log->debug("EVILRENDER: " . print_r($treeToRender, true));
         foreach ($treeToRender->getRoot() as $node_id => $subtree_nodes)
@@ -755,12 +755,12 @@ class KTDocumentFieldDispatcher extends KTAdminDispatcher {
     function _evilActionHelper($iFieldId, $bIsKeyword, $current_node) {
         $actionStr = " (";
         if ($bIsKeyword === true) {
-           $actionStr .= '<a href="?action=editTree&field_id='.$iFieldId.'&keyword_id='.$current_node.'&subaction=unlinkKeyword">unlink</a>';
+           $actionStr .= '<a href="' . KTUtil::addQueryStringSelf('action=editTree&field_id='.$iFieldId.'&keyword_id='.$current_node.'&subaction=unlinkKeyword') . '">unlink</a>';
         }
         else
         {
-           $actionStr .= '<a href="?action=editTree&field_id='.$iFieldId.'&current_node='.$current_node.'">attach keywords</a> ';
-           $actionStr .= '| <a href="?action=editTree&field_id='.$iFieldId.'&current_node='.$current_node.'&subaction=deleteCategory">delete</a>';
+           $actionStr .= '<a href="' . KTUtil::addQueryStringSlef('action=editTree&field_id=' . $iFieldId . '&current_node=' . $current_node) .'">attach keywords</a> ';
+           $actionStr .= '| <a href="' . KTUtil::addQueryStringSelf('action=editTree&field_id='.$iFieldId.'&current_node='.$current_node.'&subaction=deleteCategory') . '">delete</a>';
         }
         $actionStr .= ")";
         return $actionStr;
@@ -101,7 +101,7 @@ class KTUserAdminDispatcher extends KTAdminDispatcher {
         $oProvider =& $oRegistry->getAuthenticationProvider($sProvider);
         $this->aBreadcrumbs[] = array('url' => $_SERVER['PHP_SELF'], 'name' => _('User Management'));
-        $this->aBreadcrumbs[] = array('url' => $_SERVER['PHP_SELF'] . '?action=addUser', 'name' => _('add a new user'));
+        $this->aBreadcrumbs[] = array('url' => KTUtil::addQueryStringSelf('action=addUser'), 'name' => _('add a new user'));
         $oProvider->aBreadcrumbs = $this->aBreadcrumbs;
         $oProvider->oPage->setBreadcrumbDetails($oSource->getName());
         $oProvider->oPage->setTitle(_("Modify User Details"));
@@ -36,11 +36,10 @@ class WorkflowNavigationPortlet extends KTPortlet {
         if (is_null($this->oWorkflow)) { return _('No Workflow Selected.'); }
         $aAdminPages = array();
-        $aAdminPages[] = array('name' => _('Overview'), 'url' => KTUtil::addQueryStringSelf('action=editWorkflow&fWorkflowId=' . $this->oWorkflow->getId()));
-        $aAdminPages[] = array('name' => _('States'), 'url' => KTUtil::addQueryStringSelf('action=manageStates&fWorkflowId=' . $this->oWorkflow->getId()));
-        $aAdminPages[] = array('name' => _('Transitions'), 'url' => KTUtil::addQueryStringSelf('action=manageTransitions&fWorkflowId=' . $this->oWorkflow->getId()));
-        $aAdminPages[] = array('name' => _('Actions'), 'url' => KTUtil::addQueryStringSelf('action=manageActions&fWorkflowId=' . $this->oWorkflow->getId()));
-
+        $aAdminPages[] = array('name' => _('Overview'), 'query' => 'action=editWorkflow&fWorkflowId=' . $this->oWorkflow->getId());
+        $aAdminPages[] = array('name' => _('States'), 'query' => 'action=manageStates&fWorkflowId=' . $this->oWorkflow->getId());
+        $aAdminPages[] = array('name' => _('Transitions'), 'query' => 'action=manageTransitions&fWorkflowId=' . $this->oWorkflow->getId());
+        $aAdminPages[] = array('name' => _('Actions'), 'query' => 'action=manageActions&fWorkflowId=' . $this->oWorkflow->getId());
         $oTemplating = new KTTemplating;        
         $oTemplate = $oTemplating->loadTemplate("ktcore/workflow/admin_portlet");
@@ -208,7 +207,8 @@ class KTWorkflowDispatcher extends KTAdminDispatcher {
         }
         foreach ($aInfo['transitions_to_state'][$oState->getId()] as $aTransition) { 
-            $aT[] = sprintf('<a href="%s?action=editTransition&fWorkflowId=%d&fTransitionId=%d">%s</a>', $_SERVER['PHP_SELF'], $aInfo['workflow']->getId(), $aTransition->getId() ,$aTransition->getName()); 
+            $sUrl = KTUtil::addQueryStringSelf(sprintf('action=editTransition&fWorkflowId=%d&fTransitionId=%d', $aInfo['workflow']->getId(), $aTransition->getId())); 
+            $aT[] = sprintf('<a href="%s">%s</a>', $sUrl, $aTransition->getName()); 
         }
         return implode(', ',$aT);
@@ -302,7 +302,8 @@ class KTWorkflowDispatcher extends KTAdminDispatcher {
         $aT = array();
         foreach ($aInfo['transitions_from_state'][$oState->getId()] as $aTransition) { 
-            $aT[] = sprintf('<a href="%s?action=editTransition&fWorkflowId=%d&fTransitionId=%d">%s</a>', $_SERVER['PHP_SELF'], $aInfo['workflow']->getId(), $aTransition->getId() ,$aTransition->getName()); 
+            $sUrl = KTUtil::addQueryStringSelf(sprintf('action=editTransition&fWorkflowId=%d&fTransitionId=%d', $aInfo['workflow']->getId(), $aTransition->getId())); 
+            $aT[] = sprintf('<a href="%s">%s</a>', $sUrl, $aTransition->getName()); 
         }
         return implode(', ', $aT);
     }
@@ -109,7 +109,7 @@ class KTAuthenticationAdminPage extends KTAdminDispatcher {
         $oProvider =& $oRegistry->getAuthenticationProvider($sProvider);
         $this->aBreadcrumbs[] = array('name' => _('Authentication'), 'url' => $_SERVER['PHP_SELF']);
-        $this->aBreadcrumbs[] = array('name' => $oSource->getName(), 'url' => $_SERVER['PHP_SELF'] . "?source_id=" . $oSource->getId());
+        $this->aBreadcrumbs[] = array('name' => $oSource->getName(), 'url' => KTUtil::addQueryStringSelf("source_id=" . $oSource->getId()));
         $oProvider->aBreadcrumbs = $this->aBreadcrumbs;
         $oProvider->dispatch();
@@ -123,7 +123,7 @@ class KTAuthenticationAdminPage extends KTAdminDispatcher {
         $oProvider =& $oRegistry->getAuthenticationProvider($sProvider);
         $this->aBreadcrumbs[] = array('name' => _('Authentication'), 'url' => $_SERVER['PHP_SELF']);
-        $this->aBreadcrumbs[] = array('name' => $oSource->getName(), 'url' => $_SERVER['PHP_SELF'] . "?source_id=" . $oSource->getId());
+        $this->aBreadcrumbs[] = array('name' => $oSource->getName(), 'url' => KTUtil::addQueryStringSelf("source_id=" . $oSource->getId()));
         $oProvider->aBreadcrumbs = $this->aBreadcrumbs;
         $oProvider->dispatch();
@@ -133,7 +133,7 @@ class KTDocumentDiscussionAction extends KTDocumentAction {
         // breadcrumbs...
         $this->aBreadcrumbs[] = array(
             'name' => _('discussion'),
-            'url' => $_SERVER['PHP_SELF'] . sprintf('?fDocumentId=%d', $this->oDocument->getId()),
+            'query' => sprintf('fDocumentId=%d', $this->oDocument->getId()),
         );
         $this->aBreadcrumbs[] = array(
             'name' => $oComment->getSubject(),
@@ -36,7 +36,9 @@ class KTLDAPAuthenticationProvider extends KTAuthenticationProvider {
     }
     function showUserSource($oUser, $oSource) {
-        return '<a href="?action=editUserSource&user_id=' . $oUser->getId() .'">' . _('Edit LDAP info') . '</a>';
+        $sQuery = sprintf("action=editUserSource&user_id=%d", $oUser->getId());
+        $sUrl = KTUtil::addQueryStringSelf($sQuery);
+        return '<a href="' . $sUrl . '">' . _('Edit LDAP info') . '</a>';
     }
     function do_editUserSource() {
@@ -32,7 +32,7 @@ ensure that the Plugin is enabled.{/i18n}&lt;p&gt;&lt;/div&gt;
 <ul>
 {foreach from=$sources item=oSource}
-  <li><a href="{addQS}action=viewsource&source_id={$oSource->getId()}">{$oSource->getName()}{/addQS}</a></li>
+  <li><a href="{addQS}action=viewsource&source_id={$oSource->getId()}{/addQS}">{$oSource->getName()}</a></li>
 {/foreach}
 </ul>
 {else}
 <ul class="actionlist">
 {foreach from=$aAdminPages item=aAdminPage}
-   <li><a href="{$aAdminPage.url}">{$aAdminPage.name}</a></li>
+   <li><a href="{addQS}{$aAdminPage.query}{/addQS}">{$aAdminPage.name}</a></li>
 {/foreach}
-</ul>
 \ No newline at end of file
+</ul>