LeKoala\Blocks\BlocksLeftAndMainExtension

Complexity

Total Complexity

9

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	doPublishBlocks()	0	51	8
A	init()	0	2	1

<?php

namespace LeKoala\Blocks;

use SilverStripe\Admin\LeftAndMainExtension;
use SilverStripe\ORM\DB;

/**
 * Extend leftandmain to publish blocks
 *
 * @property \SilverStripe\Admin\LeftAndMain|\SilverStripe\Admin\ModelAdmin $owner
 */
class BlocksLeftAndMainExtension extends LeftAndMainExtension
{
    private static $allowed_actions = [

The private property $allowed_actions is not used, and could be removed.

        'doPublishBlocks'
    ];

    public function doPublishBlocks()
    {
        $owner = $this->owner;
        $ID = $owner->getRequest()->param('ID');
        $Page = BlocksPage::get()->byID($ID);

$ID of type string is incompatible with the type integer expected by parameter $id of SilverStripe\ORM\DataList::byID().

        Block::$auto_update_page = false;

        $DataCopy = [];
        $Blocks = $Page->Blocks();
        foreach ($Blocks as $Block) {
            $DataCopy[$Block->ID] = DB::query('SELECT BlockData FROM Block WHERE ID = ' . $Block->ID)->value();
            if (!$Block->BlockData) {
                $Block->BlockData = $DataCopy[$Block->ID];
            }
            $Block->write();
            // DB::prepared_query('UPDATE Block SET BlockData = ? WHERE ID = ' . $Block->ID, [$DataCopy[$Block->ID]]);
        }
        $Page->publishRecursive();

        // Publish in all other locales as well!
        if ($Page->has_extension("\\TractorCow\\Fluent\\Extension\\FluentExtension")) {
            $state = \TractorCow\Fluent\State\FluentState::singleton();

The type TractorCow\Fluent\State\FluentState was not found. Maybe you did not declare it correctly or list all dependencies?

            $currentLocale = $state->getLocale();
            $allLocales = \TractorCow\Fluent\Model\Locale::get()->exclude('Locale', $currentLocale);

The type TractorCow\Fluent\Model\Locale was not found. Maybe you did not declare it correctly or list all dependencies?

            foreach ($allLocales as $locale) {
                $state->withState(function ($state) use ($locale, $Page, $DataCopy) {
                    $state->setLocale($locale->Locale);

                    foreach ($Page->Blocks() as $Block) {
                        if (!$Block->BlockData) {
                            $Block->BlockData = $DataCopy[$Block->ID];
                        }
                        $Block->write();
                    }

                    $Page->publishRecursive();
                });
            }

            // Preserve original data
            // TODO: understand why Data is emptied
            foreach ($Blocks as $Block) {
                DB::prepared_query('UPDATE Block SET BlockData = ? WHERE ID = ' . $Block->ID, [$DataCopy[$Block->ID]]);
            }
        }

        $message = "Blocks published";

        $owner->getResponse()->addHeader('X-Status', rawurlencode($message));
        return $owner->getResponseNegotiator()->respond($owner->getRequest());
    }

    public function init()
    {
    }
}