From 59b7d147205cf225068d2964f6f01b430c37a7d5 Mon Sep 17 00:00:00 2001
From: Ingo Schommer <ingo@silverstripe.com>
Date: Tue, 16 Feb 2010 01:16:04 +0000
Subject: [PATCH] MINOR more unit tests around loose hierarchy (from r98509)

git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/branches/2.4@99062 467b73ca-7a2a-4603-9d3b-597d59a354a9
---
 core/model/SiteTree.php |  1 -
 tests/SiteTreeTest.php  | 23 +++++++++++++++++++++++
 2 files changed, 23 insertions(+), 1 deletion(-)

diff --git a/core/model/SiteTree.php b/core/model/SiteTree.php
index 090f5a7fe..883442f53 100755
--- a/core/model/SiteTree.php
+++ b/core/model/SiteTree.php
@@ -1398,7 +1398,6 @@ class SiteTree extends DataObject implements PermissionProvider,i18nEntityProvid
 		
 		// If deleting this page, delete all its children.
 		if(SiteTree::enforce_strict_hierarchy() && $children = $this->Children()) {
-		// if($children = $this->Children()) {
 			foreach($children as $child) {
 				$child->delete();
 			}
diff --git a/tests/SiteTreeTest.php b/tests/SiteTreeTest.php
index 50adb1cf3..4759d35b8 100755
--- a/tests/SiteTreeTest.php
+++ b/tests/SiteTreeTest.php
@@ -354,6 +354,29 @@ class SiteTreeTest extends SapphireTest {
 		SiteTree::enforce_strict_hierarchy(true);
 	}
 	
+	function testUnpublishDoesNotDeleteChildrenWithLooseHierachyOn() {
+		SiteTree::enforce_strict_hierarchy(false);
+		$this->logInWithPermssion('ADMIN');
+		
+		$pageAbout = $this->objFromFixture('Page', 'about');
+		$pageAbout->doPublish();
+		$pageStaff = $this->objFromFixture('Page', 'staff');
+		$pageStaff->doPublish();
+		$pageStaffDuplicate = $this->objFromFixture('Page', 'staffduplicate');
+		$pageStaffDuplicate->doPublish();
+		
+		$parentPage = $this->objFromFixture('Page', 'about');
+		$parentPage->doUnpublish();
+		
+		Versioned::reading_stage('Live');
+		$this->assertFalse(DataObject::get_by_id('Page', $pageAbout->ID));
+		$this->assertTrue(DataObject::get_by_id('Page', $pageStaff->ID) instanceof Page);
+		$this->assertTrue(DataObject::get_by_id('Page', $pageStaffDuplicate->ID) instanceof Page);
+		Versioned::reading_stage('Stage');
+		SiteTree::enforce_strict_hierarchy(true);
+	}
+	
+	
 	function testDeleteFromLiveOperatesRecursivelyStrict() {
 		$this->logInWithPermssion('ADMIN');