2011-03-18 04:23:47 +01:00
|
|
|
<?php
|
2016-06-16 06:57:19 +02:00
|
|
|
|
2017-08-09 04:53:38 +02:00
|
|
|
namespace SilverStripe\CMS\Tests\Model;
|
2017-08-09 03:25:12 +02:00
|
|
|
|
2017-02-28 03:46:19 +01:00
|
|
|
use SilverStripe\CMS\Model\SiteTree;
|
2017-03-21 05:26:46 +01:00
|
|
|
use SilverStripe\Versioned\Versioned;
|
2016-06-16 06:57:19 +02:00
|
|
|
use SilverStripe\ORM\DataObject;
|
|
|
|
use SilverStripe\ORM\DB;
|
2016-07-22 01:32:32 +02:00
|
|
|
use SilverStripe\CMS\Model\VirtualPage;
|
|
|
|
use SilverStripe\CMS\Model\RedirectorPage;
|
2016-08-23 04:36:06 +02:00
|
|
|
use SilverStripe\Assets\File;
|
|
|
|
use SilverStripe\Dev\SapphireTest;
|
2016-11-15 06:06:31 +01:00
|
|
|
use SilverStripe\Assets\Tests\Storage\AssetStoreTest\TestAssetStore;
|
2017-08-09 03:25:12 +02:00
|
|
|
use Page;
|
|
|
|
|
2011-03-18 04:23:47 +01:00
|
|
|
/**
|
2016-01-26 06:38:42 +01:00
|
|
|
* Tests {@see SiteTreeLinkTracking} broken links feature: LinkTracking
|
2011-03-18 04:23:47 +01:00
|
|
|
*/
|
2017-01-25 21:59:25 +01:00
|
|
|
class SiteTreeBrokenLinksTest extends SapphireTest
|
|
|
|
{
|
|
|
|
protected static $fixture_file = 'SiteTreeBrokenLinksTest.yml';
|
|
|
|
|
|
|
|
public function setUp()
|
|
|
|
{
|
|
|
|
parent::setUp();
|
|
|
|
|
|
|
|
Versioned::set_stage(Versioned::DRAFT);
|
|
|
|
TestAssetStore::activate('SiteTreeBrokenLinksTest');
|
|
|
|
$this->logInWithPermission('ADMIN');
|
|
|
|
}
|
|
|
|
|
|
|
|
public function tearDown()
|
|
|
|
{
|
|
|
|
TestAssetStore::reset();
|
|
|
|
parent::tearDown();
|
|
|
|
}
|
|
|
|
|
|
|
|
public function testBrokenLinksBetweenPages()
|
|
|
|
{
|
2017-08-09 00:56:08 +02:00
|
|
|
/** @var Page $obj */
|
2017-01-25 21:59:25 +01:00
|
|
|
$obj = $this->objFromFixture('Page', 'content');
|
|
|
|
|
|
|
|
$obj->Content = '<a href="[sitetree_link,id=3423423]">this is a broken link</a>';
|
|
|
|
$obj->syncLinkTracking();
|
|
|
|
$this->assertTrue($obj->HasBrokenLink, 'Page has a broken link');
|
|
|
|
|
|
|
|
$obj->Content = '<a href="[sitetree_link,id=' . $this->idFromFixture('Page', 'about') .']">this is not a broken link</a>';
|
|
|
|
$obj->syncLinkTracking();
|
|
|
|
$this->assertFalse($obj->HasBrokenLink, 'Page does NOT have a broken link');
|
|
|
|
}
|
|
|
|
|
|
|
|
public function testBrokenAnchorBetweenPages()
|
|
|
|
{
|
2017-08-09 00:56:08 +02:00
|
|
|
/** @var Page $obj */
|
2017-01-25 21:59:25 +01:00
|
|
|
$obj = $this->objFromFixture('Page', 'content');
|
|
|
|
$target = $this->objFromFixture('Page', 'about');
|
|
|
|
|
|
|
|
$obj->Content = "<a href=\"[sitetree_link,id={$target->ID}]#no-anchor-here\">this is a broken link</a>";
|
|
|
|
$obj->syncLinkTracking();
|
|
|
|
$this->assertTrue($obj->HasBrokenLink, 'Page has a broken link');
|
|
|
|
|
|
|
|
$obj->Content = "<a href=\"[sitetree_link,id={$target->ID}]#yes-anchor-here\">this is not a broken link</a>";
|
|
|
|
$obj->syncLinkTracking();
|
|
|
|
$this->assertFalse($obj->HasBrokenLink, 'Page does NOT have a broken link');
|
|
|
|
}
|
|
|
|
|
|
|
|
public function testBrokenVirtualPages()
|
|
|
|
{
|
|
|
|
$obj = $this->objFromFixture('Page', 'content');
|
|
|
|
$vp = new VirtualPage();
|
|
|
|
|
|
|
|
$vp->CopyContentFromID = $obj->ID;
|
|
|
|
$vp->syncLinkTracking();
|
|
|
|
$this->assertFalse($vp->HasBrokenLink, 'Working virtual page is NOT marked as broken');
|
|
|
|
|
|
|
|
$vp->CopyContentFromID = 12345678;
|
|
|
|
$vp->syncLinkTracking();
|
|
|
|
$this->assertTrue($vp->HasBrokenLink, 'Broken virtual page IS marked as such');
|
|
|
|
}
|
|
|
|
|
|
|
|
public function testBrokenInternalRedirectorPages()
|
|
|
|
{
|
|
|
|
$obj = $this->objFromFixture('Page', 'content');
|
|
|
|
$rp = new RedirectorPage();
|
|
|
|
|
|
|
|
$rp->RedirectionType = 'Internal';
|
|
|
|
|
|
|
|
$rp->LinkToID = $obj->ID;
|
|
|
|
$rp->syncLinkTracking();
|
|
|
|
$this->assertFalse($rp->HasBrokenLink, 'Working redirector page is NOT marked as broken');
|
|
|
|
|
|
|
|
$rp->LinkToID = 12345678;
|
|
|
|
$rp->syncLinkTracking();
|
|
|
|
$this->assertTrue($rp->HasBrokenLink, 'Broken redirector page IS marked as such');
|
|
|
|
}
|
|
|
|
|
|
|
|
public function testDeletingFileMarksBackedPagesAsBroken()
|
|
|
|
{
|
|
|
|
// Test entry
|
|
|
|
$file = new File();
|
|
|
|
$file->setFromString('test', 'test-file.txt');
|
|
|
|
$file->write();
|
|
|
|
|
2017-02-28 03:46:19 +01:00
|
|
|
/** @var Page $obj */
|
2017-01-25 21:59:25 +01:00
|
|
|
$obj = $this->objFromFixture('Page', 'content');
|
|
|
|
$obj->Content = sprintf(
|
|
|
|
'<p><a href="[file_link,id=%d]">Working Link</a></p>',
|
|
|
|
$file->ID
|
|
|
|
);
|
|
|
|
$obj->write();
|
|
|
|
$this->assertTrue($obj->publishRecursive());
|
|
|
|
// Confirm that it isn't marked as broken to begin with
|
2017-02-28 03:46:19 +01:00
|
|
|
|
|
|
|
$obj = SiteTree::get()->byID($obj->ID);
|
2017-01-25 21:59:25 +01:00
|
|
|
$this->assertEquals(0, $obj->HasBrokenFile);
|
|
|
|
|
2017-02-28 03:46:19 +01:00
|
|
|
$liveObj = Versioned::get_one_by_stage(SiteTree::class, Versioned::LIVE, "\"SiteTree\".\"ID\" = $obj->ID");
|
2017-01-25 21:59:25 +01:00
|
|
|
$this->assertEquals(0, $liveObj->HasBrokenFile);
|
|
|
|
|
|
|
|
// Delete the file
|
|
|
|
$file->delete();
|
|
|
|
|
|
|
|
// Confirm that it is marked as broken in stage
|
2017-02-28 03:46:19 +01:00
|
|
|
$obj = SiteTree::get()->byID($obj->ID);
|
2017-01-25 21:59:25 +01:00
|
|
|
$this->assertEquals(1, $obj->HasBrokenFile);
|
|
|
|
|
|
|
|
// Publishing this page marks it as broken on live too
|
|
|
|
$obj->publishRecursive();
|
2017-02-28 03:46:19 +01:00
|
|
|
$liveObj = Versioned::get_one_by_stage(SiteTree::class, Versioned::LIVE, "\"SiteTree\".\"ID\" = $obj->ID");
|
2017-01-25 21:59:25 +01:00
|
|
|
$this->assertEquals(1, $liveObj->HasBrokenFile);
|
|
|
|
}
|
|
|
|
|
|
|
|
public function testDeletingMarksBackLinkedPagesAsBroken()
|
|
|
|
{
|
|
|
|
// Set up two published pages with a link from content -> about
|
|
|
|
$linkDest = $this->objFromFixture('Page', 'about');
|
|
|
|
|
|
|
|
$linkSrc = $this->objFromFixture('Page', 'content');
|
|
|
|
$linkSrc->Content = "<p><a href=\"[sitetree_link,id=$linkDest->ID]\">about us</a></p>";
|
|
|
|
$linkSrc->write();
|
|
|
|
|
|
|
|
// Confirm no broken link
|
|
|
|
$this->assertEquals(0, (int)$linkSrc->HasBrokenLink);
|
|
|
|
|
|
|
|
// Delete page from draft
|
|
|
|
$linkDest->delete();
|
|
|
|
|
|
|
|
// Confirm draft has broken link
|
|
|
|
$linkSrc->flushCache();
|
|
|
|
$linkSrc = $this->objFromFixture('Page', 'content');
|
|
|
|
|
|
|
|
$this->assertEquals(1, (int)$linkSrc->HasBrokenLink);
|
|
|
|
}
|
|
|
|
|
|
|
|
public function testPublishingSourceBeforeDestHasBrokenLink()
|
|
|
|
{
|
|
|
|
$this->logInWithPermission('ADMIN');
|
|
|
|
|
|
|
|
// Set up two draft pages with a link from content -> about
|
2017-08-09 00:56:08 +02:00
|
|
|
/** @var Page $linkDest */
|
2017-01-25 21:59:25 +01:00
|
|
|
$linkDest = $this->objFromFixture('Page', 'about');
|
|
|
|
// Ensure that it's not on the published site
|
|
|
|
$linkDest->doUnpublish();
|
|
|
|
|
2017-08-09 00:56:08 +02:00
|
|
|
/** @var Page $linkSrc */
|
2017-01-25 21:59:25 +01:00
|
|
|
$linkSrc = $this->objFromFixture('Page', 'content');
|
|
|
|
$linkSrc->Content = "<p><a href=\"[sitetree_link,id=$linkDest->ID]\">about us</a></p>";
|
|
|
|
$linkSrc->write();
|
|
|
|
|
|
|
|
// Publish the source of the link, while the dest is still unpublished.
|
|
|
|
$linkSrc->publishRecursive();
|
|
|
|
|
2017-06-21 06:29:40 +02:00
|
|
|
// Verify that the link is not marked as broken on draft (source of truth)
|
2017-01-25 21:59:25 +01:00
|
|
|
$this->assertEquals(0, (int)$linkSrc->HasBrokenLink);
|
2017-06-21 06:29:40 +02:00
|
|
|
|
|
|
|
// Live doesn't have separate broken link tracking
|
|
|
|
$this->assertEquals(0, DB::query("SELECT \"HasBrokenLink\" FROM \"SiteTree_Live\"
|
2011-03-18 04:23:47 +01:00
|
|
|
WHERE \"ID\" = $linkSrc->ID")->value());
|
2017-01-25 21:59:25 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
public function testRestoreFixesBrokenLinks()
|
|
|
|
{
|
|
|
|
// Create page and virtual page
|
|
|
|
$p = new Page();
|
|
|
|
$p->Title = "source";
|
|
|
|
$p->write();
|
|
|
|
$pageID = $p->ID;
|
|
|
|
$this->assertTrue($p->publishRecursive());
|
|
|
|
|
|
|
|
// Content links are one kind of link to pages
|
|
|
|
$p2 = new Page();
|
|
|
|
$p2->Title = "regular link";
|
|
|
|
$p2->Content = "<a href=\"[sitetree_link,id=$p->ID]\">test</a>";
|
|
|
|
$p2->write();
|
|
|
|
$this->assertTrue($p2->publishRecursive());
|
|
|
|
|
|
|
|
// Redirector links are a third
|
|
|
|
$rp = new RedirectorPage();
|
|
|
|
$rp->Title = "redirector";
|
|
|
|
$rp->LinkType = 'Internal';
|
|
|
|
$rp->LinkToID = $p->ID;
|
|
|
|
$rp->write();
|
|
|
|
$this->assertTrue($rp->publishRecursive());
|
|
|
|
|
|
|
|
// Confirm that there are no broken links to begin with
|
|
|
|
$this->assertFalse($p2->HasBrokenLink);
|
|
|
|
$this->assertFalse($rp->HasBrokenLink);
|
|
|
|
|
2017-06-21 06:29:40 +02:00
|
|
|
// Unpublishing doesn't affect broken state on live (draft is source of truth)
|
2017-01-25 21:59:25 +01:00
|
|
|
$p->doUnpublish();
|
2017-06-21 06:29:40 +02:00
|
|
|
$p2Live = Versioned::get_one_by_stage(SiteTree::class, 'Live', '"SiteTree"."ID" = ' . $p2->ID);
|
|
|
|
$rpLive = Versioned::get_one_by_stage(SiteTree::class, 'Live', '"SiteTree"."ID" = ' . $rp->ID);
|
|
|
|
$this->assertEquals(0, $p2Live->HasBrokenLink);
|
|
|
|
$this->assertEquals(0, $rpLive->HasBrokenLink);
|
2017-01-25 21:59:25 +01:00
|
|
|
|
|
|
|
// Delete the source page, confirm that the VP, RP and page 2 have broken links on draft
|
|
|
|
$p->delete();
|
|
|
|
$p2->flushCache();
|
2017-06-21 06:29:40 +02:00
|
|
|
$p2 = DataObject::get_by_id(SiteTree::class, $p2->ID);
|
2017-01-25 21:59:25 +01:00
|
|
|
$rp->flushCache();
|
2017-06-21 06:29:40 +02:00
|
|
|
$rp = DataObject::get_by_id(SiteTree::class, $rp->ID);
|
2017-01-25 21:59:25 +01:00
|
|
|
$this->assertEquals(1, $p2->HasBrokenLink);
|
|
|
|
$this->assertEquals(1, $rp->HasBrokenLink);
|
|
|
|
|
|
|
|
// Restore the page to stage, confirm that this fixes the links
|
2017-06-21 06:29:40 +02:00
|
|
|
/** @var SiteTree $p */
|
|
|
|
$p = Versioned::get_latest_version(SiteTree::class, $pageID);
|
2017-01-25 21:59:25 +01:00
|
|
|
$p->doRestoreToStage();
|
|
|
|
|
|
|
|
$p2->flushCache();
|
2017-06-21 06:29:40 +02:00
|
|
|
$p2 = DataObject::get_by_id(SiteTree::class, $p2->ID);
|
2017-01-25 21:59:25 +01:00
|
|
|
$rp->flushCache();
|
2017-06-21 06:29:40 +02:00
|
|
|
$rp = DataObject::get_by_id(SiteTree::class, $rp->ID);
|
2017-01-25 21:59:25 +01:00
|
|
|
$this->assertFalse((bool)$p2->HasBrokenLink);
|
|
|
|
$this->assertFalse((bool)$rp->HasBrokenLink);
|
|
|
|
|
|
|
|
// Publish and confirm that the p2 and RP broken links are fixed on published
|
|
|
|
$this->assertTrue($p->publishRecursive());
|
2017-06-21 06:29:40 +02:00
|
|
|
$p2Live = Versioned::get_one_by_stage(SiteTree::class, 'Live', '"SiteTree"."ID" = ' . $p2->ID);
|
|
|
|
$rpLive = Versioned::get_one_by_stage(SiteTree::class, 'Live', '"SiteTree"."ID" = ' . $rp->ID);
|
2017-01-25 21:59:25 +01:00
|
|
|
$this->assertFalse((bool)$p2Live->HasBrokenLink);
|
|
|
|
$this->assertFalse((bool)$rpLive->HasBrokenLink);
|
|
|
|
}
|
|
|
|
|
|
|
|
public function testRevertToLiveFixesBrokenLinks()
|
|
|
|
{
|
|
|
|
// Create page and virutal page
|
2017-08-09 00:56:08 +02:00
|
|
|
$page = new Page();
|
|
|
|
$page->Title = "source";
|
|
|
|
$page->write();
|
|
|
|
$pageID = $page->ID;
|
|
|
|
$this->assertTrue($page->publishRecursive());
|
2017-01-25 21:59:25 +01:00
|
|
|
|
|
|
|
// Content links are one kind of link to pages
|
2017-08-09 00:56:08 +02:00
|
|
|
$page2 = new Page();
|
|
|
|
$page2->Title = "regular link";
|
|
|
|
$page2->Content = "<a href=\"[sitetree_link,id={$pageID}]\">test</a>";
|
|
|
|
$page2->write();
|
|
|
|
$this->assertTrue($page2->publishRecursive());
|
2017-01-25 21:59:25 +01:00
|
|
|
|
|
|
|
// Redirector links are a third
|
2017-08-09 00:56:08 +02:00
|
|
|
$redirectorPage = new RedirectorPage();
|
|
|
|
$redirectorPage->Title = "redirector";
|
|
|
|
$redirectorPage->LinkType = 'Internal';
|
|
|
|
$redirectorPage->LinkToID = $page->ID;
|
|
|
|
$redirectorPage->write();
|
|
|
|
$this->assertTrue($redirectorPage->publishRecursive());
|
2017-01-25 21:59:25 +01:00
|
|
|
|
|
|
|
// Confirm that there are no broken links to begin with
|
2017-08-09 00:56:08 +02:00
|
|
|
$this->assertFalse($page2->HasBrokenLink);
|
|
|
|
$this->assertFalse($redirectorPage->HasBrokenLink);
|
2017-01-25 21:59:25 +01:00
|
|
|
|
|
|
|
// Delete from draft and confirm that broken links are marked
|
2017-08-09 00:56:08 +02:00
|
|
|
$page->delete();
|
2017-01-25 21:59:25 +01:00
|
|
|
|
2017-08-09 00:56:08 +02:00
|
|
|
$page2->flushCache();
|
|
|
|
$page2 = DataObject::get_by_id(SiteTree::class, $page2->ID);
|
|
|
|
$redirectorPage->flushCache();
|
|
|
|
$redirectorPage = DataObject::get_by_id(SiteTree::class, $redirectorPage->ID);
|
|
|
|
$this->assertEquals(1, $page2->HasBrokenLink);
|
|
|
|
$this->assertEquals(1, $redirectorPage->HasBrokenLink);
|
2017-01-25 21:59:25 +01:00
|
|
|
|
|
|
|
// Call doRevertToLive and confirm that broken links are restored
|
2017-08-09 00:56:08 +02:00
|
|
|
/** @var Page $pageLive */
|
|
|
|
$pageLive = Versioned::get_one_by_stage(SiteTree::class, 'Live', '"SiteTree"."ID" = ' . $pageID);
|
|
|
|
$pageLive->doRevertToLive();
|
|
|
|
|
|
|
|
$page2->flushCache();
|
|
|
|
$page2 = DataObject::get_by_id(SiteTree::class, $page2->ID);
|
|
|
|
$redirectorPage->flushCache();
|
|
|
|
$redirectorPage = DataObject::get_by_id(SiteTree::class, $redirectorPage->ID);
|
|
|
|
$this->assertFalse((bool)$page2->HasBrokenLink);
|
|
|
|
$this->assertFalse((bool)$redirectorPage->HasBrokenLink);
|
2017-01-25 21:59:25 +01:00
|
|
|
}
|
|
|
|
|
|
|
|
public function testBrokenAnchorLinksInAPage()
|
|
|
|
{
|
2017-08-09 00:56:08 +02:00
|
|
|
/** @var Page $obj */
|
2017-01-25 21:59:25 +01:00
|
|
|
$obj = $this->objFromFixture('Page', 'content');
|
|
|
|
$origContent = $obj->Content;
|
|
|
|
|
|
|
|
$obj->Content = $origContent . '<a href="#no-anchor-here">this links to a non-existent in-page anchor or skiplink</a>';
|
|
|
|
$obj->syncLinkTracking();
|
|
|
|
$this->assertTrue($obj->HasBrokenLink, 'Page has a broken anchor/skiplink');
|
|
|
|
|
|
|
|
$obj->Content = $origContent . '<a href="#yes-anchor-here">this links to an existent in-page anchor/skiplink</a>';
|
|
|
|
$obj->syncLinkTracking();
|
|
|
|
$this->assertFalse($obj->HasBrokenLink, 'Page doesn\'t have a broken anchor or skiplink');
|
|
|
|
}
|
2011-03-18 04:23:47 +01:00
|
|
|
}
|