Created attachment 330905 [details] [diff] [review] sfx_affiliates.module patch Would you please review the patch which includes changes to allow rendering of affiliate links in the "Top 5 Climbers" and some notes for my own reference.
Comment on attachment 330905 [details] [diff] [review] sfx_affiliates.module patch It look okay. Maybe change <br> to <br /> if we are trying for XHTML compliant rather than HTML 4.01 + $content .= "<b>Report Affiliate Spam</b><br>Report Affiliate spam by adding comments to <a href=\"/affiliatesabuse\">this post</a> , providing as much detail as possible (why do you think this is a spammer?).";
Attachment #330905 - Flags: review?(cpollett) → review+
patch was committed to trunk in r17370 by paul http://viewvc.svn.mozilla.org/vc?view=rev&revision=17370
When do you mark bugs that have patches landed as fixed? (Not being snarky, I genuinely want to know; do you want for it to land on staging or something?)
nah, usual is to mark fixed when they hit trunk, so it can be verfied, etc. i just forgot. thanks!
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Thanks; verified FIXED on https://spreadfirefox.authstage.mozilla.com/affiliates/homepage
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.