Some slashstyle whitespace changes; two security fixes for stuff

introduced just a few minutes ago :)
scc · Mar 16, 2004 · 46249ac · 46249ac
1 parent 2278321
commit 46249ac
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/Slash/Utility/Environment/Environment.pm b/Slash/Utility/Environment/Environment.pm
@@ -1491,6 +1491,7 @@ Hashref of cleaned-up data.
 		nosubscriberbonus_present
 		ordernum pid
 		postanon_present posttype ratio retrieve
+		show_m1s show_m2s
 		seclev start startat threshold
 		thresh_count thresh_secs thresh_hps
 		uid uthreshold voters width

diff --git a/themes/slashcode/htdocs/comments.pl b/themes/slashcode/htdocs/comments.pl
@@ -372,8 +372,9 @@ sub displayComments {
 		});
 	}
 
-	if(defined $form->{show_m2s} and $user->{is_admin}){
-		$slashdb->setUser($user->{uid}, { m2_with_comm_mod => $form->{show_m2s}} );
+	if (defined($form->{show_m2s}) && $user->{is_admin}) {
+		$slashdb->setUser($user->{uid},
+			{ m2_with_comm_mod => $form->{show_m2s} });
 	}
 
 	if ($form->{cid}) {

diff --git a/themes/slashcode/htdocs/users.pl b/themes/slashcode/htdocs/users.pl
@@ -815,7 +815,9 @@ sub showInfo {
 	my($points, $nickmatch_flag, $uid, $nick);
 	my($mod_flag, $karma_flag, $n) = (0, 0, 0);
 
-	if($admin_flag and defined $form->{show_m2s} or defined $form->{show_m1s}){
+	if ($admin_flag
+		&& (defined($form->{show_m2s}) || defined($form->{show_m1s}))
+	) {
 		my $update_hr = {};
 		$update_hr->{m2_with_mod} = $form->{show_m2s} if defined $form->{show_m2s};
 		$update_hr->{mod_with_comm} = $form->{show_m1s} if defined $form->{show_m1s};