Merge pull request #1124 from Luceos/master

fixed issues with $extension visibility and typehinting

framework/core/src/Event/ExtensionWasDisabled.php

@@ -16,9 +16,9 @@ use Flarum\Extension\Extension;
 class ExtensionWasDisabled
 {
     /**
-     * @var string
+     * @var Extension
      */
-    protected $extension;
+    public $extension;
 
     /**
      * @param Extension $extension

framework/core/src/Event/ExtensionWasEnabled.php

@@ -16,9 +16,9 @@ use Flarum\Extension\Extension;
 class ExtensionWasEnabled
 {
     /**
-     * @var string
+     * @var Extension
      */
-    protected $extension;
+    public $extension;
 
     /**
      * @param Extension $extension

framework/core/src/Event/ExtensionWasUninstalled.php

@@ -11,17 +11,19 @@
 
 namespace Flarum\Event;
 
+use Flarum\Extension\Extension;
+
 class ExtensionWasUninstalled
 {
     /**
-     * @var string
+     * @var Extension
      */
-    protected $extension;
+    public $extension;
 
     /**
-     * @param string $extension
+     * @param Extension $extension
      */
-    public function __construct($extension)
+    public function __construct(Extension $extension)
     {
         $this->extension = $extension;
     }

framework/core/src/Event/ExtensionWillBeDisabled.php

@@ -16,7 +16,7 @@ use Flarum\Extension\Extension;
 class ExtensionWillBeDisabled
 {
     /**
-     * @var string
+     * @var Extension
      */
     public $extension;
 

framework/core/src/Event/ExtensionWillBeEnabled.php

@@ -16,7 +16,7 @@ use Flarum\Extension\Extension;
 class ExtensionWillBeEnabled
 {
     /**
-     * @var string
+     * @var Extension
      */
     public $extension;