Skip to content
This repository was archived by the owner on Sep 10, 2021. It is now read-only.

Commit f619b70

Browse files
committed
ENH: refs #236. api: Set exception code on folder.children if caught
1 parent a16548c commit f619b70

File tree

2 files changed

+13
-6
lines changed

2 files changed

+13
-6
lines changed

core/models/pdo/FolderModel.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ public function getSizeFiltered($folders, $userDao = null, $policy = 0)
123123
{
124124
if(!$folder instanceof FolderDao)
125125
{
126-
throw new Zend_Exception("Should be a folder" );
126+
throw new Zend_Exception("Should be a folder");
127127
}
128128
$subqueryUser = $this->database->select()
129129
->setIntegrityCheck(false)
@@ -205,7 +205,7 @@ function getRoot($folder)
205205
{
206206
if(!$folder instanceof FolderDao)
207207
{
208-
throw new Zend_Exception("Should be a folder" );
208+
throw new Zend_Exception("Should be a folder");
209209
}
210210

211211
$row = $this->database->fetchRow($this->database->select()->setIntegrityCheck(false)
@@ -247,7 +247,7 @@ function getAllChildren($folder, $userDao, $admin = false)
247247

248248
if(!$folder instanceof FolderDao)
249249
{
250-
throw new Zend_Exception("Should be a folder" );
250+
throw new Zend_Exception("Should be a folder");
251251
}
252252
$subqueryUser = $this->database->select()
253253
->setIntegrityCheck(false)
@@ -302,7 +302,7 @@ function delete($folder, $recursive = false)
302302
{
303303
if(!$folder instanceof FolderDao)
304304
{
305-
throw new Zend_Exception("Should be a folder" );
305+
throw new Zend_Exception("Should be a folder");
306306
}
307307
if(!$folder->saved)
308308
{

modules/api/controllers/components/ApiComponent.php

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -676,8 +676,15 @@ function folderChildren($args)
676676
$folder = $folderModel->load($id);
677677

678678
$userDao = $this->_getUser($args);
679-
$folders = $folderModel->getChildrenFoldersFiltered($folder, $userDao);
680-
$items = $folderModel->getItemsFiltered($folder, $userDao);
679+
try
680+
{
681+
$folders = $folderModel->getChildrenFoldersFiltered($folder, $userDao);
682+
$items = $folderModel->getItemsFiltered($folder, $userDao);
683+
}
684+
catch(Exception $e)
685+
{
686+
throw new Exception($e->getMessage(), MIDAS_INTERNAL_ERROR);
687+
}
681688

682689
return array('folders' => $folders, 'items' => $items);
683690
}

0 commit comments

Comments
 (0)