Peter M. Groen / knowledgetree

Commit 8663bd6899e6cd005454634aeb41e6d09f2dfce0

Authored by kevin_fourie
1 parent 5cdcb1fa

Merged from trunk... 

KTS-1732
"Deleting workflow state causes fatal error. "
Fixed.

Reviewed By: Conrad

KTS-1696
"when bulk deleting files in a sub folder, am redirected to root"
Fixed. Will attempt to redirect back to folder last visited.

Reviewed By: Kevin Fourie

git-svn-id: https://kt-dms.svn.sourceforge.net/svnroot/kt-dms/STABLE/branches/3.3.3-Release-Branch@6372 c91229c3-7414-0410-bfa2-8a42b809f60b
Inline Side-by-side
Showing 2 changed files with 10 additions and 2 deletions
lib/actions/bulkaction.php
  View file @8663bd6
... ... @@ -312,6 +312,10 @@ class KTBulkAction extends KTStandardDispatcher {
312 312 $sAction = 'main';
313 313  
314 314 if($sReturnAction == 'browse') {
  315 + if ($sReturnData == '')
  316 + {
  317 + $sReturnData = KTUtil::arrayGet($_REQUEST, 'fFolderId');
  318 + }
315 319 $sTargetUrl = KTBrowseUtil::getUrlForFolder(Folder::get($sReturnData));
316 320 } else if($sReturnAction == 'simpleSearch') {
317 321 $sTargetUrl = KTBrowseUtil::getSimpleSearchBaseUrl();
... ... @@ -497,4 +501,4 @@ class KTBulkActionUtil {
497 501 }
498 502 }
499 503  
500   -?>
501 504 \ No newline at end of file
  505 +?>
... ...
plugins/ktcore/KTDocumentActions.php
  View file @8663bd6
... ... @@ -933,7 +933,7 @@ class KTDocumentMoveAction extends KTDocumentAction {
933 933 $bFileClash = KTDocumentUtil::fileExists($data['browse'], $this->oDocument->getFilename());
934 934 if ($bFileClash && isset($data['filename'])) {
935 935 $filename = $data['filename'];
936   - $bFileClash = KTDocumentUtil::fileExists($this->oFolder, $filename);
  936 + $bFileClash = KTDocumentUtil::fileExists($data['browse'], $filename);
937 937 } else {
938 938 $filename = $this->oDocument->getFilename();
939 939 }
... ... @@ -1348,6 +1348,10 @@ class KTDocumentWorkflowAction extends KTDocumentAction {
1348 1348 if ($aTransitions) {
1349 1349 $aVocab = array();
1350 1350 foreach ($aTransitions as $oTransition) {
  1351 + if(is_null($oTransition) || PEAR::isError($oTransition)){
  1352 + continue;
  1353 + }
  1354 +
1351 1355 $aVocab[$oTransition->getId()] = $oTransition->showDescription();
1352 1356 }
1353 1357 $fieldOptions = array('vocab' => $aVocab);
... ...