EmailVerifiedSecondFactorRemindersCommand::execute() - Code Metrics - Inspection of "Update messages translations" - OpenConext/Stepup-Middleware - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — feature/send-vetting-reminder ( acaa4f )

unknown

created 2017-11-15 15:51 UTC

execute() B

↳ Parent: EmailVerifiedSecondFactorRemindersCommand

Complexity

Conditions	6
Paths	5

Size

Total Lines	27
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	27
rs	8.439
c	0
b	0
f	0
cc	6
eloc	17
nc	5
nop	2

<?php

/**
 * Copyright 2014 SURFnet bv
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

namespace Surfnet\StepupMiddleware\MiddlewareBundle\Console\Command;

use Assert\Assertion;
use DateInterval;
use DateTime;
use InvalidArgumentException;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\DependencyInjection\Container;

/**
 * The EmailVerifiedSecondFactorRemindersCommand can be run to send reminders to token registrants.
 *
 * The command utilizes a specific service for this task (VerifiedSecondFactorReminderService). Input validation is
 * performed on the incoming request parameters.
 */
final class EmailVerifiedSecondFactorRemindersCommand extends Command
{
    protected function configure()
    {
        $this
            ->setName('middleware:cron:email-reminder')
            ->setDescription('Sends email reminders to identities with verified tokens more than 7 days old.')
            ->addArgument('dry-run', InputArgument::OPTIONAL, 'Run in dry mode, not sending any email')
            ->addArgument(
                'date',
                InputArgument::OPTIONAL,
                'The date (Y-m-d) that should be used for sending reminder email messages, defaults to TODAY - 7'
            );
    }

    protected function execute(InputInterface $input, OutputInterface $output)
    {
        /** @var Container $container */
        $container = $this->getApplication()->getKernel()->getContainer();
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}

        $service = $container->get('surfnet_stepup_middleware_middleware.verfied_second_factor_reminder');
        $logger = $container->get('logger');

        try {
            $this->validateInput($input);
        } catch (InvalidArgumentException $e) {
            $logger->error(sprintf('Invalid arguments passed to the %s', $this->getName()), [$e->getMessage()]);
            return 1;
        }

        $date = new DateTime();
        $date->sub(new DateInterval('P7D'));
        if ($input->hasArgument('date') && !is_null($input->getArgument('date'))) {
            $date = DateTime::createFromFormat('Y-m-d', $input->getArgument('date'));
        }

        $dryRun = false;
        if ($input->hasArgument('dry-run') && !is_null($input->getArgument('dry-run'))) {
            $dryRun = $input->getArgument('dry-run');
        }
        $service->sendReminders($date, $dryRun);
    }

    private function validateInput(InputInterface $input)
    {
        if ($input->hasArgument('date')) {
            $date = $input->getArgument('date');
            Assertion::nullOrDate($date, 'Y-m-d', 'Expected date to be a string and formatted as a Y-m-d');
        }

        if ($input->hasArgument('dry-run')) {
            $dryRun = $input->getArgument('dry-run');
            Assertion::nullOrBoolean($dryRun, 'Expected dry-run parameter to be a boolean value.');
        }
    }
}


1			<?php
2
3			/**
4			* Copyright 2014 SURFnet bv
5			*
6			* Licensed under the Apache License, Version 2.0 (the "License");
7			* you may not use this file except in compliance with the License.
8			* You may obtain a copy of the License at
9			*
10			* http://www.apache.org/licenses/LICENSE-2.0
11			*
12			* Unless required by applicable law or agreed to in writing, software
13			* distributed under the License is distributed on an "AS IS" BASIS,
14			* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
15			* See the License for the specific language governing permissions and
16			* limitations under the License.
17			*/
18
19			namespace Surfnet\StepupMiddleware\MiddlewareBundle\Console\Command;
20
21			use Assert\Assertion;
22			use DateInterval;
23			use DateTime;
24			use InvalidArgumentException;
25			use Symfony\Component\Console\Command\Command;
26			use Symfony\Component\Console\Input\InputArgument;
27			use Symfony\Component\Console\Input\InputInterface;
28			use Symfony\Component\Console\Output\OutputInterface;
29			use Symfony\Component\DependencyInjection\Container;
30
31			/**
32			* The EmailVerifiedSecondFactorRemindersCommand can be run to send reminders to token registrants.
33			*
34			* The command utilizes a specific service for this task (VerifiedSecondFactorReminderService). Input validation is
35			* performed on the incoming request parameters.
36			*/
37			final class EmailVerifiedSecondFactorRemindersCommand extends Command
38			{
39			protected function configure()
40			{
41			$this
42			->setName('middleware:cron:email-reminder')
43			->setDescription('Sends email reminders to identities with verified tokens more than 7 days old.')
44			->addArgument('dry-run', InputArgument::OPTIONAL, 'Run in dry mode, not sending any email')
45			->addArgument(
46			'date',
47			InputArgument::OPTIONAL,
48			'The date (Y-m-d) that should be used for sending reminder email messages, defaults to TODAY - 7'
49			);
50			}
51
52			protected function execute(InputInterface $input, OutputInterface $output)
53			{
54			/** @var Container $container */
55			$container = $this->getApplication()->getKernel()->getContainer();
			0 ignored issues – show Bug introduced 2017-11-15 15:54 UTC by Report Bug Copy Issue Report It seems like you code against a specific sub-type and not the parent class `Symfony\Component\Console\Application` as the method `getKernel()` does only exist in the following sub-classes of `Symfony\Component\Console\Application`: `Symfony\Bundle\FrameworkBundle\Console\Application`. Maybe you want to `instanceof` check for one of these explicitly? Let’s take a look at an example: abstract class User { /** @return string / abstract public function getPassword(); } class MyUser extends User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different sub-classes of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the parent class: abstract class User { /** @return string / abstract public function getPassword(); /* @return string */ abstract public function getDisplayName(); } Loading history...
56
57			$service = $container->get('surfnet_stepup_middleware_middleware.verfied_second_factor_reminder');
58			$logger = $container->get('logger');
59
60			try {
61			$this->validateInput($input);
62			} catch (InvalidArgumentException $e) {
63			$logger->error(sprintf('Invalid arguments passed to the %s', $this->getName()), [$e->getMessage()]);
64			return 1;
65			}
66
67			$date = new DateTime();
68			$date->sub(new DateInterval('P7D'));
69			if ($input->hasArgument('date') && !is_null($input->getArgument('date'))) {
70			$date = DateTime::createFromFormat('Y-m-d', $input->getArgument('date'));
71			}
72
73			$dryRun = false;
74			if ($input->hasArgument('dry-run') && !is_null($input->getArgument('dry-run'))) {
75			$dryRun = $input->getArgument('dry-run');
76			}
77			$service->sendReminders($date, $dryRun);
78			}
79
80			private function validateInput(InputInterface $input)
81			{
82			if ($input->hasArgument('date')) {
83			$date = $input->getArgument('date');
84			Assertion::nullOrDate($date, 'Y-m-d', 'Expected date to be a string and formatted as a Y-m-d');
85			}
86
87			if ($input->hasArgument('dry-run')) {
88			$dryRun = $input->getArgument('dry-run');
89			Assertion::nullOrBoolean($dryRun, 'Expected dry-run parameter to be a boolean value.');
90			}
91			}
92			}
93

OpenConext / Stepup-Middleware

Push — feature/send-vetting-reminder ( acaa4f )

execute() B

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like