ManyImageRelationTask::yieldProductImages() - Code Metrics - Inspection of "Merge pull request #26 from muskie9/pull/productVi..." - dynamic/silverstripe-products - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 4099fa...f02648 )

by Nic

created 2019-12-11 19:39 UTC

ManyImageRelationTask::yieldProductImages() A

↳ Parent: ManyImageRelationTask

Complexity

Conditions	2
Paths	2

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	2
c	1
b	0
f	0
dl	0
loc	4
rs	10
cc	2
nc	2
nop	0

<?php

namespace Dynamic\Products\Task;

use SilverStripe\Dev\BuildTask;
use SilverStripe\ORM\DB;

/**
 * Class ManyImageRelationTask
 * @package Dynamic\Products\Task
 */
class ManyImageRelationTask extends BuildTask
{
    /**
     * @var string
     */
    protected $title = 'Product Image Relation Update Task';

    /**
     * @var string
     */
    private static $segment = 'product-image-relation-update-task';


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

    /**
     *
     */
    protected function updateRelationTable()
    {
        foreach ($this->yieldProductImages() as $record) {
            DB::prepared_query(
                'UPDATE `Product_Images` SET `FileID` = ? WHERE `ID` = ?',
                [$record['ImageID'], $record['ID']]
            );
        }
    }

    /**
     * @return \Generator
     */
    protected function yieldProductImages()
    {
        foreach (DB::query('SELECT * FROM `Product_Images`') as $record) {
            yield $record;
        }
    }
}


1			<?php
2
3			namespace Dynamic\Products\Task;
4
5			use SilverStripe\Dev\BuildTask;
6			use SilverStripe\ORM\DB;
7
8			/**
9			* Class ManyImageRelationTask
10			* @package Dynamic\Products\Task
11			*/
12			class ManyImageRelationTask extends BuildTask
13			{
14			/**
15			* @var string
16			*/
17			protected $title = 'Product Image Relation Update Task';
18
19			/**
20			* @var string
21			*/
22			private static $segment = 'product-image-relation-update-task';
			0 ignored issues – show introduced 2019-12-09 19:03 UTC by Report Bug Copy Issue Report The private property `$segment` is not used, and could be removed. Loading history...
23
24			/**
25			* @param \SilverStripe\Control\HTTPRequest $request
26			*/
27			public function run($request)
28			{
29			$this->updateRelationTable();
30			}
31
32			/**
33			*
34			*/
35			protected function updateRelationTable()
36			{
37			foreach ($this->yieldProductImages() as $record) {
38			DB::prepared_query(
39			'UPDATE `Product_Images` SET `FileID` = ? WHERE `ID` = ?',
40			[$record['ImageID'], $record['ID']]
41			);
42			}
43			}
44
45			/**
46			* @return \Generator
47			*/
48			protected function yieldProductImages()
49			{
50			foreach (DB::query('SELECT * FROM `Product_Images`') as $record) {
51			yield $record;
52			}
53			}
54			}
55

dynamic / silverstripe-products

Push — master ( 4099fa...f02648 )

ManyImageRelationTask::yieldProductImages() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like