diff options
| author | Christian Weiske <cweiske@cweiske.de> | 2015-03-17 19:53:17 +0100 |
|---|---|---|
| committer | Christian Weiske <cweiske@cweiske.de> | 2015-03-17 19:53:17 +0100 |
| commit | e19fc06310e26dab3ec372fb01676a802fc02234 (patch) | |
| tree | edece44d7c113191c1dd2d671e1d2d214a5fe190 | |
| parent | e74217a7bbd58cf76963c15c73221669e1153510 (diff) | |
| download | grauphel-e19fc06310e26dab3ec372fb01676a802fc02234.tar.gz grauphel-e19fc06310e26dab3ec372fb01676a802fc02234.zip | |
Send 404 if a note cannot be found
| -rw-r--r-- | controller/guicontroller.php | 4 | ||||
| -rw-r--r-- | controller/notescontroller.php | 5 |
2 files changed, 8 insertions, 1 deletions
diff --git a/controller/guicontroller.php b/controller/guicontroller.php index b09ef4a..8ebea8c 100644 --- a/controller/guicontroller.php +++ b/controller/guicontroller.php @@ -96,7 +96,9 @@ class GuiController extends Controller $note = $this->getNotes()->load($guid, false); if ($note === null) { - return new ErrorResponse('Note does not exist'); + $res = new ErrorResponse('Note does not exist'); + $res->setStatus(\OCP\AppFramework\Http::STATUS_NOT_FOUND); + return $res; } $converter = new \OCA\Grauphel\Converter\Html(); diff --git a/controller/notescontroller.php b/controller/notescontroller.php index 02bfc93..5618bcb 100644 --- a/controller/notescontroller.php +++ b/controller/notescontroller.php @@ -58,6 +58,11 @@ class NotesController extends Controller public function xml($guid) { $note = $this->getNotes()->load($guid, false); + if ($note === null) { + $res = new ErrorResponse('Note does not exist'); + $res->setStatus(\OCP\AppFramework\Http::STATUS_NOT_FOUND); + return $res; + } $xw = new \XMLWriter(); $xw->openMemory(); |
