Used 3x way to find if object exists. Split BlogHolder and BlogEntry record generation.

This commit is contained in:
Phill Price 2014-10-24 13:33:14 +01:00
parent 638b309b69
commit f811e7e69e

View File

@ -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,6 +163,9 @@ 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';
@ -175,7 +176,7 @@ class BlogHolder extends BlogTree implements PermissionProvider {
$blog->write();
$blog->publish("Stage", "Live");
DB::alteration_message("Blog page created","created");
DB::alteration_message("Blog entry created","created");
}
}