no more eval

live627 · live627 · commit b03480c8ef81 · 2023-12-09T21:30:43.000-07:00
Signed-off-by: John Rayes &lt;live627@gmail.com&gt;
diff --git a/Themes/default/Admin.template.php b/Themes/default/Admin.template.php
@@ -112,7 +112,6 @@ function template_admin()
 	echo '
 					<script>
 						var oAdminIndex = new smf_AdminIndex({
-							sSelf: \'oAdminCenter\',
 
 							bLoadAnnouncements: true,
 							sAnnouncementTemplate: ', Utils::JavaScriptEscape('
@@ -696,7 +695,7 @@ function doAutoSubmit()
 							document.forms.autoSubmit.cont.value = "', Lang::$txt['not_done_continue'], ' (" + countdown + ")";
 							countdown--;
 
-							setTimeout("doAutoSubmit();", 1000);
+							setTimeout(doAutoSubmit, 1000);
 						}
 					</script>';
 }
@@ -1487,7 +1486,7 @@ function doAutoSubmit()
 							document.forms.recount_form.recount_now.value = "', Lang::$txt['errors_recount_now'], ' (" + countdown + ")";
 							countdown--;
 
-							setTimeout("doAutoSubmit();", 1000);
+							setTimeout(doAutoSubmit, 1000);
 						}
 					</script>';
 	}
diff --git a/Themes/default/Calendar.template.php b/Themes/default/Calendar.template.php
@@ -983,7 +983,7 @@ function update()
 					else
 						icons[i].src = "', Utils::$context['onimg'], '";
 
-				window.setTimeout("update();", 500);
+				window.setTimeout(update, 500);
 			}
 			// Checks for variable in theArray.
 			function in_array(variable, theArray)
@@ -1078,7 +1078,7 @@ function update()
 					else
 						icons[i].src = "', Utils::$context['onimg'], '";
 
-				window.setTimeout("update();", 500);
+				window.setTimeout(update, 500);
 			}
 			// Checks for variable in theArray.
 			function in_array(variable, theArray)
@@ -1171,7 +1171,7 @@ function update()
 					else
 						icons[i].src = "', Utils::$context['onimg'], '";
 
-				window.setTimeout("update();", 500);
+				window.setTimeout(update, 500);
 			}
 			// Checks for variable in theArray.
 			function in_array(variable, theArray)
diff --git a/Themes/default/Display.template.php b/Themes/default/Display.template.php
@@ -336,7 +336,6 @@ function template_main()
 	{
 		echo '
 			var oInTopicModeration = new InTopicModeration({
-				sSelf: \'oInTopicModeration\',
 				sCheckboxContainerMask: \'in_topic_mod_check_\',
 				aMessageIds: [\'', implode('\', \'', Utils::$context['removableMessageIDs']), '\'],
 				sSessionId: smf_session_id,
@@ -362,7 +361,6 @@ function template_main()
 		// Add it to the mobile button strip as well
 		echo '
 			var oInTopicModerationMobile = new InTopicModeration({
-				sSelf: \'oInTopicModerationMobile\',
 				sCheckboxContainerMask: \'in_topic_mod_check_\',
 				aMessageIds: [\'', implode('\', \'', Utils::$context['removableMessageIDs']), '\'],
 				sSessionId: smf_session_id,
@@ -1020,7 +1018,6 @@ function insertQuoteFast(messageid)
 		echo '
 		<script>
 			var oDraftAutoSave = new smf_DraftAutoSave({
-				sSelf: \'oDraftAutoSave\',
 				sLastNote: \'draft_lastautosave\',
 				sLastID: \'id_draft\',', !empty(Utils::$context['post_box_name']) ? '
 				sSceditorID: \'' . Utils::$context['post_box_name'] . '\',' : '', '
diff --git a/Themes/default/GenericControls.template.php b/Themes/default/GenericControls.template.php
@@ -132,7 +132,6 @@ function template_control_richedit_buttons($editor_id)
 		</span>
 		<script>
 			var oDraftAutoSave = new smf_DraftAutoSave({
-				sSelf: \'oDraftAutoSave\',
 				sLastNote: \'draft_lastautosave\',
 				sLastID: \'id_draft\',
 				sSceditorID: \'', $editor_id, '\',
diff --git a/Themes/default/ManageAttachments.template.php b/Themes/default/ManageAttachments.template.php
@@ -187,7 +187,7 @@ function template_maintenance()
 		</form>
 		<script>
 			function start_progress() {
-				setTimeout(\'show_msg()\', 1000);
+				setTimeout(show_msg, 1000);
 			}
 
 			function show_msg() {
@@ -197,7 +197,7 @@ function show_msg() {
 
 			function show_progress() {
 				$(\'#show_progress\').on("load", "progress.php");
-				setTimeout(\'show_progress()\', 1500);
+				setTimeout(show_progress, 1500);
 			}
 
 		</script>
diff --git a/Themes/default/ManageBans.template.php b/Themes/default/ManageBans.template.php
@@ -209,24 +209,14 @@ function template_ban_edit()
 	if (Utils::$context['ban']['is_new'] && empty($_REQUEST['u']))
 		echo '
 		var oAddMemberSuggest = new smc_AutoSuggest({
-			sSelf: \'oAddMemberSuggest\',
 			sSessionId: smf_session_id,
 			sSessionVar: smf_session_var,
 			sSuggestId: \'user\',
 			sControlId: \'user\',
 			sSearchType: \'member\',
 			sTextDeleteItem: \'', Lang::$txt['autosuggest_delete_item'], '\',
-			bItemList: false
 		});
 
-		function onUpdateName(oAutoSuggest)
-		{
-			document.getElementById(\'user_check\').checked = true;
-			return true;
-		}
-		oAddMemberSuggest.registerCallback(\'onBeforeUpdate\', \'onUpdateName\');';
-
-	echo '
 		function confirmBan(aForm)
 		{
 			if (aForm.ban_name.value == \'\')
@@ -316,14 +306,12 @@ function template_ban_edit_trigger()
 	</div><!-- #manage_bans -->
 	<script>
 		var oAddMemberSuggest = new smc_AutoSuggest({
-			sSelf: \'oAddMemberSuggest\',
 			sSessionId: smf_session_id,
 			sSessionVar: smf_session_var,
 			sSuggestId: \'username\',
 			sControlId: \'user\',
 			sSearchType: \'member\',
 			sTextDeleteItem: \'', Lang::$txt['autosuggest_delete_item'], '\',
-			bItemList: false
 		});
 
 		function onUpdateName(oAutoSuggest)
diff --git a/Themes/default/ManageBoards.template.php b/Themes/default/ManageBoards.template.php
@@ -604,7 +604,6 @@ function template_modify_board()
 
 	<script>
 		var oModeratorSuggest = new smc_AutoSuggest({
-			sSelf: \'oModeratorSuggest\',
 			sSessionId: smf_session_id,
 			sSessionVar: smf_session_var,
 			sSuggestId: \'moderators\',
@@ -629,7 +628,6 @@ function template_modify_board()
 		});
 
 		var oModeratorGroupSuggest = new smc_AutoSuggest({
-			sSelf: \'oModeratorGroupSuggest\',
 			sSessionId: smf_session_id,
 			sSessionVar: smf_session_var,
 			sSuggestId: \'moderator_groups\',
diff --git a/Themes/default/ManageMembergroups.template.php b/Themes/default/ManageMembergroups.template.php
@@ -365,7 +365,6 @@ function template_edit_group()
 		</form>
 	<script>
 		var oModeratorSuggest = new smc_AutoSuggest({
-			sSelf: \'oModeratorSuggest\',
 			sSessionId: smf_session_id,
 			sSessionVar: smf_session_var,
 			sSuggestId: \'group_moderators\',
@@ -698,7 +697,6 @@ function template_group_members()
 		echo '
 	<script>
 		var oAddMemberSuggest = new smc_AutoSuggest({
-			sSelf: \'oAddMemberSuggest\',
 			sSessionId: \'', Utils::$context['session_id'], '\',
 			sSessionVar: \'', Utils::$context['session_var'], '\',
 			sSuggestId: \'to_suggest\',
diff --git a/Themes/default/ManageNews.template.php b/Themes/default/ManageNews.template.php
@@ -142,7 +142,6 @@ function template_email_members()
 	</script>
 	<script>
 		var oMemberSuggest = new smc_AutoSuggest({
-			sSelf: \'oMemberSuggest\',
 			sSessionId: smf_session_id,
 			sSessionVar: smf_session_var,
 			sSuggestId: \'members\',
@@ -156,7 +155,6 @@ function template_email_members()
 			aListItems: []
 		});
 		var oExcludeMemberSuggest = new smc_AutoSuggest({
-			sSelf: \'oExcludeMemberSuggest\',
 			sSessionId: \'', Utils::$context['session_id'], '\',
 			sSessionVar: \'', Utils::$context['session_var'], '\',
 			sSuggestId: \'exclude_members\',
diff --git a/Themes/default/ManagePaid.template.php b/Themes/default/ManagePaid.template.php
@@ -324,15 +324,13 @@ function template_modify_user_subscription()
 	</form>
 	<script>
 		var oAddMemberSuggest = new smc_AutoSuggest({
-			sSelf: \'oAddMemberSuggest\',
 			sSessionId: smf_session_id,
 			sSessionVar: smf_session_var,
 			sSuggestId: \'name_subscriber\',
 			sControlId: \'name_control\',
 			sSearchType: \'member\',
 			sTextDeleteItem: \'', Lang::$txt['autosuggest_delete_item'], '\',
-			bItemList: false
-			});
+		});
 	</script>';
 
 	if (!empty(Utils::$context['pending_payments']))
diff --git a/Themes/default/ManageSearch.template.php b/Themes/default/ManageSearch.template.php
@@ -304,7 +304,7 @@ function doAutoSubmit()
 			document.forms.autoSubmit.b.value = "', Lang::$txt['search_create_index_continue'], ' (" + countdown + ")";
 			countdown--;
 
-			setTimeout("doAutoSubmit();", 1000);
+			setTimeout(doAutoSubmit, 1000);
 		}
 	</script>';
 
diff --git a/Themes/default/Packages.template.php b/Themes/default/Packages.template.php
@@ -569,7 +569,6 @@ function template_browse()
 	echo '
 			<script>
 				var oAdminIndex = new smf_AdminIndex({
-					sSelf: \'oAdminCenter\',
 					bLoadAnnouncements: false,
 					bLoadVersions: false,
 					bLoadUpdateNotification: true,
@@ -1834,7 +1833,7 @@ function doAutoSubmit()
 			document.getElementById(\'cont\').value = "', Lang::$txt['not_done_continue'], ' (" + countdown + ")";
 			countdown--;
 
-			setTimeout("doAutoSubmit();", 1000);
+			setTimeout(doAutoSubmit, 1000);
 		}
 	</script>';
 }
diff --git a/Themes/default/PersonalMessage.template.php b/Themes/default/PersonalMessage.template.php
@@ -1257,7 +1257,6 @@ function onDocSent(XMLDoc)
 	echo '
 		<script>
 			var oPersonalMessageSend = new smf_PersonalMessageSend({
-				sSelf: \'oPersonalMessageSend\',
 				sSessionId: smf_session_id,
 				sSessionVar: smf_session_var,
 				sTextDeleteItem: \'', Lang::$txt['autosuggest_delete_item'], '\',
diff --git a/Themes/default/Post.template.php b/Themes/default/Post.template.php
@@ -826,7 +826,7 @@ function template_quotefast()
 			window.opener.onReceiveOpener(quote);
 
 			window.focus();
-			setTimeout("window.close();", 400);';
+			setTimeout(function() { window.close(); }, 400);';
 	}
 	echo '
 		</script>
@@ -925,7 +925,7 @@ function doAutoSubmit()
 			document.forms.autoSubmit.b.value = "', Lang::$txt['announce_continue'], ' (" + countdown + ")";
 			countdown--;
 
-			setTimeout("doAutoSubmit();", 1000);
+			setTimeout(doAutoSubmit, 1000);
 		}
 	</script>';
 }
diff --git a/Themes/default/Profile.template.php b/Themes/default/Profile.template.php
@@ -818,14 +818,12 @@ function template_editBuddies()
 	</form>
 	<script>
 		var oAddBuddySuggest = new smc_AutoSuggest({
-			sSelf: \'oAddBuddySuggest\',
 			sSessionId: smf_session_id,
 			sSessionVar: smf_session_var,
 			sSuggestId: \'new_buddy\',
 			sControlId: \'new_buddy\',
 			sSearchType: \'member\',
 			sTextDeleteItem: \'', Lang::$txt['autosuggest_delete_item'], '\',
-			bItemList: false
 		});
 	</script>';
 }
@@ -929,14 +927,12 @@ function template_editIgnoreList()
 	</form>
 	<script>
 		var oAddIgnoreSuggest = new smc_AutoSuggest({
-			sSelf: \'oAddIgnoreSuggest\',
 			sSessionId: \'', Utils::$context['session_id'], '\',
 			sSessionVar: \'', Utils::$context['session_var'], '\',
 			sSuggestId: \'new_ignore\',
 			sControlId: \'new_ignore\',
 			sSearchType: \'member\',
 			sTextDeleteItem: \'', Lang::$txt['autosuggest_delete_item'], '\',
-			bItemList: false
 		});
 	</script>';
 }
diff --git a/Themes/default/Search.template.php b/Themes/default/Search.template.php
@@ -108,7 +108,6 @@ function template_main()
 				</dd>
 			</dl>
 			<script>
-				createEventListener(window);
 				window.addEventListener("load", initSearch, false);
 			</script>
 			<input type="hidden" name="advanced" value="1">';
@@ -236,12 +235,10 @@ function template_main()
 	</form>
 	<script>
 		var oAddMemberSuggest = new smc_AutoSuggest({
-			sSelf: \'oAddMemberSuggest\',
 			sSessionId: smf_session_id,
 			sSessionVar: smf_session_var,
 			sControlId: \'userspec\',
-			sSearchType: \'member\',
-			bItemList: false
+			sSearchType: \'member\'
 		});
 	</script>';
 }
diff --git a/Themes/default/scripts/PersonalMessage.js b/Themes/default/scripts/PersonalMessage.js
@@ -28,16 +28,11 @@ smf_PersonalMessageSend.prototype.init = function()
 
 		// Make the link show the BCC control.
 		var oBccLink = document.getElementById(this.opt.sBccLinkId);
-		oBccLink.instanceRef = this;
-		oBccLink.onclick = function () {
-			this.instanceRef.showBcc();
-			return false;
-		};
+		oBccLink.onclick = this.showBcc.bind(this);
 	}
 
 	var oToControl = document.getElementById(this.opt.sToControlId);
 	this.oToAutoSuggest = new smc_AutoSuggest({
-		sSelf: this.opt.sSelf + '.oToAutoSuggest',
 		sSessionId: this.opt.sSessionId,
 		sSessionVar: this.opt.sSessionVar,
 		sSuggestId: 'to_suggest',
@@ -50,10 +45,9 @@ smf_PersonalMessageSend.prototype.init = function()
 		sItemListContainerId: 'to_item_list_container',
 		aListItems: this.opt.aToRecipients
 	});
-	this.oToAutoSuggest.registerCallback('onBeforeAddItem', this.opt.sSelf + '.callbackAddItem');
+	this.oToAutoSuggest.registerCallback('onBeforeAddItem', this.callbackAddItem.bind(this));
 
 	this.oBccAutoSuggest = new smc_AutoSuggest({
-		sSelf: this.opt.sSelf + '.oBccAutoSuggest',
 		sSessionId: this.opt.sSessionId,
 		sSessionVar: this.opt.sSessionVar,
 		sSuggestId: 'bcc_suggest',
@@ -66,23 +60,23 @@ smf_PersonalMessageSend.prototype.init = function()
 		sItemListContainerId: 'bcc_item_list_container',
 		aListItems: this.opt.aBccRecipients
 	});
-	this.oBccAutoSuggest.registerCallback('onBeforeAddItem', this.opt.sSelf + '.callbackAddItem');
-
+	this.oBccAutoSuggest.registerCallback('onBeforeAddItem', this.callbackAddItem.bind(this));
 }
 
 smf_PersonalMessageSend.prototype.showBcc = function()
 {
 	// No longer hide it, show it to the world!
 	this.oBccDiv.style.display = '';
 	this.oBccDiv2.style.display = '';
-}
 
+	return false;
+}
 
 // Prevent items to be added twice or to both the 'To' and 'Bcc'.
-smf_PersonalMessageSend.prototype.callbackAddItem = function(oAutoSuggestInstance, sSuggestId)
+smf_PersonalMessageSend.prototype.callbackAddItem = function(sItemId)
 {
-	this.oToAutoSuggest.deleteAddedItem(sSuggestId);
-	this.oBccAutoSuggest.deleteAddedItem(sSuggestId);
+	this.oToAutoSuggest.deleteAddedItem(sItemId);
+	this.oBccAutoSuggest.deleteAddedItem(sItemId);
 
 	return true;
 }
diff --git a/Themes/default/scripts/admin.js b/Themes/default/scripts/admin.js
@@ -435,7 +435,7 @@ function setPreviewTimeout()
 		previewTimeout = null;
 	}
 
-	previewTimeout = window.setTimeout("refreshPreview(true); previewTimeout = null;", 500);
+	previewTimeout = window.setTimeout(function() { refreshPreview(true); previewTimeout = null; }, 500);
 }
 
 function toggleDuration(toChange)
diff --git a/Themes/default/scripts/captcha.js b/Themes/default/scripts/captcha.js
@@ -15,18 +15,12 @@ function smfCaptcha(imageURL, uniqueID, useLibrary, letterCount)
 		// Is there anything to cycle images with - if so attach the refresh image function?
 		var cycleHandle = document.getElementById('visual_verification' + uniqueID + '_refresh');
 		if (cycleHandle)
-		{
-			createEventListener(cycleHandle);
 			cycleHandle.addEventListener('click', refreshImages, false);
-		}
 
 		// Maybe a voice is here to spread light?
 		var soundHandle = document.getElementById('visual_verification' + uniqueID + '_sound');
 		if (soundHandle)
-		{
-			createEventListener(soundHandle);
 			soundHandle.addEventListener('click', playSound, false);
-		}
 	}
 
 	// Change the images.
diff --git a/Themes/default/scripts/drafts.js b/Themes/default/scripts/drafts.js
diff --git a/Themes/default/scripts/script.js b/Themes/default/scripts/script.js
diff --git a/Themes/default/scripts/suggest.js b/Themes/default/scripts/suggest.js
diff --git a/Themes/default/scripts/theme.js b/Themes/default/scripts/theme.js
diff --git a/Themes/default/scripts/topic.js b/Themes/default/scripts/topic.js

Original file line number	Diff line number	Diff line change
`@@ -112,7 +112,6 @@ function template_admin()`
`112`	`112`	`echo '`
`113`	`113`	`<script>`
`114`	`114`	`var oAdminIndex = new smf_AdminIndex({`
`115`		`- sSelf: \'oAdminCenter\',`
`116`	`115`
`117`	`116`	`bLoadAnnouncements: true,`
`118`	`117`	`sAnnouncementTemplate: ', Utils::JavaScriptEscape('`
`@@ -696,7 +695,7 @@ function doAutoSubmit()`
`696`	`695`	`document.forms.autoSubmit.cont.value = "', Lang::$txt['not_done_continue'], ' (" + countdown + ")";`
`697`	`696`	`countdown--;`
`698`	`697`
`699`		`- setTimeout("doAutoSubmit();", 1000);`
	`698`	`+ setTimeout(doAutoSubmit, 1000);`
`700`	`699`	`}`
`701`	`700`	`</script>';`
`702`	`701`	`}`
`@@ -1487,7 +1486,7 @@ function doAutoSubmit()`
`1487`	`1486`	`document.forms.recount_form.recount_now.value = "', Lang::$txt['errors_recount_now'], ' (" + countdown + ")";`
`1488`	`1487`	`countdown--;`
`1489`	`1488`
`1490`		`- setTimeout("doAutoSubmit();", 1000);`
	`1489`	`+ setTimeout(doAutoSubmit, 1000);`
`1491`	`1490`	`}`
`1492`	`1491`	`</script>';`
`1493`	`1492`	`}`
Original file line number	Diff line number	Diff line change
`@@ -187,7 +187,7 @@ function template_maintenance()`
`187`	`187`	`</form>`
`188`	`188`	`<script>`
`189`	`189`	`function start_progress() {`
`190`		`- setTimeout(\'show_msg()\', 1000);`
	`190`	`+ setTimeout(show_msg, 1000);`
`191`	`191`	`}`
`192`	`192`
`193`	`193`	`function show_msg() {`
`@@ -197,7 +197,7 @@ function show_msg() {`
`197`	`197`
`198`	`198`	`function show_progress() {`
`199`	`199`	`$(\'#show_progress\').on("load", "progress.php");`
`200`		`- setTimeout(\'show_progress()\', 1500);`
	`200`	`+ setTimeout(show_progress, 1500);`
`201`	`201`	`}`
`202`	`202`
`203`	`203`	`</script>`