change broadcasting of ThemePref changes

I moved a MESSAGEMAN:Broadcast() call so that it only broadcasts the setting that has been saved. This seems more sensible to me, but it is possible I'm misinterpreting the original design here. Feel free to merge or not merge this commit. :)
2015-04-06 21:21:23 -04:00
parent ce8669f049
commit 3d7cb7305b
1 changed files with 5 additions and 2 deletions
@@ -57,8 +57,11 @@ local function DefaultSave( pref, choices, values )

 	return function(self, list, pn)
 		for i=1, #choices do
-			if list[i] then ThemePrefs.Set( pref, values[i] ) break end
-			MESSAGEMAN:Broadcast( msg, params )
+			if list[i] then
+				ThemePrefs.Set( pref, values[i] )
+				MESSAGEMAN:Broadcast( msg, params )
+				break
+			end
 		end
 	end
 end