Blog Documentation Book a Demo
Please upgrade here. These earlier versions are no longer being updated and have security issues.
HackerOne users: Testing against this community violates our program's Terms of Service and will result in your bounty being denied.

Email Output Screwed!

rohan10rohan10
edited August 2010 in Vanilla 2.0 - 2.8
Hello all,

When I have instances of code in the Gdn_Format::Email($this->User->Email); tags it outputs it as linked (look at status bar). How would I go about fixing this?

Picture: http://cl.ly/a39a3d08a113dbe71e1e
Tagged:

Comments

  • DanDevineDanDevine ✭
    Looks similar to the problem I reported some time ago. Have not received a solution to it so far. I see the duplicates in the Manage User page also.

    http://vanillaforums.org/discussion/12520/duplicate-email-addresses-in-manage-user-panel/#Item_1
  • rohan10rohan10
    Yes! Same!

    Are you perchance using integration to WordPress?
  • DanDevineDanDevine ✭
    edited August 2010
    No, just a Vanilla 1 user who upgraded to Vanilla 2.0. I noticed this happening when I upgraded to Vanilla 2.0.3. I believe it was fine prior to that upgrade. I do not have much experience with scripting. I am waiting for 2.0.4 to see if it solves a few of the problems I have noticed so far.
  • emkayemkay
    a quick fix for this is modifying line 161 in js/global.js:

    orig:
    $(this).html('' + email + '');

    change to:
    // $(this).html('' + email + '');
    $(this).html(email);

    note that this removes the mailto-"clickability" of email links.
    They are displayed as-is: user@email.address

    vanilla actually writes email addresses as useratemaildotadress and lets jQuery (aka javascript) take care of the translation to user@email.address.
    But something seems to over-sanitize the HTML and converts < and > into < and > so the generated e-mail link is broken.
    I just started with vanilla and I might have another look at this. But I just needed a quick fix and this does the job for me.
  • emkayemkay
    ignore the code above. vanilla sanitizes the output.
    see http://img266.imageshack.us/img266/768/img20100904.png
  • agnizagniz
    Oh, this is exactly the issue I just informed in another discussion (sorry for duplicate the subject).

    anyway, this seems to be easy (??) to be fixed (by those who are expert in codes)... but till now, no comments or solution!
  • ithcyithcy New
    There is a fix at http://www.vanillaforums.org/discussion/12745/bug-formatting-email-addresses. I'm going to close this thread to consolidate discussion about this issue.

    The fix will be included in the next stable release of Vanilla.
This discussion has been closed.