MailerModuleInstaller::upgrade() - Code Metrics - Inspection of "Remove mailerapi" - zikula/core - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Pull Request — master (#4187)

by Craig

created 2020-03-28 18:01 UTC

MailerModuleInstaller::upgrade() B

↳ Parent: MailerModuleInstaller

Complexity

Conditions	10
Paths	28

Size

Total Lines	35
Code Lines	22

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	10
eloc	22
nc	28
nop	1
dl	0
loc	35
rs	7.6666
c	0
b	0
f	0

How to fix Complexity

<?php

declare(strict_types=1);

/*
 * This file is part of the Zikula package.
 *
 * Copyright Zikula Foundation - https://ziku.la/
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Zikula\MailerModule;

use Doctrine\Persistence\ManagerRegistry;
use Symfony\Component\HttpFoundation\RequestStack;
use Symfony\Contracts\Translation\TranslatorInterface;
use Zikula\Bundle\CoreBundle\Doctrine\Helper\SchemaHelper;
use Zikula\Bundle\CoreBundle\DynamicConfigDumper;
use Zikula\Bundle\CoreBundle\HttpKernel\ZikulaHttpKernelInterface;
use Zikula\ExtensionsModule\AbstractExtension;
use Zikula\ExtensionsModule\Api\ApiInterface\VariableApiInterface;
use Zikula\ExtensionsModule\Entity\ExtensionVarEntity;
use Zikula\ExtensionsModule\Installer\AbstractExtensionInstaller;

/**
 * Installation and upgrade routines for the mailer module.
 */
class MailerModuleInstaller extends AbstractExtensionInstaller
{
    /**
     * @var ZikulaHttpKernelInterface
     */
    private $kernel;

    /**
     * @var DynamicConfigDumper
     */
    private $configDumper;

    public function __construct(
        ZikulaHttpKernelInterface $kernel,
        DynamicConfigDumper $configDumper,
        AbstractExtension $extension,
        ManagerRegistry $managerRegistry,
        SchemaHelper $schemaTool,
        RequestStack $requestStack,
        TranslatorInterface $translator,
        VariableApiInterface $variableApi
    ) {
        $this->kernel = $kernel;
        $this->configDumper = $configDumper;
        parent::__construct($extension, $managerRegistry, $schemaTool, $requestStack, $translator, $variableApi);
    }

    public function install(): bool
    {
        $this->setVars($this->getDefaults());

        // Initialisation successful
        return true;
    }

    public function upgrade(string $oldVersion): bool
    {
        // Upgrade dependent on old version number
        switch ($oldVersion) {
            case '1.3.1':
                $this->setVar('smtpsecuremethod', 'ssl');

            case '1.3.2':
                // clear old modvars
                // use manual method because getVars() is not available during system upgrade
                $modVarEntities = $this->managerRegistry->getRepository(ExtensionVarEntity::class)->findBy(['modname' => $this->name]);
                $modVars = [];
                foreach ($modVarEntities as $var) {
                    $modVars[$var['name']] = $var['value'];
                }
                $this->delVars();
                $this->setVarWithDefault('charset', $modVars['charset']);
                $this->setVarWithDefault('encoding', $modVars['encoding']);
                $this->setVarWithDefault('html', $modVars['html']);
                $this->setVarWithDefault('wordwrap', $modVars['wordwrap']);
                // new modvar for 1.4.0
                $this->setVarWithDefault('enableLogging', false);

            case '1.4.0':
            case '1.4.1':
            case '1.4.2':
            case '1.4.3':
            case '1.5.0':
            case '1.5.1':
                // all swiftmailer config changes removed from previous version upgrades above
                $this->configDumper->delConfiguration('swiftmailer');
                // future upgrade routines
        }

        // Update successful
        return true;
    }

    public function uninstall(): bool
    {
        // Delete any module variables
        $this->delVars();

        // Deletion successful
        return true;
    }

    /**
     * Default module vars.
     */
    private function getDefaults(): array
    {
        return [
            'charset' => $this->kernel->getCharset(),
            'encoding' => '8bit',
            'html' => false,
            'wordwrap' => 50,
            'enableLogging' => false
        ];
    }

    /**
     * Set the module var but if it is not set, use the default instead.
     *
     * @param mixed $value
     */
    private function setVarWithDefault(string $key, $value = null): void
    {
        if (isset($value)) {
            $this->setVar($key, $value);
        }
        $defaults = $this->getDefaults();
        $this->setVar($key, $defaults[$key]);
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			/*
6			* This file is part of the Zikula package.
7			*
8			* Copyright Zikula Foundation - https://ziku.la/
9			*
10			* For the full copyright and license information, please view the LICENSE
11			* file that was distributed with this source code.
12			*/
13
14			namespace Zikula\MailerModule;
15
16			use Doctrine\Persistence\ManagerRegistry;
17			use Symfony\Component\HttpFoundation\RequestStack;
18			use Symfony\Contracts\Translation\TranslatorInterface;
19			use Zikula\Bundle\CoreBundle\Doctrine\Helper\SchemaHelper;
20			use Zikula\Bundle\CoreBundle\DynamicConfigDumper;
21			use Zikula\Bundle\CoreBundle\HttpKernel\ZikulaHttpKernelInterface;
22			use Zikula\ExtensionsModule\AbstractExtension;
23			use Zikula\ExtensionsModule\Api\ApiInterface\VariableApiInterface;
24			use Zikula\ExtensionsModule\Entity\ExtensionVarEntity;
25			use Zikula\ExtensionsModule\Installer\AbstractExtensionInstaller;
26
27			/**
28			* Installation and upgrade routines for the mailer module.
29			*/
30			class MailerModuleInstaller extends AbstractExtensionInstaller
31			{
32			/**
33			* @var ZikulaHttpKernelInterface
34			*/
35			private $kernel;
36
37			/**
38			* @var DynamicConfigDumper
39			*/
40			private $configDumper;
41
42			public function __construct(
43			ZikulaHttpKernelInterface $kernel,
44			DynamicConfigDumper $configDumper,
45			AbstractExtension $extension,
46			ManagerRegistry $managerRegistry,
47			SchemaHelper $schemaTool,
48			RequestStack $requestStack,
49			TranslatorInterface $translator,
50			VariableApiInterface $variableApi
51			) {
52			$this->kernel = $kernel;
53			$this->configDumper = $configDumper;
54			parent::__construct($extension, $managerRegistry, $schemaTool, $requestStack, $translator, $variableApi);
55			}
56
57			public function install(): bool
58			{
59			$this->setVars($this->getDefaults());
60
61			// Initialisation successful
62			return true;
63			}
64
65			public function upgrade(string $oldVersion): bool
66			{
67			// Upgrade dependent on old version number
68			switch ($oldVersion) {
69			case '1.3.1':
70			$this->setVar('smtpsecuremethod', 'ssl');
			0 ignored issues – show Coding Style Comprehensibility introduced 2019-03-17 07:27 UTC by Report Bug Copy Issue Report Consider adding a comment if this fall-through is intended. Loading history...
71			case '1.3.2':
72			// clear old modvars
73			// use manual method because getVars() is not available during system upgrade
74			$modVarEntities = $this->managerRegistry->getRepository(ExtensionVarEntity::class)->findBy(['modname' => $this->name]);
75			$modVars = [];
76			foreach ($modVarEntities as $var) {
77			$modVars[$var['name']] = $var['value'];
78			}
79			$this->delVars();
80			$this->setVarWithDefault('charset', $modVars['charset']);
81			$this->setVarWithDefault('encoding', $modVars['encoding']);
82			$this->setVarWithDefault('html', $modVars['html']);
83			$this->setVarWithDefault('wordwrap', $modVars['wordwrap']);
84			// new modvar for 1.4.0
85			$this->setVarWithDefault('enableLogging', false);
86
87			case '1.4.0':
88			case '1.4.1':
89			case '1.4.2':
90			case '1.4.3':
91			case '1.5.0':
92			case '1.5.1':
93			// all swiftmailer config changes removed from previous version upgrades above
94			$this->configDumper->delConfiguration('swiftmailer');
95			// future upgrade routines
96			}
97
98			// Update successful
99			return true;
100			}
101
102			public function uninstall(): bool
103			{
104			// Delete any module variables
105			$this->delVars();
106
107			// Deletion successful
108			return true;
109			}
110
111			/**
112			* Default module vars.
113			*/
114			private function getDefaults(): array
115			{
116			return [
117			'charset' => $this->kernel->getCharset(),
118			'encoding' => '8bit',
119			'html' => false,
120			'wordwrap' => 50,
121			'enableLogging' => false
122			];
123			}
124
125			/**
126			* Set the module var but if it is not set, use the default instead.
127			*
128			* @param mixed $value
129			*/
130			private function setVarWithDefault(string $key, $value = null): void
131			{
132			if (isset($value)) {
133			$this->setVar($key, $value);
134			}
135			$defaults = $this->getDefaults();
136			$this->setVar($key, $defaults[$key]);
137			}
138			}
139

zikula / core

Pull Request — master (#4187)

MailerModuleInstaller::upgrade() B

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like