diff options
-rw-r--r-- | application/config/ConfigPlugin.php | 17 | ||||
-rw-r--r-- | application/config/exception/PluginConfigOrderException.php | 17 | ||||
-rw-r--r-- | composer.json | 3 | ||||
-rw-r--r-- | index.php | 1 | ||||
-rw-r--r-- | tests/config/ConfigPluginTest.php | 6 |
5 files changed, 24 insertions, 20 deletions
diff --git a/application/config/ConfigPlugin.php b/application/config/ConfigPlugin.php index 61a594d3..b3d9752b 100644 --- a/application/config/ConfigPlugin.php +++ b/application/config/ConfigPlugin.php | |||
@@ -1,5 +1,6 @@ | |||
1 | <?php | 1 | <?php |
2 | namespace Shaarli\Config; | 2 | |
3 | use Shaarli\Config\Exception\PluginConfigOrderException; | ||
3 | 4 | ||
4 | /** | 5 | /** |
5 | * Plugin configuration helper functions. | 6 | * Plugin configuration helper functions. |
@@ -110,17 +111,3 @@ function load_plugin_parameter_values($plugins, $conf) | |||
110 | 111 | ||
111 | return $out; | 112 | return $out; |
112 | } | 113 | } |
113 | |||
114 | /** | ||
115 | * Exception used if an error occur while saving plugin configuration. | ||
116 | */ | ||
117 | class PluginConfigOrderException extends \Exception | ||
118 | { | ||
119 | /** | ||
120 | * Construct exception. | ||
121 | */ | ||
122 | public function __construct() | ||
123 | { | ||
124 | $this->message = 'An error occurred while trying to save plugins loading order.'; | ||
125 | } | ||
126 | } | ||
diff --git a/application/config/exception/PluginConfigOrderException.php b/application/config/exception/PluginConfigOrderException.php new file mode 100644 index 00000000..f9d68750 --- /dev/null +++ b/application/config/exception/PluginConfigOrderException.php | |||
@@ -0,0 +1,17 @@ | |||
1 | <?php | ||
2 | |||
3 | namespace Shaarli\Config\Exception; | ||
4 | |||
5 | /** | ||
6 | * Exception used if an error occur while saving plugin configuration. | ||
7 | */ | ||
8 | class PluginConfigOrderException extends \Exception | ||
9 | { | ||
10 | /** | ||
11 | * Construct exception. | ||
12 | */ | ||
13 | public function __construct() | ||
14 | { | ||
15 | $this->message = 'An error occurred while trying to save plugins loading order.'; | ||
16 | } | ||
17 | } | ||
diff --git a/composer.json b/composer.json index 70b87bb9..57851e53 100644 --- a/composer.json +++ b/composer.json | |||
@@ -29,7 +29,8 @@ | |||
29 | "Shaarli\\Api\\": "application/api/", | 29 | "Shaarli\\Api\\": "application/api/", |
30 | "Shaarli\\Api\\Controllers\\": "application/api/controllers", | 30 | "Shaarli\\Api\\Controllers\\": "application/api/controllers", |
31 | "Shaarli\\Api\\Exceptions\\": "application/api/exceptions", | 31 | "Shaarli\\Api\\Exceptions\\": "application/api/exceptions", |
32 | "Shaarli\\Config\\": "application/config/" | 32 | "Shaarli\\Config\\": "application/config/", |
33 | "Shaarli\\Config\\Exception\\": "application/config/exception" | ||
33 | } | 34 | } |
34 | } | 35 | } |
35 | } | 36 | } |
@@ -62,6 +62,7 @@ require_once __DIR__ . '/vendor/autoload.php'; | |||
62 | require_once 'application/ApplicationUtils.php'; | 62 | require_once 'application/ApplicationUtils.php'; |
63 | require_once 'application/Cache.php'; | 63 | require_once 'application/Cache.php'; |
64 | require_once 'application/CachedPage.php'; | 64 | require_once 'application/CachedPage.php'; |
65 | require_once 'application/config/ConfigPlugin.php'; | ||
65 | require_once 'application/FeedBuilder.php'; | 66 | require_once 'application/FeedBuilder.php'; |
66 | require_once 'application/FileUtils.php'; | 67 | require_once 'application/FileUtils.php'; |
67 | require_once 'application/HttpUtils.php'; | 68 | require_once 'application/HttpUtils.php'; |
diff --git a/tests/config/ConfigPluginTest.php b/tests/config/ConfigPluginTest.php index 22ab927b..deb02c9e 100644 --- a/tests/config/ConfigPluginTest.php +++ b/tests/config/ConfigPluginTest.php | |||
@@ -1,9 +1,7 @@ | |||
1 | <?php | 1 | <?php |
2 | namespace Shaarli\Config; | 2 | namespace Shaarli\Config; |
3 | 3 | ||
4 | /** | 4 | use Shaarli\Config\Exception\PluginConfigOrderException; |
5 | * Config' tests | ||
6 | */ | ||
7 | 5 | ||
8 | require_once 'application/config/ConfigPlugin.php'; | 6 | require_once 'application/config/ConfigPlugin.php'; |
9 | 7 | ||
@@ -41,7 +39,7 @@ class ConfigPluginTest extends \PHPUnit_Framework_TestCase | |||
41 | /** | 39 | /** |
42 | * Test save_plugin_config with invalid data. | 40 | * Test save_plugin_config with invalid data. |
43 | * | 41 | * |
44 | * @expectedException Shaarli\Config\PluginConfigOrderException | 42 | * @expectedException Shaarli\Config\Exception\PluginConfigOrderException |
45 | */ | 43 | */ |
46 | public function testSavePluginConfigInvalid() | 44 | public function testSavePluginConfigInvalid() |
47 | { | 45 | { |