EMTTretQuote::inject_in() - Code Metrics - it-blaster/typograph-bundle - Measure and Improve Code Quality continuously with Scrutinizer

EMTTretQuote::inject_in() A
last analyzed 2016-10-17 04:34 UTC

↳ Parent: EMTTretQuote

Complexity

Conditions	2
Paths	2

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

1		<?php
2
3		/**
4		* Evgeny Muravjev Typograph, http://mdash.ru
5		* Version: 3.4 Gold Master
6		* Release Date: September 20, 2014
7		* Authors: Evgeny Muravjev & Alexander Drutsa
8		*/
9
10		namespace Fenrizbes\TypographBundle\EMT;
11
12		/**
13		* @see EMTTret
14		*/
15
16		class EMTTretQuote extends EMTTret
17		{
18		/**
19		* Базовые параметры тофа
20		*
21		* @var array
22		*/
23		public $title = "Кавычки";
24
25
26		public $rules = array(
27		'quotes_outside_a' => array(
28		'description' => 'Кавычки вне тэга <a>',
29		//'pattern' => '/(\<%%\_\_.+?\>)\"(.+?)\"(\<\/%%\_\_.+?\>)/s',
		0 ignored issues – show Unused Code Comprehensibility introduced 2016-10-17 04:34 UTC by Report Bug Copy Issue Report `67%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
30		'pattern' => '/(\<%%\_\_[^\>]+\>)\"(.+?)\"(\<\/%%\_\_[^\>]+\>)/s',
31		'replacement' => '"\1\2\3"'
32		),
33
34		'open_quote' => array(
35		'description' => 'Открывающая кавычка',
36		'pattern' => '/(^\|\(\|\s\|\>\|-)((\"\|\\\")+)(\S+)/iue',
37		'replacement' => '$m[1] . str_repeat(self::QUOTE_FIRS_OPEN, substr_count($m[2],"\"") ) . $m[4]'
38		),
39		'close_quote' => array(
40		'description' => 'Закрывающая кавычка',
41		'pattern' => '/([a-zа-яё0-9]\|\.\|\&hellip\;\|\!\|\?\|\>\|\)\|\:\|\+\|\%\|\@\|\#\|\$\|\*)((\"\|\\\")+)(\.\|\&hellip\;\|\;\|\:\|\?\|\!\|\,\|\s\|\)\|\<\/\|\<\|$)/uie',
42		'replacement' => '$m[1] . str_repeat(self::QUOTE_FIRS_CLOSE, substr_count($m[2],"\"") ) . $m[4]'
43		),
44		'close_quote_adv' => array(
45		'description' => 'Закрывающая кавычка особые случаи',
46		//'pattern' => '/([a-zа-яё0-9]\|\.\|\&hellip\;\|\!\|\?\|\>\|\)\|\:)((\"\|\\\"\|\&laquo\;)+)(\<.+?\>)(\.\|\&hellip\;\|\;\|\:\|\?\|\!\|\,\|\s\|\)\|\<\/\|$)/uie',
		0 ignored issues – show Unused Code Comprehensibility introduced 2016-10-17 04:34 UTC by Report Bug Copy Issue Report `67%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
47		'pattern' =>
48		array(
49		'/([a-zа-яё0-9]\|\.\|\&hellip\;\|\!\|\?\|\>\|\)\|\:\|\+\|\%\|\@\|\#\|\$\|\*)((\"\|\\\"\|\&laquo\;)+)(\<[^\>]+\>)(\.\|\&hellip\;\|\;\|\:\|\?\|\!\|\,\|\)\|\<\/\|$\| )/uie',
50		'/([a-zа-яё0-9]\|\.\|\&hellip\;\|\!\|\?\|\>\|\)\|\:\|\+\|\%\|\@\|\#\|\$\|\*)(\s+)((\"\|\\\")+)(\s+)(\.\|\&hellip\;\|\;\|\:\|\?\|\!\|\,\|\)\|\<\/\|$\| )/uie',
51		'/\>(\&laquo\;)\.($\|\s\|\<)/ui',
52		'/\>(\&laquo\;),($\|\s\|\<\|\S)/ui',
53		'/\>(\&laquo\;):($\|\s\|\<\|\S)/ui',
54		'/\>(\&laquo\;);($\|\s\|\<\|\S)/ui',
55		'/\>(\&laquo\;)\)($\|\s\|\<\|\S)/ui',
56		'/((\"\|\\\")+)$/uie',
57		),
58		'replacement' =>
59		array(
60		'$m[1] . str_repeat(self::QUOTE_FIRS_CLOSE, substr_count($m[2],"\"")+substr_count($m[2],"«") ) . $m[4]. $m[5]',
61		'$m[1] .$m[2]. str_repeat(self::QUOTE_FIRS_CLOSE, substr_count($m[3],"\"")+substr_count($m[3],"«") ) . $m[5]. $m[6]',
62		'>».\2',
63		'>»,\2',
64		'>»:\2',
65		'>»;\2',
66		'>»)\2',
67		'str_repeat(self::QUOTE_FIRS_CLOSE, substr_count($m[1],"\"") )',
68		),
69		),
70		'open_quote_adv' => array(
71		'description' => 'Открывающая кавычка особые случаи',
72		'pattern' => '/(^\|\(\|\s\|\>)(\"\|\\\")(\s)(\S+)/iue',
73		'replacement' => '$m[1] . self::QUOTE_FIRS_OPEN .$m[4]'
74		),
75		'close_quote_adv_2' => array(
76		'description' => 'Закрывающая кавычка последний шанс',
77		'pattern' => '/(\S)((\"\|\\\")+)(\.\|\&hellip\;\|\;\|\:\|\?\|\!\|\,\|\s\|\)\|\<\/\|\<\|$)/uie',
78		'replacement' => '$m[1] . str_repeat(self::QUOTE_FIRS_CLOSE, substr_count($m[2],"\"") ) . $m[4]'
79		),
80		'quotation' => array(
81		'description' => 'Внутренние кавычки-лапки и дюймы',
82		'function' => 'build_sub_quotations'
83		),
84		);
85
86		protected function inject_in($pos, $text, &$thetext)
87		{
88		for($i=0;$i<strlen($text);$i++) $thetext[$pos+$i] = $text[$i];
89		}
90
91		protected function build_sub_quotations()
92		{
93		global $__ax,$__ay;
		0 ignored issues – show Compatibility Best Practice introduced 2016-10-17 04:34 UTC by Report Bug Copy Issue Report Use of `global` functionality is not recommended; it makes your code harder to test, and less reusable. Instead of relying on `global` state, we recommend one of these alternatives: 1. Pass all data via parameters function myFunction($a, $b) { // Do something } 2. Create a class that maintains your state class MyClass { private $a; private $b; public function __construct($a, $b) { $this->a = $a; $this->b = $b; } public function myFunction() { // Do something } } Loading history...
94
95		$exp = strpos($this->_text, "</cA===>")!==false ? "</cA===>" : (strpos($this->_text,"\r\n")!==false ? "\r\n\r\n" :"\n\n");
96
97		$texts_in = explode($exp, $this->_text);
98		$texts_out = array();
99
100		foreach($texts_in as $textx) {
101
102		$okposstack = array('0');
103		$okpos = 0;
104		$level = 0;
105		$off = 0;
106		while(true)
107		{
108		$p = EMTLib::strpos_ex($textx, array("«", "»"), $off);
109		if($p===false) break;
110	View Code Duplication	if($p['str'] == "«")
		0 ignored issues – show Duplication introduced 2016-10-17 04:34 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
111		{
112		if($level>0) if(!$this->is_on('no_bdquotes')) $this->inject_in($p['pos'], self::QUOTE_CRAWSE_OPEN, $textx);
113		$level++;
114		}
115	View Code Duplication	if($p['str'] == "»")
		0 ignored issues – show Duplication introduced 2016-10-17 04:34 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
116		{
117		$level--;
118		if($level>0) if(!$this->is_on('no_bdquotes')) $this->inject_in($p['pos'], self::QUOTE_CRAWSE_CLOSE, $textx);
119		}
120		$off = $p['pos']+strlen($p['str']);
121		if($level == 0)
122		{
123		$okpos = $off;
124		array_push($okposstack, $okpos);
125		} elseif($level<0) // уровень стал меньше нуля
126		{
127		if(!$this->is_on('no_inches'))
128		{
129		do{
130		$lokpos = array_pop($okposstack);
131		$k = substr($textx, $lokpos, $off-$lokpos);
132		$k = str_replace(self::QUOTE_CRAWSE_OPEN, self::QUOTE_FIRS_OPEN, $k);
133		$k = str_replace(self::QUOTE_CRAWSE_CLOSE, self::QUOTE_FIRS_CLOSE, $k);
134		//$k = preg_replace("/(^\|[^0-9])([0-9]+)\&raquo\;/ui", '\1\2″', $k, 1, $amount);
		0 ignored issues – show Unused Code Comprehensibility introduced 2016-10-17 04:34 UTC by Report Bug Copy Issue Report `62%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
135
136		$amount = 0;
137		$__ax = preg_match_all("/(^\|[^0-9])([0-9]+)\&raquo\;/ui", $k, $m);
138		$__ay = 0;
139		if($__ax)
140		{
141		$k = preg_replace_callback("/(^\|[^0-9])([0-9]+)\&raquo\;/ui",
142		create_function('$m','global $__ax,$__ay; $__ay++; if($__ay==$__ax){ return $m[1].$m[2]."″";} return $m[0];'),
		0 ignored issues – show Security Best Practice introduced 2016-10-17 04:34 UTC by Report Bug Copy Issue Report The use of `create_function` is highly discouraged, better use a closure. `create_function` can pose a great security vulnerability as it is similar to `eval`, and could be used for arbitrary code execution. We highly recommend to use a closure instead. // Instead of $function = create_function('$a, $b', 'return $a + $b'); // Better use $function = function($a, $b) { return $a + $b; } Loading history...
143		$k);
144		$amount = 1;
145		}
146
147
148
149		} while(($amount==0) && count($okposstack));
150
151		// успешно сделали замену
152		if($amount == 1)
153		{
154		// заново просмотрим содержимое
155		$textx = substr($textx, 0, $lokpos). $k . substr($textx, $off);
156		$off = $lokpos;
157		$level = 0;
158		continue;
159		}
160
161		// иначе просто заменим последнюю явно на " от отчаяния
162		if($amount == 0)
163		{
164		// говорим, что всё в порядке
165		$level = 0;
166		$textx = substr($textx, 0, $p['pos']). '"' . substr($textx, $off);
167		$off = $p['pos'] + strlen('"');
168		$okposstack = array($off);
169		continue;
170		}
171		}
172		}
173
174
175		}
176		// не совпало количество, отменяем все подкавычки
177		if($level != 0 ){
178
179		// закрывающих меньше, чем надо
180		if($level>0)
181		{
182		$k = substr($textx, $okpos);
183		$k = str_replace(self::QUOTE_CRAWSE_OPEN, self::QUOTE_FIRS_OPEN, $k);
184		$k = str_replace(self::QUOTE_CRAWSE_CLOSE, self::QUOTE_FIRS_CLOSE, $k);
185		$textx = substr($textx, 0, $okpos). $k;
186		}
187		}
188		$texts_out[] = $textx;
189		}
190		$this->_text = implode($exp, $texts_out);
191		}
192		}
193

it-blaster / typograph-bundle

EMTTretQuote::inject_in() A last analyzed 2016-10-17 04:34 UTC

Complexity

Size

Duplication

Importance

1. Pass all data via parameters

2. Create a class that maintains your state

Duplication Side-by-Side

Filter issues like

EMTTretQuote::inject_in() A
last analyzed 2016-10-17 04:34 UTC