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

[dw-free] "Read more" in latest shown unnecessarily triggered by entries with certain HTML

[commit: http://hg.dwscoalition.org/dw-free/rev/8f0046eee5f5]

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

Defensive programming.

Patch by [personal profile] fu.

Files modified:
  • cgi-bin/ljtextutil.pl
--------------------------------------------------------------------------------
diff -r c9cda49a744f -r 8f0046eee5f5 cgi-bin/ljtextutil.pl
--- a/cgi-bin/ljtextutil.pl	Sat Sep 04 18:06:34 2010 +0800
+++ b/cgi-bin/ljtextutil.pl	Sat Sep 04 18:17:38 2010 +0800
@@ -560,7 +560,7 @@ sub html_trim {
                 # truncate and stop parsing
                 $content = LJ::text_trim($content, undef, ($char_max - $content_len));
                 $out .= $content;
-                $$truncated = 1;
+                $$truncated = 1 if ref $truncated;
                 last;
             }
 
--------------------------------------------------------------------------------

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