Remove unused path helpers

Author: iMattPro

config/services.yml

@@ -33,7 +33,6 @@ services:
             - '@log'
             - '@user_loader'
             - '@user'
-            - '@path_helper'
             - '%core.root_path%'
             - '%core.php_ext%'
             - '%tables.phpbb.wpn.notification_push%'
@@ -62,5 +61,4 @@ services:
         class: phpbb\webpushnotifications\controller\manifest
         arguments:
             - '@config'
-            - '@path_helper'
             - '@user'

controller/manifest.php

@@ -11,7 +11,6 @@
 namespace phpbb\webpushnotifications\controller;
 
 use phpbb\config\config;
-use phpbb\path_helper;
 use phpbb\user;
 use phpbb\webpushnotifications\ext;
 use Symfony\Component\HttpFoundation\JsonResponse;
@@ -21,23 +20,18 @@ class manifest
 	/** @var config */
 	protected $config;
 
-	/** @var path_helper */
-	protected $path_helper;
-
 	/** @var user */
 	protected $user;
 
 	/**
 	 * Constructor for webpush controller
 	 *
 	 * @param config $config
-	 * @param path_helper $path_helper
 	 * @param user $user
 	 */
-	public function __construct(config $config, path_helper $path_helper, user $user)
+	public function __construct(config $config, user $user)
 	{
 		$this->config = $config;
-		$this->path_helper = $path_helper;
 		$this->user = $user;
 	}
 

notification/method/webpush.php

@@ -17,7 +17,6 @@
 use phpbb\log\log_interface;
 use phpbb\notification\method\base;
 use phpbb\notification\type\type_interface;
-use phpbb\path_helper;
 use phpbb\user;
 use phpbb\user_loader;
 use phpbb\webpushnotifications\form\form_helper;
@@ -44,9 +43,6 @@ class webpush extends base implements extended_method_interface
 	/** @var user */
 	protected $user;
 
-	/** @var path_helper */
-	protected $path_helper;
-
 	/** @var string */
 	protected $phpbb_root_path;
 
@@ -73,21 +69,19 @@ class webpush extends base implements extended_method_interface
 	 * @param log_interface $log
 	 * @param user_loader $user_loader
 	 * @param user $user
-	 * @param path_helper $path_helper
 	 * @param string $phpbb_root_path
 	 * @param string $php_ext
 	 * @param string $notification_webpush_table
 	 * @param string $push_subscriptions_table
 	 */
-	public function __construct(config $config, driver_interface $db, log_interface $log, user_loader $user_loader, user $user, path_helper $path_helper,
+	public function __construct(config $config, driver_interface $db, log_interface $log, user_loader $user_loader, user $user,
 								string $phpbb_root_path, string $php_ext, string $notification_webpush_table, string $push_subscriptions_table)
 	{
 		$this->config = $config;
 		$this->db = $db;
 		$this->log = $log;
 		$this->user_loader = $user_loader;
 		$this->user = $user;
-		$this->path_helper = $path_helper;
 		$this->phpbb_root_path = $phpbb_root_path;
 		$this->php_ext = $php_ext;
 		$this->notification_webpush_table = $notification_webpush_table;

tests/controller/controller_manifest_test.php

@@ -15,14 +15,12 @@
 class controller_manifest_test extends \phpbb_test_case
 {
 	protected $config;
-	protected $path_helper;
 	protected $user;
 	protected $manifest;
 
 	protected function setUp(): void
 	{
-		global $phpbb_root_path, $phpEx;
-		global $config, $user, $request, $symfony_request;
+		global $config, $user, $phpbb_root_path, $phpEx;
 
 		parent::setUp();
 
@@ -38,15 +36,8 @@ protected function setUp(): void
 		$lang_loader = new \phpbb\language\language_file_loader($phpbb_root_path, $phpEx);
 		$language = new \phpbb\language\language($lang_loader);
 		$user = $this->user = new \phpbb\user($language, '\phpbb\datetime');
-		$symfony_request = $this->createMock(\phpbb\symfony_request::class);
-		$request = $this->request = $this->createMock(\phpbb\request\request_interface::class);
-		$this->path_helper = new \phpbb\path_helper($symfony_request, new \phpbb\filesystem\filesystem(), $this->request, $phpbb_root_path, $phpEx);
-
-		$this->manifest = new \phpbb\webpushnotifications\controller\manifest(
-			$this->config,
-			$this->path_helper,
-			$this->user
-		);
+
+		$this->manifest = new \phpbb\webpushnotifications\controller\manifest($this->config, $this->user);
 	}
 
 	public function manifest_data()

tests/event/listener_test.php

@@ -95,11 +95,6 @@ protected function setUp(): void
 			$this->user
 		);
 
-		$path_helper = $this->getMockBuilder('\phpbb\path_helper')
-			->disableOriginalConstructor()
-			->setMethods(array())
-			->getMock();
-
 		$this->notifications = $this->getMockBuilder('\phpbb\notification\manager')
 			->disableOriginalConstructor()
 			->getMock();
@@ -113,7 +108,6 @@ protected function setUp(): void
 			new \phpbb\log\dummy(),
 			$user_loader,
 			$this->user,
-			$path_helper,
 			$phpbb_root_path,
 			$phpEx,
 			'phpbb_wpn_notification_push',

tests/notification/notification_method_webpush_test.php

@@ -128,19 +128,6 @@ protected function setUp(): void
 			$phpEx
 		);
 
-		$request = new \phpbb_mock_request;
-		$symfony_request = new \phpbb\symfony_request(
-			$request
-		);
-		$filesystem = new \phpbb\filesystem\filesystem();
-		$phpbb_path_helper = new \phpbb\path_helper(
-			$symfony_request,
-			$filesystem,
-			$request,
-			$phpbb_root_path,
-			$phpEx
-		);
-
 		$log_table = 'phpbb_log';
 		$this->log = new \phpbb\log\log($this->db, $user, $auth, $this->phpbb_dispatcher, $phpbb_root_path, 'adm/', $phpEx, $log_table);
 
@@ -158,7 +145,6 @@ protected function setUp(): void
 		$phpbb_container->set('log', $this->log);
 		$phpbb_container->set('text_formatter.utils', new \phpbb\textformatter\s9e\utils());
 		$phpbb_container->set('dispatcher', $this->phpbb_dispatcher);
-		$phpbb_container->set('path_helper', $phpbb_path_helper);
 		$phpbb_container->setParameter('core.root_path', $phpbb_root_path);
 		$phpbb_container->setParameter('core.php_ext', $phpEx);
 		$phpbb_container->setParameter('tables.notifications', 'phpbb_notifications');
@@ -174,7 +160,6 @@ protected function setUp(): void
 			$phpbb_container->get('log'),
 			$phpbb_container->get('user_loader'),
 			$phpbb_container->get('user'),
-			$phpbb_container->get('path_helper'),
 			$phpbb_root_path,
 			$phpEx,
 			$phpbb_container->getParameter('tables.phpbb.wpn.notification_push'),