From d182a2c26cd82d05868998bd4e7306166275638a Mon Sep 17 00:00:00 2001
From: Kijin Sung <kijin@kijinsung.com>
Date: Mon, 11 Sep 2017 21:10:40 +0900
Subject: [PATCH] Fix all remaining skins making unnecessary use of
 window.open()

---
 layouts/simple_world/layout.html                          | 2 +-
 modules/member/m.skins/default/comment_list.html          | 2 +-
 modules/member/m.skins/default/document_list.html         | 2 +-
 modules/member/m.skins/default/saved_list.html            | 2 +-
 modules/member/m.skins/default/scrapped_list.html         | 2 +-
 modules/member/skins/default/comment_list.html            | 2 +-
 modules/member/skins/simple_world/comment_list.html       | 2 +-
 modules/member/skins/simple_world/document_list.html      | 2 +-
 modules/member/skins/simple_world/saved_list.html         | 2 +-
 modules/member/skins/simple_world/scrapped_list.html      | 2 +-
 widgets/content/skins/simple_rectangle/title_content.html | 4 ++--
 11 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/layouts/simple_world/layout.html b/layouts/simple_world/layout.html
index e793972a7..76d784406 100644
--- a/layouts/simple_world/layout.html
+++ b/layouts/simple_world/layout.html
@@ -166,7 +166,7 @@
 				<li loop="$lang_supported=>$key,$val" cond="$key!= $lang_type"><button type="button" onclick="doChangeLangType('{$key}');return false;">{$val}</button></li>
 			</ul>
 		</div>
-		<p cond="!$layout_info->FOOTER">Powered by <a href="https://www.rhymix.org/">Rhymix</a>.</p>
+		<p cond="!$layout_info->FOOTER">Powered by <a href="https://www.rhymix.org/" target="_blank" rel="noopener">Rhymix</a>.</p>
 		<p cond="$layout_info->FOOTER">{$layout_info->FOOTER}</p>
 	</div>
 </footer>
\ No newline at end of file
diff --git a/modules/member/m.skins/default/comment_list.html b/modules/member/m.skins/default/comment_list.html
index 3bffeb5d6..5d29c7be6 100644
--- a/modules/member/m.skins/default/comment_list.html
+++ b/modules/member/m.skins/default/comment_list.html
@@ -3,7 +3,7 @@
 	<h2 class="hx h2">{$member_title = $lang->cmd_view_own_comment}</h2>
 	<ul class="lt">
 		<li loop="$comment_list => $no,$comment">
-			<a href="{getUrl('','document_srl',$comment->document_srl)}#comment_{$comment->comment_srl}" onclick="window.open(this.href);return false;">
+			<a href="{getUrl('','document_srl',$comment->document_srl)}#comment_{$comment->comment_srl}" target="_blank">
 				{$comment->getSummary() ?: $lang->msg_no_text_comment}
 				<span class="memberInfo">{$comment->getRegdate("Y-m-d")}</span>
 			</a>
diff --git a/modules/member/m.skins/default/document_list.html b/modules/member/m.skins/default/document_list.html
index cd0db8d18..a6f61db7a 100644
--- a/modules/member/m.skins/default/document_list.html
+++ b/modules/member/m.skins/default/document_list.html
@@ -3,7 +3,7 @@
 	<h2 class="hx h2">{$member_title = $lang->cmd_view_own_document}</h2>
 	<ul class="lt">
 		<li loop="$document_list => $no,$oDocument">
-			<a href="{getUrl('','document_srl',$oDocument->document_srl)}" onclick="window.open(this.href);return false;">
+			<a href="{getUrl('','document_srl',$oDocument->document_srl)}" target="_blank">
 				{$oDocument->getTitleText()}
 				<span class="memberInfo">{$oDocument->getRegdate("Y-m-d")}</span>
 			</a>
diff --git a/modules/member/m.skins/default/saved_list.html b/modules/member/m.skins/default/saved_list.html
index c2a0cc3b6..924da88ef 100644
--- a/modules/member/m.skins/default/saved_list.html
+++ b/modules/member/m.skins/default/saved_list.html
@@ -3,7 +3,7 @@
 	<h2 class="hx h2">{$member_title = $lang->cmd_view_saved_document}</h2>
 	<ul class="lt">
 		<li loop="$document_list=>$no,$val">
-			<a href="{getUrl('','document_srl',$val->document_srl)}" onclick="window.open(this.href);return false;">
+			<a href="{getUrl('','document_srl',$val->document_srl)}" target="_blank">
 				{$val->getTitle()}
 				<span class="memberInfo">{$val->getRegdate("Y-m-d")}</span>
 			</a>
diff --git a/modules/member/m.skins/default/scrapped_list.html b/modules/member/m.skins/default/scrapped_list.html
index 1c1ae0cc9..8b4011ec1 100644
--- a/modules/member/m.skins/default/scrapped_list.html
+++ b/modules/member/m.skins/default/scrapped_list.html
@@ -3,7 +3,7 @@
 		<h2 class="hx h2">{$member_title = $lang->cmd_view_scrapped_document}</h2>
 		<ul class="lt">
 			<li loop="$document_list=>$no,$val">
-				<a href="{getUrl('','document_srl',$val->document_srl)}" onclick="window.open(this.href);return false;">
+				<a href="{getUrl('','document_srl',$val->document_srl)}" target="_blank">
 					{htmlspecialchars($val->title, ENT_COMPAT | ENT_HTML401, 'UTF-8', false)}
 					<span class="memberInfo">{$val->nick_name}  ({zdate($val->regdate,"Y-m-d")})</span>
 				</a>
diff --git a/modules/member/skins/default/comment_list.html b/modules/member/skins/default/comment_list.html
index 23a766b57..47a015d04 100644
--- a/modules/member/skins/default/comment_list.html
+++ b/modules/member/skins/default/comment_list.html
@@ -15,7 +15,7 @@
 		<tr loop="$comment_list => $no,$comment">
 			<td>{$no}</td>
 			<td>
-				<a href="{getUrl('','document_srl',$comment->document_srl)}#comment_{$comment->comment_srl}" onclick="window.open(this.href);return false;">{$comment->getSummary() ?: $lang->msg_no_text_comment}</a>
+				<a href="{getUrl('','document_srl',$comment->document_srl)}#comment_{$comment->comment_srl}" target="_blank">{$comment->getSummary() ?: $lang->msg_no_text_comment}</a>
 			</td>
 			<td>{$comment->getRegdate("Y-m-d")}</td>
 		</tr>
diff --git a/modules/member/skins/simple_world/comment_list.html b/modules/member/skins/simple_world/comment_list.html
index 10c95023a..62eebdad4 100644
--- a/modules/member/skins/simple_world/comment_list.html
+++ b/modules/member/skins/simple_world/comment_list.html
@@ -7,7 +7,7 @@
 	<div>
 		<ul class="rx_sw_list">
 			<li loop="$comment_list => $key, $item">
-				<a href="{getUrl('','document_srl',$item->document_srl)}#comment_{$item->comment_srl}" class="cont_a no_delete" onclick="window.open(this.href);return false;">
+				<a href="{getUrl('','document_srl',$item->document_srl)}#comment_{$item->comment_srl}" class="cont_a no_delete" target="_blank">
 					<span class="content_basic">
 						<span class="content_title">
 							{$item->getSummary() ?: lang('comment.msg_no_text_comment')}
diff --git a/modules/member/skins/simple_world/document_list.html b/modules/member/skins/simple_world/document_list.html
index 99835ddd8..9d5c5b413 100644
--- a/modules/member/skins/simple_world/document_list.html
+++ b/modules/member/skins/simple_world/document_list.html
@@ -7,7 +7,7 @@
 	<div>
 		<ul class="rx_sw_list">
 			<li loop="$document_list => $key, $item">
-				<a href="{getUrl('','document_srl',$item->document_srl)}" class="cont_a no_delete" onclick="window.open(this.href);return false;">
+				<a href="{getUrl('','document_srl',$item->document_srl)}" class="cont_a no_delete" target="_blank">
 					<span class="content_basic">
 						<span class="content_title">
 							{escape($item->getTitle())}
diff --git a/modules/member/skins/simple_world/saved_list.html b/modules/member/skins/simple_world/saved_list.html
index 25f7077dd..0500a1410 100644
--- a/modules/member/skins/simple_world/saved_list.html
+++ b/modules/member/skins/simple_world/saved_list.html
@@ -5,7 +5,7 @@
 	<div>
 		<ul class="rx_sw_list">
 			<li loop="$document_list => $key, $item">
-				<a href="{getUrl('','document_srl',$item->document_srl)}" class="cont_a" onclick="window.open(this.href);return false;">
+				<a href="{getUrl('','document_srl',$item->document_srl)}" class="cont_a" target="_blank">
 					<span class="content_basic">
 						<span class="content_title">
 							{escape($item->getTitle())}
diff --git a/modules/member/skins/simple_world/scrapped_list.html b/modules/member/skins/simple_world/scrapped_list.html
index 99d57af9f..2984a3f10 100644
--- a/modules/member/skins/simple_world/scrapped_list.html
+++ b/modules/member/skins/simple_world/scrapped_list.html
@@ -5,7 +5,7 @@
 	<div>
 		<ul class="rx_sw_list">
 			<li loop="$document_list => $key, $item">
-				<a href="{getUrl('','document_srl',$item->document_srl)}" class="cont_a" onclick="window.open(this.href);return false;">
+				<a href="{getUrl('','document_srl',$item->document_srl)}" class="cont_a" target="_blank">
 					<span class="content_basic">
 						<span class="content_title">
 							{escape($item->title)}
diff --git a/widgets/content/skins/simple_rectangle/title_content.html b/widgets/content/skins/simple_rectangle/title_content.html
index ca6b5349d..74c6c9106 100644
--- a/widgets/content/skins/simple_rectangle/title_content.html
+++ b/widgets/content/skins/simple_rectangle/title_content.html
@@ -29,7 +29,7 @@
                     <!--@end-->
 
                 <!--@else if($v=='nickname')-->
-                    <a <!--@if($item->getMemberSrl())-->href="#" onclick="return false;" class="author member_{$item->getMemberSrl()}"<!--@elseif($item->getAuthorSite())-->href="{$item->getAuthorSite()}" onclick="window.open(this.href); return false;" class="author member"<!--@else-->href="#" onclick="return false;" class="author member"<!--@end--> >{$item->getNickName()}</a>
+                    <a <!--@if($item->getMemberSrl())-->href="#" onclick="return false;" class="author member_{$item->getMemberSrl()}"<!--@elseif($item->getAuthorSite())-->href="{$item->getAuthorSite()}" target="_blank" rel="noopener" class="author member"<!--@else-->href="#" onclick="return false;" class="author member"<!--@end--> >{$item->getNickName()}</a>
 
                 <!--@else if($v=='regdate')-->
                     <span class="date">{$item->getRegdate("Y-m-d")}</span> <span class="hour">{$item->getRegdate("H:i")}</span>
@@ -84,7 +84,7 @@
 				<!--@end-->
 			</td>
 		<!--@else if($v=='nickname')-->
-			<td><a <!--@if($item->getMemberSrl())-->href="#" onclick="return false;" class="author member_{$item->getMemberSrl()}"<!--@elseif($item->getAuthorSite())-->href="{$item->getAuthorSite()}" onclick="window.open(this.href); return false;" class="author member"<!--@else-->href="#" onclick="return false;" class="author member"<!--@end--> >{$item->getNickName()}</a></td>
+			<td><a <!--@if($item->getMemberSrl())-->href="#" onclick="return false;" class="author member_{$item->getMemberSrl()}"<!--@elseif($item->getAuthorSite())-->href="{$item->getAuthorSite()}" target="_blank" rel="noopener" class="author member"<!--@else-->href="#" onclick="return false;" class="author member"<!--@end--> >{$item->getNickName()}</a></td>
 		<!--@else if($v=='regdate')-->
 			<td class="time"><span class="date">{$item->getRegdate("Y-m-d")}</span> <span class="hour">{$item->getRegdate("H:i")}</span></td>
 		<!--@end-->