Skip to content

Commit 2abc52e

Browse files
committed
more fixing for insight
1 parent 5460f42 commit 2abc52e

6 files changed

+9
-15
lines changed

.gitignore

-9
Original file line numberDiff line numberDiff line change
@@ -24,15 +24,6 @@ composer.phar
2424

2525
/.idea
2626

27-
# OS generated files #
28-
######################
29-
.DS_Store
30-
.DS_Store?
31-
._*
32-
.Spotlight-V100
33-
.Trashes
34-
ehthumbs.db
35-
Thumbs.db
3627

3728
# Keep .gitkeep #
3829
#################

.scrutinizer.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
checks:
22
php:
33
code_rating: true
4-
duplication: true
4+
duplication: true

Command/BaseBootstrapSymlinkCommand.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,7 @@ protected function getBootstrapPathsFromUser()
217217
;
218218
$this->output->writeln(array(
219219
'',
220-
$this->getHelperSet()->get('formatter')->formatBlock($text, $style = 'bg=blue;fg=white', true),
220+
$this->getHelperSet()->get('formatter')->formatBlock($text, 'bg=blue;fg=white', true),
221221
'',
222222
));
223223

Form/Extension/StaticTextExtension.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ public function configureOptions(OptionsResolver $resolver)
6060
public function buildView(FormView $view, FormInterface $form, array $options)
6161
{
6262
$view->vars['static_text'] = $options['static_text'];
63-
if ($options['static_text'] == true) {
63+
if ($options['static_text'] === true) {
6464
$view->vars['disabled'] = true;
6565
}
6666
}

Form/Extension/WidgetCollectionFormTypeExtension.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ public function __construct(array $options)
4646
public function buildView(FormView $view, FormInterface $form, array $options)
4747
{
4848
if (in_array('collection', $view->vars['block_prefixes'])) {
49-
if ($options['widget_add_btn'] != null && !is_array($options['widget_add_btn'])) {
49+
if ($options['widget_add_btn'] !== null && !is_array($options['widget_add_btn'])) {
5050
throw new InvalidArgumentException('The "widget_add_btn" option must be an "array".');
5151
}
5252

@@ -59,7 +59,7 @@ public function buildView(FormView $view, FormInterface $form, array $options)
5959
}
6060

6161
if ($view->parent && in_array('collection', $view->parent->vars['block_prefixes'])) {
62-
if ($options['widget_remove_btn'] != null && !is_array($options['widget_remove_btn'])) {
62+
if ($options['widget_remove_btn'] !== null && !is_array($options['widget_remove_btn'])) {
6363
throw new InvalidArgumentException('The "widget_remove_btn" option must be an "array".');
6464
}
6565

Form/Type/FormActionsType.php

+4-1
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,10 @@ public function buildForm(FormBuilderInterface $builder, array $options)
4141
protected function createButton($builder, $name, $config)
4242
{
4343
$options = (isset($config['options'])) ? $config['options'] : array();
44-
$button = $builder->add($name, $config['type'], $options);
44+
45+
$builder->add($name, $config['type'], $options);
46+
47+
return $builder;
4548
}
4649

4750
/**

0 commit comments

Comments
 (0)