Fix function line breaks

src/Abstracts/Carrier.php

@@ -167,7 +167,6 @@ public function __construct($slug = null, $name = null)
 	 */
 	public function __clone()
 	{
-
 		$fields = [];
 
 		foreach ($this->formFields as $rawName => $field) {
@@ -222,18 +221,13 @@ public function hash()
 	 */
 	public function addFormField(Interfaces\Fillable $field)
 	{
-
-		if (
-			in_array(
-				$field->getRawName(),
-				$this->restrictedFields,
-				true
-			)
-		) {
+		if (in_array($field->getRawName(), $this->restrictedFields, true)) {
 			throw new \Exception(
-				'You cannot use restricted field name. Restricted names: ' . implode(
-					', ',
-					$this->restrictedFields
+				sprintf(
+					'%s %s, %s',
+					'You cannot use restricted field name.',
+					'Restricted names:',
+					implode(', ', $this->restrictedFields)
 				)
 			);
 		}
@@ -304,15 +298,9 @@ public function getRecipientsField()
 
 		// Setup the field data if it's available.
 		if (!empty($this->recipientsResolvedData)) {
-			$this->setFieldData(
-				$field,
-				$this->recipientsResolvedData
-			);
+			$this->setFieldData($field, $this->recipientsResolvedData);
 		} else {
-			$this->setFieldData(
-				$field,
-				$this->recipientsData
-			);
+			$this->setFieldData($field, $this->recipientsData);
 		}
 
 		return $field;
@@ -383,10 +371,7 @@ public function resolveFields(Triggerable $trigger)
 				continue;
 			}
 
-			$resolved = $this->resolveValue(
-				$field->getValue(),
-				$trigger
-			);
+			$resolved = $this->resolveValue($field->getValue(), $trigger);
 			$field->setValue($resolved);
 		}
 
@@ -421,29 +406,17 @@ protected function resolveValue($value, Triggerable $trigger)
 			$resolved = [];
 
 			foreach ($value as $key => $val) {
-				$key = $this->resolveValue(
-					$key,
-					$trigger
-				);
-				$val = $this->resolveValue(
-					$val,
-					$trigger
-				);
+				$key = $this->resolveValue($key, $trigger);
+				$val = $this->resolveValue($val, $trigger);
 				$resolved[$key] = $val;
 			}
 
 			return $resolved;
 		}
 
-		$value = apply_filters(
-			'notification/carrier/field/resolving',
-			$value
-		);
+		$value = apply_filters('notification/carrier/field/resolving', $value);
 
-		$resolved = Resolver::resolve(
-			$value,
-			$trigger
-		);
+		$resolved = Resolver::resolve($value, $trigger);
 
 		// Unused tags.
 		$stripMergeTags = apply_filters(
@@ -470,17 +443,9 @@ protected function resolveValue($value, Triggerable $trigger)
 			: do_shortcode($resolved);
 
 		// Unescape escaped {.
-		$resolved = str_replace(
-			'!{',
-			'{',
-			$resolved
-		);
+		$resolved = str_replace('!{', '{', $resolved);
 
-		return apply_filters(
-			'notification/carrier/field/value/resolved',
-			$resolved,
-			null
-		);
+		return apply_filters('notification/carrier/field/value/resolved', $resolved, null);
 	}
 
 	/**
@@ -555,10 +520,7 @@ public function setData($data)
 				continue;
 			}
 
-			$this->setFieldData(
-				$field,
-				$data[$field->getRawName()]
-			);
+			$this->setFieldData($field, $data[$field->getRawName()]);
 		}
 
 		// Set recipients data.

src/Abstracts/Field.php

@@ -107,21 +107,11 @@ abstract class Field implements Interfaces\Fillable
 	 */
 	public function __construct($params = [])
 	{
-
 		if (!isset($params['label'], $params['name'])) {
-			trigger_error(
-				'Field requires label and name',
-				E_USER_ERROR
-			);
+			trigger_error('Field requires label and name', E_USER_ERROR);
 		}
 
-		$this->fieldTypeHtml = substr(
-			strrchr(
-				static::class,
-				'\\'
-			),
-			1
-		);
+		$this->fieldTypeHtml = substr(strrchr(static::class, '\\'), 1);
 
 		$this->label = $params['label'];
 		$this->name = $params['name'];
@@ -204,11 +194,7 @@ public function getValue()
 		$value = is_string($this->value)
 			? stripslashes($this->value)
 			: $this->value;
-		return apply_filters(
-			'notification/field/' . $this->getRawName() . '/value',
-			$value,
-			$this
-		);
+		return apply_filters('notification/field/' . $this->getRawName() . '/value', $value, $this);
 	}
 
 	/**

src/Abstracts/MergeTag.php

@@ -89,12 +89,8 @@ abstract class MergeTag implements Interfaces\Taggable
 	 */
 	public function __construct($params = [])
 	{
-
 		if (!isset($params['slug'], $params['name'], $params['resolver'])) {
-			trigger_error(
-				'Merge tag requires resolver',
-				E_USER_ERROR
-			);
+			trigger_error('Merge tag requires resolver', E_USER_ERROR);
 		}
 
 		if (!empty($params['slug'])) {
@@ -154,32 +150,22 @@ abstract public function sanitize($value);
 	 */
 	public function resolve()
 	{
-
 		if ($this->isResolved()) {
 			return $this->getValue();
 		}
 
 		try {
-			$value = call_user_func(
-				$this->resolver,
-				$this->getTrigger()
-			);
+			$value = call_user_func($this->resolver, $this->getTrigger());
 		} catch (\Throwable $t) {
 			$value = null;
-			trigger_error(
-				esc_html($t->getMessage()),
-				E_USER_NOTICE
-			);
+			trigger_error(esc_html($t->getMessage()), E_USER_NOTICE);
 		}
 
 		if (!empty($value) && !$this->validate($value)) {
 			$errorType = (defined('WP_DEBUG') && WP_DEBUG)
 				? E_USER_ERROR
 				: E_USER_NOTICE;
-			trigger_error(
-				'Resolved value is a wrong type',
-				$errorType
-			);
+			trigger_error('Resolved value is a wrong type', $errorType);
 		}
 
 		$this->resolved = true;
@@ -249,12 +235,8 @@ public function setTrigger(Interfaces\Triggerable $trigger)
 	 */
 	public function setResolver($resolver)
 	{
-
 		if (!is_callable($resolver)) {
-			trigger_error(
-				'Merge tag resolver has to be callable',
-				E_USER_ERROR
-			);
+			trigger_error('Merge tag resolver has to be callable', E_USER_ERROR);
 		}
 
 		$this->resolver = $resolver;

src/Abstracts/Recipient.php

@@ -39,7 +39,6 @@ abstract class Recipient implements Interfaces\Receivable
 	 */
 	public function __construct($params = [])
 	{
-
 		if (!empty($params['slug'])) {
 			$this->setSlug($params['slug']);
 		}
@@ -49,10 +48,7 @@ public function __construct($params = [])
 		}
 
 		if (!isset($params['default_value'])) {
-			trigger_error(
-				'Recipient requires default_value',
-				E_USER_ERROR
-			);
+			trigger_error('Recipient requires default_value', E_USER_ERROR);
 		}
 
 		$this->defaultValue = $params['default_value'];

src/Abstracts/Resolver.php

@@ -38,16 +38,9 @@ abstract class Resolver implements Interfaces\Resolvable
 	 */
 	public function getSlug()
 	{
-		$prepared = str_replace(
-			'\\',
-			'-',
-			static::class
-		);
-		$prepared = str_replace(
-			'BracketSpace-Notification-',
-			'',
-			$prepared
-		);
+		$prepared = str_replace('\\', '-', static::class);
+		$prepared = str_replace('BracketSpace-Notification-', '', $prepared);
+
 		return sanitize_title_with_dashes($prepared);
 	}
 

src/Abstracts/Trigger.php

@@ -98,7 +98,6 @@ public function mergeTags()
 	 */
 	public function setupMergeTags()
 	{
-
 		if ($this->mergeTagsAdded) {
 			return;
 		}
@@ -107,10 +106,7 @@ public function setupMergeTags()
 
 		$this->mergeTagsAdded = true;
 
-		do_action(
-			'notification/trigger/merge_tags',
-			$this
-		);
+		do_action('notification/trigger/merge_tags', $this);
 	}
 
 	/**
@@ -141,10 +137,7 @@ public function clearMergeTags()
 	public function addAction($tag, $priority = 10, $acceptedArgs = 1)
 	{
 		if (empty($tag)) {
-			trigger_error(
-				'Action tag cannot be empty',
-				E_USER_ERROR
-			);
+			trigger_error('Action tag cannot be empty', E_USER_ERROR);
 		}
 
 		array_push(
@@ -167,12 +160,8 @@ public function addAction($tag, $priority = 10, $acceptedArgs = 1)
 	 */
 	public function removeAction($tag, $priority = 10, $deprecated = null)
 	{
-
 		if (empty($tag)) {
-			trigger_error(
-				'Action tag cannot be empty',
-				E_USER_ERROR
-			);
+			trigger_error('Action tag cannot be empty', E_USER_ERROR);
 		}
 
 		foreach ($this->actions as $actionIndex => $action) {
@@ -244,7 +233,6 @@ public function addQuickMergeTag($propertyName, $label, $group = null)
 	 */
 	public function removeMergeTag($mergeTagSlug)
 	{
-
 		foreach ($this->mergeTags as $index => $mergeTag) {
 			if ($mergeTag->getSlug() === $mergeTagSlug) {
 				unset($this->mergeTags[$index]);
@@ -266,7 +254,6 @@ public function removeMergeTag($mergeTagSlug)
 	 */
 	public function getMergeTags($type = 'all', $grouped = false)
 	{
-
 		if (!$this->mergeTagsAdded) {
 			$this->setupMergeTags();
 		}

src/Admin/CheckRestApi.php

@@ -43,18 +43,14 @@ public function testRestApi()
 			)
 		);
 
-		$message = json_decode(
-			wp_remote_retrieve_body($response),
-			true
-		);
+		$message = json_decode(wp_remote_retrieve_body($response), true);
 
 		$isAvailable = false;
 
 		if (
-			is_array($message) && array_key_exists(
-				'data',
-				$message
-			) && $message['data'] === 'RestApi'
+			is_array($message) &&
+			array_key_exists('data', $message) &&
+			$message['data'] === 'RestApi'
 		) {
 			return;
 		}