common: common Alert function to format error messages

[sheet.git] / common.inc.plp
diff --git a/common.inc.plp b/common.inc.plp

index 6330329cc907f5c27f400326037a39c255d521eb..e4cc5f99f5635a4920d007ddb5433878c13c991c 100644 (file)
--- a/common.inc.plp
+++ b/common.inc.plp
@@ -8,10 +8,17 @@ use open ':std' => ':utf8';
  
  use File::stat 'stat';
  use HTTP::Date;
+use Encode qw( decode_utf8 );
+
+sub Alert {
+       my ($title, $debug) = @_;
+       $title .= ": <em>$debug</em>" if $Dev and defined $debug;
+       say "<p class=error>$title</p>\n";
+}
  
  $PLP::ERROR = sub {
         my ($text, $html) = @_;
-       print '<p class="error"><strong>Fatal error</strong>: '.$html."</p>\n\n";
+       Alert("<strong>Fatal error</strong>: $html.");
         warn $text;
  };
  
@@ -20,6 +27,11 @@ BEGIN {
         our $Time = [Time::HiRes::gettimeofday];
  }
  
+# user request
+our $Dev = $ENV{HTTP_HOST} =~ /\bdev\./;
+our ($file) = $ENV{SCRIPT_FILENAME} =~ m{ ([^/]+) \.plp$ }x;
+our $Request = decode_utf8($ENV{PATH_INFO} =~ s{^/}{}r);
+
  our $style;
  our $showkeys = !exists $get{keys} ? undef :
         ($get{keys} ne '0' && ($get{keys} || 'always'));
@@ -54,7 +66,7 @@ sub stylesheet {
  }
  
  sub checkmodified {
-       my $lastmod;
+       my $lastmod = 0;
         for (@_) {
                 my $mod = stat $_ or next;
                 $mod = $mod->mtime or next;
@@ -85,7 +97,9 @@ sub Html {
         $meta->{stylesheet} ||= [qw'light dark circus mono red terse'];
         $meta->{charset} ||= 'utf-8';
  
-       ref $_ eq 'ARRAY' or $_ = [$_] for $meta->{raw};
+       # convert options to arrays
+       ref $_ eq 'ARRAY' or $_ = [$_]
+               for grep {$_} $meta->{raw}, $meta->{description}, $meta->{keywords};
  
         # optional amends
         push @{ $meta->{raw} }, (
@@ -97,33 +111,26 @@ sub Html {
                 '<script type="text/javascript" src="/keys.js?1.6"></script>',
         ) if $meta->{keys};
  
-       # flatten arrays
-       ref $_ eq 'ARRAY' and $_ = join ' ',  @$_ for $meta->{description};
-       ref $_ eq 'ARRAY' and $_ = join ', ', @$_ for $meta->{keywords};
-       ref $_ eq 'ARRAY' and $_ = join "\n", @$_ for $meta->{raw};
-
-       # other vars
-       my $Dev = $ENV{HTTP_HOST} =~ /\bdev\./;
-       my ($file) = $ENV{SCRIPT_FILENAME} =~ m{ ([^/]+) \.plp$ }x;
-
         # leading output
         $header{content_type} = "text/html; charset=$meta->{charset}";
-       print <<"EOT";
-<!DOCTYPE html>
-<html lang="en">
-
-<head>
-<meta http-equiv="content-type" content="$header{content_type}">
-<title>$meta->{title}</title>
-<meta name="description" content="$meta->{description}">
-<meta name="keywords" content="$meta->{keywords}">
-<meta name="viewport" content="width=device-width, initial-scale=1">
-<link rel="icon" type="image/png" href="/clip.png">
-EOT
+       say '<!DOCTYPE html>';
+       say '<html lang="en">';
+       say '';
+       say '<head>';
+       say sprintf '<meta http-equiv="content-type" content="%s">', $header{content_type};
+       say sprintf '<title>%s</title>', $meta->{title};
+       say sprintf '<meta name="description" content="%s">', EscapeHTML($_)
+               for join(' ', @{ $meta->{description} }) || ();
+       say sprintf '<meta name="keywords" content="%s">', EscapeHTML($_)
+               for join(', ', @{ $meta->{keywords} }) || ();
+       say '<meta name="viewport" content="width=device-width, initial-scale=1">';
+       say '<link rel="icon" type="image/png" href="/clip.png">';
         say for stylesheet($meta->{stylesheet});
-       say for $meta->{raw} || ();
+       say for map { @{$_} } $meta->{raw} || ();
         say '<meta name="robots" content="noindex">' if $Dev;
-       say qq{</head>\n\n<body id="$file">};
+       say '</head>';
+       say '';
+       say sprintf '<body id="%s">', $file;
  
         # development version indicator
         printf '<p style="%s">beta</p>', join('; ',