remove_bbcodes_legacy::set_data() - Code Metrics - Inspection of "Test" - iMattPro/topicpreview - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#28)

by Matt

created 2016-02-04 20:46 UTC

remove_bbcodes_legacy::set_data() A

↳ Parent: remove_bbcodes_legacy

Complexity

Conditions	4
Paths	3

Size

Total Lines	10
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	10
rs	9.2
cc	4
eloc	5
nc	3
nop	0

<?php
/**
 *
 * Topic Preview
 *
 * @copyright (c) 2016 Matt Friedman
 * @license GNU General Public License, version 2 (GPL-2.0)
 *
 */

namespace vse\topicpreview\core\tools;

use phpbb\config\config;

class remove_bbcodes_legacy extends base
{
	/** @var config */
	protected $config;

	/** @var string Regex data of BBCodes to remove */
	protected $data;

	/**
	 * Constructor
	 *
	 * @param config $config
	 * @access public
	 */
	public function __construct(config $config)
	{
		$this->config = $config;
	}

	/**
	 * @inheritdoc
	 */
	public function run()
	{
		return $this->set_data()->remove_bbcodes();
	}

	/**
	 * @inheritdoc
	 */
	public function set_data()
	{
		if (!isset($this->data) || is_array($this->data))
		{
			$strip_bbcodes = (!empty($this->config['topic_preview_strip_bbcodes'])) ? 'flash|' . trim($this->config['topic_preview_strip_bbcodes']) : 'flash';
			$this->data = '#\[(' . $strip_bbcodes . ')[^\[\]]*\]((?:(?!\[\1[^\[\]]*\]).)*)\[\/\1[^\[\]]*\]#Usi';
		}

		return $this;
	}

	/**
	 * Remove specified BBCodes and their contents
	 * Uses recursion to handle nested BBCodes
	 *
	 * @return string Stripped message text
	 * @access protected
	 */
	protected function remove_bbcodes()
	{
		if (preg_match($this->data, $this->text))
		{
			$this->text = preg_replace($this->data, '', $this->text);
			return $this->remove_bbcodes();
		}

		return $this->text;
	}
}


1			<?php
2			/**
3			*
4			* Topic Preview
5			*
6			* @copyright (c) 2016 Matt Friedman
7			* @license GNU General Public License, version 2 (GPL-2.0)
8			*
9			*/
10
11			namespace vse\topicpreview\core\tools;
12
13			use phpbb\config\config;
14
15			class remove_bbcodes_legacy extends base
16			{
17			/** @var config */
18			protected $config;
19
20			/** @var string Regex data of BBCodes to remove */
21			protected $data;
22
23			/**
24			* Constructor
25			*
26			* @param config $config
27			* @access public
28			*/
29			public function __construct(config $config)
30			{
31			$this->config = $config;
32			}
33
34			/**
35			* @inheritdoc
36			*/
37			public function run()
38			{
39			return $this->set_data()->remove_bbcodes();
40			}
41
42			/**
43			* @inheritdoc
44			*/
45			public function set_data()
46			{
47			if (!isset($this->data) \|\| is_array($this->data))
48			{
49			$strip_bbcodes = (!empty($this->config['topic_preview_strip_bbcodes'])) ? 'flash\|' . trim($this->config['topic_preview_strip_bbcodes']) : 'flash';
50			$this->data = '#\[(' . $strip_bbcodes . ')[^\[\]]\]((?:(?!\[\1[^\[\]]\]).))\[\/\1[^\[\]]\]#Usi';
51			}
52
53			return $this;
54			}
55
56			/**
57			* Remove specified BBCodes and their contents
58			* Uses recursion to handle nested BBCodes
59			*
60			* @return string Stripped message text
61			* @access protected
62			*/
63			protected function remove_bbcodes()
64			{
65			if (preg_match($this->data, $this->text))
66			{
67			$this->text = preg_replace($this->data, '', $this->text);
68			return $this->remove_bbcodes();
69			}
70
71			return $this->text;
72			}
73			}
74

iMattPro / topicpreview

Pull Request — master (#28)

remove_bbcodes_legacy::set_data() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like