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-03-19 04:56 am

[dw-free] Remove rogue appearance of ext_ when adding OpenID accounts to circle

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

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

Fix some appearances of ext_ on identity accounts.

Patch by [personal profile] rho.

Files modified:
  • htdocs/manage/circle/add.bml
--------------------------------------------------------------------------------
diff -r 04db81932d89 -r 1e6ced387e4e htdocs/manage/circle/add.bml
--- a/htdocs/manage/circle/add.bml	Thu Mar 19 04:47:00 2009 +0000
+++ b/htdocs/manage/circle/add.bml	Thu Mar 19 04:56:09 2009 +0000
@@ -20,6 +20,7 @@
     my $user = $POST{'user'} || $GET{'user'};
     my $u = LJ::load_user($user);
     my $userid = $u->{userid};
+    my $username = $u->name_html;
     $body = "";
 
     LJ::need_res(qw(js/core.js js/dom.js js/colorpicker.js));
@@ -213,7 +214,7 @@
     ## let them pick the colors
     my $color_text = "<?p <table><tr><td valign=\"top\"><i>$ML{'.optional'}:</i> ".
              "<input type=\"checkbox\" id=\"color_switch\" value=\"\" onClick=\"color_display(this);\"> </td><td><label for=\"color_switch\">" .
-             BML::ml('.colors.text', {'user'=>$user}) .
+             BML::ml( '.colors.text', { user => $username } ) .
              "</label><br /><span style=\"font-size: 7pt;\">($ML{'.colors.disclaimer'})</span></td></tr></table> p?>";
     $body .= "<script>\n";
     $body .= "function color_display(check) {\n";
@@ -229,7 +230,7 @@
 
     $body .= "<noscript>\n";
     $body .= "<?p <table><tr><td valign='top'><i>$ML{'.optional'}:</i> </td><td>" .
-             BML::ml('.colors.text', {'user'=>$user}) .
+             BML::ml( '.colors.text', { user => $username } ) .
              "<br /><span style='font-size: 7pt;'>($ML{'.colors.disclaimer'})</span></td></tr></table> p?>";
     $body .= "</noscript>\n";
 
@@ -420,7 +421,7 @@
    <?p
    <script language='JavaScript' type='text/javascript'>
    <!--
-   document.write("<span style='border: 1px solid #666666; padding: 10px; background-color: $bgvalue\; color: $fgvalue\;' id='preview'>$user</span>");
+   document.write("<span style='border: 1px solid #666666; padding: 10px; background-color: $bgvalue\; color: $fgvalue\;' id='preview'>$username</span>");
    // -->
    </script>
    p?>
--------------------------------------------------------------------------------