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

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