Bugzilla – Attachment 81979 Details for
Bug 21740
Fixed-length fields show _ instead of @ when editing subfields
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 21740: (QA follow-up) Make tests aware of special hrefs
Bug-21740-QA-follow-up-Make-tests-aware-of-special.patch (text/plain), 1.63 KB, created by
Tomás Cohen Arazi (tcohen)
on 2018-11-06 13:20:56 UTC
(
hide
)
Description:
Bug 21740: (QA follow-up) Make tests aware of special hrefs
Filename:
MIME Type:
Creator:
Tomás Cohen Arazi (tcohen)
Created:
2018-11-06 13:20:56 UTC
Size:
1.63 KB
patch
obsolete
>From 7706a1014183db5d828cea793b69f6ccb7eaf7c5 Mon Sep 17 00:00:00 2001 >From: Tomas Cohen Arazi <tomascohen@theke.io> >Date: Tue, 6 Nov 2018 10:20:38 -0300 >Subject: [PATCH] Bug 21740: (QA follow-up) Make tests aware of special hrefs > >Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> >--- > t/lib/QA/TemplateFilters.pm | 17 ++++++++++++++++- > 1 file changed, 16 insertions(+), 1 deletion(-) > >diff --git a/t/lib/QA/TemplateFilters.pm b/t/lib/QA/TemplateFilters.pm >index f2009cc929..b1f444e86d 100644 >--- a/t/lib/QA/TemplateFilters.pm >+++ b/t/lib/QA/TemplateFilters.pm >@@ -70,7 +70,22 @@ sub _process_tt_content { > if $line =~ m{\[%(\s|-|~)*USE raw(\s|-|~)*%\]}; # Does [% Use raw %] exist? > > my $e; >- if ( $line =~ qr{<a href="([^"]+)} ) { >+ if ( $line =~ qr{<a href="#.*([^"]+)} ) { # it's an href, but tagetting JavaScript usage (like bootstrap) >+ my $to_uri_escape = $1; >+ while ( >+ $to_uri_escape =~ m{ >+ \[% >+ (?<pre_chomp>(\s|\-|~)*) >+ (?<tt_block>[^%\-~]+) >+ (?<post_chomp>(\s|\-|~)*) >+ %\]}gmxs >+ ) >+ { >+ ( $new_line, $e ) = process_tt_block($new_line, { %+, filter => 'html_entity' }); >+ push @errors, { line => $line, line_number => $line_number, error => $e } if $e; >+ } >+ } >+ elsif ( $line =~ qr{<a href="([^"]+)} ) { > my $to_uri_escape = $1; > while ( > $to_uri_escape =~ m{ >-- >2.19.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 21740
:
81733
|
81920
|
81923
|
81925
|
81979
|
82232
|
82233