mark: A photo of Mark kneeling on top of the Taal Volcano in the Philippines. It was a long hike. (Default)
Mark Smith ([staff profile] mark) wrote in [site community profile] changelog2009-04-21 05:45 am

[dw-free] include crosspost option on update.bml

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

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

Don't use button for link. Fix some trailing whitespace. Fix some gnarly
relative URLs.

Patch by [personal profile] allen.

Files modified:
  • cgi-bin/DW/Setting/XPostAccounts.pm
--------------------------------------------------------------------------------
diff -r 9d3ec3a36560 -r fabe0f04f412 cgi-bin/DW/Setting/XPostAccounts.pm
--- a/cgi-bin/DW/Setting/XPostAccounts.pm	Tue Apr 21 05:41:47 2009 +0000
+++ b/cgi-bin/DW/Setting/XPostAccounts.pm	Tue Apr 21 05:45:34 2009 +0000
@@ -3,7 +3,7 @@ use strict;
 use strict;
 use warnings;
 
-use Digest::MD5 qw(md5_hex);  
+use Digest::MD5 qw(md5_hex);
 
 sub tags { qw(xpost_option_server xpost_option_username xpost_option_password xpost_option_xpostbydefault) }
 
@@ -29,7 +29,7 @@ sub label {
 }
 
 # option. this is where all of the entered info is displayed.  in this case,
-# shows both the existing configured ExternalAccounts, as well as the UI for 
+# shows both the existing configured ExternalAccounts, as well as the UI for
 # adding new accounts.
 sub option {
     my ($class, $u, $errs, $args, %opts) = @_;
@@ -79,7 +79,7 @@ sub option {
                 id       => "${key}xpostbydefault[${acctid}]",
                 selected => $externalacct->xpostbydefault
             }) . "</td>";
-            $ret .= "<td style='text-align: center;'><a href = '../externalaccount.bml?acctid=${acctid}'>" . $class->ml('setting.xpost.option.change') . "</a></td>\n";
+            $ret .= "<td style='text-align: center;'><a href='$LJ::SITEROOT/manage/externalaccount.bml?acctid=${acctid}'>" . $class->ml('setting.xpost.option.change') . "</a></td>\n";
             $ret .= "<td class='checkbox'>" . LJ::html_check({
                 name     => "${key}delete[${acctid}]",
                 value    => 1,
@@ -92,14 +92,14 @@ sub option {
         $ret .= "<tr><td>" . $class->ml('setting.xpost.noaccounts') . "</td></tr>\n";
     }
     $ret .= "</table>\n";
-    
+
     # show account usage.
     my $max_accounts = LJ::get_cap($u, "xpost_accounts");
     $ret .= "<p style='text-align: center;'>" . $class->ml('setting.xpost.message.usage', { current => scalar @accounts, max => $max_accounts });
 
     # add account
     if (scalar @accounts < $max_accounts) {
-        $ret .= "<div class='xpost_add'><a href = '../externalaccount.bml'><button>" . $class->ml('setting.xpost.btn.add') . "</button></a></div>\n";
+        $ret .= "<div class='xpost_add'><a href='$LJ::SITEROOT/manage/externalaccount.bml'>" . $class->ml('setting.xpost.btn.add') . "</a></div>\n";
     }
 
     # disable comments on crosspost
@@ -142,7 +142,7 @@ sub save {
 
     # reset disable comments
     $u->set_prop( opt_xpost_disable_comments => $class->get_arg($args, "xpostdisablecomments") ? "1" : "0");
-    
+
     return 1;
 }
 
--------------------------------------------------------------------------------

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