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;
             }
 
--------------------------------------------------------------------------------