Tweet::tweet() - Code Metrics - magium/twitter - Measure and Improve Code Quality continuously with Scrutinizer

Tweet::tweet() A
last analyzed 2018-02-12 14:01 UTC

↳ Parent: Tweet

Complexity

Conditions	1
Paths	1

Size

Total Lines	15
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

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

<?php

namespace Magium\Twitter\Actions;

use Magium\Twitter\Themes\Twitter;
use Magium\WebDriver\ExpectedCondition;
use Magium\WebDriver\WebDriver;

class Tweet
{

    const ACTION = 'Magium\Twitter\Actions\Tweet';

    protected $theme;
    protected $webDriver;

    public function __construct(
        Twitter $theme,
        WebDriver $webDriver
    )
    {
        $this->theme = $theme;
        $this->webDriver = $webDriver;
    }

    public function tweet($text)
    {

        $element = $this->webDriver->byXpath($this->theme->getTweetBoxXpath());
        $element->click();

        $this->webDriver->getKeyboard()->sendKeys($text);

        $this->webDriver->wait(5)->until(ExpectedCondition::elementExists($this->theme->getTweetButton(), WebDriver::BY_XPATH));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $element = $this->webDriver->byXpath($this->theme->getTweetButton());

        $this->webDriver->wait(5)->until(ExpectedCondition::visibilityOf($element));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $element->click();

    }

}

1			<?php
2
3			namespace Magium\Twitter\Actions;
4
5			use Magium\Twitter\Themes\Twitter;
6			use Magium\WebDriver\ExpectedCondition;
7			use Magium\WebDriver\WebDriver;
8
9			class Tweet
10			{
11
12			const ACTION = 'Magium\Twitter\Actions\Tweet';
13
14			protected $theme;
15			protected $webDriver;
16
17			public function __construct(
18			Twitter $theme,
19			WebDriver $webDriver
20			)
21			{
22			$this->theme = $theme;
23			$this->webDriver = $webDriver;
24			}
25
26			public function tweet($text)
27			{
28
29			$element = $this->webDriver->byXpath($this->theme->getTweetBoxXpath());
30			$element->click();
31
32			$this->webDriver->getKeyboard()->sendKeys($text);
33
34			$this->webDriver->wait(5)->until(ExpectedCondition::elementExists($this->theme->getTweetButton(), WebDriver::BY_XPATH));
			0 ignored issues – show Documentation introduced 2017-01-27 23:23 UTC by Report Bug Copy Issue Report `\Magium\WebDriver\Expect...er\WebDriver::BY_XPATH)` is of type `object<Facebook\WebDrive...riverExpectedCondition>`, but the function expects a `callable`. 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...
35			$element = $this->webDriver->byXpath($this->theme->getTweetButton());
36
37			$this->webDriver->wait(5)->until(ExpectedCondition::visibilityOf($element));
			0 ignored issues – show Documentation introduced 2017-01-27 23:23 UTC by Report Bug Copy Issue Report `\Magium\WebDriver\Expect...:visibilityOf($element)` is of type `object<Magium\WebDriver\ExpectedCondition>`, but the function expects a `callable`. 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...
38			$element->click();
39
40			}
41
42			}

magium / twitter

Tweet::tweet() A last analyzed 2018-02-12 14:01 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

Tweet::tweet() A
last analyzed 2018-02-12 14:01 UTC