From 77f46da53abea0eb6c8e2a6bbea4512ddd2e7d93 Mon Sep 17 00:00:00 2001 From: Ferran Recio Date: Wed, 12 Aug 2020 16:57:17 +0200 Subject: [PATCH] MDL-69143 contentbank: catching Exception on file upload --- contentbank/classes/contenttype.php | 4 ++-- contentbank/tests/contenttype_test.php | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/contentbank/classes/contenttype.php b/contentbank/classes/contenttype.php index cd1fb418503..699cee0cdb7 100644 --- a/contentbank/classes/contenttype.php +++ b/contentbank/classes/contenttype.php @@ -28,7 +28,7 @@ use core\event\contentbank_content_created; use core\event\contentbank_content_deleted; use core\event\contentbank_content_viewed; use stored_file; -use file_exception; +use Exception; use moodle_url; /** @@ -107,7 +107,7 @@ abstract class contenttype { $content = $this->create_content($record); try { $content->import_file($file); - } catch (file_exception $e) { + } catch (Exception $e) { $this->delete_content($content); throw $e; } diff --git a/contentbank/tests/contenttype_test.php b/contentbank/tests/contenttype_test.php index 9c80d8fbfe7..1fee5be6a31 100644 --- a/contentbank/tests/contenttype_test.php +++ b/contentbank/tests/contenttype_test.php @@ -28,7 +28,7 @@ namespace core_contentbank; use stdClass; use context_system; use context_user; -use file_exception; +use Exception; use contenttype_testable\contenttype as contenttype; /** * Test for content bank contenttype class. @@ -283,7 +283,7 @@ class core_contenttype_contenttype_testcase extends \advanced_testcase { try { $content = $contenttype->upload_content($dummyfile); $this->assertTrue(false); - } catch (file_exception $e) { + } catch (Exception $e) { $this->assertTrue(true); } $this->assertEquals($cbcontents, $DB->count_records('contentbank_content'));