dynamic / silverstripe-flexslider

src/Task/DefaultSlideTypeTask.php

<?php

namespace Dynamic\Flexslider\Task;

use Dynamic\FlexSlider\Model\SlideImage;
use SilverStripe\Dev\BuildTask;
use SilverStripe\ORM\DB;

/**
 * Class DefaultSlideTypeTask
 * @package Dynamic\Flexslider\Task
 */
class DefaultSlideTypeTask extends BuildTask
{
    /**
     * @var string
     */
    private static $segment = 'default-slide-type-task';

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

    /**
     * @var string
     */
    protected $title = 'Flexslider - Default Slide Type Task';

    /**
     * @param \SilverStripe\Control\HTTPRequest $request
     */
    public function run($request)
    {
        $this->setDefaults();
    }

    /**
     *
     */
    protected function setDefaults()
    {
        $default = SlideImage::singleton()->config()->get('defaults');

        if (isset($default['SlideType'])) {
            $baseTable = SlideImage::singleton()->baseTable();

            $tables = [
                $baseTable,
                "{$baseTable}_Versions",
                "{$baseTable}_Live",
            ];

            foreach ($tables as $table) {
                $query = DB::query("SELECT * FROM \"{$table}\" WHERE \"SlideType\" IS NULL");

                foreach ($this->yieldSingle($query) as $record) {
                    DB::prepared_query(
                        "UPDATE \"{$table}\" SET \"SlideType\" = ? WHERE \"ID\" = ?",
                        [$default['SlideType'], $record['ID']]
                    );
                }
            }
        }
    }

    /**
     * @param $list
     * @return \Generator
     */
    protected function yieldSingle($list)
    {
        foreach ($list as $item) {
            yield $item;
        }
    }
}