In-Portal Issue Tracker

Index: core/kernel/db/db_tag_processor.php
===================================================================
--- core/kernel/db/db_tag_processor.php	(revision 13134)
+++ core/kernel/db/db_tag_processor.php	(working copy)
@@ -786,6 +786,7 @@
 
 		$first_chars = $this->SelectParam($params,'first_chars,cut_first');
 		if ($first_chars) {
+			$value = strip_tags($value, $this->SelectParam($params, 'allowed_tags')); // apply strip_tags before cutting 
 			$needs_cut = mb_strlen($value) > $first_chars;
 			$value = mb_substr($value, 0, $first_chars);
 			if ($needs_cut) $value .= ' ...';

Index: core/kernel/db/db_tag_processor.php
===================================================================
--- core/kernel/db/db_tag_processor.php	(revision 13152)
+++ core/kernel/db/db_tag_processor.php	(working copy)
@@ -786,6 +786,7 @@
 
 		$first_chars = $this->SelectParam($params,'first_chars,cut_first');
 		if ($first_chars) {
+			$value = strip_tags($value, $this->SelectParam($params, 'allowed_tags')); // apply strip_tags before cutting 
 			$needs_cut = mb_strlen($value) > $first_chars;
 			$value = mb_substr($value, 0, $first_chars);
 			if ($needs_cut) $value .= ' ...';
@@ -1672,12 +1673,13 @@
 		if ($this->Application->ConfigValue('UseSmallHeader') && isset($params['group_title']) && $params['group_title']) {
 			$title .= ' - '.$params['group_title'];
 		}
-
+		 
 		$cut_first = array_key_exists('cut_first', $params) && $params['cut_first'];
-
-		if ($cut_first && mb_strlen($title) > $cut_first) {
-			if (!preg_match('/<a href="(.*)">(.*)<\/a>/',$title)) {
-				$title = mb_substr($title, 0, $cut_first).' ...';
+		
+		if ( $cut_first && !preg_match('/<a href="(.*)">(.*)<\/a>/', $title) ) { // skip cut part if urls present in title
+			if (mb_strlen($title) > $cut_first) {
+				$allowed_tags = array_key_exists('allowed_tags', $params) && $params['allowed_tags'];				
+				$title = mb_substr( strip_tags($title, $allowed_tags) , 0, $cut_first) . ' ...'; // apply strip_tags before cutting
 			}
 		}
 

Index: db/db_tag_processor.php
===================================================================
--- db/db_tag_processor.php	(revision 13130)
+++ db/db_tag_processor.php	(working copy)
@@ -785,10 +785,13 @@
 		}
 
 		$first_chars = $this->SelectParam($params,'first_chars,cut_first');
+
 		if ($first_chars) {
-			$needs_cut = mb_strlen($value) > $first_chars;
-			$value = mb_substr($value, 0, $first_chars);
-			if ($needs_cut) $value .= ' ...';
+			$stripped_value = strip_tags($value, $this->SelectParam($params, 'allowed_tags'));
+
+			if (mb_strlen($stripped_value) > $first_chars) {
+				$value = mb_substr($stripped_value, 0, $first_chars) . ' ...';
+			}
 		}
 
 		if (array_key_exists('nl2br', $params) && $params['nl2br']) {
@@ -1673,11 +1676,14 @@
 			$title .= ' - '.$params['group_title'];
 		}
 
-		$cut_first = array_key_exists('cut_first', $params) && $params['cut_first'];
+		$first_chars = $this->SelectParam($params, 'first_chars,cut_first');
 
-		if ($cut_first && mb_strlen($title) > $cut_first) {
-			if (!preg_match('/<a href="(.*)">(.*)<\/a>/',$title)) {
-				$title = mb_substr($title, 0, $cut_first).' ...';
+		if ($first_chars && !preg_match('/<a href="(.*)".*>(.*)<\/a>/', $title)) {
+			// don't cut titles, that contain phrase translation links
+			$stripped_title = strip_tags($title, $this->SelectParam($params, 'allowed_tags'));
+
+			if (mb_strlen($stripped_title) > $first_chars) {
+				$title = mb_substr($stripped_title, 0, $first_chars) . ' ...';
 			}
 		}