Tweak for a couple more warnings.

        Patch by fu. 

diffs (49 lines):

diff -r fe774f46a075 -r 624bda6a8380 cgi-bin/LJ/Talk.pm
--- a/cgi-bin/LJ/Talk.pm        Wed Nov 17 14:20:56 2010 +0800
+++ b/cgi-bin/LJ/Talk.pm        Wed Nov 17 14:56:53 2010 +0800
@@ -186,17 +186,15 @@ sub init
     my $ju = undef;
     my $item = undef;        # hashref; journal item conversation is in
 
-    $form->{$_} ||= 0 foreach qw( itemid thread replyto );
-
     # defaults, to be changed later:
-    $init->{'itemid'} = $form->{'itemid'}+0;
+    $init->{'itemid'} = ( $form->{'itemid'} || 0 )+0;
     $init->{'ditemid'} = $init->{'itemid'};
-    $init->{'thread'} = $form->{'thread'}+0;
+    $init->{'thread'} = ( $form->{'thread'} || 0 )+0;
     $init->{'dthread'} = $init->{'thread'};
     $init->{'clustered'} = 0;
-    $init->{'replyto'} = $form->{'replyto'}+0;
+    $init->{'replyto'} = ( $form->{'replyto'} || 0 )+0;
     $init->{'style'} = $form->{'style'}
-        if $form->{'style'} =~ /^(?:mine|light)$/;
+        if $form->{style} && $form->{style} =~ /^(?:mine|light)$/;
 
     if ($journal) {
         # they specified a journal argument, which indicates new style.
diff -r fe774f46a075 -r 624bda6a8380 htdocs/talkpost_do.bml
--- a/htdocs/talkpost_do.bml    Wed Nov 17 14:20:56 2010 +0800
+++ b/htdocs/talkpost_do.bml    Wed Nov 17 14:56:53 2010 +0800
@@ -170,13 +170,14 @@ body<=
     }
 
 
-    my $talkurl = $init->{talkurl};
+    my ( $talkurl, $entryu, $parent, $comment, $item );
 
-    my $entryu   = $init->{entryu};
-    my $journalu = $init->{journalu};
-    my $parent   = $init->{parent};
-    my $comment  = $init->{comment};
-    my $item     = $init->{item};
+    $talkurl = $init->{talkurl};
+    $entryu   = $init->{entryu};
+    $journalu = $init->{journalu};
+    $parent   = $init->{parent};
+    $comment  = $init->{comment};
+    $item     = $init->{item};
 
     my $unscreen_parent = $POST{unscreen_parent} ? 1 : 0;
 
_______________________________________________
dw-changelog mailing list
[email protected]
http://lists.dwscoalition.org/mailman/listinfo/dw-changelog

Reply via email to