Piou-piou / RibsAdminBundle

Service/Version.php

Indentation +167 added lines, -167 removed lines

@@ -13,171 +13,171 @@
 
 class Version
 {
-    /**
-     * @var EntityManagerInterface
-     */
-    private $em;
-
-    /**
-     * @var HttpClientInterface
-     */
-    private $client;
-
-    /**
-     * @var Package
-     */
-    private $package;
-
-    private $messages = [];
-
-    /**
-     * Version constructor.
-     * @param EntityManagerInterface $em
-     * @param HttpClientInterface $client
-     */
-    public function __construct(EntityManagerInterface $em, HttpClientInterface $client)
-    {
-        $this->em = $em;
-        $this->client = $client;
-    }
-
-    /**
-     * @return array
-     */
-    public function getMessages(): array
-    {
-        return $this->messages;
-    }
-
-    /**
-     * @param Package $package
-     */
-    public function setPackageEntity(Package $package) {
-        $this->package = $package;
-    }
-
-    /**
-     * @return mixed|null
-     * @throws ClientExceptionInterface
-     * @throws RedirectionExceptionInterface
-     * @throws ServerExceptionInterface
-     * @throws TransportExceptionInterface
-     */
-    private function getComposerLockJson()
-    {
-        if ($this->package && !$this->package->isIsLocal()) {
-            $response = $this->client->request("GET", $this->package->getProjectUrl().$this->package->getComposerLockUrl());
-            $composer_lock = $response->getStatusCode() == 200 ? $response->getContent() : null;
-        } else {
-            $composer_lock = file_get_contents('../composer.lock');
-        }
-
-        if ($composer_lock) {
-            return json_decode($composer_lock, true);
-        }
-
-        return null;
-    }
-
-    /**
-     * @param $package_name
-     * @return mixed|null
-     * @throws ClientExceptionInterface
-     * @throws RedirectionExceptionInterface
-     * @throws ServerExceptionInterface
-     * @throws TransportExceptionInterface
-     */
-    public function getPackage($package_name)
-    {
-        $composer_lock = $this->getComposerLockJson();
-        if ($composer_lock) {
-            $packages = $composer_lock["packages"];
-            $key = array_search($package_name, array_column($packages, 'name'));
-
-            if ($key) {
-                return $packages[$key];
-            }
-        }
-
-        $this->messages["composer_lock"] = "Composer lock not found at " . $this->package->getProjectUrl();
-
-        return null;
-    }
-
-    /**
-     * @param $package_name
-     * @return mixed|null
-     */
-    public function getVersion($package_name)
-    {
-        return $this->getPackage($package_name) ? $this->getPackage($package_name)["version"] : null;
-    }
-
-    /**
-     * @param $package_name
-     * @return mixed|null
-     * @throws Exception
-     */
-    public function getVersionDate($package_name)
-    {
-        $string_date = $this->getPackage($package_name) ? explode("T", $this->getPackage($package_name)["time"])[0] : null;
-        $version_date = null;
-
-        if ($string_date) {
-            $version_date = new \DateTime($string_date);
-        }
-
-        return $version_date;
-    }
-
-    /**
-     * @param $package_name
-     * @return false|int|string|null
-     * @throws ClientExceptionInterface
-     * @throws RedirectionExceptionInterface
-     * @throws ServerExceptionInterface
-     * @throws TransportExceptionInterface
-     */
-    public function getLastPackagistVersion($package_name)
-    {
-        if (!strpos($package_name, "/")) {
-            return false;
-        }
-
-        $packgist_url = "https://repo.packagist.org/p/".$package_name.".json";
-
-        $response = $this->client->request("GET", $packgist_url);
-
-        if ($response->getStatusCode() == 200) {
-            $content = json_decode($response->getContent(), true);
-            if (is_array($content) && $content["packages"] && $content["packages"][$package_name]) {
-                return array_key_first($content["packages"][$package_name]);
-            }
-        }
-    }
-
-    /**
-     * @param $package_name
-     * @throws Exception
-     */
-    public function save($package_name)
-    {
-        $package = $this->em->getRepository(Package::class)->findOneBy(["package_name" => $package_name]);
-
-        if (!$package) {
-            $package = new Package();
-            $package->setProjectName($package_name);
-            $package->setPackageName($package_name);
-            $package->setProjectUrl($package_name);
-            $package->setCheckVersionUrl($package_name);
-        }
-
-        $package->setVersion($this->getVersion($package_name));
-        $package->setVersionDate($this->getVersionDate($package_name));
-        $package->setLastPackagistVersion($this->getLastPackagistVersion($package_name));
-        $package->setLastCheck(new \DateTime());
-
-        $this->em->persist($package);
-        $this->em->flush();
-    }
+				/**
+				 * @var EntityManagerInterface
+				 */
+				private $em;
+
+				/**
+				 * @var HttpClientInterface
+				 */
+				private $client;
+
+				/**
+				 * @var Package
+				 */
+				private $package;
+
+				private $messages = [];
+
+				/**
+				 * Version constructor.
+				 * @param EntityManagerInterface $em
+				 * @param HttpClientInterface $client
+				 */
+				public function __construct(EntityManagerInterface $em, HttpClientInterface $client)
+				{
+								$this->em = $em;
+								$this->client = $client;
+				}
+
+				/**
+				 * @return array
+				 */
+				public function getMessages(): array
+				{
+								return $this->messages;
+				}
+
+				/**
+				 * @param Package $package
+				 */
+				public function setPackageEntity(Package $package) {
+								$this->package = $package;
+				}
+
+				/**
+				 * @return mixed|null
+				 * @throws ClientExceptionInterface
+				 * @throws RedirectionExceptionInterface
+				 * @throws ServerExceptionInterface
+				 * @throws TransportExceptionInterface
+				 */
+				private function getComposerLockJson()
+				{
+								if ($this->package && !$this->package->isIsLocal()) {
+												$response = $this->client->request("GET", $this->package->getProjectUrl().$this->package->getComposerLockUrl());
+												$composer_lock = $response->getStatusCode() == 200 ? $response->getContent() : null;
+								} else {
+												$composer_lock = file_get_contents('../composer.lock');
+								}
+
+								if ($composer_lock) {
+												return json_decode($composer_lock, true);
+								}
+
+								return null;
+				}
+
+				/**
+				 * @param $package_name
+				 * @return mixed|null
+				 * @throws ClientExceptionInterface
+				 * @throws RedirectionExceptionInterface
+				 * @throws ServerExceptionInterface
+				 * @throws TransportExceptionInterface
+				 */
+				public function getPackage($package_name)
+				{
+								$composer_lock = $this->getComposerLockJson();
+								if ($composer_lock) {
+												$packages = $composer_lock["packages"];
+												$key = array_search($package_name, array_column($packages, 'name'));
+
+												if ($key) {
+																return $packages[$key];
+												}
+								}
+
+								$this->messages["composer_lock"] = "Composer lock not found at " . $this->package->getProjectUrl();
+
+								return null;
+				}
+
+				/**
+				 * @param $package_name
+				 * @return mixed|null
+				 */
+				public function getVersion($package_name)
+				{
+								return $this->getPackage($package_name) ? $this->getPackage($package_name)["version"] : null;
+				}
+
+				/**
+				 * @param $package_name
+				 * @return mixed|null
+				 * @throws Exception
+				 */
+				public function getVersionDate($package_name)
+				{
+								$string_date = $this->getPackage($package_name) ? explode("T", $this->getPackage($package_name)["time"])[0] : null;
+								$version_date = null;
+
+								if ($string_date) {
+												$version_date = new \DateTime($string_date);
+								}
+
+								return $version_date;
+				}
+
+				/**
+				 * @param $package_name
+				 * @return false|int|string|null
+				 * @throws ClientExceptionInterface
+				 * @throws RedirectionExceptionInterface
+				 * @throws ServerExceptionInterface
+				 * @throws TransportExceptionInterface
+				 */
+				public function getLastPackagistVersion($package_name)
+				{
+								if (!strpos($package_name, "/")) {
+												return false;
+								}
+
+								$packgist_url = "https://repo.packagist.org/p/".$package_name.".json";
+
+								$response = $this->client->request("GET", $packgist_url);
+
+								if ($response->getStatusCode() == 200) {
+												$content = json_decode($response->getContent(), true);
+												if (is_array($content) && $content["packages"] && $content["packages"][$package_name]) {
+																return array_key_first($content["packages"][$package_name]);
+												}
+								}
+				}
+
+				/**
+				 * @param $package_name
+				 * @throws Exception
+				 */
+				public function save($package_name)
+				{
+								$package = $this->em->getRepository(Package::class)->findOneBy(["package_name" => $package_name]);
+
+								if (!$package) {
+												$package = new Package();
+												$package->setProjectName($package_name);
+												$package->setPackageName($package_name);
+												$package->setProjectUrl($package_name);
+												$package->setCheckVersionUrl($package_name);
+								}
+
+								$package->setVersion($this->getVersion($package_name));
+								$package->setVersionDate($this->getVersionDate($package_name));
+								$package->setLastPackagistVersion($this->getLastPackagistVersion($package_name));
+								$package->setLastCheck(new \DateTime());
+
+								$this->em->persist($package);
+								$this->em->flush();
+				}
 }

Controller/PackageController.php

Indentation +142 added lines, -142 removed lines

@@ -16,147 +16,147 @@
 
 class PackageController extends AbstractController
 {
-    /**
-     * @Route("/packages/", name="ribsadmin_packages")
-     * @param EntityManagerInterface $em
-     * @return Response
-     */
-    public function index(EntityManagerInterface $em): Response
-    {
-        $packages = $em->getRepository(Package::class)->findAll();
+				/**
+				 * @Route("/packages/", name="ribsadmin_packages")
+				 * @param EntityManagerInterface $em
+				 * @return Response
+				 */
+				public function index(EntityManagerInterface $em): Response
+				{
+								$packages = $em->getRepository(Package::class)->findAll();
         
-        return $this->render("@RibsAdmin/packages/list.html.twig", ["packages" => $packages]);
-    }
-
-    /**
-     * @Route("/packages/create/", name="ribsadmin_packages_create")
-     * @Route("/packages/show/{guid}", name="ribsadmin_packages_show")
-     * @Route("/packages/edit/{guid}", name="ribsadmin_packages_edit")
-     * @param Request $request
-     * @param EntityManagerInterface $em
-     * @param string|null $guid
-     * @return Response
-     */
-    public function edit(Request $request, EntityManagerInterface $em, string $guid = null): Response
-    {
-        $disabled_form = strpos($request->get("_route"), "_show") ? true : false;
-        if (!$guid) {
-            $package = new Package();
-            $text = "created";
-        } else {
-            $package = $em->getRepository(Package::class)->findOneBy(["guid" => $guid]);
-            $text = "edited";
-        }
-
-        $form = $this->createForm(\PiouPiou\RibsAdminBundle\Form\Package::class, $package, ["disabled" => $disabled_form]);
-        $form->handleRequest($request);
-
-        if ($form->isSubmitted() && $form->isValid()) {
-            /** @var Package $data */
-            $data = $form->getData();
-            $em->persist($data);
-            $em->flush();
-            $this->addFlash("success-flash", "Package " . $data->getProjectName() . " was " . $text);
-
-            return $this->redirectToRoute("ribsadmin_packages");
-        }
-
-        return $this->render("@RibsAdmin/packages/edit.html.twig", [
-            "form" => $form->createView(),
-            "form_errors" => $form->getErrors(),
-            "package" => $package
-        ]);
-    }
-
-    /**
-     * @Route("/packages/delete/{guid}", name="ribsadmin_packages_delete")
-     * @param EntityManagerInterface $em
-     * @param string $guid
-     * @return RedirectResponse
-     */
-    public function delete(EntityManagerInterface $em, string $guid): RedirectResponse
-    {
-        $package = $em->getRepository(Package::class)->findOneBy(["guid" => $guid]);
-
-        if ($package) {
-            $em->remove($package);
-            $em->flush();
-            $this->addFlash("success-flash", "The project package was deleted");
-        } else {
-            $this->addFlash("error-flash", "The project package was not found");
-        }
-
-        return $this->redirectToRoute("ribsadmin_packages");
-    }
-
-    /**
-     * @Route("/packages/update/{guid}", name="ribsadmin_packages_update")
-     * @param EntityManagerInterface $em
-     * @param Version $version
-     * @param string $guid
-     * @return RedirectResponse
-     * @throws Exception
-     */
-    public function updatePackage(EntityManagerInterface $em, Version $version, string $guid): RedirectResponse
-    {
-        $package = $em->getRepository(Package::class)->findOneBy(["guid" => $guid]);
-
-        if ($package) {
-            $version->setPackageEntity($package);
-            $version->save($package->getPackageName());
-
-            if ($version->getMessages()) {
-                $message = "<ul>";
-                $message .= "<li>The project package was not well updated</li>";
-                foreach ($version->getMessages() as $version_message) {
-                    $message .= "<li>".$version_message."</li>";
-                }
-                $message .= "</ul>";
-
-                $this->addFlash("info-flash", $message);
-            }
-
-            $this->addFlash("success-flash", "The project package was updated");
-        } else {
-            $this->addFlash("error-flash", "The project package was not found");
-        }
-
-        return $this->redirectToRoute("ribsadmin_packages");
-    }
-
-    /**
-     * @Route("/packages/send-package/{package_name}", name="ribsadmin_packages_send", requirements={"package_name"=".+"})
-     * @param EntityManagerInterface $em
-     * @param Version $version
-     * @param string $package_name
-     * @return mixed|null
-     */
-    public function sendPackageInformations(EntityManagerInterface $em, Version $version, string $package_name): JsonResponse
-    {
-        $package = $em->getRepository(Package::class)->findOneBy(["package_name" => $package_name]);
-
-        if ($package) {
-            $version->setPackageEntity($package);
-        }
-
-        return new JsonResponse([
-            "package" => $version->getPackage($package_name),
-            "package_date" => $version->getPackageDate($package_name)
-        ]);
-    }
-
-    /**
-     * @Route("/packages/send-composer-lock/", name="ribsadmin_packages_send_composer_lock")
-     * @return JsonResponse
-     */
-    public function sendComposerJson(): JsonResponse
-    {
-        $composer_lock = file_get_contents('../composer.lock');
-
-        if ($composer_lock) {
-            $composer_lock = json_decode($composer_lock);
-        }
-
-        return new JsonResponse($composer_lock);
-    }
+								return $this->render("@RibsAdmin/packages/list.html.twig", ["packages" => $packages]);
+				}
+
+				/**
+				 * @Route("/packages/create/", name="ribsadmin_packages_create")
+				 * @Route("/packages/show/{guid}", name="ribsadmin_packages_show")
+				 * @Route("/packages/edit/{guid}", name="ribsadmin_packages_edit")
+				 * @param Request $request
+				 * @param EntityManagerInterface $em
+				 * @param string|null $guid
+				 * @return Response
+				 */
+				public function edit(Request $request, EntityManagerInterface $em, string $guid = null): Response
+				{
+								$disabled_form = strpos($request->get("_route"), "_show") ? true : false;
+								if (!$guid) {
+												$package = new Package();
+												$text = "created";
+								} else {
+												$package = $em->getRepository(Package::class)->findOneBy(["guid" => $guid]);
+												$text = "edited";
+								}
+
+								$form = $this->createForm(\PiouPiou\RibsAdminBundle\Form\Package::class, $package, ["disabled" => $disabled_form]);
+								$form->handleRequest($request);
+
+								if ($form->isSubmitted() && $form->isValid()) {
+												/** @var Package $data */
+												$data = $form->getData();
+												$em->persist($data);
+												$em->flush();
+												$this->addFlash("success-flash", "Package " . $data->getProjectName() . " was " . $text);
+
+												return $this->redirectToRoute("ribsadmin_packages");
+								}
+
+								return $this->render("@RibsAdmin/packages/edit.html.twig", [
+												"form" => $form->createView(),
+												"form_errors" => $form->getErrors(),
+												"package" => $package
+								]);
+				}
+
+				/**
+				 * @Route("/packages/delete/{guid}", name="ribsadmin_packages_delete")
+				 * @param EntityManagerInterface $em
+				 * @param string $guid
+				 * @return RedirectResponse
+				 */
+				public function delete(EntityManagerInterface $em, string $guid): RedirectResponse
+				{
+								$package = $em->getRepository(Package::class)->findOneBy(["guid" => $guid]);
+
+								if ($package) {
+												$em->remove($package);
+												$em->flush();
+												$this->addFlash("success-flash", "The project package was deleted");
+								} else {
+												$this->addFlash("error-flash", "The project package was not found");
+								}
+
+								return $this->redirectToRoute("ribsadmin_packages");
+				}
+
+				/**
+				 * @Route("/packages/update/{guid}", name="ribsadmin_packages_update")
+				 * @param EntityManagerInterface $em
+				 * @param Version $version
+				 * @param string $guid
+				 * @return RedirectResponse
+				 * @throws Exception
+				 */
+				public function updatePackage(EntityManagerInterface $em, Version $version, string $guid): RedirectResponse
+				{
+								$package = $em->getRepository(Package::class)->findOneBy(["guid" => $guid]);
+
+								if ($package) {
+												$version->setPackageEntity($package);
+												$version->save($package->getPackageName());
+
+												if ($version->getMessages()) {
+																$message = "<ul>";
+																$message .= "<li>The project package was not well updated</li>";
+																foreach ($version->getMessages() as $version_message) {
+																				$message .= "<li>".$version_message."</li>";
+																}
+																$message .= "</ul>";
+
+																$this->addFlash("info-flash", $message);
+												}
+
+												$this->addFlash("success-flash", "The project package was updated");
+								} else {
+												$this->addFlash("error-flash", "The project package was not found");
+								}
+
+								return $this->redirectToRoute("ribsadmin_packages");
+				}
+
+				/**
+				 * @Route("/packages/send-package/{package_name}", name="ribsadmin_packages_send", requirements={"package_name"=".+"})
+				 * @param EntityManagerInterface $em
+				 * @param Version $version
+				 * @param string $package_name
+				 * @return mixed|null
+				 */
+				public function sendPackageInformations(EntityManagerInterface $em, Version $version, string $package_name): JsonResponse
+				{
+								$package = $em->getRepository(Package::class)->findOneBy(["package_name" => $package_name]);
+
+								if ($package) {
+												$version->setPackageEntity($package);
+								}
+
+								return new JsonResponse([
+												"package" => $version->getPackage($package_name),
+												"package_date" => $version->getPackageDate($package_name)
+								]);
+				}
+
+				/**
+				 * @Route("/packages/send-composer-lock/", name="ribsadmin_packages_send_composer_lock")
+				 * @return JsonResponse
+				 */
+				public function sendComposerJson(): JsonResponse
+				{
+								$composer_lock = file_get_contents('../composer.lock');
+
+								if ($composer_lock) {
+												$composer_lock = json_decode($composer_lock);
+								}
+
+								return new JsonResponse($composer_lock);
+				}
 }

Spacing +1 added lines, -1 removed lines

@@ -109,7 +109,7 @@
                 $message = "<ul>";
                 $message .= "<li>The project package was not well updated</li>";
                 foreach ($version->getMessages() as $version_message) {
-                    $message .= "<li>".$version_message."</li>";
+                    $message .= "<li>" . $version_message . "</li>";
                 }
                 $message .= "</ul>";