kareila: (Default)
kareila ([personal profile] kareila) wrote in [site community profile] changelog2010-11-10 07:28 am

[dw-free] LJ::Event::XPostSuccess->new has mislabelled variables

[commit: http://hg.dwscoalition.org/dw-free/rev/2006aa9eb358]

http://bugs.dwscoalition.org/show_bug.cgi?id=3218

Use correct names for argument variables.

Patch by [personal profile] kareila.

Files modified:
  • cgi-bin/LJ/Event/XPostSuccess.pm
--------------------------------------------------------------------------------
diff -r b5dd05a21e20 -r 2006aa9eb358 cgi-bin/LJ/Event/XPostSuccess.pm
--- a/cgi-bin/LJ/Event/XPostSuccess.pm	Wed Nov 10 14:53:34 2010 +0800
+++ b/cgi-bin/LJ/Event/XPostSuccess.pm	Wed Nov 10 01:27:54 2010 -0600
@@ -19,9 +19,9 @@ use Carp qw(croak);
 use Carp qw(croak);
 
 sub new {
-    my ($class, $u, $ditemid, $acctid) = @_;
+    my ( $class, $u, $acctid, $ditemid ) = @_;
     croak 'Not an LJ::User' unless LJ::isu($u);
-    return $class->SUPER::new($u, $ditemid, $acctid);
+    return $class->SUPER::new( $u, $acctid, $ditemid );
 }
 
 # for this to be on for all users
--------------------------------------------------------------------------------