Skip to content

Commit

Permalink
Merge pull request #3353 from acrobat/fix-phpcs-issues
Browse files Browse the repository at this point in the history
[AllBundles] Fix php-cs-fixer issues
  • Loading branch information
acrobat authored Mar 23, 2024
2 parents c1ce5eb + d37c748 commit bf9d238
Show file tree
Hide file tree
Showing 24 changed files with 72 additions and 72 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ public function buildListActions()
$listRoute = [
'path' => 'kunstmaanadminbundle_admin_exception_resolve_all',
'params' => [],
];
];

$this->addListAction(
new SimpleListAction(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,8 @@ public function checkAction(Request $request, $id, $repository)
public static function getSubscribedServices(): array
{
return [
'kunstmaan_entity.admin_entity.entity_version_lock_service' => EntityVersionLockService::class,
'translator' => TranslatorInterface::class,
] + parent::getSubscribedServices();
'kunstmaan_entity.admin_entity.entity_version_lock_service' => EntityVersionLockService::class,
'translator' => TranslatorInterface::class,
] + parent::getSubscribedServices();
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -53,8 +53,8 @@ public function testApply($comparator, $whereClause, $value, $testValue)
public static function applyDataProvider(): array
{
return [
['in', 'IN (:var_enumeration)', [1, 2], true],
['notin', 'NOT IN (:var_enumeration)', [1, 2], true],
['in', 'IN (:var_enumeration)', [1, 2], true],
['notin', 'NOT IN (:var_enumeration)', [1, 2], true],
];
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -85,8 +85,8 @@ public function testApplyReturnsNull()

$badData = [
'value' => [
'date' => 'oopsNotADate',
'time' => 'oopsNotATime',
'date' => 'oopsNotADate',
'time' => 'oopsNotATime',
],
'comparator' => 'true',
];
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -53,8 +53,8 @@ public function testApply($comparator, $whereClause, $value, $testValue)
public static function applyDataProvider(): array
{
return [
['in', 'IN(:var_enumeration)', [1, 2], true],
['notin', 'NOT IN(:var_enumeration)', [1, 2], true],
['in', 'IN(:var_enumeration)', [1, 2], true],
['notin', 'NOT IN(:var_enumeration)', [1, 2], true],
];
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -111,9 +111,9 @@ public function getAccounts()

foreach ($accounts as $account) {
$data[$account->getName()] = [
'accountId' => $account->getId(),
'accountName' => $account->getName(),
];
'accountId' => $account->getId(),
'accountName' => $account->getName(),
];
}
ksort($data);

Expand Down Expand Up @@ -183,9 +183,9 @@ public function getProperties($accountId = false)
$profiles = $this->getProfiles($accountId, $property->getId());
if (\count($profiles) > 0) {
$data[$property->getName()] = [
'propertyId' => $property->getId(),
'propertyName' => $property->getName() . ' (' . $property->getWebsiteUrl() . ')',
];
'propertyId' => $property->getId(),
'propertyName' => $property->getName() . ' (' . $property->getWebsiteUrl() . ')',
];
}
}
ksort($data);
Expand Down Expand Up @@ -262,10 +262,10 @@ public function getProfiles($accountId = false, $propertyId = false)
if (\is_array($profiles->getItems())) {
foreach ($profiles->getItems() as $profile) {
$data[$profile->name] = [
'profileId' => $profile->id,
'profileName' => $profile->name,
'created' => $profile->created,
];
'profileId' => $profile->id,
'profileName' => $profile->name,
'created' => $profile->created,
];
}
}
ksort($data);
Expand Down Expand Up @@ -352,14 +352,14 @@ public function getProfileSegments()
foreach ($profileSegments as $segment) {
if ($segment->type == 'BUILT_IN') {
$builtin[] = [
'name' => $segment->name,
'query' => $segment->segmentId,
];
'name' => $segment->name,
'query' => $segment->segmentId,
];
} else {
$own[] = [
'name' => $segment->name,
'query' => $segment->segmentId,
];
'name' => $segment->name,
'query' => $segment->segmentId,
];
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,9 +42,9 @@ public function findDefaultOverviews($config)

return $em->getRepository(AnalyticsOverview::class)
->findBy([
'config' => $config,
'segment' => null,
]);
'config' => $config,
'segment' => null,
]);
}

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -193,13 +193,13 @@ public function deleteAction(Request $request, $id)
public static function getSubscribedServices(): array
{
return [
'kunstmaan_admin.acl.helper' => AclHelper::class,
'kunstmaan_adminlist.factory' => AdminListFactory::class,
'kunstmaan_adminlist.service.export' => ExportService::class,
'request_stack' => RequestStack::class,
'translator' => TranslatorInterface::class,
'logger' => LoggerInterface::class,
'kunstmaan_utilities.slugifier' => SlugifierInterface::class,
] + parent::getSubscribedServices();
'kunstmaan_admin.acl.helper' => AclHelper::class,
'kunstmaan_adminlist.factory' => AdminListFactory::class,
'kunstmaan_adminlist.service.export' => ExportService::class,
'request_stack' => RequestStack::class,
'translator' => TranslatorInterface::class,
'logger' => LoggerInterface::class,
'kunstmaan_utilities.slugifier' => SlugifierInterface::class,
] + parent::getSubscribedServices();
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -95,9 +95,9 @@ protected function doExecute()

$this->assistant->writeSection('PageParts successfully created', 'bg=green;fg=black');
$this->assistant->writeLine([
'Make sure you update your database first before you test the pagepart:',
' Directly update your database: <comment>bin/console doctrine:schema:update --force</comment>',
' Create a Doctrine migration and run it: <comment>bin/console doctrine:migrations:diff && bin/console doctrine:migrations:migrate</comment>', ]
'Make sure you update your database first before you test the pagepart:',
' Directly update your database: <comment>bin/console doctrine:schema:update --force</comment>',
' Create a Doctrine migration and run it: <comment>bin/console doctrine:migrations:diff && bin/console doctrine:migrations:migrate</comment>', ]
);

return 0;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -81,9 +81,9 @@ protected function execute(InputInterface $input, OutputInterface $output)
$generator->generate($bundle, $prefix, $rootDir, $createPage, $output);

$output->writeln([
'Make sure you update your database first before you test the pagepart:',
' Directly update your database: <comment>bin/console doctrine:schema:update --force</comment>',
' Create a Doctrine migration and run it: <comment>bin/console doctrine:migrations:diff && bin/console doctrine:migrations:migrate</comment>', ]
'Make sure you update your database first before you test the pagepart:',
' Directly update your database: <comment>bin/console doctrine:schema:update --force</comment>',
' Create a Doctrine migration and run it: <comment>bin/console doctrine:migrations:diff && bin/console doctrine:migrations:migrate</comment>', ]
);

if ($createPage) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ public function testCorrectParametersHaveBeenSet()
$this->load([
'popup_types' => ['abc'],
'debug' => true,
]);
]);

$this->assertContainerBuilderHasParameter('kunstmaan_lead_generation.popup_types');
$this->assertContainerBuilderHasParameter('kunstmaan_lead_generation.debug', true);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,8 @@ public function __construct($redirectUrl)
public function getUrlFor($item)
{
return [
'path' => 'KunstmaanMediaBundle_media_delete',
'params' => ['mediaId' => $item->getId(), 'redirectUrl' => $this->redirectUrl],
'path' => 'KunstmaanMediaBundle_media_delete',
'params' => ['mediaId' => $item->getId(), 'redirectUrl' => $this->redirectUrl],
];
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@ class MediaEditItemAction implements ItemActionInterface
public function getUrlFor($item)
{
return [
'path' => 'KunstmaanMediaBundle_media_show',
'params' => ['mediaId' => $item->getId()],
'path' => 'KunstmaanMediaBundle_media_show',
'params' => ['mediaId' => $item->getId()],
];
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ public function getBlockPrefix()
public function configureOptions(OptionsResolver $resolver)
{
$resolver->setDefaults([
'data_class' => 'Kunstmaan\MediaPagePartBundle\Entity\AudioPagePart',
'data_class' => 'Kunstmaan\MediaPagePartBundle\Entity\AudioPagePart',
]);
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ public function getBlockPrefix()
public function configureOptions(OptionsResolver $resolver)
{
$resolver->setDefaults([
'data_class' => 'Kunstmaan\MediaPagePartBundle\Entity\DownloadPagePart',
'data_class' => 'Kunstmaan\MediaPagePartBundle\Entity\DownloadPagePart',
]);
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ public function getBlockPrefix()
public function configureOptions(OptionsResolver $resolver)
{
$resolver->setDefaults([
'data_class' => 'Kunstmaan\MediaPagePartBundle\Entity\ImagePagePart',
'data_class' => 'Kunstmaan\MediaPagePartBundle\Entity\ImagePagePart',
]);
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ public function getBlockPrefix()
public function configureOptions(OptionsResolver $resolver)
{
$resolver->setDefaults([
'data_class' => 'Kunstmaan\MediaPagePartBundle\Entity\SlidePagePart',
'data_class' => 'Kunstmaan\MediaPagePartBundle\Entity\SlidePagePart',
]);
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ public function getBlockPrefix()
public function configureOptions(OptionsResolver $resolver)
{
$resolver->setDefaults([
'data_class' => 'Kunstmaan\MediaPagePartBundle\Entity\VideoPagePart',
'data_class' => 'Kunstmaan\MediaPagePartBundle\Entity\VideoPagePart',
]);
}
}
2 changes: 1 addition & 1 deletion src/Kunstmaan/NodeBundle/Form/PageAdminType.php
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ public function buildForm(FormBuilderInterface $builder, array $options)
public function configureOptions(OptionsResolver $resolver)
{
$resolver->setDefaults([
'data_class' => 'Kunstmaan\NodeBundle\Entity\AbstractPage',
'data_class' => 'Kunstmaan\NodeBundle\Entity\AbstractPage',
]);
}

Expand Down
8 changes: 4 additions & 4 deletions src/Kunstmaan/PagePartBundle/Form/HeaderPagePartAdminType.php
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,9 @@ public function buildForm(FormBuilderInterface $builder, array $options)
'niv',
ChoiceType::class,
[
'label' => 'pagepart.header.type',
'choices' => ['Header 1' => '1', 'Header 2' => '2', 'Header 3' => '3', 'Header 4' => '4', 'Header 5' => '5', 'Header 6' => '6'],
'required' => true,
'label' => 'pagepart.header.type',
'choices' => ['Header 1' => '1', 'Header 2' => '2', 'Header 3' => '3', 'Header 4' => '4', 'Header 5' => '5', 'Header 6' => '6'],
'required' => true,
]
);
$builder->add('title', TextType::class, [
Expand All @@ -48,7 +48,7 @@ public function configureOptions(OptionsResolver $resolver)
{
$resolver->setDefaults(
[
'data_class' => 'Kunstmaan\PagePartBundle\Entity\HeaderPagePart',
'data_class' => 'Kunstmaan\PagePartBundle\Entity\HeaderPagePart',
]
);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ public function configureOptions(OptionsResolver $resolver)
{
$resolver->setDefaults(
[
'data_class' => 'Kunstmaan\PagePartBundle\Entity\LinePagePart',
'data_class' => 'Kunstmaan\PagePartBundle\Entity\LinePagePart',
]
);
}
Expand Down
12 changes: 6 additions & 6 deletions src/Kunstmaan/PagePartBundle/PagePartAdmin/Builder.php
Original file line number Diff line number Diff line change
Expand Up @@ -13,12 +13,12 @@ class Builder
public function getPageParts()
{
$pageParts = [['name' => 'Header', 'class' => 'Kunstmaan\PagePartBundle\Entity\HeaderPagePart'],
['name' => 'Text', 'class' => 'Kunstmaan\PagePartBundle\Entity\TextPagePart'],
['name' => 'Link', 'class' => 'Kunstmaan\PagePartBundle\Entity\LinkPagePart'],
['name' => 'Raw HTML', 'class' => 'Kunstmaan\PagePartBundle\Entity\RawHTMLPagePart'],
['name' => 'Line', 'class' => 'Kunstmaan\PagePartBundle\Entity\LinePagePart'],
['name' => 'TOC', 'class' => 'Kunstmaan\PagePartBundle\Entity\TocPagePart'],
['name' => 'Link To Top', 'class' => 'Kunstmaan\PagePartBundle\Entity\ToTopPagePart'], ];
['name' => 'Text', 'class' => 'Kunstmaan\PagePartBundle\Entity\TextPagePart'],
['name' => 'Link', 'class' => 'Kunstmaan\PagePartBundle\Entity\LinkPagePart'],
['name' => 'Raw HTML', 'class' => 'Kunstmaan\PagePartBundle\Entity\RawHTMLPagePart'],
['name' => 'Line', 'class' => 'Kunstmaan\PagePartBundle\Entity\LinePagePart'],
['name' => 'TOC', 'class' => 'Kunstmaan\PagePartBundle\Entity\TocPagePart'],
['name' => 'Link To Top', 'class' => 'Kunstmaan\PagePartBundle\Entity\ToTopPagePart'], ];

return $pageParts;
}
Expand Down
2 changes: 1 addition & 1 deletion src/Kunstmaan/SeoBundle/Form/SeoType.php
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@ public function getBlockPrefix()
public function configureOptions(OptionsResolver $resolver)
{
$resolver->setDefaults([
'data_class' => 'Kunstmaan\SeoBundle\Entity\Seo',
'data_class' => 'Kunstmaan\SeoBundle\Entity\Seo',
]);
}
}
12 changes: 6 additions & 6 deletions src/Kunstmaan/TaggingBundle/Form/TagsAdminType.php
Original file line number Diff line number Diff line change
Expand Up @@ -44,12 +44,12 @@ public function getDefaultOptions(array $options)
}

return [
'choices' => $result,
'multiple' => true,
'required' => false,
'attr' => [
'class' => 'js-advanced-select form-control advanced-select',
],
'choices' => $result,
'multiple' => true,
'required' => false,
'attr' => [
'class' => 'js-advanced-select form-control advanced-select',
],
];
}

Expand Down

0 comments on commit bf9d238

Please sign in to comment.