fu: Close-up of Fu, bringing a scoop of water to her mouth (Default)
fu ([personal profile] fu) wrote in [site community profile] changelog2011-02-24 01:56 pm

[dw-free] Warning in the logs when there are no included files in JS tests

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

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

Fix warnings.

Patch by [personal profile] fu.

Files modified:
  • cgi-bin/DW/Controller/Dev.pm
--------------------------------------------------------------------------------
diff -r c8a1ed6b6719 -r f4d51a6cd107 cgi-bin/DW/Controller/Dev.pm
--- a/cgi-bin/DW/Controller/Dev.pm	Thu Feb 24 21:50:20 2011 +0800
+++ b/cgi-bin/DW/Controller/Dev.pm	Thu Feb 24 21:56:24 2011 +0800
@@ -58,7 +58,7 @@ sub tests_handler {
     my $testcontent = eval{ DW::Template->template_string( "dev/tests/${test}.js" ) } || "";
     if ( $testcontent ) {
         $testcontent =~ m#/\*\s*INCLUDE:\s*(.*?)\*/#s;
-        my $match = $1;
+        my $match = $1 || "";
         for my $res ( split( /\n+/, $match ) ) {
 
             # skip things that don't look like names (could just be an empty line)
--------------------------------------------------------------------------------

Post a comment in response:

This account has disabled anonymous posting.
If you don't have an account you can create one now.
No Subject Icon Selected
More info about formatting

If you are unable to use this captcha for any reason, please contact us by email at support@dreamwidth.org