From 9ce1ab17248825e70723311c5172bfe5a41769e8 Mon Sep 17 00:00:00 2001
From: Jerome Bakker <jeabakker@coldtrick.com>
Date: Wed, 29 Jan 2020 12:13:08 +0100
Subject: [PATCH] changed: moved start.php to Bootstrap and elgg-plugin.php

---
 classes/ColdTrick/TargetBlank/Bootstrap.php   | 15 ++++++++
 classes/ColdTrick/TargetBlank/Javascript.php  | 21 +++++++++++
 .../ColdTrick/TargetBlank/PluginSettings.php  | 28 +++++++++++++++
 elgg-plugin.php                               | 15 ++++++++
 start.php                                     | 35 -------------------
 5 files changed, 79 insertions(+), 35 deletions(-)
 create mode 100644 classes/ColdTrick/TargetBlank/Bootstrap.php
 create mode 100644 classes/ColdTrick/TargetBlank/Javascript.php
 create mode 100644 classes/ColdTrick/TargetBlank/PluginSettings.php
 delete mode 100644 start.php

diff --git a/classes/ColdTrick/TargetBlank/Bootstrap.php b/classes/ColdTrick/TargetBlank/Bootstrap.php
new file mode 100644
index 0000000..d0f90ad
--- /dev/null
+++ b/classes/ColdTrick/TargetBlank/Bootstrap.php
@@ -0,0 +1,15 @@
+<?php
+
+namespace ColdTrick\TargetBlank;
+
+use Elgg\DefaultPluginBootstrap;
+
+class Bootstrap extends DefaultPluginBootstrap {
+
+	/**
+	 * {@inheritDoc}
+	 */
+	public function init() {
+		elgg_require_js('target_blank/target_blank');
+	}
+}
diff --git a/classes/ColdTrick/TargetBlank/Javascript.php b/classes/ColdTrick/TargetBlank/Javascript.php
new file mode 100644
index 0000000..8add337
--- /dev/null
+++ b/classes/ColdTrick/TargetBlank/Javascript.php
@@ -0,0 +1,21 @@
+<?php
+
+namespace ColdTrick\TargetBlank;
+
+class Javascript {
+
+	/**
+	 * Add the link suffix to the global JS data
+	 *
+	 * @param \Elgg\Hook $hook 'elgg.data', 'page'
+	 *
+	 * @return array
+	 */
+	public function setLinkSuffix(\Elgg\Hook $hook) {
+		$result = $hook->getValue();
+		
+		$result['target_blank']['link_suffix'] = elgg_get_plugin_setting('link_suffix', 'target_blank');
+		
+		return $result;
+	}
+}
diff --git a/classes/ColdTrick/TargetBlank/PluginSettings.php b/classes/ColdTrick/TargetBlank/PluginSettings.php
new file mode 100644
index 0000000..1939caa
--- /dev/null
+++ b/classes/ColdTrick/TargetBlank/PluginSettings.php
@@ -0,0 +1,28 @@
+<?php
+
+namespace ColdTrick\TargetBlank;
+
+class PluginSettings {
+
+	/**
+	 * Save the unfilterd input of the link suffix input
+	 *
+	 * @param \Elgg\Hook $hook 'setting', 'plugin'
+	 *
+	 * @return void|string
+	 */
+	public static function saveLinkSuffix(\Elgg\Hook $hook) {
+		if ($hook->getParam('plugin_id') !== 'target_blank') {
+			return;
+		}
+		
+		if ($hook->getParam('name') !== 'link_suffix') {
+			return;
+		}
+		
+		// get unfiltered input
+		$params = (array) get_input('params', [], false);
+		
+		return elgg_extract('link_suffix', $params);
+	}
+}
diff --git a/elgg-plugin.php b/elgg-plugin.php
index 5a2be29..92a7bf8 100644
--- a/elgg-plugin.php
+++ b/elgg-plugin.php
@@ -1,6 +1,21 @@
 <?php
 
+use ColdTrick\TargetBlank\Bootstrap;
+
 return [
+	'bootstrap' => Bootstrap::class,
+	'hooks' => [
+		'elgg.data' => [
+			'page' => [
+				'\ColdTrick\TargetBlank\Javascript::setLinkSuffix' => [],
+			],
+		],
+		'setting' => [
+			'plugin' => [
+				'\ColdTrick\TargetBlank\PluginSettings::saveLinkSuffix' => [],
+			],
+		],
+	],
 	'routes' => [
 		'target_blank:test' => [
 			'path' => '/target_blank/test',
diff --git a/start.php b/start.php
deleted file mode 100644
index 724fdfb..0000000
--- a/start.php
+++ /dev/null
@@ -1,35 +0,0 @@
-<?php
-/**
- * Open external links in a new window.
- */
-
-/**
- * Inits the plugin
- *
- * @return void
- */
-function target_blank_init() {
-	elgg_require_js('target_blank/target_blank');
-	
-	elgg_register_plugin_hook_handler('elgg.data', 'page', function (\Elgg\Hook $hook) {
-		$result = $hook->getValue();
-		$result['target_blank']['link_suffix'] = elgg_get_plugin_setting('link_suffix', 'target_blank');
-		
-		return $result;
-	});
-
-	elgg_register_plugin_hook_handler('setting', 'plugin', function (\Elgg\Hook $hook) {
-		if ($hook->getParam('plugin_id') !== 'target_blank') {
-			return;
-		}
-		
-		if ($hook->getParam('name') !== 'link_suffix') {
-			return;
-		}
-		
-		$params = (array) get_input('params', [], false);
-		return elgg_extract('link_suffix', $params);
-	});
-}
-
-elgg_register_event_handler('init', 'system', 'target_blank_init');