EnginesCheck::checkImplementation() - Code Metrics - Inspection of "Merge branch 'release/1.97.20'" - netz98/n98-magerun - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 36e8c5...2454a8 )

by Tom

created 2016-05-06 09:36 UTC

EnginesCheck::checkImplementation() A

↳ Parent: EnginesCheck

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	14
rs	9.4285
cc	2
eloc	9
nc	2
nop	2

<?php
/*
 * this file is part of magerun
 *
 * @author Tom Klingenberg <https://github.com/ktomk>
 */

namespace N98\Magento\Command\System\Check\MySQL;

use N98\Magento\Command\System\Check\Result;
use Varien_Db_Adapter_Interface;

class EnginesCheck extends ResourceCheck
{
    /**
     * @param Result $result
     * @param Varien_Db_Adapter_Interface $dbAdapter
     * @return void
     */
    protected function checkImplementation(Result $result, Varien_Db_Adapter_Interface $dbAdapter)
    {
        $innodbFound = $this->checkInnodbEngine($dbAdapter);

        if ($innodbFound) {
            $result->setStatus(Result::STATUS_OK);
            $result->setMessage("<info>Required MySQL Storage Engine <comment>InnoDB</comment> found.</info>");
        } else {
            $result->setStatus(Result::STATUS_ERROR);
            $result->setMessage(
                "<error>Required MySQL Storage Engine <comment>InnoDB</comment> not found!</error>"
            );
        }
    }

    /**
     * @param Varien_Db_Adapter_Interface $dbAdapter
     * @return bool
     */
    private function checkInnodbEngine(Varien_Db_Adapter_Interface $dbAdapter)
    {
        $innodbFound = false;

        $engines = $dbAdapter->fetchAll('SHOW ENGINES');

        foreach ($engines as $engine) {
            if (strtolower($engine['Engine']) === 'innodb') {
                $innodbFound = true;
                break;
            }
        }

        return $innodbFound;
    }
}


1			<?php
2			/*
3			* this file is part of magerun
4			*
5			* @author Tom Klingenberg <https://github.com/ktomk>
6			*/
7
8			namespace N98\Magento\Command\System\Check\MySQL;
9
10			use N98\Magento\Command\System\Check\Result;
11			use Varien_Db_Adapter_Interface;
12
13			class EnginesCheck extends ResourceCheck
14			{
15			/**
16			* @param Result $result
17			* @param Varien_Db_Adapter_Interface $dbAdapter
18			* @return void
19			*/
20			protected function checkImplementation(Result $result, Varien_Db_Adapter_Interface $dbAdapter)
21			{
22			$innodbFound = $this->checkInnodbEngine($dbAdapter);
23
24			if ($innodbFound) {
25			$result->setStatus(Result::STATUS_OK);
26			$result->setMessage("<info>Required MySQL Storage Engine <comment>InnoDB</comment> found.</info>");
27			} else {
28			$result->setStatus(Result::STATUS_ERROR);
29			$result->setMessage(
30			"<error>Required MySQL Storage Engine <comment>InnoDB</comment> not found!</error>"
31			);
32			}
33			}
34
35			/**
36			* @param Varien_Db_Adapter_Interface $dbAdapter
37			* @return bool
38			*/
39			private function checkInnodbEngine(Varien_Db_Adapter_Interface $dbAdapter)
40			{
41			$innodbFound = false;
42
43			$engines = $dbAdapter->fetchAll('SHOW ENGINES');
44
45			foreach ($engines as $engine) {
46			if (strtolower($engine['Engine']) === 'innodb') {
47			$innodbFound = true;
48			break;
49			}
50			}
51
52			return $innodbFound;
53			}
54			}
55

netz98 / n98-magerun

Push — master ( 36e8c5...2454a8 )

EnginesCheck::checkImplementation() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like