summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--.gitignore1
-rw-r--r--.htaccess21
-rw-r--r--Bugzilla.pm6
-rw-r--r--Bugzilla/CGI.pm10
-rw-r--r--Bugzilla/Constants.pm9
-rw-r--r--Bugzilla/Install/AssetManager.pm218
-rw-r--r--Bugzilla/Install/Filesystem.pm42
-rw-r--r--Bugzilla/Template.pm153
-rwxr-xr-xchecksetup.pl6
-rw-r--r--extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl4
-rw-r--r--extensions/BugmailFilter/template/en/default/account/prefs/bugmail_filter.html.tmpl6
-rw-r--r--extensions/ComponentWatching/template/en/default/account/prefs/component_watch.html.tmpl2
-rw-r--r--extensions/ProdCompSearch/template/en/default/prodcompsearch/form.html.tmpl2
-rw-r--r--template/en/default/attachment/diff-header.html.tmpl4
-rw-r--r--template/en/default/bug/comments.html.tmpl3
-rw-r--r--template/en/default/flag/list.html.tmpl2
-rw-r--r--template/en/default/global/header.html.tmpl64
-rw-r--r--template/en/default/list/list-simple.html.tmpl2
-rw-r--r--template/en/default/reports/duplicates-simple.html.tmpl4
-rw-r--r--template/en/default/rest.html.tmpl2
-rw-r--r--template/en/default/search/boolean-charts.html.tmpl4
21 files changed, 325 insertions, 240 deletions
diff --git a/.gitignore b/.gitignore
index 18f2d0a17..5287b1ea0 100644
--- a/.gitignore
+++ b/.gitignore
@@ -14,6 +14,7 @@
/skins/custom
/graphs
/data
+/assets
/localconfig
/localconfig.*
/index.html
diff --git a/.htaccess b/.htaccess
index 3d128ac9a..d8cf8f0c0 100644
--- a/.htaccess
+++ b/.htaccess
@@ -2,27 +2,6 @@
<FilesMatch (\.pm|\.pl|\.tmpl|\.swf|localconfig.*|cpanfile)$>
deny from all
</FilesMatch>
-<IfModule mod_expires.c>
-<IfModule mod_headers.c>
-<IfModule mod_env.c>
- <FilesMatch (\.js|\.css)$>
- ExpiresActive On
- # According to RFC 2616, "1 year in the future" means "never expire".
- # We change the name of the file's URL whenever its modification date
- # changes, so browsers can cache any individual JS or CSS URL forever.
- # However, since all JS and CSS URLs involve a ? in them (for the changing
- # name) we have to explicitly set an Expires header or browsers won't
- # *ever* cache them.
- ExpiresDefault "now plus 1 years"
- Header append Cache-Control "public"
- </FilesMatch>
-
- # This lets Bugzilla know that we are properly sending Cache-Control
- # and Expires headers for CSS and JS files.
- SetEnv BZ_CACHE_CONTROL 1
-</IfModule>
-</IfModule>
-</IfModule>
# Allow ZeroClipboard for access to the clipboard
<Files "ZeroClipboard.swf">
diff --git a/Bugzilla.pm b/Bugzilla.pm
index 0bdb8d443..b6dcd58ab 100644
--- a/Bugzilla.pm
+++ b/Bugzilla.pm
@@ -36,6 +36,7 @@ use Bugzilla::Flag;
use Bugzilla::Hook;
use Bugzilla::Install::Localconfig qw(read_localconfig);
use Bugzilla::Install::Util qw(init_console include_languages);
+use Bugzilla::Install::AssetManager;
use Bugzilla::Memcached;
use Bugzilla::Template;
use Bugzilla::Token;
@@ -257,6 +258,11 @@ sub extensions {
return $cache->{extensions};
}
+sub asset_manager {
+ state $asset_manager = Bugzilla::Install::AssetManager->new;
+ return $asset_manager;
+}
+
sub cgi {
return $_[0]->request_cache->{cgi} ||= new Bugzilla::CGI();
}
diff --git a/Bugzilla/CGI.pm b/Bugzilla/CGI.pm
index 2402e997b..65e36a24f 100644
--- a/Bugzilla/CGI.pm
+++ b/Bugzilla/CGI.pm
@@ -441,12 +441,20 @@ sub header {
%headers = ('-type' => shift(@_));
}
else {
- %headers = @_;
+ # we always want headers to be in lower case, to avoid
+ # for instance -Cache_Control vs. -cache_control.
+ my %tmp = @_;
+ foreach my $key (keys %tmp) {
+ my $lc_key = lc $key;
+ warn "duplicate header: $key" if exists $headers{$lc_key};
+ $headers{$lc_key} = $tmp{$key};
+ }
}
if ($self->{'_content_disp'}) {
$headers{'-content_disposition'} = $self->{'_content_disp'};
}
+ $headers{'-cache_control'} //= 'no-cache, no-store, must-revalidate';
if (!$user->id && $user->authorizer->can_login
&& !$self->cookie('Bugzilla_login_request_cookie'))
diff --git a/Bugzilla/Constants.pm b/Bugzilla/Constants.pm
index 9c8f39b4a..4cb838f94 100644
--- a/Bugzilla/Constants.pm
+++ b/Bugzilla/Constants.pm
@@ -26,8 +26,6 @@ use Memoize;
bz_locations
- CONCATENATE_ASSETS
-
IS_NULL
NOT_NULL
@@ -219,11 +217,6 @@ use constant REST_DOC => "http://www.bugzilla.org/docs/tip/en/html/api/";
use constant REMOTE_FILE => 'http://updates.bugzilla.org/bugzilla-update.xml';
use constant LOCAL_FILE => 'bugzilla-update.xml'; # Relative to datadir.
-# When true CSS and JavaScript assets will be concatanted and minified at
-# run-time, to reduce the number of requests required to render a page.
-# Setting this to a false value can help debugging.
-use constant CONCATENATE_ASSETS => 1;
-
# These are unique values that are unlikely to match a string or a number,
# to be used in criteria for match() functions and other things. They start
# and end with spaces because most Bugzilla stuff has trim() called on it,
@@ -702,7 +695,7 @@ sub _bz_locations {
# The script should really generate these graphs directly...
'webdotdir' => "$datadir/webdot",
'extensionsdir' => "$libpath/extensions",
- 'assetsdir' => "$datadir/assets",
+ 'assetsdir' => "$libpath/assets",
# error_reports store error/warnings destined for sentry
'error_reports' => "$libpath/error_reports",
};
diff --git a/Bugzilla/Install/AssetManager.pm b/Bugzilla/Install/AssetManager.pm
new file mode 100644
index 000000000..e314e2ccb
--- /dev/null
+++ b/Bugzilla/Install/AssetManager.pm
@@ -0,0 +1,218 @@
+# This Source Code Form is subject to the terms of the Mozilla Public
+# License, v. 2.0. If a copy of the MPL was not distributed with this
+# file, You can obtain one at http://mozilla.org/MPL/2.0/.
+#
+# This Source Code Form is "Incompatible With Secondary Licenses", as
+# defined by the Mozilla Public License, v. 2.0.
+
+package Bugzilla::Install::AssetManager;
+
+use 5.10.1;
+use strict;
+use warnings;
+
+use Moo;
+use MooX::StrictConstructor;
+use Type::Utils;
+use Types::Standard qw(Bool Str ArrayRef);
+
+use Digest::SHA ();
+use File::Copy qw(cp);
+use File::Find qw(find);
+use File::Basename qw(dirname);
+use File::Spec;
+use JSON::XS ();
+use MIME::Base64 qw(encode_base64);
+use File::Slurp;
+use List::MoreUtils qw(any all);
+use Carp;
+
+use Bugzilla::Constants qw(bz_locations);
+
+our $VERSION = 1;
+
+my $SHA_VERSION = '224';
+
+my $ABSOLUTE_DIR = declare as Str, where { File::Spec->file_name_is_absolute($_) && -d $_ }
+message {"must be an absolute path to a directory"};
+
+has 'base_dir' => ( is => 'lazy', isa => $ABSOLUTE_DIR );
+has 'asset_dir' => ( is => 'lazy', isa => $ABSOLUTE_DIR );
+has 'source_dirs' => ( is => 'lazy' );
+has 'state' => ( is => 'lazy' );
+has 'state_file' => ( is => 'lazy' );
+has 'json' => ( is => 'lazy' );
+
+sub asset_file {
+ my ( $self, $file, $relative_to ) = @_;
+ $relative_to //= $self->base_dir;
+ my $asset_file = $self->state->{asset_map}->{$file}
+ or return $file;
+
+ return File::Spec->abs2rel( File::Spec->catfile( $self->asset_dir, $asset_file ), $relative_to );
+}
+
+sub asset_files {
+ my ( $self, @files ) = @_;
+
+ return \@files;
+}
+
+sub asset_sri {
+ my ( $self, $asset_file ) = @_;
+ my ($hex) = $asset_file =~ m!([[:xdigit:]]+)\.\w+$!;
+ my $data = pack "H*", $hex;
+ return "sha$SHA_VERSION-" . encode_base64( $data, "" );
+}
+
+sub compile_file {
+ my ( $self, $file ) = @_;
+ return unless -f $file;
+ my $base_dir = $self->base_dir;
+ my $asset_dir = $self->asset_dir;
+ my $asset_map = $self->state->{asset_map};
+
+ my $key = File::Spec->abs2rel( $file, $base_dir );
+ return if $asset_map->{$key};
+
+ if ( $file =~ /\.(jpe?g|png|gif|ico|woff|js)$/i ) {
+ my $ext = $1;
+ my $digest = $self->_digest_file($file) or die "invalid digest for $file";
+ my $asset_file = File::Spec->catfile( $asset_dir, "$digest.$ext" );
+ cp( $file, $asset_file ) or die "failed to copy $file to $asset_file: $!";
+ if ( $digest eq $self->_digest_file($asset_file) ) {
+ $asset_map->{$key} = File::Spec->abs2rel( $asset_file, $asset_dir );
+ }
+ else {
+ die "failed to write $asset_file";
+ }
+ }
+ elsif ( $file =~ /\.css$/ ) {
+ my $content = read_file($file);
+
+ $content =~ s{(?<!=)url\(([^\)]+)\)}{$self->_css_url_rewrite($1, $file)}eig;
+ my $digest = $self->_digest_string($content);
+ my $asset_file = File::Spec->catfile( $asset_dir, "$digest.css" );
+ write_file( $asset_file, $content ) or die "failed to write $asset_file: $!";
+ if ( $digest eq $self->_digest_file($asset_file) ) {
+ $asset_map->{$key} = File::Spec->abs2rel( $asset_file, $asset_dir );
+ }
+ else {
+ die "failed to write $asset_file";
+ }
+ }
+}
+
+sub compile_all {
+ my ($self) = @_;
+ my $asset_map = $self->state->{asset_map} = {};
+
+ my $wanted = sub {
+ $self->compile_file($File::Find::name);
+ };
+
+ find( { wanted => $wanted, no_chdir => 1 }, @{ $self->source_dirs } );
+
+ $self->_save_state();
+}
+
+sub _css_url_rewrite {
+ my ( $self, $url, $file ) = @_;
+ my $dir = dirname($file);
+
+ # rewrite relative urls as the unified stylesheet lives in a different
+ # directory from the source
+ $url =~ s/(^['"]|['"]$)//g;
+ if ( $url =~ m!^(/|data:)! ) {
+ return 'url(' . $url . ')';
+ }
+ else {
+ my $url_file = File::Spec->rel2abs( $url, $dir );
+ my $ref_file = File::Spec->abs2rel( $url_file, $self->base_dir );
+ $self->compile_file($url_file);
+ return sprintf( "url(%s)", $self->asset_file( $ref_file, $self->asset_dir ) );
+ }
+}
+
+sub _new_digest { Digest::SHA->new($SHA_VERSION) }
+
+sub _digest_file {
+ my ( $self, $file ) = @_;
+ my $digest = $self->_new_digest;
+ $digest->addfile( $file, "b" );
+ return $digest->hexdigest;
+}
+
+sub _digest_string {
+ my ( $self, $string ) = @_;
+ my $digest = $self->_new_digest;
+ $digest->add($string);
+ return $digest->hexdigest;
+}
+
+sub _build_base_dir {
+ Cwd::realpath( File::Spec->rel2abs( bz_locations->{cgi_path} ) );
+}
+
+sub _build_asset_dir {
+ my ($self) = @_;
+ my $dir
+ = Cwd::realpath( File::Spec->rel2abs( bz_locations->{assetsdir} ) );
+
+ if ( $dir && -d $dir ) {
+ my $version_dir = File::Spec->catdir( $dir, "v" . $self->VERSION );
+ unless ( -d $version_dir ) {
+ mkdir $version_dir or die "mkdir $version_dir failed: $!";
+ }
+ return $version_dir;
+ }
+ else {
+ return $dir;
+ }
+}
+
+sub _build_source_dirs {
+ my ($self) = @_;
+ my $base = $self->base_dir;
+ my $ext_dir = "$base/extensions";
+ opendir my $ext_dir_handle, $ext_dir or die "unable to open $ext_dir: $!";
+ my @dirs = grep { -d $_ } map {"$ext_dir/$_/web"} grep { !/^\.\.?$/ } readdir $ext_dir_handle;
+ closedir $ext_dir_handle;
+
+ return [ "$base/skins", "$base/js", grep { -d $_ } @dirs ];
+}
+
+sub _build_state_file {
+ my ($self) = @_;
+ return $self->asset_dir . "/state.json";
+}
+
+sub _build_state {
+ my ($self) = @_;
+ my $state;
+ if ( open my $fh, '<:bytes', $self->state_file ) {
+ local $/ = undef;
+ my $json = <$fh>;
+ close $fh;
+ $state = $self->json->decode($json);
+ }
+ else {
+ $state = {};
+ }
+
+ $state->{asset_map} //= {};
+
+ return $state;
+}
+
+sub _build_json { JSON::XS->new->canonical->utf8 }
+
+sub _save_state {
+ my ($self) = @_;
+ open my $fh, '>:bytes', $self->state_file
+ or die "unable to write state file: $!";
+ print $fh $self->json->encode( $self->state );
+ close $fh;
+}
+
+1;
diff --git a/Bugzilla/Install/Filesystem.pm b/Bugzilla/Install/Filesystem.pm
index 22ec34a95..162e324f7 100644
--- a/Bugzilla/Install/Filesystem.pm
+++ b/Bugzilla/Install/Filesystem.pm
@@ -31,6 +31,7 @@ use File::Path;
use File::Basename;
use File::Copy qw(move);
use File::Spec;
+use File::stat;
use Cwd ();
use File::Slurp;
use IO::File;
@@ -81,12 +82,16 @@ EOT
use constant HT_ASSETS_DIR => <<'EOT';
# Allow access to .css and js files
-<FilesMatch \.(css|js)$>
- Allow from all
+<FilesMatch state\.json$>
+ Deny from all
</FilesMatch>
+FileETag None
+Header set Cache-Control "public, immutable, max-age=31536000"
+Header set Content-Security-Policy "default-src 'none';"
+
# And no directory listings, either.
-Deny from all
+Options -Indexes
EOT
use constant INDEX_HTML => <<'EOT';
@@ -344,7 +349,7 @@ sub FILESYSTEM {
$attachdir => DIR_CGI_WRITE,
$graphsdir => DIR_CGI_WRITE | DIR_ALSO_WS_SERVE,
$webdotdir => DIR_CGI_WRITE | DIR_ALSO_WS_SERVE,
- $assetsdir => DIR_CGI_WRITE | DIR_ALSO_WS_SERVE,
+ $assetsdir => DIR_WS_SERVE,
$template_cache => DIR_CGI_WRITE,
$error_reports => DIR_CGI_WRITE,
# Directories that contain content served directly by the web server.
@@ -446,8 +451,13 @@ sub FILESYSTEM {
"$webdotdir/.htaccess" => { perms => WS_SERVE,
contents => HT_WEBDOT_DIR },
"$assetsdir/.htaccess" => { perms => WS_SERVE,
- contents => HT_ASSETS_DIR },
+ contents => HT_ASSETS_DIR },
);
+ my $mtime = stat(__FILE__)->mtime;
+ foreach my $file (keys %htaccess) {
+ my $file_stat = stat($file);
+ $htaccess{$file}{overwrite} = $file_stat && $mtime > $file_stat->mtime;
+ }
Bugzilla::Hook::process('install_filesystem', {
files => \%files,
@@ -561,7 +571,6 @@ sub update_filesystem {
_remove_empty_css_files();
_convert_single_file_skins();
- _remove_dynamic_assets();
}
sub _css_url_fix {
@@ -627,27 +636,6 @@ sub _convert_single_file_skins {
}
}
-# delete all automatically generated css/js files to force recreation at the
-# next request.
-sub _remove_dynamic_assets {
- my @files = (
- glob(bz_locations()->{assetsdir} . '/*.css'),
- glob(bz_locations()->{assetsdir} . '/*.js'),
- );
- foreach my $file (@files) {
- unlink($file);
- }
-
- # remove old skins/assets directory
- my $old_path = bz_locations()->{skinsdir} . '/assets';
- if (-d $old_path) {
- foreach my $file (glob("$old_path/*.css")) {
- unlink($file);
- }
- rmdir($old_path);
- }
-}
-
sub create_htaccess {
_create_files(%{FILESYSTEM()->{htaccess}});
diff --git a/Bugzilla/Template.pm b/Bugzilla/Template.pm
index 5bef599d4..d72feea67 100644
--- a/Bugzilla/Template.pm
+++ b/Bugzilla/Template.pm
@@ -401,7 +401,13 @@ sub css_files {
unshift @requested_css, "skins/yui.css" unless $no_yui;
- my @css_sets = map { _css_link_set($_) } @requested_css;
+ my @css_sets = map { _css_link_set($_) } sort {
+ my $first_a = $a =~ m!js/jquery!;
+ my $first_b = $b =~ m!js/jquery!;
+ return -1 if $first_a && !$first_b;
+ return 1 if !$first_a && $first_b;
+ return 0;
+ } @requested_css;
my %by_type = (standard => [], skin => [], custom => []);
foreach my $set (@css_sets) {
@@ -410,18 +416,13 @@ sub css_files {
}
}
- # build unified
- $by_type{unified_standard_skin} = _concatenate_css($by_type{standard},
- $by_type{skin});
- $by_type{unified_custom} = _concatenate_css($by_type{custom});
-
return \%by_type;
}
sub _css_link_set {
my ($file_name) = @_;
- my %set = (standard => mtime_filter($file_name));
+ my %set = (standard => $file_name);
# We use (?:^|/) to allow Extensions to use the skins system if they want.
if ($file_name !~ m{(?:^|/)skins/standard/}) {
@@ -432,127 +433,19 @@ sub _css_link_set {
my $cgi_path = bz_locations()->{'cgi_path'};
my $skin_file_name = $file_name;
$skin_file_name =~ s{(?:^|/)skins/standard/}{skins/contrib/$skin/};
- if (my $mtime = _mtime("$cgi_path/$skin_file_name")) {
- $set{skin} = mtime_filter($skin_file_name, $mtime);
+ if (-f "$cgi_path/$skin_file_name") {
+ $set{skin} = $skin_file_name;
}
my $custom_file_name = $file_name;
$custom_file_name =~ s{(?:^|/)skins/standard/}{skins/custom/};
- if (my $custom_mtime = _mtime("$cgi_path/$custom_file_name")) {
- $set{custom} = mtime_filter($custom_file_name, $custom_mtime);
+ if (-f "$cgi_path/$custom_file_name") {
+ $set{custom} = $custom_file_name;
}
return \%set;
}
-sub _concatenate_css {
- my @sources = map { @$_ } @_;
- return unless @sources;
-
- my %files =
- map {
- (my $file = $_) =~ s/(^[^\?]+)\?.+/$1/;
- $_ => $file;
- } @sources;
-
- my $cgi_path = bz_locations()->{cgi_path};
- my $skins_path = bz_locations()->{assetsdir};
-
- # build minified files
- my @minified;
- foreach my $source (@sources) {
- next unless -e "$cgi_path/$files{$source}";
- my $file = $skins_path . '/' . md5_hex($source) . '.css';
- if (!-e $file) {
- my $content = read_file("$cgi_path/$files{$source}");
-
- # minify
- $content =~ s{/\*.*?\*/}{}sg; # comments
- $content =~ s{(^\s+|\s+$)}{}mg; # leading/trailing whitespace
- $content =~ s{\n}{}g; # single line
-
- # rewrite urls
- $content =~ s{url\(([^\)]+)\)}{_css_url_rewrite($source, $1)}eig;
-
- write_file($file, "/* $files{$source} */\n" . $content . "\n");
- }
- push @minified, $file;
- }
-
- # concat files
- my $file = $skins_path . '/' . md5_hex(join(' ', @sources)) . '.css';
- if (!-e $file) {
- my $content = '';
- foreach my $source (@minified) {
- $content .= read_file($source);
- }
- write_file($file, $content);
- }
-
- $file =~ s/^\Q$cgi_path\E\///o;
- return mtime_filter($file);
-}
-
-sub _css_url_rewrite {
- my ($source, $url) = @_;
- # rewrite relative urls as the unified stylesheet lives in a different
- # directory from the source
- $url =~ s/(^['"]|['"]$)//g;
- if (substr($url, 0, 1) eq '/' || substr($url, 0, 5) eq 'data:') {
- return 'url(' . $url . ')';
- }
- return 'url(../../' . dirname($source) . '/' . $url . ')';
-}
-
-sub _concatenate_js {
- return @_ unless CONCATENATE_ASSETS;
- my ($sources) = @_;
- return [] unless $sources;
- $sources = ref($sources) ? $sources : [ $sources ];
-
- my %files =
- map {
- (my $file = $_) =~ s/(^[^\?]+)\?.+/$1/;
- $_ => $file;
- } @$sources;
-
- my $cgi_path = bz_locations()->{cgi_path};
- my $skins_path = bz_locations()->{assetsdir};
-
- # build minified files
- my @minified;
- foreach my $source (@$sources) {
- next unless -e "$cgi_path/$files{$source}";
- my $file = $skins_path . '/' . md5_hex($source) . '.js';
- if (!-e $file) {
- my $content = read_file("$cgi_path/$files{$source}");
-
- # minimal minification
- $content =~ s#/\*.*?\*/##sg; # block comments
- $content =~ s#(^ +| +$)##gm; # leading/trailing spaces
- $content =~ s#^//.+$##gm; # single line comments
- $content =~ s#\n{2,}#\n#g; # blank lines
- $content =~ s#(^\s+|\s+$)##g; # whitespace at the start/end of file
-
- write_file($file, "/* $files{$source} */\n" . $content . "\n");
- }
- push @minified, $file;
- }
-
- # concat files
- my $file = $skins_path . '/' . md5_hex(join(' ', @$sources)) . '.js';
- if (!-e $file) {
- my $content = '';
- foreach my $source (@minified) {
- $content .= read_file($source);
- }
- write_file($file, $content);
- }
-
- $file =~ s/^\Q$cgi_path\E\///o;
- return [ $file ];
-}
-
# YUI dependency resolution
sub yui_resolve_deps {
my ($yui, $yui_deps) = @_;
@@ -900,7 +793,7 @@ sub create {
html_light => \&Bugzilla::Util::html_light_quote,
email => \&Bugzilla::Util::email_filter,
-
+
mtime => \&mtime_filter,
# iCalendar contentline filter
@@ -983,19 +876,12 @@ sub create {
# Function for retrieving global parameters.
'Param' => sub { return Bugzilla->params->{$_[0]}; },
- 'bugzilla_version' => sub {
- my $version = Bugzilla->VERSION;
- if (my @ver = $version =~ /^(\d{4})(\d{2})(\d{2})\.(\d+)$/s) {
- if ($ver[3] eq '1') {
- return join('.', @ver[0,1,2]);
- }
- else {
- return join('.', @ver);
- }
- }
- else {
- return $version;
- }
+ asset_file => sub {
+ return Bugzilla->asset_manager->asset_file($_[0]);
+ },
+
+ asset_files => sub {
+ return Bugzilla->asset_manager->asset_files(@{ $_[0] });
},
json_encode => sub {
@@ -1091,7 +977,6 @@ sub create {
'css_files' => \&css_files,
yui_resolve_deps => \&yui_resolve_deps,
- concatenate_js => \&_concatenate_js,
# Whether or not keywords are enabled, in this Bugzilla.
'use_keywords' => sub { return Bugzilla::Keyword->any_exist; },
diff --git a/checksetup.pl b/checksetup.pl
index 598432c93..d689602fd 100755
--- a/checksetup.pl
+++ b/checksetup.pl
@@ -52,6 +52,7 @@ GetOptions(\%switch, 'help|h|?',
'no-templates|t', 'verbose|v|no-silent',
'cpanm:s', 'check-modules',
'make-admin=s', 'reset-password=s', 'version|V',
+ 'no-assets',
'no-permissions|p');
# Print the help message if that switch was selected.
@@ -190,6 +191,11 @@ my %old_params = update_params();
Bugzilla::Template::precompile_templates(!$silent)
unless $switch{'no-templates'};
+unless ($switch{'no-assets'}) {
+ say "Compiling assets..." unless $silent;
+ Bugzilla->asset_manager->compile_all;
+}
+
###########################################################################
# Set proper rights (--CHMOD--)
###########################################################################
diff --git a/extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl b/extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl
index 5ece78805..18faf19cc 100644
--- a/extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl
+++ b/extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl
@@ -298,7 +298,7 @@
<button type="button" id="mode-btn" class="major">
<span id="mode-btn-readonly" title="Enable editing fields for [% terms.bug %] metadata">Edit [% terms.Bug %]</span>
<span id="mode-btn-loading">
- <img id="edit-throbber" src="extensions/BugModal/web/throbber.gif" width="16" height="11">
+ <img id="edit-throbber" src="[% asset_file('extensions/BugModal/web/throbber.gif') FILTER html %]" width="16" height="11">
Fetching
</span>
</button>
@@ -402,7 +402,7 @@
%]
<button id="product-search" type="button" class="minor">Search</button>
<button id="product-search-cancel" type="button" class="minor" style="display:none">X</button>
- <img id="product-throbber" src="extensions/BugModal/web/throbber.gif"
+ <img id="product-throbber" src="[% asset_file('extensions/BugModal/web/throbber.gif') FILTER html %]"
width="16" height="11" style="display:none">
<img id="product-search-error" class="tt" src="extensions/BugModal/web/error.png"
width="16" height="16" style="display:none">
diff --git a/extensions/BugmailFilter/template/en/default/account/prefs/bugmail_filter.html.tmpl b/extensions/BugmailFilter/template/en/default/account/prefs/bugmail_filter.html.tmpl
index e7e0ed749..e09a16e39 100644
--- a/extensions/BugmailFilter/template/en/default/account/prefs/bugmail_filter.html.tmpl
+++ b/extensions/BugmailFilter/template/en/default/account/prefs/bugmail_filter.html.tmpl
@@ -6,10 +6,10 @@
# defined by the Mozilla Public License, v. 2.0.
#%]
-<link href="[% "extensions/BugmailFilter/web/style/bugmail-filter.css" FILTER mtime %]"
+<link href="[% asset_file("extensions/BugmailFilter/web/style/bugmail-filter.css") FILTER html %]"
rel="stylesheet" type="text/css">
<script type="text/javascript"
- src="[% "extensions/BugmailFilter/web/js/bugmail-filter.js" FILTER mtime %]"></script>
+ src="[% asset_file("extensions/BugmailFilter/web/js/bugmail-filter.js") FILTER html %]"></script>
[% SET selectable_products = user.get_selectable_products %]
[% SET dont_show_button = 1 %]
@@ -26,7 +26,7 @@ var cpts = new Array();
[% n = n + 1 %]
[% END %]
</script>
-<script type="text/javascript" src="[% 'js/productform.js' FILTER mtime FILTER html %]">
+<script type="text/javascript" src="[% asset_file('js/productform.js') FILTER html %]">
</script>
<hr>
diff --git a/extensions/ComponentWatching/template/en/default/account/prefs/component_watch.html.tmpl b/extensions/ComponentWatching/template/en/default/account/prefs/component_watch.html.tmpl
index 5e27c1247..8252cdb9e 100644
--- a/extensions/ComponentWatching/template/en/default/account/prefs/component_watch.html.tmpl
+++ b/extensions/ComponentWatching/template/en/default/account/prefs/component_watch.html.tmpl
@@ -44,7 +44,7 @@ var watch_users = new Array();
[% END %]
[% END %]
</script>
-<script type="text/javascript" src="[% 'js/productform.js' FILTER mtime FILTER html %]">
+<script type="text/javascript" src="[% asset_file('js/productform.js') FILTER html %]">
</script>
<script>
diff --git a/extensions/ProdCompSearch/template/en/default/prodcompsearch/form.html.tmpl b/extensions/ProdCompSearch/template/en/default/prodcompsearch/form.html.tmpl
index c232f677d..bbd0cfc40 100644
--- a/extensions/ProdCompSearch/template/en/default/prodcompsearch/form.html.tmpl
+++ b/extensions/ProdCompSearch/template/en/default/prodcompsearch/form.html.tmpl
@@ -33,7 +33,7 @@
<div class="pcs-header">
[% input_label FILTER none %]&nbsp;
<img id="[% id FILTER html %]-throbber"
- src="extensions/ProdCompSearch/web/images/throbber.gif"
+ src="[% asset_file('extensions/ProdCompSearch/web/images/throbber.gif') FILTER html %]"
style="display:none" width="16" height="11">
<span class="pcs-message" id="[% id FILTER html %]-no_results" style="display:none">
No components found
diff --git a/template/en/default/attachment/diff-header.html.tmpl b/template/en/default/attachment/diff-header.html.tmpl
index b407b4f3a..3e56ab2f4 100644
--- a/template/en/default/attachment/diff-header.html.tmpl
+++ b/template/en/default/attachment/diff-header.html.tmpl
@@ -57,9 +57,9 @@ Interdiff of #[% oldid %] and #[% newid %] for [% terms.bug %] #[% bugid %]
[% ELSE %]
<html>
<head>
- <link href="[% 'skins/standard/attachment.css' FILTER mtime %]"
+ <link href="[% asset_file('skins/standard/attachment.css') FILTER html %]"
rel="stylesheet" type="text/css">
- <script src="[% 'js/attachment.js' FILTER mtime %]"
+ <script src="[% asset_file('js/attachment.js') FILTER html %]"
type="text/javascript"></script>
</head>
<body onload="[% onload FILTER html %]">
diff --git a/template/en/default/bug/comments.html.tmpl b/template/en/default/bug/comments.html.tmpl
index 2f8658c9c..eb7d240b6 100644
--- a/template/en/default/bug/comments.html.tmpl
+++ b/template/en/default/bug/comments.html.tmpl
@@ -22,8 +22,7 @@
[% PROCESS bug/time.html.tmpl %]
-<script src="[% 'js/comments.js' FILTER mtime %]" type="text/javascript">
-</script>
+<script src="[% asset_file('js/comments.js') FILTER html %]" type="text/javascript"></script>
<script type="text/javascript">
<!--
diff --git a/template/en/default/flag/list.html.tmpl b/template/en/default/flag/list.html.tmpl
index 55184c47b..09113bbf3 100644
--- a/template/en/default/flag/list.html.tmpl
+++ b/template/en/default/flag/list.html.tmpl
@@ -30,7 +30,7 @@
[% DEFAULT flag_table_id = "flags" %]
-<script src="[% 'js/flag.js' FILTER mtime %]" type="text/javascript"></script>
+<script src="[% asset_file('js/flag.js') FILTER html %]" type="text/javascript"></script>
<table id="[% flag_table_id FILTER html %]">
[% UNLESS flag_no_header %]
diff --git a/template/en/default/global/header.html.tmpl b/template/en/default/global/header.html.tmpl
index 107c69069..57088ac2a 100644
--- a/template/en/default/global/header.html.tmpl
+++ b/template/en/default/global/header.html.tmpl
@@ -52,7 +52,6 @@
style_urls = []
no_yui = 0
jquery = []
- jquery_css = []
generate_api_token = 0
%]
@@ -73,7 +72,7 @@
"js/jquery/ui/jquery-ui-structure-min.css",
"js/jquery/ui/jquery-ui-theme-min.css",
] %]
-[% style_urls.import(jquery_css, jq_css_urls) FILTER null %]
+[% CALL style_urls.import(jq_css_urls) %]
[%# Add our required jQuery plugins %]
[% jquery.push("cookie", "devbridgeAutocomplete") %]
@@ -137,30 +136,25 @@
[% PROCESS 'global/setting-descs.none.tmpl' %]
[% SET css_sets = css_files(style_urls.unique, no_yui) %]
- [% IF constants.CONCATENATE_ASSETS %]
- [% PROCESS format_css_link asset_url = css_sets.unified_standard_skin %]
- [% ELSE %]
- [% FOREACH asset_url = css_sets.standard %]
- [% PROCESS format_css_link %]
- [% END %]
- [% FOREACH asset_url = css_sets.skin %]
+ [% SET css_standard_and_skin = css_sets.standard %]
+ [% CALL css_standard_and_skin.import(css_sets.skin) %]
+ [% IF style %]
+ [% FOREACH asset_url = asset_files(css_standard_and_skin) %]
[% PROCESS format_css_link %]
[% END %]
- [% END %]
- [% IF style %]
<style type="text/css">
[% style %]
</style>
- [% END %]
- [% IF css_sets.unified_custom %]
- [% IF constants.CONCATENATE_ASSETS %]
- [% PROCESS format_css_link asset_url = css_sets.unified_custom %]
- [% ELSE %]
- [% FOREACH asset_url = css_sets.custom %]
- [% PROCESS format_css_link %]
- [% END %]
+ [% FOREACH asset_url = asset_files(css_sets.custom) %]
+ [% PROCESS format_css_link %]
+ [% END %]
+ [% ELSE %]
+ [% SET css_all = [] %]
+ [% CALL css_all.import(css_standard_and_skin, css_sets.custom) %]
+ [% FOREACH asset_url = asset_files(css_all) %]
+ [% PROCESS format_css_link %]
[% END %]
[% END %]
@@ -170,10 +164,6 @@
[% END %]
[% starting_js_urls.push('js/global.js') %]
- [% FOREACH asset_url = concatenate_js(starting_js_urls) %]
- [% PROCESS format_js_link %]
- [% END %]
-
[% inline_javascript = BLOCK %]
[% IF NOT no_yui %]
YAHOO.namespace('bugzilla');
@@ -188,14 +178,26 @@
[% javascript %]
[% END %]
[% END %]
+
[% IF inline_javascript.search("\\S") %]
+ [% FOREACH asset_url = asset_files(starting_js_urls) %]
+ [% PROCESS format_js_link %]
+ [% END %]
+
<script [% script_nonce FILTER none %]>
[% inline_javascript FILTER none %]
</script>
- [% END %]
- [% FOREACH asset_url = concatenate_js(javascript_urls) %]
- [% PROCESS format_js_link %]
+ [% FOREACH asset_url = asset_files(javascript_urls) %]
+ [% PROCESS format_js_link %]
+ [% END %]
+ [% ELSE %]
+ [% SET js_all = [] %]
+ [% CALL js_all.import(starting_js_urls) %]
+ [% CALL js_all.import(javascript_urls) %]
+ [% FOREACH asset_url = asset_files(js_all) %]
+ [% PROCESS format_js_link %]
+ [% END %]
[% END %]
[%# this puts the live bookmark up on firefox for the Atom feed %]
@@ -310,7 +312,7 @@
</td>
<td id="moz_tab">
<a href="https://www.mozilla.org/" title="Mozilla - Home of the Mozilla Project">
- <img src="images/tabzilla.png" border="0" height="42" width="154"></a>
+ <img src="[% asset_file('images/tabzilla.png') FILTER html %]" border="0" height="42" width="154"></a>
</td>
</tr>
</table>
@@ -396,9 +398,9 @@
[% END %]
[% BLOCK format_css_link %]
- <link href="[% asset_url FILTER html %]" rel="stylesheet" type="text/css">
-[% END %]
+ <link data-file="[% asset_url FILTER html %]" href="[% asset_file(asset_url) FILTER html %]" rel="stylesheet" type="text/css">
+[%+ END %]
[% BLOCK format_js_link %]
- <script [% script_nonce FILTER none %] type="text/javascript" src="[% asset_url FILTER mtime FILTER html %]"></script>
-[% END %]
+ <script [% script_nonce FILTER none %] data-file="[% asset_url FILTER html %]" type="text/javascript" src="[% asset_file(asset_url) FILTER html %]"></script>
+[%+ END %]
diff --git a/template/en/default/list/list-simple.html.tmpl b/template/en/default/list/list-simple.html.tmpl
index f4c3549ed..4c186e207 100644
--- a/template/en/default/list/list-simple.html.tmpl
+++ b/template/en/default/list/list-simple.html.tmpl
@@ -39,7 +39,7 @@
<head>
<title>[% title FILTER html %]</title>
<base href="[% urlbase FILTER html %]">
- <link href="[% 'skins/standard/buglist.css' FILTER mtime %]"
+ <link href="[% asset_file('skins/standard/buglist.css') FILTER html %]"
rel="stylesheet" type="text/css">
</head>
diff --git a/template/en/default/reports/duplicates-simple.html.tmpl b/template/en/default/reports/duplicates-simple.html.tmpl
index a08522790..88f14b910 100644
--- a/template/en/default/reports/duplicates-simple.html.tmpl
+++ b/template/en/default/reports/duplicates-simple.html.tmpl
@@ -39,9 +39,9 @@
<head>
<title>[% title FILTER html %]</title>
- <link href="[% 'skins/standard/global.css' FILTER mtime %]"
+ <link href="[% asset_file('skins/standard/global.css') FILTER html %]"
rel="stylesheet" type="text/css">
- <link href="[% 'skins/standard/duplicates.css' FILTER mtime %]"
+ <link href="[% asset_file('skins/standard/duplicates.css') FILTER html %]"
rel="stylesheet" type="text/css">
</head>
diff --git a/template/en/default/rest.html.tmpl b/template/en/default/rest.html.tmpl
index 0b8321dd1..bc9c5b6fd 100644
--- a/template/en/default/rest.html.tmpl
+++ b/template/en/default/rest.html.tmpl
@@ -10,7 +10,7 @@
<html>
<head>
<title>Bugzilla::REST::API</title>
- <link href="[% urlbase FILTER none %][% 'skins/standard/global.css' FILTER mtime %]"
+ <link href="[% urlbase FILTER none %][% asset_file('skins/standard/global.css') FILTER html %]"
rel="stylesheet" type="text/css">
</head>
<body>
diff --git a/template/en/default/search/boolean-charts.html.tmpl b/template/en/default/search/boolean-charts.html.tmpl
index 3fb1f8eae..e3b149969 100644
--- a/template/en/default/search/boolean-charts.html.tmpl
+++ b/template/en/default/search/boolean-charts.html.tmpl
@@ -76,8 +76,8 @@
TUI_alternates['custom_search_advanced'] = "Show Advanced Features";
TUI_hide_default('custom_search_advanced');
</script>
- <script type="text/javascript" src="[% 'js/custom-search.js' FILTER mtime %]"></script>
- <script type="text/javascript" src="[% 'js/history.js/native.history.js' FILTER mtime %]"></script>
+ <script type="text/javascript" src="[% asset_file('js/custom-search.js') FILTER html %]"></script>
+ <script type="text/javascript" src="[% asset_file('js/history.js/native.history.js') FILTER html %]"></script>
<script type="text/javascript">
redirect_html4_browsers();
[%# These are alternative labels for the AND and OR options in and_all_select %]