BigPipeContext::prepareBigPipeNoJsCookie() - Code Metrics - Inspection of "Merge branch 'follow/259' of https://github.com/za..." - jhedstrom/drupalextension - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — 259-follow-up ( 71585a )

by Jonathan

created 2017-12-12 19:47 UTC

BigPipeContext::prepareBigPipeNoJsCookie() A

↳ Parent: BigPipeContext

Complexity

Conditions	3
Paths	3

Size

Total Lines	14
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	14
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	8
nc	3
nop	0

<?php

namespace Drupal\DrupalExtension\Context;

use Drupal\big_pipe\Render\Placeholder\BigPipeStrategy;
use Behat\Mink\Exception\UnsupportedDriverActionException;

/**
 * Big Pipe context.
 */
class BigPipeContext extends RawDrupalContext
{

    /**
     * Prepares Big Pipe NOJS cookie if needed.
     *
     * @BeforeScenario
     */
    public function prepareBigPipeNoJsCookie()
    {
        try {
            // Check if JavaScript can be executed by Driver.
            $this->getSession()->getDriver()->executeScript('true');
        } catch (UnsupportedDriverActionException $e) {
            // Set NOJS cookie.
            $this
              ->getSession()
              ->setCookie(BigPipeStrategy::NOJS_COOKIE, true);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        } catch (\Exception $e) {
            // Mute exceptions.
        }
    }
}


1			<?php
2
3			namespace Drupal\DrupalExtension\Context;
4
5			use Drupal\big_pipe\Render\Placeholder\BigPipeStrategy;
6			use Behat\Mink\Exception\UnsupportedDriverActionException;
7
8			/**
9			* Big Pipe context.
10			*/
11			class BigPipeContext extends RawDrupalContext
12			{
13
14			/**
15			* Prepares Big Pipe NOJS cookie if needed.
16			*
17			* @BeforeScenario
18			*/
19			public function prepareBigPipeNoJsCookie()
20			{
21			try {
22			// Check if JavaScript can be executed by Driver.
23			$this->getSession()->getDriver()->executeScript('true');
24			} catch (UnsupportedDriverActionException $e) {
25			// Set NOJS cookie.
26			$this
27			->getSession()
28			->setCookie(BigPipeStrategy::NOJS_COOKIE, true);
			0 ignored issues – show Documentation introduced 2016-11-15 21:07 UTC by Report Bug Copy Issue Report `true` is of type `boolean`, but the function expects a `string\|null`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
29			} catch (\Exception $e) {
30			// Mute exceptions.
31			}
32			}
33			}
34

jhedstrom / drupalextension

Push — 259-follow-up ( 71585a )

BigPipeContext::prepareBigPipeNoJsCookie() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like