fu: Close-up of Fu, bringing a scoop of water to her mouth (Default)
fu ([personal profile] fu) wrote in [site community profile] changelog2010-12-10 07:17 am

[dw-free] Show 'display cross post links' on cross-posting summary page

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

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

Adds checkbox for the "show crosspost link" option to the general Other
Sites view, so that it's more visible.

Patch by Erin of [personal profile] teamdestroyer.

Files modified:
  • bin/upgrading/en.dat
  • cgi-bin/DW/Setting/XPostAccounts.pm
--------------------------------------------------------------------------------
diff -r 50c3866d1cec -r 18a19f24b6a8 bin/upgrading/en.dat
--- a/bin/upgrading/en.dat	Fri Dec 10 15:05:11 2010 +0800
+++ b/bin/upgrading/en.dat	Fri Dec 10 15:16:55 2010 +0800
@@ -3079,6 +3079,8 @@ setting.xpost.option.footer.when.never=N
 
 setting.xpost.option.password=Password
 
+setting.xpost.option.recordlink=Display Crosspost Link
+
 setting.xpost.option.server=Server
 
 setting.xpost.option.username=Username
diff -r 50c3866d1cec -r 18a19f24b6a8 cgi-bin/DW/Setting/XPostAccounts.pm
--- a/cgi-bin/DW/Setting/XPostAccounts.pm	Fri Dec 10 15:05:11 2010 +0800
+++ b/cgi-bin/DW/Setting/XPostAccounts.pm	Fri Dec 10 15:16:55 2010 +0800
@@ -69,6 +69,7 @@ sub option {
         $ret .= "<th>" . $class->ml('setting.xpost.option.username') . "</th>\n";
         $ret .= "<th>" . $class->ml('setting.xpost.option.server') . "</th>\n";
         $ret .= "<th>" . $class->ml('setting.xpost.option.xpostbydefault') . "</th>\n";
+        $ret .= "<th>" . $class->ml('setting.xpost.option.recordlink') . "</th>\n";
         $ret .= "<th>" . $class->ml('setting.xpost.option.change') . "</th>\n";
         $ret .= "<th>" . $class->ml('setting.xpost.option.delete') . "</th>\n";
         $ret .= "</tr>\n";
@@ -84,6 +85,12 @@ sub option {
                 value    => 1,
                 id       => "${key}xpostbydefault[${acctid}]",
                 selected => $externalacct->xpostbydefault
+            }) . "</td>";
+            $ret .= "<td class='checkbox'>" . LJ::html_check({
+                name     => "${key}recordlink[${acctid}]",
+                value    => 1,
+                id       => "${key}recordlink[${acctid}]",
+                selected => $externalacct->recordlink
             }) . "</td>";
             $ret .= "<td style='text-align: center;'><a href='$LJ::SITEROOT/manage/externalaccount?acctid=${acctid}'>" . $class->ml('setting.xpost.option.change') . "</a></td>\n";
             $ret .= "<td class='checkbox'>" . LJ::html_check({
@@ -281,6 +288,10 @@ sub save {
                 if ($class->get_arg($args, "xpostbydefault[$acctid]") ne $account->{'xpostbydefault'}) {
                     $account->set_xpostbydefault($class->get_arg($args, "xpostbydefault[$acctid]"));
                 }
+                # check to see if we need to reset the recordlink
+                if ($class->get_arg($args, "recordlink[$acctid]") ne $account->{'recordlink'}) {
+                    $account->set_recordlink($class->get_arg($args, "recordlink[$acctid]"));
+                }
             }
         }
     }
--------------------------------------------------------------------------------

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