Text::prepare_options() - Code Metrics - Inspection of "Merge pull request #2993 from Murilo2508/inputmask..." - z-song/laravel-admin - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( adb78f...0848e3 )

by Song

created 2019-02-08 08:35 UTC

Text::prepare_options() A

↳ Parent: Text

Complexity

Conditions	4
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	4
nc	4
nop	1
dl	0
loc	20
rs	9.6
c	0
b	0
f	0

<?php

namespace Encore\Admin\Form\Field;

use Encore\Admin\Form\Field;

class Text extends Field
{
    use PlainInput;

    /**
     * Render this filed.
     *
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
     */
    public function render()
    {
        $this->initPlainInput();

        $this->prepend('<i class="fa fa-pencil fa-fw"></i>')
            ->defaultAttribute('type', 'text')
            ->defaultAttribute('id', $this->id)
            ->defaultAttribute('name', $this->elementName ?: $this->formatName($this->column))
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
            ->defaultAttribute('value', old($this->column, $this->value()))
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
            ->defaultAttribute('class', 'form-control '.$this->getElementClassString())
            ->defaultAttribute('placeholder', $this->getPlaceholder());

        $this->addVariables([
            'prepend' => $this->prepend,
            'append'  => $this->append,
        ]);

        return parent::render();

    }

    /**
     * Add inputmask to an elements.
     *
     * @param array $options
     *
     * @return $this
     */
    public function inputmask($options)
    {
        $options = $this->json_encode_options($options);

        $this->script = "$('{$this->getElementClassSelector()}').inputmask($options);";

        return $this;
    }

    /**
     * Encode options to Json.
     *
     * @param array $options
     *
     * @return $json

     */
    protected function json_encode_options($options)
    {
        $data = $this->prepare_options($options);

        $json = json_encode($data['options']);

        $json = str_replace($data['toReplace'], $data['original'], $json);

        return $json;
    }

    /**
     * Prepare options.
     *
     * @param array $options
     *
     * @return array
     */
    protected function prepare_options($options)
    {
        $original = [];
        $toReplace = [];

        foreach ($options as $key => &$value) {
            if (is_array($value)) {
                $subArray = $this->prepare_options($value);
                $value = $subArray['options'];
                $original = array_merge($original, $subArray['original']);
                $toReplace = array_merge($toReplace, $subArray['toReplace']);
            } elseif (preg_match('/function.*?/', $value)) {
                $original[] = $value;
                $value = "%{$key}%";
                $toReplace[] = "\"{$value}\"";
            }
        }

        return compact('original', 'toReplace', 'options');    
    }
  
    public function datalist( $entries = [] )
    {
        $this->defaultAttribute('list', "list-{$this->id}");

        $datalist = "<datalist id=\"list-{$this->id}\">";
        foreach($entries as $k => $v) {
            $datalist .= "<option value=\"{$k}\">{$v}</option>";
        }
        $datalist .= "</datalist>";
        $this->append($datalist);
    }
}


1			<?php
2
3			namespace Encore\Admin\Form\Field;
4
5			use Encore\Admin\Form\Field;
6
7			class Text extends Field
8			{
9			use PlainInput;
10
11			/**
12			* Render this filed.
13			*
14			* @return \Illuminate\Contracts\View\Factory\|\Illuminate\View\View
15			*/
16			public function render()
17			{
18			$this->initPlainInput();
19
20			$this->prepend('<i class="fa fa-pencil fa-fw"></i>')
21			->defaultAttribute('type', 'text')
22			->defaultAttribute('id', $this->id)
23			->defaultAttribute('name', $this->elementName ?: $this->formatName($this->column))
			0 ignored issues – show Bug introduced 2017-09-19 15:53 UTC by Report Bug Copy Issue Report It seems like `$this->column` can also be of type `array`; however, `Encore\Admin\Form\Field::formatName()` does only seem to accept `string`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /** * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
24			->defaultAttribute('value', old($this->column, $this->value()))
			0 ignored issues – show Bug introduced 2017-09-19 15:53 UTC by Report Bug Copy Issue Report It seems like `$this->column` can also be of type `array`; however, `old()` does only seem to accept `string\|null`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /** * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
25			->defaultAttribute('class', 'form-control '.$this->getElementClassString())
26			->defaultAttribute('placeholder', $this->getPlaceholder());
27
28			$this->addVariables([
29			'prepend' => $this->prepend,
30			'append' => $this->append,
31			]);
32
33			return parent::render();
			0 ignored issues – show Bug Compatibility introduced 2019-02-04 18:06 UTC by Report Bug Copy Issue Report The expression `parent::render();` of type `string\|Illuminate\View\V...\Contracts\View\Factory` adds the type `Illuminate\Contracts\View\Factory` to the return on line 33 which is incompatible with the return type declared by the interface `Illuminate\Contracts\Support\Renderable::render` of type `string`. Loading history...
34			}
35
36			/**
37			* Add inputmask to an elements.
38			*
39			* @param array $options
40			*
41			* @return $this
42			*/
43			public function inputmask($options)
44			{
45			$options = $this->json_encode_options($options);
46
47			$this->script = "$('{$this->getElementClassSelector()}').inputmask($options);";
48
49			return $this;
50			}
51
52			/**
53			* Encode options to Json.
54			*
55			* @param array $options
56			*
57			* @return $json
			0 ignored issues – show Documentation introduced 2019-02-04 18:15 UTC by Report Bug Copy Issue Report The doc-type `$json` could not be parsed: Unknown type name "$json" at position 0. (view supported doc-types) This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types. Loading history...
58			*/
59			protected function json_encode_options($options)
60			{
61			$data = $this->prepare_options($options);
62
63			$json = json_encode($data['options']);
64
65			$json = str_replace($data['toReplace'], $data['original'], $json);
66
67			return $json;
68			}
69
70			/**
71			* Prepare options.
72			*
73			* @param array $options
74			*
75			* @return array
76			*/
77			protected function prepare_options($options)
78			{
79			$original = [];
80			$toReplace = [];
81
82			foreach ($options as $key => &$value) {
83			if (is_array($value)) {
84			$subArray = $this->prepare_options($value);
85			$value = $subArray['options'];
86			$original = array_merge($original, $subArray['original']);
87			$toReplace = array_merge($toReplace, $subArray['toReplace']);
88			} elseif (preg_match('/function.*?/', $value)) {
89			$original[] = $value;
90			$value = "%{$key}%";
91			$toReplace[] = "\"{$value}\"";
92			}
93			}
94
95			return compact('original', 'toReplace', 'options');
96			}
97
98			public function datalist( $entries = [] )
99			{
100			$this->defaultAttribute('list', "list-{$this->id}");
101
102			$datalist = "<datalist id=\"list-{$this->id}\">";
103			foreach($entries as $k => $v) {
104			$datalist .= "<option value=\"{$k}\">{$v}</option>";
105			}
106			$datalist .= "</datalist>";
107			$this->append($datalist);
108			}
109			}
110

z-song / laravel-admin

Push — master ( adb78f...0848e3 )

Text::prepare_options() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like