superdesk / web-publisher

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

Doc Comments -84 removed lines

@@ -133,87 +133,3 @@
                 return new SingleResourceResponse([
                     'status' => 401,
                     'message' => <<<'MESSAGE'
-Unauthorized (user not found in Superdesk). 
-Make sure that Publisher can talk to Superdesk instance. Set it's address in "SUPERDESK_SERVERS" environment variable.
-MESSAGE,
-                ], new ResponseContext(401));
-            }
-
-            $publisherUser = $userProvider->findOneByEmail($superdeskUser['email']);
-            if (null === $publisherUser) {
-                try {
-                    $publisherUser = $userProvider->loadUserByUsername($superdeskUser['username']);
-                } catch (UsernameNotFoundException $e) {
-                    $publisherUser = null;
-                }
-            }
-
-            if (null === $publisherUser) {
-                /** @var UserInterface $publisherUser */
-                $publisherUser = $userManager->createUser();
-                $publisherUser->setUsername($superdeskUser['username']);
-                $publisherUser->setEmail($superdeskUser['email']);
-                $publisherUser->setRoles(['ROLE_INTERNAL_API']);
-                $publisherUser->setFirstName(\array_key_exists('first_name', $superdeskUser) ? $superdeskUser['first_name'] : 'Anon.');
-                $publisherUser->setLastName(\array_key_exists('last_name', $superdeskUser) ? $superdeskUser['last_name'] : '');
-                $publisherUser->setPassword(password_hash(random_bytes(36), PASSWORD_BCRYPT));
-                $userManager->updateUser($publisherUser);
-            }
-
-            if (null !== $publisherUser) {
-                return $this->returnApiTokenResponse($publisherUser, str_replace('Basic ', '', $formData['token']));
-            }
-        }
-
-        return new SingleResourceResponse([
-            'status' => 401,
-            'message' => 'Unauthorized',
-        ], new ResponseContext(401));
-    }
-
-    private function returnApiTokenResponse(UserInterface $user, string $token = null): SingleResourceResponseInterface
-    {
-        /** @var ApiKeyInterface $apiKey */
-        $apiKey = $this->generateOrGetApiKey($user, $token);
-
-        return new SingleResourceResponse([
-            'token' => [
-                'api_key' => $apiKey->getApiKey(),
-                'valid_to' => $apiKey->getValidTo(),
-            ],
-            'user' => $user,
-        ]);
-    }
-
-    private function generateOrGetApiKey(UserInterface $user, $token): ?ApiKeyInterface
-    {
-        $apiKey = null;
-        if (null !== $token) {
-            $apiKey = $this->apiKeyRepository->getValidToken($token)->getQuery()->getOneOrNullResult();
-        } else {
-            $validKeys = $this->apiKeyRepository->getValidTokenForUser($user)->getQuery()->getResult();
-            if (count($validKeys) > 0) {
-                $apiKey = reset($validKeys);
-            }
-        }
-
-        if (null === $apiKey) {
-            $apiKey = $this->apiKeyFactory->create($user, $token);
-
-            try {
-                $lock = $this->lockFactory->createLock(md5(json_encode(['type' => 'user_api_key', 'user' => $user->getId()])), 2);
-                if (!$lock->acquire()) {
-                    throw new RuntimeException('Other api key is created right now for this user');
-                }
-                $this->apiKeyRepository->add($apiKey);
-                $lock->release();
-            } catch (RuntimeException $e) {
-                sleep(2);
-
-                return $this->generateOrGetApiKey($user, $token);
-            }
-        }
-
-        return $apiKey;
-    }
-}

src/SWP/Bundle/CoreBundle/Theme/Model/ThemeInterface.php

Doc Comments +7 added lines

@@ -20,6 +20,7 @@ 
 {
     /**
      * @param string $name
+     * @return void
      */
     public function setName($name);
 
@@ -50,6 +51,7 @@ 
 
     /**
      * @param array $settings
+     * @return void
      */
     public function setSettings(array $settings): void;
 
@@ -60,6 +62,7 @@ 
 
     /**
      * @param \SplFileInfo|null $file
+     * @return void
      */
     public function setLogo(?\SplFileInfo $file): void;
 
@@ -75,8 +78,12 @@ 
 
     /**
      * @param string|null $path
+     * @return void
      */
     public function setLogoPath(?string $path): void;
 
+    /**
+     * @return void
+     */
     public function setGeneratedData(array $data): void;
 }