src/Controller/SnipController.php

@@ -22,8 +22,7 @@ class SnipController extends AbstractController
 {
     public function __construct(
         private readonly SnipRepository $repository,
-    ) {
-    }
+    ) {}
 
     #[Route('/', name: '_index')]
     public function index(#[MapQueryCached] SnipFilterRequest $request): Response
@@ -67,7 +66,8 @@ class SnipController extends AbstractController
             ->setVary(['Accept', 'Accept-Encoding'])
             ->setEtag(md5($response->getContent()))
             ->setTtl(3600)
-            ->setClientTtl(300);
+            ->setClientTtl(300)
+        ;
 
         if (!$request->isNoCache()) {
             $response->isNotModified($request);
@@ -90,9 +90,11 @@ class SnipController extends AbstractController
             $this->addFlash('error', 'Snip is not the latest version, changes will not be saved.');
         }
 
-        $form = $this->createForm(SnipType::class, $snip)
-            ->add('Save', SubmitType::class);
-        $form->get('content')->setData($snip->getActiveText());
+        $form = $this->createForm(SnipType::class, $snip);
+        $form->add('Save', SubmitType::class);
+        if ($snip->getId()) {
+            $form->get('content')->setData($snip->getActiveText());
+        }
 
         $form->handleRequest($request);
         if ($form->isSubmitted() && $form->isValid()) {
@@ -126,31 +128,10 @@ class SnipController extends AbstractController
     {
         $snip = new Snip();
         $snip->setCreatedAtNow()
-            ->setCreatedBy($this->getUser());
+             ->setCreatedBy($this->getUser())
+        ;
 
-        $form = $this->createForm(SnipType::class, $snip);
-        $form->add('Create', SubmitType::class);
-
-        $form->handleRequest($request);
-        if ($form->isSubmitted() && $form->isValid()) {
-            $this->repository->save($snip);
-            $contentService->update(
-                $snip,
-                $form->get('content')->getData(),
-                $form->get('contentName')->getData()
-            );
-
-            $this->addFlash('success', sprintf('Snip "%s" created', $snip));
-
-            return $this->redirectToRoute('snip_single', [
-                'snip' => $snip->getId(),
-            ]);
-        }
-
-        return $this->render('snip/create.html.twig', [
-            'snip' => $snip,
-            'form' => $form->createView(),
-        ]);
+        return $this->edit($snip, $request, $contentService);
     }
 
     #[Route('/delete/{snip}', name: '_delete')]

src/Service/SnipParser/Markdown/MarkdownParser.php

@@ -33,6 +33,7 @@ class MarkdownParser extends AbstractParser
                 ],
                 Link::class => [
                     'class' => 'btn btn-sm btn-secondary',
+                    'target' => '_blank',
                 ],
             ],
         ];
@@ -61,7 +62,7 @@ class MarkdownParser extends AbstractParser
         foreach ($linkNodes as $linkNode) {
             $url = $linkNode->getUrl();
 
-            if (!is_numeric($url)) {
+            if (!is_int($url)) {
                 continue;
             }
 

templates/snip/create.html.twig

@@ -1,7 +0,0 @@
-{% extends 'base/one.column.html.twig' %}
-
-{% set title = 'Create Snip' %}
-
-{% block body %}
-    {{ form(form) }}
-{% endblock %}

templates/snip/edit.html.twig

@@ -1,6 +1,10 @@
 {% extends 'snip/base.html.twig' %}
 
-{% set title %}{{ snip }} - Edit{% endset %}
+{% if snip.id %}
+    {% set title %}{{ snip }} - Edit{% endset %}
+{% else %}
+    {% set title = 'Create Snip' %}
+{% endif %}
 {% set active = 'edit' %}
 
 {% block buttons %}