Housekeeping - define new user methods in correct section.
         Don't check keep_args twice in create_url.

        Patch by kareila. 

diffs (46 lines):

diff -r e8e390fd9d08 -r 5ce9fca619c5 cgi-bin/LJ/User.pm
--- a/cgi-bin/LJ/User.pm        Fri Dec 31 14:51:57 2010 +0800
+++ b/cgi-bin/LJ/User.pm        Sat Jan 01 00:05:39 2011 -0600
@@ -5526,14 +5526,6 @@ sub revoke_priv_all {
     return 1;
 }
 
-sub opt_viewentrystyle {
-    return ( $_[0]->prop( 'opt_viewentrystyle' ) || "O" );
-}
-
-sub opt_viewjournalstyle {
-    return ( $_[0]->prop( 'opt_viewjournalstyle' ) || "O" );
-}
-
 ########################################################################
 ###  24. Styles and S2-Related Functions
 
@@ -5647,6 +5639,14 @@ sub opt_embedplaceholders {
         my $imagelinks = $u->prop('opt_imagelinks');
         return $imagelinks;
     }
+}
+
+sub opt_viewentrystyle {
+    return ( $_[0]->prop( 'opt_viewentrystyle' ) || "O" );
+}
+
+sub opt_viewjournalstyle {
+    return ( $_[0]->prop( 'opt_viewjournalstyle' ) || "O" );
 }
 
 sub set_default_style {
diff -r e8e390fd9d08 -r 5ce9fca619c5 cgi-bin/weblib.pl
--- a/cgi-bin/weblib.pl Fri Dec 31 14:51:57 2010 +0800
+++ b/cgi-bin/weblib.pl Sat Jan 01 00:05:39 2011 -0600
@@ -1229,9 +1229,6 @@ sub create_url {
     }
 
     $opts{keep_args} = [ keys %$orig_args ] if defined $opts{keep_args} and 
$opts{keep_args} == 1;
-    $opts{keep_args} = [] if ref $opts{keep_args} ne 'ARRAY';
-
-    $opts{keep_args} = [ keys %$orig_args ] if $opts{keep_args} == 1;
     $opts{keep_args} = [] if ref $opts{keep_args} ne 'ARRAY';
 
     # Move over arguments that we need to keep
_______________________________________________
dw-changelog mailing list
[email protected]
http://lists.dwscoalition.org/mailman/listinfo/dw-changelog

Reply via email to