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
--------------------------------------------------------------------------------

Post a comment in response:

This account has disabled anonymous posting.
If you don't have an account you can create one now.
HTML doesn't work in the subject.
More info about formatting

If you are unable to use this captcha for any reason, please contact us by email at support@dreamwidth.org