diff --git a/e107_core/url/news/sef_noid_url.php b/e107_core/url/news/sef_noid_url.php
index 72bf4336f..5ce292cd6 100644
--- a/e107_core/url/news/sef_noid_url.php
+++ b/e107_core/url/news/sef_noid_url.php
@@ -124,7 +124,7 @@ class core_news_sef_noid_url extends eUrlConfig
 						// news/Category/Category-Name?page=xxx
 						// news/Short/Category-Name?page=xxx
 						$r[0] = $route[1] == 'category' ? 'Category' :  'Short';
-						$r[1] = $params['name'] ? $params['name'] : $params['id'];
+						$r[1] = !empty($params['name']) ? $params['name'] : $params['id'];
 						if($page) $parm = array('page' => $page);  
 					}
 				break;
diff --git a/e107_plugins/banner/banner_template.php b/e107_plugins/banner/banner_template.php
index 602f274bf..b9ece302b 100644
--- a/e107_plugins/banner/banner_template.php
+++ b/e107_plugins/banner/banner_template.php
@@ -64,7 +64,8 @@ if(!isset($BANNER_TABLE))
 			<td colspan='2' class='forumheader3' style='text-align:center'>{BANNER_TABLE_ACTIVE}</td>
 			<td colspan='4' class='forumheader3' style='text-align:center'>{BANNER_TABLE_STARTDATE} {BANNER_TABLE_ENDDATE}</td>";
 
-			if($BANNER_TABLE_IP){
+			if(!empty($BANNER_TABLE_IP)) // review
+			{
 				$BANNER_TABLE .= "
 				</tr>
 				<tr>
@@ -104,7 +105,7 @@ if(!isset($BANNER_MENU_END))
 // ##### ------------------------------------ v2.x ------------------------------------------------------
 
 
-$BANNER_TEMPLATE['menu']['start'] =		"<div class='banner-menu text-center'>{SETIMAGE: w=800}"; 
+$BANNER_TEMPLATE['menu']['start'] =		"<div class='banner-menu text-center'>{SETIMAGE: w=800}";
 $BANNER_TEMPLATE['menu']['item'] =		"<div class='banner-menu-item'>{BANNER}<br /><br /></div>"; 
 $BANNER_TEMPLATE['menu']['end'] = 		"</div>";