Completed
Push — master ( 3e9785...ac0672 )
by Song
08:02
created

Field::formatAttributes()   A

Complexity

Conditions 2
Paths 2

Size

Total Lines 10
Code Lines 5

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 2
eloc 5
nc 2
nop 0
dl 0
loc 10
rs 9.4285
c 0
b 0
f 0
1
<?php
0 ignored issues
show
Coding Style Compatibility introduced by
For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 16 and the first side effect is on line 948.

The PSR-1: Basic Coding Standard recommends that a file should either introduce new symbols, that is classes, functions, constants or similar, or have side effects. Side effects are anything that executes logic, like for example printing output, changing ini settings or writing to a file.

The idea behind this recommendation is that merely auto-loading a class should not change the state of an application. It also promotes a cleaner style of programming and makes your code less prone to errors, because the logic is not spread out all over the place.

To learn more about the PSR-1, please see the PHP-FIG site on the PSR-1.

Loading history...
2
3
namespace Encore\Admin\Form;
4
5
use Encore\Admin\Admin;
6
use Encore\Admin\Form;
7
use Illuminate\Contracts\Support\Arrayable;
8
use Illuminate\Contracts\Support\Renderable;
9
use Illuminate\Support\Arr;
10
use Illuminate\Support\Facades\Validator;
11
use Illuminate\Support\Traits\Macroable;
12
13
/**
14
 * Class Field.
15
 */
16
class Field implements Renderable
17
{
18
    use Macroable;
19
20
    const FILE_DELETE_FLAG = '_file_del_';
21
22
    /**
23
     * Element id.
24
     *
25
     * @var array|string
26
     */
27
    protected $id;
28
29
    /**
30
     * Element value.
31
     *
32
     * @var mixed
33
     */
34
    protected $value;
35
36
    /**
37
     * Field original value.
38
     *
39
     * @var mixed
40
     */
41
    protected $original;
42
43
    /**
44
     * Field default value.
45
     *
46
     * @var mixed
47
     */
48
    protected $default;
49
50
    /**
51
     * Element label.
52
     *
53
     * @var string
54
     */
55
    protected $label = '';
56
57
    /**
58
     * Column name.
59
     *
60
     * @var string|array
61
     */
62
    protected $column = '';
63
64
    /**
65
     * Form element name.
66
     *
67
     * @var string
68
     */
69
    protected $elementName = [];
70
71
    /**
72
     * Form element classes.
73
     *
74
     * @var array
75
     */
76
    protected $elementClass = [];
77
78
    /**
79
     * Variables of elements.
80
     *
81
     * @var array
82
     */
83
    protected $variables = [];
84
85
    /**
86
     * Options for specify elements.
87
     *
88
     * @var array
89
     */
90
    protected $options = [];
91
92
    /**
93
     * Validation rules.
94
     *
95
     * @var string|\Closure
96
     */
97
    protected $rules = '';
98
99
    /**
100
     * @var callable
101
     */
102
    protected $validator;
103
104
    /**
105
     * Validation messages.
106
     *
107
     * @var array
108
     */
109
    protected $validationMessages = [];
110
111
    /**
112
     * Css required by this field.
113
     *
114
     * @var array
115
     */
116
    protected static $css = [];
117
118
    /**
119
     * Js required by this field.
120
     *
121
     * @var array
122
     */
123
    protected static $js = [];
124
125
    /**
126
     * Script for field.
127
     *
128
     * @var string
129
     */
130
    protected $script = '';
131
132
    /**
133
     * Element attributes.
134
     *
135
     * @var array
136
     */
137
    protected $attributes = [];
138
139
    /**
140
     * Parent form.
141
     *
142
     * @var Form
143
     */
144
    protected $form = null;
145
146
    /**
147
     * View for field to render.
148
     *
149
     * @var string
150
     */
151
    protected $view = '';
152
153
    /**
154
     * Help block.
155
     *
156
     * @var array
157
     */
158
    protected $help = [];
159
160
    /**
161
     * Key for errors.
162
     *
163
     * @var mixed
164
     */
165
    protected $errorKey;
166
167
    /**
168
     * Placeholder for this field.
169
     *
170
     * @var string|array
171
     */
172
    protected $placeholder;
173
174
    /**
175
     * Width for label and field.
176
     *
177
     * @var array
178
     */
179
    protected $width = [
180
        'label' => 2,
181
        'field' => 8,
182
    ];
183
184
    /**
185
     * If the form horizontal layout.
186
     *
187
     * @var bool
188
     */
189
    protected $horizontal = true;
190
191
    /**
192
     * @var bool
193
     */
194
    protected $display = true;
195
196
    /**
197
     * Field constructor.
198
     *
199
     * @param $column
200
     * @param array $arguments
201
     */
202
    public function __construct($column, $arguments = [])
203
    {
204
        $this->column = $column;
205
        $this->label = $this->formatLabel($arguments);
206
        $this->id = $this->formatId($column);
207
    }
208
209
    /**
210
     * Get assets required by this field.
211
     *
212
     * @return array
213
     */
214
    public static function getAssets()
215
    {
216
        return [
217
            'css' => static::$css,
218
            'js'  => static::$js,
219
        ];
220
    }
221
222
    /**
223
     * Format the field element id.
224
     *
225
     * @param string|array $column
226
     *
227
     * @return string|array
228
     */
229
    protected function formatId($column)
230
    {
231
        return str_replace('.', '_', $column);
232
    }
233
234
    /**
235
     * Format the label value.
236
     *
237
     * @param array $arguments
238
     *
239
     * @return string
240
     */
241
    protected function formatLabel($arguments = [])
242
    {
243
        $column = is_array($this->column) ? current($this->column) : $this->column;
244
245
        $label = isset($arguments[0]) ? $arguments[0] : ucfirst($column);
246
247
        return str_replace(['.', '_'], ' ', $label);
248
    }
249
250
    /**
251
     * Format the name of the field.
252
     *
253
     * @param string $column
254
     *
255
     * @return array|mixed|string
256
     */
257
    protected function formatName($column)
258
    {
259
        if (is_string($column)) {
260
            $name = explode('.', $column);
261
262
            if (count($name) == 1) {
263
                return $name[0];
264
            }
265
266
            $html = array_shift($name);
267
            foreach ($name as $piece) {
268
                $html .= "[$piece]";
269
            }
270
271
            return $html;
272
        }
273
274
        if (is_array($this->column)) {
275
            $names = [];
276
            foreach ($this->column as $key => $name) {
277
                $names[$key] = $this->formatName($name);
278
            }
279
280
            return $names;
281
        }
282
283
        return '';
284
    }
285
286
    /**
287
     * Set form element name.
288
     *
289
     * @param string $name
290
     *
291
     * @return $this
292
     *
293
     * @author Edwin Hui
294
     */
295
    public function setElementName($name)
296
    {
297
        $this->elementName = $name;
298
299
        return $this;
300
    }
301
302
    /**
303
     * Fill data to the field.
304
     *
305
     * @param array $data
306
     *
307
     * @return void
308
     */
309 View Code Duplication
    public function fill($data)
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
310
    {
311
        // Field value is already setted.
0 ignored issues
show
Unused Code Comprehensibility introduced by
37% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
312
//        if (!is_null($this->value)) {
313
//            return;
314
//        }
315
316
        if (is_array($this->column)) {
317
            foreach ($this->column as $key => $column) {
318
                $this->value[$key] = array_get($data, $column);
319
            }
320
321
            return;
322
        }
323
324
        $this->value = array_get($data, $this->column);
325
    }
326
327
    /**
328
     * Set original value to the field.
329
     *
330
     * @param array $data
331
     *
332
     * @return void
333
     */
334 View Code Duplication
    public function setOriginal($data)
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
335
    {
336
        if (is_array($this->column)) {
337
            foreach ($this->column as $key => $column) {
338
                $this->original[$key] = array_get($data, $column);
339
            }
340
341
            return;
342
        }
343
344
        $this->original = array_get($data, $this->column);
345
    }
346
347
    /**
348
     * @param Form $form
349
     *
350
     * @return $this
351
     */
352
    public function setForm(Form $form = null)
353
    {
354
        $this->form = $form;
355
356
        return $this;
357
    }
358
359
    /**
360
     * Set width for field and label.
361
     *
362
     * @param int $field
363
     * @param int $label
364
     *
365
     * @return $this
366
     */
367
    public function setWidth($field = 8, $label = 2)
368
    {
369
        $this->width = [
370
            'label' => $label,
371
            'field' => $field,
372
        ];
373
374
        return $this;
375
    }
376
377
    /**
378
     * Set the field options.
379
     *
380
     * @param array $options
381
     *
382
     * @return $this
383
     */
384 View Code Duplication
    public function options($options = [])
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
385
    {
386
        if ($options instanceof Arrayable) {
387
            $options = $options->toArray();
388
        }
389
390
        $this->options = array_merge($this->options, $options);
391
392
        return $this;
393
    }
394
395
    /**
396
     * Get or set rules.
397
     *
398
     * @param null  $rules
399
     * @param array $messages
400
     *
401
     * @return $this
402
     */
403
    public function rules($rules = null, $messages = [])
404
    {
405
        if ($rules instanceof \Closure) {
406
            $this->rules = $rules;
407
        }
408
409
        if (is_array($rules)) {
410
            $thisRuleArr = array_filter(explode('|', $this->rules));
411
412
            $this->rules = array_merge($thisRuleArr, $rules);
0 ignored issues
show
Documentation Bug introduced by
It seems like array_merge($thisRuleArr, $rules) of type array is incompatible with the declared type string|object<Closure> of property $rules.

Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property.

Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property..

Loading history...
413
        } elseif (is_string($rules)) {
414
            $rules = array_filter(explode('|', "{$this->rules}|$rules"));
415
416
            $this->rules = implode('|', $rules);
417
        }
418
419
        $this->validationMessages = $messages;
420
421
        return $this;
422
    }
423
424
    /**
425
     * Get field validation rules.
426
     *
427
     * @return string
428
     */
429
    protected function getRules()
430
    {
431
        if ($this->rules instanceof \Closure) {
432
            return $this->rules->call($this, $this->form);
433
        }
434
435
        return $this->rules;
436
    }
437
438
    /**
439
     * Remove a specific rule.
440
     *
441
     * @param string $rule
442
     *
443
     * @return void
444
     */
445
    protected function removeRule($rule)
446
    {
447
        $this->rules = str_replace($rule, '', $this->rules);
448
    }
449
450
    /**
451
     * Set field validator.
452
     *
453
     * @param callable $validator
454
     *
455
     * @return $this
456
     */
457
    public function validator(callable $validator)
458
    {
459
        $this->validator = $validator;
460
461
        return $this;
462
    }
463
464
    /**
465
     * Get key for error message.
466
     *
467
     * @return string
468
     */
469
    public function getErrorKey()
470
    {
471
        return $this->errorKey ?: $this->column;
472
    }
473
474
    /**
475
     * Set key for error message.
476
     *
477
     * @param string $key
478
     *
479
     * @return $this
480
     */
481
    public function setErrorKey($key)
482
    {
483
        $this->errorKey = $key;
484
485
        return $this;
486
    }
487
488
    /**
489
     * Set or get value of the field.
490
     *
491
     * @param null $value
492
     *
493
     * @return mixed
494
     */
495 View Code Duplication
    public function value($value = null)
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
496
    {
497
        if (is_null($value)) {
498
            return is_null($this->value) ? $this->getDefault() : $this->value;
499
        }
500
501
        $this->value = $value;
502
503
        return $this;
504
    }
505
506
    /**
507
     * Set default value for field.
508
     *
509
     * @param $default
510
     *
511
     * @return $this
512
     */
513
    public function default($default)
0 ignored issues
show
Coding Style introduced by
Possible parse error: non-abstract method defined as abstract
Loading history...
Coding Style introduced by
It is generally advisable to only define one property per statement.

Only declaring a single property per statement allows you to later on add doc comments more easily.

It is also recommended by PSR2, so it is a common style that many people expect.

Loading history...
514
    {
515
        $this->default = $default;
516
517
        return $this;
0 ignored issues
show
Coding Style introduced by
The visibility should be declared for property $this.

The PSR-2 coding standard requires that all properties in a class have their visibility explicitly declared. If you declare a property using

class A {
    var $property;
}

the property is implicitly global.

To learn more about the PSR-2, please see the PHP-FIG site on the PSR-2.

Loading history...
518
    }
519
520
    /**
521
     * Get default value.
522
     *
523
     * @return mixed
524
     */
525
    public function getDefault()
526
    {
527
        if ($this->default instanceof \Closure) {
528
            return call_user_func($this->default, $this->form);
529
        }
530
531
        return $this->default;
532
    }
533
534
    /**
535
     * Set help block for current field.
536
     *
537
     * @param string $text
538
     * @param string $icon
539
     *
540
     * @return $this
541
     */
542
    public function help($text = '', $icon = 'fa-info-circle')
543
    {
544
        $this->help = compact('text', 'icon');
545
546
        return $this;
547
    }
548
549
    /**
550
     * Get column of the field.
551
     *
552
     * @return string|array
553
     */
554
    public function column()
555
    {
556
        return $this->column;
557
    }
558
559
    /**
560
     * Get label of the field.
561
     *
562
     * @return string
563
     */
564
    public function label()
565
    {
566
        return $this->label;
567
    }
568
569
    /**
570
     * Get original value of the field.
571
     *
572
     * @return mixed
573
     */
574
    public function original()
575
    {
576
        return $this->original;
577
    }
578
579
    /**
580
     * Get validator for this field.
581
     *
582
     * @param array $input
583
     *
584
     * @return bool|Validator
585
     */
586
    public function getValidator(array $input)
587
    {
588
        if ($this->validator) {
589
            return $this->validator->call($this, $input);
590
        }
591
592
        $rules = $attributes = [];
593
594
        if (!$fieldRules = $this->getRules()) {
595
            return false;
596
        }
597
598
        if (is_string($this->column)) {
599
            if (!array_has($input, $this->column)) {
600
                return false;
601
            }
602
603
            $input = $this->sanitizeInput($input, $this->column);
604
605
            $rules[$this->column] = $fieldRules;
606
            $attributes[$this->column] = $this->label;
607
        }
608
609
        if (is_array($this->column)) {
610
            foreach ($this->column as $key => $column) {
611
                if (!array_key_exists($column, $input)) {
612
                    continue;
613
                }
614
                $input[$column.$key] = array_get($input, $column);
615
                $rules[$column.$key] = $fieldRules;
616
                $attributes[$column.$key] = $this->label."[$column]";
617
            }
618
        }
619
620
        return Validator::make($input, $rules, $this->validationMessages, $attributes);
0 ignored issues
show
Bug Best Practice introduced by
The return type of return \Illuminate\Suppo...Messages, $attributes); (Illuminate\Contracts\Validation\Validator) is incompatible with the return type documented by Encore\Admin\Form\Field::getValidator of type boolean|Illuminate\Support\Facades\Validator.

If you return a value from a function or method, it should be a sub-type of the type that is given by the parent type f.e. an interface, or abstract method. This is more formally defined by the Lizkov substitution principle, and guarantees that classes that depend on the parent type can use any instance of a child type interchangably. This principle also belongs to the SOLID principles for object oriented design.

Let’s take a look at an example:

class Author {
    private $name;

    public function __construct($name) {
        $this->name = $name;
    }

    public function getName() {
        return $this->name;
    }
}

abstract class Post {
    public function getAuthor() {
        return 'Johannes';
    }
}

class BlogPost extends Post {
    public function getAuthor() {
        return new Author('Johannes');
    }
}

class ForumPost extends Post { /* ... */ }

function my_function(Post $post) {
    echo strtoupper($post->getAuthor());
}

Our function my_function expects a Post object, and outputs the author of the post. The base class Post returns a simple string and outputting a simple string will work just fine. However, the child class BlogPost which is a sub-type of Post instead decided to return an object, and is therefore violating the SOLID principles. If a BlogPost were passed to my_function, PHP would not complain, but ultimately fail when executing the strtoupper call in its body.

Loading history...
621
    }
622
623
    /**
624
     * Sanitize input data.
625
     *
626
     * @param array  $input
627
     * @param string $column
628
     *
629
     * @return array
630
     */
631
    protected function sanitizeInput($input, $column)
632
    {
633
        if ($this instanceof Field\MultipleSelect) {
634
            $value = array_get($input, $column);
635
            array_set($input, $column, array_filter($value));
636
        }
637
638
        return $input;
639
    }
640
641
    /**
642
     * Add html attributes to elements.
643
     *
644
     * @param array|string $attribute
645
     * @param mixed        $value
646
     *
647
     * @return $this
648
     */
649
    public function attribute($attribute, $value = null)
650
    {
651
        if (is_array($attribute)) {
652
            $this->attributes = array_merge($this->attributes, $attribute);
653
        } else {
654
            $this->attributes[$attribute] = (string) $value;
655
        }
656
657
        return $this;
658
    }
659
660
    /**
661
     * Set the field as readonly mode.
662
     *
663
     * @return Field
664
     */
665
    public function readOnly()
666
    {
667
        return $this->attribute('disabled', true);
668
    }
669
670
    /**
671
     * Set field placeholder.
672
     *
673
     * @param string $placeholder
674
     *
675
     * @return Field
676
     */
677
    public function placeholder($placeholder = '')
678
    {
679
        $this->placeholder = $placeholder;
680
681
        return $this;
682
    }
683
684
    /**
685
     * Get placeholder.
686
     *
687
     * @return string
688
     */
689
    public function getPlaceholder()
690
    {
691
        return $this->placeholder ?: trans('admin.input').' '.$this->label;
692
    }
693
694
    /**
695
     * Prepare for a field value before update or insert.
696
     *
697
     * @param $value
698
     *
699
     * @return mixed
700
     */
701
    public function prepare($value)
702
    {
703
        return $value;
704
    }
705
706
    /**
707
     * Format the field attributes.
708
     *
709
     * @return string
710
     */
711
    protected function formatAttributes()
712
    {
713
        $html = [];
714
715
        foreach ($this->attributes as $name => $value) {
716
            $html[] = $name.'="'.e($value).'"';
717
        }
718
719
        return implode(' ', $html);
720
    }
721
722
    /**
723
     * @return $this
724
     */
725
    public function disableHorizontal()
726
    {
727
        $this->horizontal = false;
728
729
        return $this;
730
    }
731
732
    /**
733
     * @return array
734
     */
735
    public function getViewElementClasses()
736
    {
737
        if ($this->horizontal) {
738
            return [
739
                'label'      => "col-sm-{$this->width['label']}",
740
                'field'      => "col-sm-{$this->width['field']}",
741
                'form-group' => 'form-group ',
742
            ];
743
        }
744
745
        return ['label' => '', 'field' => '', 'form-group' => ''];
746
    }
747
748
    /**
749
     * Set form element class.
750
     *
751
     * @param string|array $class
752
     *
753
     * @return $this
754
     */
755
    public function setElementClass($class)
756
    {
757
        $this->elementClass = (array) $class;
758
759
        return $this;
760
    }
761
762
    /**
763
     * Get element class.
764
     *
765
     * @return array
766
     */
767
    protected function getElementClass()
768
    {
769
        if (!$this->elementClass) {
0 ignored issues
show
Bug Best Practice introduced by
The expression $this->elementClass of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent.

Consider making the comparison explicit by using empty(..) or ! empty(...) instead.

Loading history...
770
            $name = $this->elementName ?: $this->formatName($this->column);
0 ignored issues
show
Bug introduced by
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...
771
772
            $this->elementClass = (array) str_replace(['[', ']'], '_', $name);
773
        }
774
775
        return $this->elementClass;
776
    }
777
778
    /**
779
     * Get element class string.
780
     *
781
     * @return mixed
782
     */
783 View Code Duplication
    protected function getElementClassString()
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
784
    {
785
        $elementClass = $this->getElementClass();
786
787
        if (Arr::isAssoc($elementClass)) {
788
            $classes = [];
789
790
            foreach ($elementClass as $index => $class) {
791
                $classes[$index] = is_array($class) ? implode(' ', $class) : $class;
792
            }
793
794
            return $classes;
795
        }
796
797
        return implode(' ', $elementClass);
798
    }
799
800
    /**
801
     * Get element class selector.
802
     *
803
     * @return string
804
     */
805 View Code Duplication
    protected function getElementClassSelector()
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
806
    {
807
        $elementClass = $this->getElementClass();
808
809
        if (Arr::isAssoc($elementClass)) {
810
            $classes = [];
811
812
            foreach ($elementClass as $index => $class) {
813
                $classes[$index] = '.'.(is_array($class) ? implode('.', $class) : $class);
814
            }
815
816
            return $classes;
817
        }
818
819
        return '.'.implode('.', $elementClass);
820
    }
821
822
    /**
823
     * Add the element class.
824
     *
825
     * @param $class
826
     *
827
     * @return $this
828
     */
829
    public function addElementClass($class)
830
    {
831
        if (is_array($class) || is_string($class)) {
832
            $this->elementClass = array_merge($this->elementClass, (array) $class);
833
834
            $this->elementClass = array_unique($this->elementClass);
835
        }
836
837
        return $this;
838
    }
839
840
    /**
841
     * Remove element class.
842
     *
843
     * @param $class
844
     *
845
     * @return $this
846
     */
847
    public function removeElementClass($class)
848
    {
849
        $delClass = [];
850
851
        if (is_string($class) || is_array($class)) {
852
            $delClass = (array) $class;
853
        }
854
855
        foreach ($delClass as $del) {
856
            if (($key = array_search($del, $this->elementClass))) {
857
                unset($this->elementClass[$key]);
858
            }
859
        }
860
861
        return $this;
862
    }
863
864
    /**
865
     * Add variables to field view.
866
     *
867
     * @param array $variables
868
     * @return $this
869
     */
870
    protected function addVariables(array $variables = [])
871
    {
872
        $this->variables = array_merge($this->variables, $variables);
873
874
        return $this;
875
    }
876
877
    /**
878
     * Get the view variables of this field.
879
     *
880
     * @return array
881
     */
882
    protected function variables()
883
    {
884
        return array_merge($this->variables, [
885
            'id'          => $this->id,
886
            'name'        => $this->elementName ?: $this->formatName($this->column),
0 ignored issues
show
Bug introduced by
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...
887
            'help'        => $this->help,
888
            'class'       => $this->getElementClassString(),
889
            'value'       => $this->value(),
890
            'label'       => $this->label,
891
            'viewClass'   => $this->getViewElementClasses(),
892
            'column'      => $this->column,
893
            'errorKey'    => $this->getErrorKey(),
894
            'attributes'  => $this->formatAttributes(),
895
            'placeholder' => $this->getPlaceholder(),
896
        ]);
897
    }
898
899
    /**
900
     * Get view of this field.
901
     *
902
     * @return string
903
     */
904
    public function getView()
905
    {
906
        if (!empty($this->view)) {
907
            return $this->view;
908
        }
909
910
        $class = explode('\\', get_called_class());
911
912
        return 'admin::form.'.strtolower(end($class));
913
    }
914
915
    /**
916
     * Get script of current field.
917
     *
918
     * @return string
919
     */
920
    public function getScript()
921
    {
922
        return $this->script;
923
    }
924
925
    /**
926
     * Render this filed.
927
     *
928
     * @return \Illuminate\Contracts\View\Factory|\Illuminate\View\View
929
     */
930
    public function render()
931
    {
932
        if (!$this->display) {
933
            return '';
934
        }
935
936
        Admin::script($this->script);
937
938
        return view($this->getView(), $this->variables());
0 ignored issues
show
Bug Compatibility introduced by
The expression view($this->getView(), $this->variables()); of type Illuminate\View\View|Ill...\Contracts\View\Factory adds the type Illuminate\Contracts\View\Factory to the return on line 938 which is incompatible with the return type declared by the interface Illuminate\Contracts\Support\Renderable::render of type string.
Loading history...
939
    }
940
941
    /**
942
     * @return string
943
     */
944
    public function __toString()
945
    {
946
        return $this->render()->render();
0 ignored issues
show
Bug introduced by
The method render does only exist in Illuminate\View\View, but not in Illuminate\Contracts\View\Factory.

It seems like the method you are trying to call exists only in some of the possible types.

Let’s take a look at an example:

class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}

Available Fixes

  1. Add an additional type-check:

    /**
     * @param A|B $x
     */
    function someFunction($x)
    {
        $x->foo();
    
        if ($x instanceof B) {
            $x->bar();
        }
    }
    
  2. Only allow a single type to be passed if the variable comes from a parameter:

    function someFunction(B $x) { /** ... */ }
    
Loading history...
947
    }
948
}
949