X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=src%2FWallabag%2FCoreBundle%2FController%2FTagController.php;h=e8e9ecbee0893f2620e3191ff9ea562e67b93a85;hb=f2e5fdc3666a2a6525b4202ab48df05efeebaf5c;hp=7b34939d2495847074fbf270f436ff6849d4580e;hpb=9b5edf33a00490b033692efca9987a35225835ba;p=github%2Fwallabag%2Fwallabag.git diff --git a/src/Wallabag/CoreBundle/Controller/TagController.php b/src/Wallabag/CoreBundle/Controller/TagController.php index 7b34939d..e8e9ecbe 100644 --- a/src/Wallabag/CoreBundle/Controller/TagController.php +++ b/src/Wallabag/CoreBundle/Controller/TagController.php @@ -13,6 +13,7 @@ class TagController extends Controller { /** * @param Request $request + * @param Entry $entry * * @Route("/new-tag/{entry}", requirements={"entry" = "\d+"}, name="new_tag") * @@ -20,39 +21,31 @@ class TagController extends Controller */ public function addTagFormAction(Request $request, Entry $entry) { - $tag = new Tag(); - $form = $this->createForm(NewTagType::class, $tag); + $form = $this->createForm(NewTagType::class, new Tag()); $form->handleRequest($request); if ($form->isValid()) { - $existingTag = $this->getDoctrine() - ->getRepository('WallabagCoreBundle:Tag') - ->findOneByLabel($tag->getLabel()); + $this->get('wallabag_core.content_proxy')->assignTagsToEntry( + $entry, + $form->get('label')->getData() + ); $em = $this->getDoctrine()->getManager(); - - if (is_null($existingTag)) { - $entry->addTag($tag); - $em->persist($tag); - } elseif (!$existingTag->hasEntry($entry)) { - $entry->addTag($existingTag); - $em->persist($existingTag); - } - + $em->persist($entry); $em->flush(); $this->get('session')->getFlashBag()->add( 'notice', - 'Tag added' + 'flashes.tag.notice.tag_added' ); - return $this->redirect($this->generateUrl('view', array('id' => $entry->getId()))); + return $this->redirect($this->generateUrl('view', ['id' => $entry->getId()])); } - return $this->render('WallabagCoreBundle:Tag:new_form.html.twig', array( + return $this->render('WallabagCoreBundle:Tag:new_form.html.twig', [ 'form' => $form->createView(), 'entry' => $entry, - )); + ]); } /** @@ -90,9 +83,9 @@ class TagController extends Controller return $this->render( 'WallabagCoreBundle:Tag:tags.html.twig', - array( + [ 'tags' => $tags, - ) + ] ); } }