diff options
author | Samuel Lidén Borell | 2023-01-07 10:32:07 +0100 |
---|---|---|
committer | Yigit Sever | 2023-07-21 03:04:19 +0300 |
commit | b6ffc623fec1149c2be40ce94e160b2a5c542bf4 (patch) | |
tree | 0daa7e279f7c1691914866ca452c451ac65ddfa0 | |
parent | 2dcb46a217e141d949dbce66ee79201658cfb391 (diff) | |
download | cgit-b6ffc623fec1149c2be40ce94e160b2a5c542bf4.tar.gz cgit-b6ffc623fec1149c2be40ce94e160b2a5c542bf4.tar.bz2 cgit-b6ffc623fec1149c2be40ce94e160b2a5c542bf4.zip |
config: make empty js= omit script tag
According to the cgitrc man page, an empty js= value should cause the
script tag to be omitted. But instead, a script tag with an empty URL
is emitted. The same applies to css. So, skip emitting a tag if the
specified string is empty.
Signed-off-by: Samuel Lidén Borell <samuel@kodafritt.se>
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
-rw-r--r-- | ui-shared.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/ui-shared.c b/ui-shared.c index 959ed8b..25bd3f4 100644 --- a/ui-shared.c +++ b/ui-shared.c | |||
@@ -770,6 +770,10 @@ static void print_rel_vcs_link(const char *url) | |||
770 | 770 | ||
771 | static int emit_css_link(struct string_list_item *s, void *arg) | 771 | static int emit_css_link(struct string_list_item *s, void *arg) |
772 | { | 772 | { |
773 | /* Do not emit anything if css= is specified. */ | ||
774 | if (s && *s->string == '\0') | ||
775 | return 0; | ||
776 | |||
773 | html("<link rel='stylesheet' type='text/css' href='"); | 777 | html("<link rel='stylesheet' type='text/css' href='"); |
774 | if (s) | 778 | if (s) |
775 | html_attr(s->string); | 779 | html_attr(s->string); |
@@ -782,6 +786,10 @@ static int emit_css_link(struct string_list_item *s, void *arg) | |||
782 | 786 | ||
783 | static int emit_js_link(struct string_list_item *s, void *arg) | 787 | static int emit_js_link(struct string_list_item *s, void *arg) |
784 | { | 788 | { |
789 | /* Do not emit anything if js= is specified. */ | ||
790 | if (s && *s->string == '\0') | ||
791 | return 0; | ||
792 | |||
785 | html("<script type='text/javascript' src='"); | 793 | html("<script type='text/javascript' src='"); |
786 | if (s) | 794 | if (s) |
787 | html_attr(s->string); | 795 | html_attr(s->string); |