Commit 509ce67d authored by Chris Graham's avatar Chris Graham
Browse files

Fixed MANTIS-4721 (Fix broken links)

parent 58513d0b

_tests/tests/unit_tests/special_links.php

+4 −4
Original line number Diff line number Diff line
@@ -32,7 +32,7 @@ class special_links_test_set extends cms_test_case 


    public function testLookupLinks()

    {

        $this->assertTrue(strpos(http_download_file('http://whatismyipaddress.com/ip/12.34.56.78', null, false), 'AT&T Services') !== false, 'External link not working, fix test and use within Composr (separate) [LOOKUP_SCREEN.tpl, COMMANDR_WHOIS.tpl]');

        $this->assertTrue(strpos(http_download_file('https://whatismyipaddress.com/ip/12.34.56.78', null, false), 'AT&T Services') !== false, 'External link not working, fix test and use within Composr (separate) [LOOKUP_SCREEN.tpl, COMMANDR_WHOIS.tpl]');

        $this->assertTrue(strpos(http_download_file('https://ping.eu/ping/?host=12.34.56.78', null, false), 'Ping') !== false, 'External link not working, fix test and use within Composr (separate) [LOOKUP_SCREEN.tpl, COMMANDR_WHOIS.tpl]');

        $this->assertTrue(strpos(http_download_file('https://ping.eu/traceroute/?host=12.34.56.78', null, false), 'Traceroute') !== false, 'External link not working, fix test and use within Composr (separate) [LOOKUP_SCREEN.tpl, COMMANDR_WHOIS.tpl]');

    }
@@ -52,12 +52,12 @@ class special_links_test_set extends cms_test_case 
            'https://www.authoritylabs.com/resources/' => true,

            'https://validator.w3.org/' => true,

            'https://jigsaw.w3.org/css-validator/' => true,

            'https://achecker.ca/checker/index.php' => true,

            'https://search.google.com/structured-data/testing-tool/' => true,

            'https://achecker.achecks.ca/checker/index.php' => true,

            'https://developers.google.com/search/docs/advanced/structured-data' => true,

            'https://developers.facebook.com/tools/debug/' => true,

            'https://www.woorank.com/' => true,

            'https://website.grader.com/' => true,

            'https://developers.google.com/speed/pagespeed/insights/' => true,

            'https://pagespeed.web.dev/' => true,

            'https://www.ssllabs.com/ssltest/' => true,

        );

        foreach ($urls as $url => $test_no_redirecting) {

sources_custom/hooks/systems/health_checks/mistakes_build.php

+2 −2
Original line number Diff line number Diff line
@@ -61,9 +61,9 @@ class Hook_health_check_mistakes_build extends Hook_Health_Check 


        $this->state_check_manual('Check [url="HTML5 validation"]https://validator.w3.org/[/url] (take warnings with a pinch of salt, not every suggestion is appropriate)');

        $this->state_check_manual('Check [url="CSS validation"]https://jigsaw.w3.org/css-validator/[/url] (take warnings with a pinch of salt, not every suggestion is appropriate)');

        $this->state_check_manual('Check [url="WCAG validation"]https://achecker.ca/checker/index.php[/url] (take warnings with a pinch of salt, not every suggestion is appropriate)');

        $this->state_check_manual('Check [url="WCAG validation"]https://achecker.achecks.ca/checker/index.php[/url] (take warnings with a pinch of salt, not every suggestion is appropriate)');



        $this->state_check_manual('Check [url="schema.org/microformats validation"]https://search.google.com/structured-data/testing-tool/[/url] on any key pages you want to be semantic');

        $this->state_check_manual('Check [url="schema.org/microformats validation"]https://developers.google.com/search/docs/advanced/structured-data[/url] on any key pages you want to be semantic');

        $this->state_check_manual('Check [url="OpenGraph metadata"]https://developers.facebook.com/tools/debug/[/url] on any key pages you expect to be shared');



        $this->state_check_manual('Do a [url="general check"]https://www.woorank.com/[/url] (take warnings with a pinch of salt, not every suggestion is appropriate)');

sources_custom/hooks/systems/health_checks/performance.php

+1 −1
Original line number Diff line number Diff line
@@ -62,7 +62,7 @@ class Hook_health_check_performance extends Hook_Health_Check 
            return;

        }



        $this->state_check_manual('Check for [url="speed issues"]https://developers.google.com/speed/pagespeed/insights/[/url] (take warnings with a pinch of salt, not every suggestion is appropriate)');

        $this->state_check_manual('Check for [url="speed issues"]https://pagespeed.web.dev/[/url] (take warnings with a pinch of salt, not every suggestion is appropriate)');

    }



    /**

themes/default/templates/COMMANDR_WHOIS.tpl

+1 −1
Original line number Diff line number Diff line
@@ -26,7 +26,7 @@ 
			<th>{!ACTIONS}</th>

			<td>

				<ul class="actions_list">

					<li><a href="http://whatismyipaddress.com/ip/{IP*}">Reverse-DNS/WHOIS/Geo-Lookup</a></li>

					<li><a href="https://whatismyipaddress.com/ip/{IP*}">Reverse-DNS/WHOIS/Geo-Lookup</a></li>

					<li><a href="https://ping.eu/ping/?host={IP*}">Ping</a></li>

					<li><a href="https://ping.eu/traceroute/?host={IP*}">Tracert</a></li>

				</ul>

themes/default/templates/LOOKUP_SCREEN.tpl

+1 −1
Original line number Diff line number Diff line
@@ -77,7 +77,7 @@ 
					<!-- If you like new windows, add this... title="{!LINK_NEW_WINDOW}" target="_blank"  -->

					<nav>

						<ul class="actions_list">

							<li><a rel="external" href="http://whatismyipaddress.com/ip/{IP*}">Reverse-DNS/WHOIS/Geo-Lookup</a></li>

							<li><a rel="external" href="https://whatismyipaddress.com/ip/{IP*}">Reverse-DNS/WHOIS/Geo-Lookup</a></li>

							<li><a rel="external" href="https://ping.eu/ping/?host={IP*}">Ping</a></li>

							<li><a rel="external" href="https://ping.eu/traceroute/?host={IP*}">Tracert</a></li>

						</ul>