TwitterEmbedTweetBlockService::configureEditForm() - Code Metrics - Inspection of "Remove old sonata dependencies" - sonata-project/SonataSeoBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#356)

by Christian

created 2019-12-09 20:49 UTC

TwitterEmbedTweetBlockService::configureEditForm() A

↳ Parent: TwitterEmbedTweetBlockService

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	53
rs	9.0254
c	0
b	0
f	0
cc	1
nc	1
nop	2

How to fix Long Method

<?php

declare(strict_types=1);

/*
 * This file is part of the Sonata Project package.
 *
 * (c) Thomas Rabaix <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Sonata\SeoBundle\Block\Social;

use GuzzleHttp\Exception\GuzzleException;
use Psr\Http\Client\ClientExceptionInterface;
use Psr\Http\Client\ClientInterface;
use Psr\Http\Message\RequestFactoryInterface;
use Sonata\BlockBundle\Block\BlockContextInterface;
use Sonata\BlockBundle\Block\Service\EditableBlockService;
use Sonata\BlockBundle\Form\Mapper\FormMapper;
use Sonata\BlockBundle\Meta\MetadataInterface;
use Sonata\BlockBundle\Model\BlockInterface;
use Sonata\Form\Type\ImmutableArrayType;
use Sonata\Form\Validator\ErrorElement;
use Symfony\Component\Form\Extension\Core\Type\CheckboxType;
use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
use Symfony\Component\Form\Extension\Core\Type\IntegerType;
use Symfony\Component\Form\Extension\Core\Type\TextareaType;
use Symfony\Component\Form\Extension\Core\Type\TextType;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\OptionsResolver\OptionsResolver;
use Twig\Environment;

/**
 * This block service allows to embed a tweet by requesting the Twitter API.
 *
 * @see https://dev.twitter.com/docs/api/1/get/statuses/oembed
 *
 * @author Hugo Briand <[email protected]>
 */
class TwitterEmbedTweetBlockService extends BaseTwitterButtonBlockService implements EditableBlockService
{
    public const TWITTER_OEMBED_URI = 'https://api.twitter.com/1/statuses/oembed.json';
    private const TWEET_URL_PATTERN = '%^(https://)(www.)?(twitter.com/)(.*)(/status)(es)?(/)([0-9]*)$%i';
    private const TWEET_ID_PATTERN = '%^([0-9]*)$%';

    /**
     * @var ClientInterface|null
     */
    private $httpClient;

    /**
     * @var RequestFactoryInterface|null
     */
    private $messageFactory;

    public function __construct(
        Environment $twig,
        ClientInterface $httpClient = null,
        RequestFactoryInterface $messageFactory = null
    ) {
        parent::__construct($twig);

        $this->httpClient = $httpClient;
        $this->messageFactory = $messageFactory;
    }

    public function execute(BlockContextInterface $blockContext, Response $response = null): Response
    {
        return $this->renderResponse($blockContext->getTemplate(), [
            'block' => $blockContext->getBlock(),
            'tweet' => $this->loadTweet($blockContext),
        ], $response);
    }

    public function configureSettings(OptionsResolver $resolver): void
    {
        $resolver->setDefaults([
            'template' => '@SonataSeo/Block/block_twitter_embed.html.twig',
            'tweet' => '',
            'maxwidth' => null,
            'hide_media' => false,
            'hide_thread' => false,
            'omit_script' => false,
            'align' => 'none',
            'related' => null,
            'lang' => null,
        ]);
    }

    public function configureCreateForm(FormMapper $form, BlockInterface $block): void
    {
        $this->configureEditForm($form, $block);
    }

    public function configureEditForm(FormMapper $form, BlockInterface $block): void
    {
        $form->add('settings', ImmutableArrayType::class, [
            'keys' => [
                ['tweet', TextareaType::class, [
                    'required' => true,
                    'label' => 'form.label_tweet',
                    'sonata_help' => 'form.help_tweet',
                ]],
                ['maxwidth', IntegerType::class, [
                    'required' => false,
                    'label' => 'form.label_maxwidth',
                    'sonata_help' => 'form.help_maxwidth',
                ]],
                ['hide_media', CheckboxType::class, [
                    'required' => false,
                    'label' => 'form.label_hide_media',
                    'sonata_help' => 'form.help_hide_media',
                ]],
                ['hide_thread', CheckboxType::class, [
                    'required' => false,
                    'label' => 'form.label_hide_thread',
                    'sonata_help' => 'form.help_hide_thread',
                ]],
                ['omit_script', CheckboxType::class, [
                    'required' => false,
                    'label' => 'form.label_omit_script',
                    'sonata_help' => 'form.help_omit_script',
                ]],
                ['align', ChoiceType::class, [
                    'required' => false,
                    'choices' => [
                        'left' => 'form.label_align_left',
                        'right' => 'form.label_align_right',
                        'center' => 'form.label_align_center',
                        'none' => 'form.label_align_none',
                    ],
                    'label' => 'form.label_align',
                ]],
                ['related', TextType::class, [
                    'required' => false,
                    'label' => 'form.label_related',
                    'sonata_help' => 'form.help_related',
                ]],
                ['lang', ChoiceType::class, [
                    'required' => true,
                    'choices' => $this->languageList,
                    'label' => 'form.label_lang',
                ]],
            ],
            'translation_domain' => 'SonataSeoBundle',
        ]);
    }

    public function validate(ErrorElement $errorElement, BlockInterface $block): void
    {
    }

    public function getMetadata(): MetadataInterface
    {
        return new \Sonata\BlockBundle\Meta\Metadata('sonata.seo.block.twitter.embed', null, null, 'SonataSeoBundle', [
            'class' => 'fa fa-twitter',
        ]);
    }

    /**
     * Returns supported API parameters from settings.
     *
     * @return array
     */
    protected function getSupportedApiParams()
    {
        return [
            'maxwidth',
            'hide_media',
            'hide_thread',
            'omit_script',
            'align',
            'related',
            'lang',
            'url',
            'id',
        ];
    }

    /**
     * Builds the API query URI based on $settings.
     *
     * @param bool $uriMatched
     *
     * @return string
     */
    protected function buildUri($uriMatched, array $settings)
    {
        $apiParams = $settings;
        $supportedParams = $this->getSupportedApiParams();

        if ($uriMatched) {
            // We matched the uri
            $apiParams['url'] = $settings['tweet'];
        } else {
            $apiParams['id'] = $settings['tweet'];
        }

        unset($apiParams['tweet']);

        $parameters = [];
        foreach ($apiParams as $key => $value) {
            if ($value && \in_array($key, $supportedParams, true)) {
                $parameters[] = $key.'='.$value;
            }
        }

        return sprintf('%s?%s', self::TWITTER_OEMBED_URI, implode('&', $parameters));
    }

    /**
     * Loads twitter tweet.
     */
    private function loadTweet(BlockContextInterface $blockContext): ?string
    {
        $uriMatched = preg_match(self::TWEET_URL_PATTERN, $blockContext->getSetting('tweet'));

        if (!$uriMatched || !preg_match(self::TWEET_ID_PATTERN, $blockContext->getSetting('tweet'))) {
            return null;
        }

        if (null !== $this->httpClient && null !== $this->messageFactory) {
            try {
                $response = $this->httpClient->sendRequest(
                    $this->messageFactory->createRequest(
                        'GET',
                        $this->buildUri($uriMatched, $blockContext->getSettings())
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
                    )
                );
            } catch (ClientExceptionInterface $e) {
                // log error
                return null;
            }

            $apiTweet = json_decode($response->getBody(), true);

            return $apiTweet['html'];
        }

        // NEXT_MAJOR: Remove the old guzzle implementation

        // We matched an URL or an ID, we'll need to ask the API
        if (false === class_exists('GuzzleHttp\Client')) {
            throw new \RuntimeException(
                'The guzzle http client library is required to call the Twitter API.'.
                'Make sure to add psr/http-client or guzzlehttp/guzzle to your composer.json.'
            );
        }

        @trigger_error(
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
            'The direct Guzzle implementation is deprecated since 2.x and will be removed with the next major release.',
            E_USER_DEPRECATED
        );

        // TODO cache API result
        $client = new \GuzzleHttp\Client();
        $client->setConfig(['curl.options' => [CURLOPT_CONNECTTIMEOUT_MS => 1000]]);

        try {
            $request = $client->get($this->buildUri($uriMatched, $blockContext->getSettings()));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            $apiTweet = json_decode($request->send()->getBody(true), true);


            return $apiTweet['html'];
        } catch (GuzzleException $e) {
            // log error
            return null;
        }
        // END NEXT_MAJOR
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			/*
6			* This file is part of the Sonata Project package.
7			*
8			* (c) Thomas Rabaix <[email protected]>
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 Sonata\SeoBundle\Block\Social;
15
16			use GuzzleHttp\Exception\GuzzleException;
17			use Psr\Http\Client\ClientExceptionInterface;
18			use Psr\Http\Client\ClientInterface;
19			use Psr\Http\Message\RequestFactoryInterface;
20			use Sonata\BlockBundle\Block\BlockContextInterface;
21			use Sonata\BlockBundle\Block\Service\EditableBlockService;
22			use Sonata\BlockBundle\Form\Mapper\FormMapper;
23			use Sonata\BlockBundle\Meta\MetadataInterface;
24			use Sonata\BlockBundle\Model\BlockInterface;
25			use Sonata\Form\Type\ImmutableArrayType;
26			use Sonata\Form\Validator\ErrorElement;
27			use Symfony\Component\Form\Extension\Core\Type\CheckboxType;
28			use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
29			use Symfony\Component\Form\Extension\Core\Type\IntegerType;
30			use Symfony\Component\Form\Extension\Core\Type\TextareaType;
31			use Symfony\Component\Form\Extension\Core\Type\TextType;
32			use Symfony\Component\HttpFoundation\Response;
33			use Symfony\Component\OptionsResolver\OptionsResolver;
34			use Twig\Environment;
35
36			/**
37			* This block service allows to embed a tweet by requesting the Twitter API.
38			*
39			* @see https://dev.twitter.com/docs/api/1/get/statuses/oembed
40			*
41			* @author Hugo Briand <[email protected]>
42			*/
43			class TwitterEmbedTweetBlockService extends BaseTwitterButtonBlockService implements EditableBlockService
44			{
45			public const TWITTER_OEMBED_URI = 'https://api.twitter.com/1/statuses/oembed.json';
46			private const TWEET_URL_PATTERN = '%^(https://)(www.)?(twitter.com/)(.)(/status)(es)?(/)([0-9])$%i';
47			private const TWEET_ID_PATTERN = '%^([0-9]*)$%';
48
49			/**
50			* @var ClientInterface\|null
51			*/
52			private $httpClient;
53
54			/**
55			* @var RequestFactoryInterface\|null
56			*/
57			private $messageFactory;
58
59			public function __construct(
60			Environment $twig,
61			ClientInterface $httpClient = null,
62			RequestFactoryInterface $messageFactory = null
63			) {
64			parent::__construct($twig);
65
66			$this->httpClient = $httpClient;
67			$this->messageFactory = $messageFactory;
68			}
69
70			public function execute(BlockContextInterface $blockContext, Response $response = null): Response
71			{
72			return $this->renderResponse($blockContext->getTemplate(), [
73			'block' => $blockContext->getBlock(),
74			'tweet' => $this->loadTweet($blockContext),
75			], $response);
76			}
77
78			public function configureSettings(OptionsResolver $resolver): void
79			{
80			$resolver->setDefaults([
81			'template' => '@SonataSeo/Block/block_twitter_embed.html.twig',
82			'tweet' => '',
83			'maxwidth' => null,
84			'hide_media' => false,
85			'hide_thread' => false,
86			'omit_script' => false,
87			'align' => 'none',
88			'related' => null,
89			'lang' => null,
90			]);
91			}
92
93			public function configureCreateForm(FormMapper $form, BlockInterface $block): void
94			{
95			$this->configureEditForm($form, $block);
96			}
97
98			public function configureEditForm(FormMapper $form, BlockInterface $block): void
99			{
100			$form->add('settings', ImmutableArrayType::class, [
101			'keys' => [
102			['tweet', TextareaType::class, [
103			'required' => true,
104			'label' => 'form.label_tweet',
105			'sonata_help' => 'form.help_tweet',
106			]],
107			['maxwidth', IntegerType::class, [
108			'required' => false,
109			'label' => 'form.label_maxwidth',
110			'sonata_help' => 'form.help_maxwidth',
111			]],
112			['hide_media', CheckboxType::class, [
113			'required' => false,
114			'label' => 'form.label_hide_media',
115			'sonata_help' => 'form.help_hide_media',
116			]],
117			['hide_thread', CheckboxType::class, [
118			'required' => false,
119			'label' => 'form.label_hide_thread',
120			'sonata_help' => 'form.help_hide_thread',
121			]],
122			['omit_script', CheckboxType::class, [
123			'required' => false,
124			'label' => 'form.label_omit_script',
125			'sonata_help' => 'form.help_omit_script',
126			]],
127			['align', ChoiceType::class, [
128			'required' => false,
129			'choices' => [
130			'left' => 'form.label_align_left',
131			'right' => 'form.label_align_right',
132			'center' => 'form.label_align_center',
133			'none' => 'form.label_align_none',
134			],
135			'label' => 'form.label_align',
136			]],
137			['related', TextType::class, [
138			'required' => false,
139			'label' => 'form.label_related',
140			'sonata_help' => 'form.help_related',
141			]],
142			['lang', ChoiceType::class, [
143			'required' => true,
144			'choices' => $this->languageList,
145			'label' => 'form.label_lang',
146			]],
147			],
148			'translation_domain' => 'SonataSeoBundle',
149			]);
150			}
151
152			public function validate(ErrorElement $errorElement, BlockInterface $block): void
153			{
154			}
155
156			public function getMetadata(): MetadataInterface
157			{
158			return new \Sonata\BlockBundle\Meta\Metadata('sonata.seo.block.twitter.embed', null, null, 'SonataSeoBundle', [
159			'class' => 'fa fa-twitter',
160			]);
161			}
162
163			/**
164			* Returns supported API parameters from settings.
165			*
166			* @return array
167			*/
168			protected function getSupportedApiParams()
169			{
170			return [
171			'maxwidth',
172			'hide_media',
173			'hide_thread',
174			'omit_script',
175			'align',
176			'related',
177			'lang',
178			'url',
179			'id',
180			];
181			}
182
183			/**
184			* Builds the API query URI based on $settings.
185			*
186			* @param bool $uriMatched
187			*
188			* @return string
189			*/
190			protected function buildUri($uriMatched, array $settings)
191			{
192			$apiParams = $settings;
193			$supportedParams = $this->getSupportedApiParams();
194
195			if ($uriMatched) {
196			// We matched the uri
197			$apiParams['url'] = $settings['tweet'];
198			} else {
199			$apiParams['id'] = $settings['tweet'];
200			}
201
202			unset($apiParams['tweet']);
203
204			$parameters = [];
205			foreach ($apiParams as $key => $value) {
206			if ($value && \in_array($key, $supportedParams, true)) {
207			$parameters[] = $key.'='.$value;
208			}
209			}
210
211			return sprintf('%s?%s', self::TWITTER_OEMBED_URI, implode('&', $parameters));
212			}
213
214			/**
215			* Loads twitter tweet.
216			*/
217			private function loadTweet(BlockContextInterface $blockContext): ?string
218			{
219			$uriMatched = preg_match(self::TWEET_URL_PATTERN, $blockContext->getSetting('tweet'));
220
221			if (!$uriMatched \|\| !preg_match(self::TWEET_ID_PATTERN, $blockContext->getSetting('tweet'))) {
222			return null;
223			}
224
225			if (null !== $this->httpClient && null !== $this->messageFactory) {
226			try {
227			$response = $this->httpClient->sendRequest(
228			$this->messageFactory->createRequest(
229			'GET',
230			$this->buildUri($uriMatched, $blockContext->getSettings())
			0 ignored issues – show Documentation introduced 2018-06-01 17:49 UTC by Report Bug Copy Issue Report `$uriMatched` is of type `integer`, but the function expects a `boolean`. 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...
231			)
232			);
233			} catch (ClientExceptionInterface $e) {
234			// log error
235			return null;
236			}
237
238			$apiTweet = json_decode($response->getBody(), true);
239
240			return $apiTweet['html'];
241			}
242
243			// NEXT_MAJOR: Remove the old guzzle implementation
244
245			// We matched an URL or an ID, we'll need to ask the API
246			if (false === class_exists('GuzzleHttp\Client')) {
247			throw new \RuntimeException(
248			'The guzzle http client library is required to call the Twitter API.'.
249			'Make sure to add psr/http-client or guzzlehttp/guzzle to your composer.json.'
250			);
251			}
252
253			@trigger_error(
			0 ignored issues – show Security Best Practice introduced 2019-01-15 18:44 UTC by Report Bug Copy Issue Report It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended. If you suppress an error, we recommend checking for the error condition explicitly: // For example instead of @mkdir($dir); // Better use if (@mkdir($dir) === false) { throw new \RuntimeException('The directory '.$dir.' could not be created.'); } Loading history...
254			'The direct Guzzle implementation is deprecated since 2.x and will be removed with the next major release.',
255			E_USER_DEPRECATED
256			);
257
258			// TODO cache API result
259			$client = new \GuzzleHttp\Client();
260			$client->setConfig(['curl.options' => [CURLOPT_CONNECTTIMEOUT_MS => 1000]]);
261
262			try {
263			$request = $client->get($this->buildUri($uriMatched, $blockContext->getSettings()));
			0 ignored issues – show Documentation introduced 2018-06-01 14:14 UTC by Report Bug Copy Issue Report `$uriMatched` is of type `integer`, but the function expects a `boolean`. 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...
264			$apiTweet = json_decode($request->send()->getBody(true), true);
			0 ignored issues – show Bug introduced 2018-06-01 15:42 UTC by Report Bug Copy Issue Report The method `send()` does not seem to exist on `object<Psr\Http\Message\ResponseInterface>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
265
266			return $apiTweet['html'];
267			} catch (GuzzleException $e) {
268			// log error
269			return null;
270			}
271			// END NEXT_MAJOR
272			}
273			}
274

sonata-project / SonataSeoBundle

Pull Request — master (#356)

TwitterEmbedTweetBlockService::configureEditForm() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like