Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: includes/testcases.tmpl

Issue 29874607: Issue 6769 - Avoid hardcoding the URL (Closed)
Patch Set: Created Sept. 4, 2018, 4:23 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: includes/testcases.tmpl
diff --git a/includes/testcases.md b/includes/testcases.tmpl
similarity index 92%
rename from includes/testcases.md
rename to includes/testcases.tmpl
index ca07c42e1111f8bd998673f1b19d6057dc37869d..197eb659468c9c8af387463896ded551372ac9b0 100644
--- a/includes/testcases.md
+++ b/includes/testcases.tmpl
@@ -2,8 +2,8 @@
<h2>Testpages Filter Subscription</h2>
<p>This subscription includes all of the filters required to verify the following test cases.</p>
<p>You can add this subscription and then click through to each test case page and check that the page state matches the test case description. Alternatively you could add a test cases filter(s) manually in order to check it in isolation.</p>
- <a class="site-button" href="/abp-testcase-subscription.txt" target="_blank">View</a>
- <a id="subscribe-button" class="site-button" href="abp:subscribe?location=http%3A%2F%2Ftestpages.adblockplus.org%2Fabp-testcase-subscription.txt&amp;title=ABP%20Testcase%20Subscription">Subscribe</a>
+ <a class="site-button" href="/en/abp-testcase-subscription.txt" target="_blank">View</a>
+ <a id="subscribe-button" class="site-button" href="abp:subscribe?location={{ (site_url + "/en/abp-testcase-subscription.txt").replace(":", "%3A").replace("/", "%2F") }}&amp;title=ABP%20Testcase%20Subscription">Subscribe</a>
</section>
<section class="site-panel">
@@ -148,4 +148,4 @@
</a>
</li>
</ul>
-</section>
+</section>

Powered by Google App Engine
This is Rietveld