From 9ceb4ca05dc7b91d95ea69742111998bd9b594be Mon Sep 17 00:00:00 2001 From: Phill Price Date: Fri, 24 Oct 2014 14:08:02 +0100 Subject: [PATCH] Used 3x way to find if object exists. Split BlogHolder and BlogEntry record generation. --- code/BlogHolder.php | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/code/BlogHolder.php b/code/BlogHolder.php index 48edefc..547d5a3 100644 --- a/code/BlogHolder.php +++ b/code/BlogHolder.php @@ -124,9 +124,7 @@ class BlogHolder extends BlogTree implements PermissionProvider { // Skip creation of default records if(!self::config()->create_default_pages) return; - $blogHolder = DataObject::get_one('BlogHolder'); - //TODO: This does not check for whether this blogholder is an orphan or not - if(!$blogHolder) { + if(!BlogHolder::get()->exists()) { $blogholder = new BlogHolder(); $blogholder->Title = "Blog"; $blogholder->URLSegment = "blog"; @@ -165,17 +163,20 @@ class BlogHolder extends BlogTree implements PermissionProvider { } } + DB::alteration_message("Blog holder created","created"); + } + if(!BlogEntry::get()->exists()) { $blog = new BlogEntry(); $blog->Title = _t('BlogHolder.SUCTITLE', "SilverStripe blog module successfully installed"); $blog->URLSegment = 'sample-blog-entry'; $blog->Tags = _t('BlogHolder.SUCTAGS',"silverstripe, blog"); $blog->Content = _t('BlogHolder.SUCCONTENT',"

Congratulations, the SilverStripe blog module has been successfully installed. This blog entry can be safely deleted. You can configure aspects of your blog in the CMS.

"); $blog->Status = "Published"; - $blog->ParentID = $blogholder->ID; + $blog->ParentID = BlogHolder::get()->first()->ID; $blog->write(); $blog->publish("Stage", "Live"); - DB::alteration_message("Blog page created","created"); + DB::alteration_message("Blog entry created","created"); } }