Fix DI, remove unwanted plugin-wide Constants

simpleshop-cz.php

@@ -27,4 +27,5 @@
 /**
  * Start plugin
  */
+SimpleShop::$pluginMainFile = __FILE__;
 SimpleShop::getInstance();

src/Admin.php

@@ -21,10 +21,14 @@ class Admin {
 	 */
 	private $pluginDirUrl;
 
+	/**
+	 * @param Plugin $loader
+	 * @param string $pluginMainFile
+	 */
 	public function __construct(Plugin $loader) {
 		$this->loader = $loader;
 
-		$this->pluginDirUrl = plugin_dir_url(__DIR__ . '/../');
+		$this->pluginDirUrl = plugin_dir_url($loader->get_plugin_main_file());
 
 		add_action( 'admin_menu', [ $this, 'add_settings_page' ] );
 		add_filter( 'manage_edit-ssc_group_columns', [ $this, 'ssc_group_columns' ] );

src/Plugin.php

@@ -31,19 +31,22 @@ class Plugin {
 	/**
 	 * @var string
 	 */
-	private $pluginDirPath;
+	private $pluginMainFile;
+
+	/**
+	 * @param string $mainFile Filename of main plugin path. Used for native WP functions
+	 */
+	public function __construct( $mainFile ) {
+		$this->pluginMainFile = $mainFile;
 
-	public function __construct() {
 		$this->init();
 		$this->init_i18n();
 
-		$this->pluginDirPath = plugin_dir_path( __DIR__ . '/../' );
-
 		$this->secure_key = $this->load_api_key();
 		$this->email      = $this->load_email();
 
 		add_action( 'tgmpa_register', [ $this, 'register_required_plugins' ] );
-		register_activation_hook( $this->pluginDirPath, [ $this, 'ssc_activation_hook' ] );
+		register_activation_hook( plugin_dir_path( $this->pluginMainFile ), [ $this, 'ssc_activation_hook' ] );
 	}
 
 	private function init() {
@@ -157,7 +160,11 @@ public function init_i18n() {
 	}
 
 	public function load_textdomain_i18n() {
-		$plugin_rel_path     = str_replace( WP_PLUGIN_DIR . '/', '', $this->pluginDirPath . 'languages/' );
+		$plugin_rel_path = str_replace( WP_PLUGIN_DIR . '/', '', plugin_dir_path( $this->pluginMainFile ) . 'languages/' );
 		load_plugin_textdomain( 'simpleshop-cz', false, $plugin_rel_path );
 	}
+
+	public function get_plugin_main_file() {
+		return $this->pluginMainFile;
+	}
 }

src/SimpleShop.php

@@ -9,6 +9,11 @@
 namespace Redbit\SimpleShop\WpPlugin;
 
 class SimpleShop {
+	/**
+	 * @var string
+	 */
+	public static $pluginMainFile = __DIR__ . '/../simpleshop-cz.php';
+
 	/**
 	 * @var Plugin|null
 	 */
@@ -29,6 +34,6 @@ public static function getInstance() {
 	 * @return Plugin
 	 */
 	protected static function factory() {
-		return new Plugin();
+		return new Plugin(self::$pluginMainFile);
 	}
 }