In-Portal Issue Tracker

Index: admin_templates/incs/close_popup.tpl
===================================================================
--- admin_templates/incs/close_popup.tpl	(revision 14103)
+++ admin_templates/incs/close_popup.tpl	(working copy)
@@ -30,7 +30,11 @@
 				if ($opener) {
 					// using popups & close_popup called (from anywhere)
 					try {
-						window_close( function() { onAfterWindowClose($opener); } );
+						window_close(
+							function() {
+								$opener.onAfterWindowClose($redirect_url, <inp2:m_if check="adm_OpenNewWindow" diff="1">true<inp2:m_else/>false</inp2:m_if>, <inp2:m_if check="m_Get" name="skip_refresh">true<inp2:m_else/>false</inp2:m_if>);
+							}
+						);
 					}
 					catch (err) {
 						// another website is opened in parent window
@@ -43,32 +47,6 @@
 				}
 			}
 
-			function onAfterWindowClose($opener) {
-				<inp2:m_if check="adm_OpenNewWindow" diff="1">
-					var $ru = $redirect_url;
-
-					// setTimeout allows to call method indirectly. Without it whole idea won't work 2nd time (try adding 2 relations one after another)
-					$opener.setTimeout(
-						function() {
-							$opener.openSelector('adm', $ru.replace('%5C', '\\') + '&merge_opener_stack=1');
-						},
-						200
-					);
-				<inp2:m_else/>
-					<inp2:m_if check="m_Get" name="skip_refresh">
-						var $skip_refresh = true;
-					<inp2:m_else/>
-						var $skip_refresh = isset($opener.$skip_refresh) && $opener.$skip_refresh ? true : false;
-					</inp2:m_if>
-
-					$opener.focus();
-
-					if (!$skip_refresh) {
-						$opener.location.href = $redirect_url.replace('%5C', '\\');
-					}
-				</inp2:m_if>
-			}
-
 			// copied from "js/script.js" because it's not included here due performance reasons
 			function getFrame($name) {
 				var $main_window = window;
Index: admin_templates/js/script.js
===================================================================
--- admin_templates/js/script.js	(revision 14147)
+++ admin_templates/js/script.js	(working copy)
@@ -1785,6 +1785,28 @@
 	return getFrame('main').TB.remove();
 }
 
+function onAfterWindowClose($redirect_url, $open_new_window, $force_skip_refresh) {
+	if ($open_new_window) {
+		var $ru = $redirect_url;
+
+		// setTimeout allows to call method indirectly. Without it whole idea won't work 2nd time (try adding 2 relations one after another)
+		setTimeout(
+			function() {
+				openSelector('adm', $ru.replace('%5C', '\\') + '&merge_opener_stack=1');
+			},
+			200
+		);
+
+		return ;
+	}
+
+	window.focus();
+
+	if ( !(($force_skip_refresh === true) || (typeof $skip_refresh != 'undefined' && $skip_refresh)) ) {
+		window.location.href = $redirect_url.replace('%5C', '\\');
+	}
+}
+
 function get_control($mask, $field, $append, $prepend) {
 	$append = $append !== undefined ? '_' + $append : '';
 	$prepend = $prepend !== undefined ? $prepend + '_' : '';