Parcourir la source

Throw LogicException instead of RuntimeException in Pico::setConfig()

Daniel Rudolf il y a 9 ans
Parent
commit
a138d23daf
2 fichiers modifiés avec 5 ajouts et 3 suppressions
  1. 2 0
      CHANGELOG.md
  2. 3 3
      lib/Pico.php

+ 2 - 0
CHANGELOG.md

@@ -9,6 +9,8 @@ Released: -
         all contributors and users which made this possible!
 * [New] Adding `$queryData` parameter to `Pico::getPageUrl()` method
 * [Changed] Moving `LICENSE` to `LICENSE.md`
+* [Changed] Throw `LogicException` instead of `RuntimeException` when calling
+            `Pico::setConfig()` after processing has started
 ```
 
 ### Version 1.0.0-beta.2

+ 3 - 3
lib/Pico.php

@@ -267,7 +267,7 @@ class Pico
      * the rendered contents.
      *
      * @return string           rendered Pico contents
-     * @throws RuntimeException thrown when a not recoverable error occurs
+     * @throws Exception thrown when a not recoverable error occurs
      */
     public function run()
     {
@@ -518,12 +518,12 @@ class Pico
      * @see    Pico::getConfig()
      * @param  mixed[] $config  array with config variables
      * @return void
-     * @throws RuntimeException thrown if Pico already started processing
+     * @throws LogicException thrown if Pico already started processing
      */
     public function setConfig(array $config)
     {
         if ($this->locked) {
-            throw new RuntimeException("You cannot modify Pico's config after processing has started");
+            throw new LogicException("You cannot modify Pico's config after processing has started");
         }
 
         $this->config = $config;