superdesk / web-publisher

src/SWP/Bundle/CoreBundle/Theme/Loader/TenantAwareThemeLoader.php

Spacing +3 added lines, -3 removed lines

@@ -161,7 +161,7 @@ 
     {
         $tenantCode = substr($themeName, strpos($themeName, '@') + 1);
 
-        return array_map(function ($parentName) use ($themeName, $existingThemes, $tenantCode) {
+        return array_map(function($parentName) use ($themeName, $existingThemes, $tenantCode) {
             $parentName .= '@'.$tenantCode;
             if (!isset($existingThemes[$parentName])) {
                 throw new ThemeLoadingFailedException(sprintf(
@@ -182,7 +182,7 @@ 
      */
     private function convertAuthorsArraysToAuthorsObjects(array $authorsArrays)
     {
-        return array_map(function (array $authorArray) {
+        return array_map(function(array $authorArray) {
             return $this->themeAuthorFactory->createFromArray($authorArray);
         }, $authorsArrays);
     }
@@ -194,7 +194,7 @@ 
      */
     private function convertScreenshotsArraysToScreenshotsObjects(array $screenshotsArrays)
     {
-        return array_map(function (array $screenshotArray) {
+        return array_map(function(array $screenshotArray) {
             return $this->themeScreenshotFactory->createFromArray($screenshotArray);
         }, $screenshotsArrays);
     }

src/SWP/Bundle/TemplatesSystemBundle/Form/Type/WidgetType.php

Spacing +4 added lines, -4 removed lines

@@ -39,10 +39,10 @@ 
 
         $builder->get('visible')
             ->addModelTransformer(new CallbackTransformer(
-                function ($value) {
+                function($value) {
                     return $value;
                 },
-                function ($value) {
+                function($value) {
                     if (is_bool($value) || in_array($value, ['true', 'false', '1', '0', null])) {
                         return filter_var($value, FILTER_VALIDATE_BOOLEAN);
                     }
@@ -53,14 +53,14 @@ 
 
         $builder->get('parameters')
             ->addModelTransformer(new CallbackTransformer(
-                function ($value) {
+                function($value) {
                     if (is_array($value) && !empty($value)) {
                         return json_encode($value);
                     }
 
                     return $value;
                 },
-                function ($value) {
+                function($value) {
                     if (is_string($value)) {
                         return json_decode($value, true);
                     }

src/SWP/Bundle/FixturesBundle/DataFixtures/ORM/LoadArticlesData.php

Doc Comments +9 added lines

@@ -41,6 +41,10 @@ 
         $manager->flush();
     }
 
+    /**
+     * @param string $env
+     * @param ObjectManager $manager
+     */
     public function loadRoutes($env, $manager)
     {
         $routes = [
@@ -195,6 +199,7 @@ 
     /**
      * Sets articles manually (not via Alice) for test env due to fatal error:
      * Method PHPCRProxies\__CG__\Doctrine\ODM\PHPCR\Document\Generic::__toString() must not throw an exception.
+     * @param string $env
      */
     public function loadArticles($env, ObjectManager $manager)
     {
@@ -446,6 +451,10 @@ 
         ];
     }
 
+    /**
+     * @param string $fakeImage
+     * @param string $targetFile
+     */
     private function cropAndResizeImage($fakeImage, array $rendition, $targetFile)
     {
         $image = imagecreatefromjpeg($fakeImage);

src/SWP/Bundle/CoreBundle/Serializer/TenantHandler.php

Indentation +7 added lines, -7 removed lines

@@ -74,17 +74,17 @@
         }
 
         return [
-           'id' => $tenant->getId(),
-           'subdomain' => $tenant->getSubdomain(),
-           'domainName' => $tenant->getDomainName(),
-           'code' => $tenantCode,
-           'name' => $tenant->getName(),
-           'ampEnabled' => $tenant->isAmpEnabled(),
+            'id' => $tenant->getId(),
+            'subdomain' => $tenant->getSubdomain(),
+            'domainName' => $tenant->getDomainName(),
+            'code' => $tenantCode,
+            'name' => $tenant->getName(),
+            'ampEnabled' => $tenant->isAmpEnabled(),
             '_links' => [
                 'self' => [
                     'href' => $this->router->generate('swp_api_core_get_tenant', ['code' => $tenantCode]),
                 ],
             ],
-       ];
+        ];
     }
 }

src/SWP/Bundle/CoreBundle/Factory/ApiKeyFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
      * @param UserInterface $user
      * @param string|null   $apiKeyValue
      *
-     * @return mixed
+     * @return ApiKeyInterface
      */
     public function create($user, $apiKeyValue = null)
     {

src/SWP/Bundle/CoreBundle/Controller/ThemesController.php

Spacing +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
         $currentTheme = $tenant->getThemeName();
         $themes = array_filter(
             $this->get('sylius.repository.theme')->findAll(),
-            function ($element) use (&$tenantCode, $currentTheme) {
+            function($element) use (&$tenantCode, $currentTheme) {
                 if (strpos($element->getName(), ThemeHelper::SUFFIX_SEPARATOR.$tenantCode)) {
                     return true;
                 }

Unused Use Statements -1 removed lines

@@ -17,7 +17,6 @@
 use Knp\Component\Pager\Pagination\SlidingPagination;
 use SWP\Bundle\CoreBundle\Form\Type\ThemeInstallType;
 use SWP\Bundle\CoreBundle\Form\Type\ThemeUploadType;
-use SWP\Bundle\CoreBundle\Model\Tenant;
 use SWP\Bundle\CoreBundle\Model\TenantInterface;
 use SWP\Bundle\CoreBundle\Theme\Helper\ThemeHelper;
 use SWP\Component\Common\Response\ResponseContext;

src/SWP/Bundle/FixturesBundle/AbstractFixture.php

Doc Comments +7 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
     /**
      * Loads Alice fixtures.
      *
-     * @param array|string  $paths      Fixtures path(s)
+     * @param string[]  $paths      Fixtures path(s)
      * @param ObjectManager $manager    Object manager
      * @param array         $parameters Extra parameters
      */
@@ -112,6 +112,11 @@ 
         return $this->getTenantPrefix(self::DEFAULT_TENANT_DOMAIN).'/'.ltrim($id, '/');
     }
 
+    /**
+     * @param string $id
+     *
+     * @return null|\SWP\Bundle\ContentBundle\Model\RouteInterface
+     */
     public function getRouteByName($id)
     {
         return $this->container->get('swp.provider.route')->getRouteByName($id);
@@ -120,7 +125,7 @@ 
     /**
      * Gets current tenant's prefix.
      *
-     * @param string $subdomain
+     * @param string $domain
      *
      * @return string
      */

src/SWP/Bundle/ContentBundle/Doctrine/ORM/ArticleRepository.php

Unused Use Statements -1 removed lines

@@ -17,7 +17,6 @@
 namespace SWP\Bundle\ContentBundle\Doctrine\ORM;
 
 use Doctrine\ORM\QueryBuilder;
-use Elastica\Query;
 use SWP\Bundle\ContentBundle\Model\ArticleSourceReference;
 use SWP\Component\Common\Criteria\Criteria;
 use SWP\Bundle\ContentBundle\Doctrine\ArticleRepositoryInterface;

src/SWP/Bundle/CoreBundle/Theme/Generator/ThemeContainersGenerator.php

Doc Comments +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
     /**
      * @param array $containerData
      *
-     * @return mixed
+     * @return null|\SWP\Component\TemplatesSystem\Gimme\Model\ContainerInterface
      *
      * @throws \Exception
      */