seoController::_addNofollow() - Code Metrics - Inspection of "SEO모듈 댓글에 포함된 링크 추적 방지 기능 추가" - xpressengine/xe-core - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — develop (#2395)

by звезда

created 2019-04-23 11:47 UTC

seoController::_addNofollow() A

↳ Parent: seoController

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	1
dl	0
loc	4
rs	10
c	0
b	0
f	0

<?php
class seoController extends seo
{
	function getBrowserTitle($document_title = null)
	{
		$site_module_info = Context::get('site_module_info');
		if ($site_module_info->site_srl != 0) return Context::getBrowserTitle();

		$config = $this->getConfig();
		if ($config->use_optimize_title != 'Y') return Context::getBrowserTitle();

		$current_module_info = Context::get('current_module_info');
		$is_index = ($current_module_info->module_srl == $site_module_info->module_srl) ? true : false;

		$piece = array();
		$piece['[:site_name:]'] = $config->site_name;
		$piece['[:site_slogan:]'] = $config->site_slogan;
		$piece['[:module_title:]'] = $current_module_info->browser_title;
		if ($document_title) $piece['[:document_title:]'] = $document_title;

		if ($config->use_optimize_title == 'Y') {
			$title = array();
			if ($piece['[:document_title:]']) {
				$title[] = $piece['[:document_title:]'];
				$title[] = $piece['[:module_title:]'];
				$title[] = $piece['[:site_name:]'];
			} else {
				if ($is_index) {
					$title[] = $piece['[:site_name:]'];
					if ($piece['[:site_slogan:]']) $title[] = $piece['[:site_slogan:]'];
				} else {
					$title[] = $piece['[:module_title:]'];
					$title[] = $piece['[:site_name:]'];
				}
			}
			$title = implode(' - ', $title);
		}

		return $title;
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
	}

	function triggerBeforeDisplay(&$output_content)
	{
		if (Context::getResponseMethod() != 'HTML') return;
		if (Context::get('module') == 'admin') return;

		require_once(_XE_PATH_ . 'libs/idna_convert/idna_convert.class.php');

		$oCacheHandler = CacheHandler::getInstance('object', NULL, TRUE);

		$locales = array(
			'de' => 'de_DE',
			'en' => 'en_US',
			'es' => 'es_ES',
			'fr' => 'fr_FR',
			'ja' => 'ja_JP',
			'jp' => 'ja_JP',
			'ko' => 'ko_KR',
			'mn' => 'mn_MN',
			'ru' => 'ru_RU',
			'tr' => 'tr_TR',
			'vi' => 'vi_VN',
			'zh-CN' => 'zh_CN',
			'zh-TW' => 'zh_TW',
		);

		$oModuleModel = getModel('module');
		$config = $this->getConfig();
		$IDN = new idna_convert(array('idn_version' => 2008));
		$request_uri = $IDN->encode(Context::get('request_uri'));

		$logged_info = Context::get('logged_info');
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
		$current_module_info = Context::get('current_module_info');
		$site_module_info = Context::get('site_module_info');
		$document_srl = Context::get('document_srl');
		$is_article = false;
		$single_image = false;
		$is_index = ($current_module_info->module_srl == $site_module_info->module_srl) ? true : false;

		$piece = new stdClass;
		$piece->document_title = null;
		$piece->type = 'website';
		$piece->url = getFullUrl('');
		$piece->title = Context::getBrowserTitle();
		$piece->description = $config->site_description;
		$piece->keywords = $config->site_keywords;
		$piece->tags = array();
		$piece->image = array();
		$piece->author = null;

		if(stristr($_SERVER['HTTP_USER_AGENT'], 'facebookexternalhit') != FALSE) {

			$single_image = true;
		}


		if($current_module_info->module_srl !== $site_module_info->module_srl) {
			$mdoulePartConfig = $oModuleModel->getModulePartConfig('seo', $current_module_info->module_srl);
			if($mdoulePartConfig && isset($mdoulePartConfig->meta_description) && trim($mdoulePartConfig->meta_description)) {
				$piece->description = trim($mdoulePartConfig->meta_description);
			}
		}

		if ($document_srl) {
			$oDocument = Context::get('oDocument');
			if (!is_a($oDocument, 'documentItem')) {
				$oDocumentModel = getModel('document');
				$oDocument = $oDocumentModel->getDocument($document_srl);
			}

			if (is_a($oDocument, 'documentItem') && $document_srl == $oDocument->document_srl) {
				$is_article = true;
			}
		}

		// 문서 데이터 수집
		if ($is_article) {
			if (!$oDocument->isSecret()) {
				$piece->document_title = $oDocument->getTitleText();
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
				$piece->url = getFullUrl('', 'mid', $current_module_info->mid, 'document_srl',$document_srl);
				$piece->type = 'article';
				$piece->description = trim(str_replace('&nbsp;', ' ', $oDocument->getContentText(400)));
				$piece->author = $oDocument->getNickName();
				$tags = $oDocument->get('tag_list');
				if (count($tags)) $piece->tags = $tags;

				$document_images = false;
				if($oCacheHandler->isSupport()) {
					$cache_key_document_images = 'seo:document_images:' . $document_srl;
					$document_images = $oCacheHandler->get($cache_key_document_images);
				}

				if($document_images === false && $oDocument->hasUploadedFiles()) {
					$image_ext = array('bmp', 'gif', 'jpg', 'jpeg', 'png');
					$document_images = array();

					foreach ($oDocument->getUploadedFiles() as $file) {
						if ($file->isvalid != 'Y') continue;

						$ext = array_pop(explode('.', $file->uploaded_filename));


						if (!in_array(strtolower($ext), $image_ext)) continue;
						list($width, $height) = @getimagesize($file->uploaded_filename);
						if($width < 100 && $height < 100) continue;

						$image = array(
							'filepath' => $file->uploaded_filename,
							'width' => $width,
							'height' => $height
						);

						if($file->cover_image === 'Y') {
							array_unshift($document_images, $image);
						} else {
							$document_images[] = $image;
						}
					}

					if($oCacheHandler->isSupport()) {
						$oCacheHandler->put($cache_key_document_images, $document_images);
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
					}
				}

				if($document_images) $piece->image = $document_images;
			} else {
				$piece->url = getFullUrl('', 'mid', $current_module_info->mid);
			}
		} else {
			if (!$is_index) {
				$page = (Context::get('page') > 1) ? Context::get('page') : null;
				$piece->url = getNotEncodedFullUrl('mid', $current_module_info->mid, 'page',$page);
			}
		}

		$piece->title = $this->getBrowserTitle($piece->document_title);

		if($oCacheHandler->isSupport()) {
			$cache_key = 'seo:site_image';
			$site_image = $oCacheHandler->get($cache_key);
			if($site_image) {
				$site_image['url'] = $config->site_image_url;
			}
			$piece->image[] = $site_image;
		}

		$this->addLink('canonical', $piece->url);
		$this->addMeta('keywords', $piece->keywords, 'name');
		$this->addMeta('description', $piece->description, 'name');

		// Open Graph
		$this->addMeta('og:locale', $locales[Context::getLangType()]);
		$this->addMeta('og:type', $piece->type);
		$this->addMeta('og:url', $piece->url);
		$this->addMeta('og:site_name', $config->site_name);
		$this->addMeta('og:title', $piece->title);
		$this->addMeta('og:description', $piece->description);
		if($is_article) {
			if(Context::getLangType() !== $oDocument->getLangCode()) {
				$this->addMeta('og:locale:alternate', $locales[$oDocument->getLangCode()]);
			}
			$this->addMeta('article:published_time', $oDocument->getRegdate('c'));
			$this->addMeta('article:modified_time', $oDocument->getUpdate('c'));
			foreach ($piece->tags as $tag) {
				$this->addMeta('article:tag', $tag);
			}
		}

		foreach ($piece->image as $img) {
$collection = json_decode($data, true);
if ( ! is_array($collection)) {
    throw new \RuntimeException('$collection must be an array.');
}

foreach ($collection as $item) { /** ... */ }
			if(!$img['url']) {
				if(!$img['filepath']) continue;
				$img['url'] = $request_uri . $img['filepath'];
			}

			$this->addMeta('og:image', $img['url']);
			$this->addMeta('og:image:width', $img['width']);
			$this->addMeta('og:image:height', $img['height']);
			if($single_image) break;
		}

		$this->canonical_url = $piece->url;

		$this->applySEO();

		if ($config->use_optimize_title == 'Y') Context::setBrowserTitle($piece->title);

		if($config->link_nofollow == 'Y') {
			$temp_output = (string)$output_content;
			$temp_output = preg_replace_callback('!<\!--BeforeComment\([0-9]+,[0-9]+\)-->(.+?)<\!--AfterComment\([0-9]+,[0-9]+\)-->!is', array($this, 'replaceCommentHyperlink'), $temp_output);
			$output_content = $temp_output;
		}
	}

	private function replaceCommentHyperlink($matches)
	{
		return preg_replace_callback('!<a((?:\s(?:href|target|rel)=(?:"|\')?[^"\']+(?:"|\')?)+)>(.+?)<\/a>!is', array($this, '_addNofollow'), $matches[0]);
	}

	private function _addNofollow($matches)
	{
		return '<a' . preg_replace('!\srel=(?:"|\')?([^"\']+)(?:"|\')?!is', '', $matches[1]) . ' rel="nofollow">' . $matches[2] . '</a>';
	}

	function triggerAfterFileDeleteFile($data)
	{
		$document_srl = $data->upload_target_srl;
		if(!$document_srl) return $this->makeObject();

		$this->deleteCacheDocumentImages($document_srl);
	}

	function triggerAfterDocumentUpdateDocument($data)
	{
		$document_srl = $data->document_srl;
		$this->deleteCacheDocumentImages($document_srl);
	}

	function triggerAfterDocumentDeleteDocument($data)
	{
		$document_srl = $data->document_srl;
		$this->deleteCacheDocumentImages($document_srl);
	}

	private function deleteCacheDocumentImages($document_srl)
	{
		$oCacheHandler = CacheHandler::getInstance('object', NULL, TRUE);
		if($oCacheHandler->isSupport()) {
			$cache_key_document_images = 'seo:document_images:' . $document_srl;
			$oCacheHandler->delete($cache_key_document_images);
		}
	}
}
/* !End of file */


GitHub Access Token became invalid

Pull Request — develop (#2395)

seoController::_addNofollow() A

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Available Fixes

1			<?php
2			class seoController extends seo
3			{
4			function getBrowserTitle($document_title = null)
5			{
6			$site_module_info = Context::get('site_module_info');
7			if ($site_module_info->site_srl != 0) return Context::getBrowserTitle();
8
9			$config = $this->getConfig();
10			if ($config->use_optimize_title != 'Y') return Context::getBrowserTitle();
11
12			$current_module_info = Context::get('current_module_info');
13			$is_index = ($current_module_info->module_srl == $site_module_info->module_srl) ? true : false;
14
15			$piece = array();
16			$piece['[:site_name:]'] = $config->site_name;
17			$piece['[:site_slogan:]'] = $config->site_slogan;
18			$piece['[:module_title:]'] = $current_module_info->browser_title;
19			if ($document_title) $piece['[:document_title:]'] = $document_title;
20
21			if ($config->use_optimize_title == 'Y') {
22			$title = array();
23			if ($piece['[:document_title:]']) {
24			$title[] = $piece['[:document_title:]'];
25			$title[] = $piece['[:module_title:]'];
26			$title[] = $piece['[:site_name:]'];
27			} else {
28			if ($is_index) {
29			$title[] = $piece['[:site_name:]'];
30			if ($piece['[:site_slogan:]']) $title[] = $piece['[:site_slogan:]'];
31			} else {
32			$title[] = $piece['[:module_title:]'];
33			$title[] = $piece['[:site_name:]'];
34			}
35			}
36			$title = implode(' - ', $title);
37			}
38
39			return $title;
			0 ignored issues – show Bug introduced 2018-01-02 09:19 UTC by Report Bug Copy Issue Report The variable `$title` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
40			}
41
42			function triggerBeforeDisplay(&$output_content)
43			{
44			if (Context::getResponseMethod() != 'HTML') return;
45			if (Context::get('module') == 'admin') return;
46
47			require_once(_XE_PATH_ . 'libs/idna_convert/idna_convert.class.php');
48
49			$oCacheHandler = CacheHandler::getInstance('object', NULL, TRUE);
50
51			$locales = array(
52			'de' => 'de_DE',
53			'en' => 'en_US',
54			'es' => 'es_ES',
55			'fr' => 'fr_FR',
56			'ja' => 'ja_JP',
57			'jp' => 'ja_JP',
58			'ko' => 'ko_KR',
59			'mn' => 'mn_MN',
60			'ru' => 'ru_RU',
61			'tr' => 'tr_TR',
62			'vi' => 'vi_VN',
63			'zh-CN' => 'zh_CN',
64			'zh-TW' => 'zh_TW',
65			);
66
67			$oModuleModel = getModel('module');
68			$config = $this->getConfig();
69			$IDN = new idna_convert(array('idn_version' => 2008));
70			$request_uri = $IDN->encode(Context::get('request_uri'));
71
72			$logged_info = Context::get('logged_info');
			0 ignored issues – show Unused Code introduced 2015-11-22 14:33 UTC by Report Bug Copy Issue Report `$logged_info` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
73			$current_module_info = Context::get('current_module_info');
74			$site_module_info = Context::get('site_module_info');
75			$document_srl = Context::get('document_srl');
76			$is_article = false;
77			$single_image = false;
78			$is_index = ($current_module_info->module_srl == $site_module_info->module_srl) ? true : false;
79
80			$piece = new stdClass;
81			$piece->document_title = null;
82			$piece->type = 'website';
83			$piece->url = getFullUrl('');
84			$piece->title = Context::getBrowserTitle();
85			$piece->description = $config->site_description;
86			$piece->keywords = $config->site_keywords;
87			$piece->tags = array();
88			$piece->image = array();
89			$piece->author = null;
90
91			if(stristr($_SERVER['HTTP_USER_AGENT'], 'facebookexternalhit') != FALSE) {
			0 ignored issues – show Bug Best Practice introduced 2018-01-02 09:19 UTC by Report Bug Copy Issue Report It seems like you are loosely comparing `stristr($_SERVER['HTTP_U... 'facebookexternalhit')` of type `string` to the boolean `FALSE`. If you are specifically checking for a non-empty string, consider using the more explicit `!== ''` instead. Loading history...
92			$single_image = true;
93			}
94
95
96			if($current_module_info->module_srl !== $site_module_info->module_srl) {
97			$mdoulePartConfig = $oModuleModel->getModulePartConfig('seo', $current_module_info->module_srl);
98			if($mdoulePartConfig && isset($mdoulePartConfig->meta_description) && trim($mdoulePartConfig->meta_description)) {
99			$piece->description = trim($mdoulePartConfig->meta_description);
100			}
101			}
102
103			if ($document_srl) {
104			$oDocument = Context::get('oDocument');
105			if (!is_a($oDocument, 'documentItem')) {
106			$oDocumentModel = getModel('document');
107			$oDocument = $oDocumentModel->getDocument($document_srl);
108			}
109
110			if (is_a($oDocument, 'documentItem') && $document_srl == $oDocument->document_srl) {
111			$is_article = true;
112			}
113			}
114
115			// 문서 데이터 수집
116			if ($is_article) {
117			if (!$oDocument->isSecret()) {
118			$piece->document_title = $oDocument->getTitleText();
			0 ignored issues – show Bug introduced 2018-01-02 09:19 UTC by Report Bug Copy Issue Report The variable `$oDocument` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
119			$piece->url = getFullUrl('', 'mid', $current_module_info->mid, 'document_srl',$document_srl);
120			$piece->type = 'article';
121			$piece->description = trim(str_replace(' ', ' ', $oDocument->getContentText(400)));
122			$piece->author = $oDocument->getNickName();
123			$tags = $oDocument->get('tag_list');
124			if (count($tags)) $piece->tags = $tags;
125
126			$document_images = false;
127			if($oCacheHandler->isSupport()) {
128			$cache_key_document_images = 'seo:document_images:' . $document_srl;
129			$document_images = $oCacheHandler->get($cache_key_document_images);
130			}
131
132			if($document_images === false && $oDocument->hasUploadedFiles()) {
133			$image_ext = array('bmp', 'gif', 'jpg', 'jpeg', 'png');
134			$document_images = array();
135
136			foreach ($oDocument->getUploadedFiles() as $file) {
137			if ($file->isvalid != 'Y') continue;
138
139			$ext = array_pop(explode('.', $file->uploaded_filename));
			0 ignored issues – show Bug introduced 2018-01-02 09:19 UTC by Report Bug Copy Issue Report `explode('.', $file->uploaded_filename)` cannot be passed to `array_pop()` as the parameter `$array` expects a reference. Loading history...
140
141			if (!in_array(strtolower($ext), $image_ext)) continue;
142			list($width, $height) = @getimagesize($file->uploaded_filename);
143			if($width < 100 && $height < 100) continue;
144
145			$image = array(
146			'filepath' => $file->uploaded_filename,
147			'width' => $width,
148			'height' => $height
149			);
150
151			if($file->cover_image === 'Y') {
152			array_unshift($document_images, $image);
153			} else {
154			$document_images[] = $image;
155			}
156			}
157
158			if($oCacheHandler->isSupport()) {
159			$oCacheHandler->put($cache_key_document_images, $document_images);
			0 ignored issues – show Bug introduced 2018-01-02 09:19 UTC by Report Bug Copy Issue Report The variable `$cache_key_document_images` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
160			}
161			}
162
163			if($document_images) $piece->image = $document_images;
164			} else {
165			$piece->url = getFullUrl('', 'mid', $current_module_info->mid);
166			}
167			} else {
168			if (!$is_index) {
169			$page = (Context::get('page') > 1) ? Context::get('page') : null;
170			$piece->url = getNotEncodedFullUrl('mid', $current_module_info->mid, 'page',$page);
171			}
172			}
173
174			$piece->title = $this->getBrowserTitle($piece->document_title);
175
176			if($oCacheHandler->isSupport()) {
177			$cache_key = 'seo:site_image';
178			$site_image = $oCacheHandler->get($cache_key);
179			if($site_image) {
180			$site_image['url'] = $config->site_image_url;
181			}
182			$piece->image[] = $site_image;
183			}
184
185			$this->addLink('canonical', $piece->url);
186			$this->addMeta('keywords', $piece->keywords, 'name');
187			$this->addMeta('description', $piece->description, 'name');
188
189			// Open Graph
190			$this->addMeta('og:locale', $locales[Context::getLangType()]);
191			$this->addMeta('og:type', $piece->type);
192			$this->addMeta('og:url', $piece->url);
193			$this->addMeta('og:site_name', $config->site_name);
194			$this->addMeta('og:title', $piece->title);
195			$this->addMeta('og:description', $piece->description);
196			if($is_article) {
197			if(Context::getLangType() !== $oDocument->getLangCode()) {
198			$this->addMeta('og:locale:alternate', $locales[$oDocument->getLangCode()]);
199			}
200			$this->addMeta('article:published_time', $oDocument->getRegdate('c'));
201			$this->addMeta('article:modified_time', $oDocument->getUpdate('c'));
202			foreach ($piece->tags as $tag) {
203			$this->addMeta('article:tag', $tag);
204			}
205			}
206
207			foreach ($piece->image as $img) {
			0 ignored issues – show Bug introduced 2018-01-02 09:19 UTC by Report Bug Copy Issue Report The expression `$piece->image` of type `array\|boolean` is not guaranteed to be traversable. How about adding an additional type check? There are different options of fixing this problem. If you want to be on the safe side, you can add an additional type-check: $collection = json_decode($data, true); if ( ! is_array($collection)) { throw new \RuntimeException('$collection must be an array.'); } foreach ($collection as $item) { /** ... / } If you are sure that the expression is traversable, you might want to add a doc comment cast* to improve IDE auto-completion and static analysis: /** @var array $collection / $collection = json_decode($data, true); foreach ($collection as $item) { /* .. / } Mark the issue as a false-positive*: Just hover the remove button, in the top-right corner of this issue for more options. Loading history...
208			if(!$img['url']) {
209			if(!$img['filepath']) continue;
210			$img['url'] = $request_uri . $img['filepath'];
211			}
212
213			$this->addMeta('og:image', $img['url']);
214			$this->addMeta('og:image:width', $img['width']);
215			$this->addMeta('og:image:height', $img['height']);
216			if($single_image) break;
217			}
218
219			$this->canonical_url = $piece->url;
220
221			$this->applySEO();
222
223			if ($config->use_optimize_title == 'Y') Context::setBrowserTitle($piece->title);
224
225			if($config->link_nofollow == 'Y') {
226			$temp_output = (string)$output_content;
227			$temp_output = preg_replace_callback('!<\!--BeforeComment\([0-9]+,[0-9]+\)-->(.+?)<\!--AfterComment\([0-9]+,[0-9]+\)-->!is', array($this, 'replaceCommentHyperlink'), $temp_output);
228			$output_content = $temp_output;
229			}
230			}
231
232			private function replaceCommentHyperlink($matches)
233			{
234			return preg_replace_callback('!<a((?:\s(?:href\|target\|rel)=(?:"\|\')?[^"\']+(?:"\|\')?)+)>(.+?)<\/a>!is', array($this, '_addNofollow'), $matches[0]);
235			}
236
237			private function _addNofollow($matches)
238			{
239			return '<a' . preg_replace('!\srel=(?:"\|\')?([^"\']+)(?:"\|\')?!is', '', $matches[1]) . ' rel="nofollow">' . $matches[2] . '</a>';
240			}
241
242			function triggerAfterFileDeleteFile($data)
243			{
244			$document_srl = $data->upload_target_srl;
245			if(!$document_srl) return $this->makeObject();
246
247			$this->deleteCacheDocumentImages($document_srl);
248			}
249
250			function triggerAfterDocumentUpdateDocument($data)
251			{
252			$document_srl = $data->document_srl;
253			$this->deleteCacheDocumentImages($document_srl);
254			}
255
256			function triggerAfterDocumentDeleteDocument($data)
257			{
258			$document_srl = $data->document_srl;
259			$this->deleteCacheDocumentImages($document_srl);
260			}
261
262			private function deleteCacheDocumentImages($document_srl)
263			{
264			$oCacheHandler = CacheHandler::getInstance('object', NULL, TRUE);
265			if($oCacheHandler->isSupport()) {
266			$cache_key_document_images = 'seo:document_images:' . $document_srl;
267			$oCacheHandler->delete($cache_key_document_images);
268			}
269			}
270			}
271			/* !End of file */
272

xpressengine / xe-core

GitHub Access Token became invalid

Pull Request — develop (#2395)

seoController::_addNofollow() A

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like