Browse Source

! The saving of preferences was copied/pasted from elsewhere... it doesn't work properly and can actually screw up other preferences. In addition, if we're not saving the settings (since there's not really a need to), we might as well not specify something that the code doesn't insist on.

Signed-off-by: Peter Spicer <[email protected]>
Peter Spicer 11 years ago
parent
commit
7da77647c6
2 changed files with 4 additions and 18 deletions
  1. 2 9
      Themes/default/ManageNews.template.php
  2. 2 9
      Themes/default/Packages.template.php

+ 2 - 9
Themes/default/ManageNews.template.php

@@ -50,7 +50,7 @@ function template_email_members()
 
 			<div id="advanced_panel_header" class="cat_bar">
 				<h3 class="catbg">
-					<img id="advanced_panel_toggle" class="panel_toggle" style="display: none;" src="', $settings['images_url'], '/', empty($context['show_advanced_options']) ? 'collapse' : 'expand', '.png" alt="*" />
+					<img id="advanced_panel_toggle" class="panel_toggle" style="display: none;" src="', $settings['images_url'], '/expand.png" alt="*" />
 					<a href="#" id="advanced_panel_link" >', $txt['advanced'], '</a>
 				</h3>
 			</div>
@@ -142,14 +142,7 @@ function template_email_members()
 					msgExpanded: ', JavaScriptEscape($txt['advanced']), ',
 					msgCollapsed: ', JavaScriptEscape($txt['advanced']), '
 				}
-			],
-			oThemeOptions: {
-				bUseThemeSettings: ', $context['user']['is_guest'] ? 'false' : 'true', ',
-				sOptionName: \'admin_preferences\',
-				sSessionVar: smf_session_var,
-				sSessionId: smf_session_id,
-				sThemeId: \'1\'
-			}
+			]
 		});
 	// ]]></script>
 	<script type="text/javascript" src="', $settings['default_theme_url'], '/scripts/suggest.js?alp21"></script>

+ 2 - 9
Themes/default/Packages.template.php

@@ -644,7 +644,7 @@ function template_browse()
 	<script type="text/javascript"><!-- // --><![CDATA[
 		var oAdvancedPanelToggle = new smc_Toggle({
 			bToggleEnabled: true,
-			bCurrentlyCollapsed: ', empty($context['show_advanced_options']) ? 'true' : 'false', ',
+			bCurrentlyCollapsed: true,
 			aSwappableContainers: [
 				\'advanced_panel_div\'
 			],
@@ -663,14 +663,7 @@ function template_browse()
 					msgExpanded: ', JavaScriptEscape($txt['package_advanced_button']), ',
 					msgCollapsed: ', JavaScriptEscape($txt['package_advanced_button']), '
 				}
-			],
-			oThemeOptions: {
-				bUseThemeSettings: true,
-				sOptionName: \'admin_preferences\',
-				sSessionVar: smf_session_var,
-				sSessionId: smf_session_id,
-				sThemeId: \'1\'
-			}
+			]
 		});
 	// ]]></script>
 	<script type="text/javascript" src="', $settings['default_theme_url'], '/scripts/suggest.js?alp21"></script>