Modernize private property declarations in src/plugin

src/plugin/DiskResourceProvider.php

@@ -38,9 +38,7 @@ use const DIRECTORY_SEPARATOR;
  * to enable special types of access.
  */
 class DiskResourceProvider implements ResourceProvider{
-
+	private string $file;
-	/** @var string */
-	private $file;
 
 	public function __construct(string $path){
 		$this->file = rtrim(str_replace(DIRECTORY_SEPARATOR, "/", $path), "/") . "/";

src/plugin/PharPluginLoader.php

@@ -31,13 +31,9 @@ use function substr;
  * Handles different types of plugins
  */
 class PharPluginLoader implements PluginLoader{
-
+	public function __construct(
-	/** @var \DynamicClassLoader */
+		private \DynamicClassLoader $loader
-	private $loader;
+	){}
-
-	public function __construct(\DynamicClassLoader $loader){
-		$this->loader = $loader;
-	}
 
 	public function canLoadPlugin(string $path) : bool{
 		$ext = ".phar";

src/plugin/PluginBase.php

@@ -47,41 +47,22 @@ use function strtolower;
 use function trim;
 
 abstract class PluginBase implements Plugin, CommandExecutor{
+	private bool $isEnabled = false;
 
-	/** @var PluginLoader */
+	private ?Config $config = null;
-	private $loader;
+	private string $configFile;
 
-	/** @var Server */
+	private PluginLogger $logger;
-	private $server;
+	private TaskScheduler $scheduler;
 
-	/** @var bool */
+	public function __construct(
-	private $isEnabled = false;
+		private PluginLoader $loader,
-
+		private Server $server,
-	/** @var PluginDescription */
+		private PluginDescription $description,
-	private $description;
+		private string $dataFolder,
-
+		private string $file,
-	/** @var string */
+		private ResourceProvider $resourceProvider
-	private $dataFolder;
+	){
-	/** @var Config|null */
-	private $config = null;
-	/** @var string */
-	private $configFile;
-	/** @var string */
-	private $file;
-
-	/** @var PluginLogger */
-	private $logger;
-
-	/** @var TaskScheduler */
-	private $scheduler;
-
-	/** @var ResourceProvider */
-	private $resourceProvider;
-
-	public function __construct(PluginLoader $loader, Server $server, PluginDescription $description, string $dataFolder, string $file, ResourceProvider $resourceProvider){
-		$this->loader = $loader;
-		$this->server = $server;
-		$this->description = $description;
 		$this->dataFolder = rtrim($dataFolder, "/" . DIRECTORY_SEPARATOR) . "/";
 		//TODO: this is accessed externally via reflection, not unused
 		$this->file = rtrim($file, "/" . DIRECTORY_SEPARATOR) . "/";
@@ -90,7 +71,6 @@ abstract class PluginBase implements Plugin, CommandExecutor{
 		$prefix = $this->getDescription()->getPrefix();
 		$this->logger = new PluginLogger($server->getLogger(), $prefix !== "" ? $prefix : $this->getName());
 		$this->scheduler = new TaskScheduler($this->getFullName());
-		$this->resourceProvider = $resourceProvider;
 
 		$this->onLoad();
 

src/plugin/PluginGraylist.php

@@ -32,9 +32,8 @@ use function is_string;
 class PluginGraylist{
 
 	/** @var string[] */
-	private $plugins;
+	private array $plugins;
-	/** @var bool */
+	private bool $isWhitelist = false;
-	private $isWhitelist = false;
 
 	/**
 	 * @param string[] $plugins

src/plugin/PluginLogger.php

@@ -34,7 +34,7 @@ class PluginLogger extends \PrefixedLogger implements \AttachableLogger{
 	 * @var \Closure[]
 	 * @phpstan-var LoggerAttachment[]
 	 */
-	private $attachments = [];
+	private array $attachments = [];
 
 	/**
 	 * @phpstan-param LoggerAttachment $attachment

src/plugin/PluginManager.php

@@ -69,10 +69,6 @@ use function strtolower;
  * Manages all the plugins
  */
 class PluginManager{
-
-	/** @var Server */
-	private $server;
-
 	/** @var Plugin[] */
 	protected $plugins = [];
 
@@ -87,14 +83,11 @@ class PluginManager{
 	 */
 	protected $fileAssociations = [];
 
-	/** @var string|null */
+	public function __construct(
-	private $pluginDataDirectory;
+		private Server $server,
-	/** @var PluginGraylist|null */
+		private ?string $pluginDataDirectory,
-	private $graylist;
+		private ?PluginGraylist $graylist = null
-
+	){
-	public function __construct(Server $server, ?string $pluginDataDirectory, ?PluginGraylist $graylist = null){
-		$this->server = $server;
-		$this->pluginDataDirectory = $pluginDataDirectory;
 		if($this->pluginDataDirectory !== null){
 			if(!file_exists($this->pluginDataDirectory)){
 				@mkdir($this->pluginDataDirectory, 0777, true);
@@ -102,8 +95,6 @@ class PluginManager{
 				throw new \RuntimeException("Plugin data path $this->pluginDataDirectory exists and is not a directory");
 			}
 		}
-
-		$this->graylist = $graylist;
 	}
 
 	public function getPlugin(string $name) : ?Plugin{