Merge pull request #273 from 4c0n/feature/allow-for-setting-page-title

Adding option to set the page title

Author: bitbager

features/admin/adding_page.feature

@@ -8,6 +8,16 @@ Feature: Adding new page
         Given the store operates on a single channel in "United States"
         And I am logged in as an administrator
 
+    @ui @javascript @title
+    Scenario: Adding page with title
+        When I go to the create page page
+        And I fill the code with "page_with_title"
+        And I fill the slug with "page_with_title"
+        And I fill the name with "page_with_title"
+        And I fill the content with "This is a page with a title set by us."
+        And I add it
+        Then I should be notified that the page has been created
+
     @ui @javascript
     Scenario: Adding page
         When I go to the create page page

features/shop/displaying_page.feature

@@ -48,3 +48,24 @@ Feature: Displaying pages
         And I should also see "iPhone 8" and "iPhone X" products associated with this page
         And I should also see "Blog" and "General" sections associated with this page
         And I should also see page image
+
+    @ui @javascript @title
+    Scenario: Displaying page with title
+        Given there is a page in the store
+        And this page has "some_page_with_title" code
+        And this page has "some_page_with_title" name
+        And this page also has "some_page_with_title" slug
+        And this page also has "Lorem ipsum" content
+        And this page also has "title" title
+        When I go to this page
+        Then I should see page title "title"
+
+    @ui @javascript @title
+    Scenario: Displaying page with default title
+        Given there is a page in the store
+        And this page has "title" code
+        And this page has "Title" name
+        And this page also has "title" slug
+        And this page also has "Lorem ipsum" content
+        When I go to this page
+        Then I should see page title "Sylius"

spec/Entity/PageTranslationSpec.php

@@ -65,5 +65,8 @@ function it_allows_access_via_properties(PageImageInterface $pageImage): void
 
         $this->setDescriptionWhenLinked('description linked');
         $this->getDescriptionWhenLinked()->shouldReturn('description linked');
+
+        $this->setTitle('title');
+        $this->getTitle()->shouldReturn('title');
     }
 }

src/Entity/Page.php

@@ -156,6 +156,16 @@ public function setImage(?ImageInterface $image): void
         $this->getPageTranslation()->setImage($image);
     }
 
+    public function getTitle(): ?string
+    {
+        return $this->getPageTranslation()->getTitle();
+    }
+
+    public function setTitle(?string $title): void
+    {
+        $this->getPageTranslation()->setTitle($title);
+    }
+
     /**
      * @return PageTranslationInterface|TranslationInterface|null
      */

src/Entity/PageInterface.php

@@ -63,4 +63,8 @@ public function setBreadcrumb(?string $breadcrumb): void;
     public function getDescriptionWhenLinked(): ?string;
 
     public function setDescriptionWhenLinked(?string $descriptionWhenLinked): void;
+
+    public function getTitle(): ?string;
+
+    public function setTitle(?string $title): void;
 }

src/Entity/PageTranslation.php

@@ -46,6 +46,9 @@ class PageTranslation extends AbstractTranslation implements PageTranslationInte
     /** @var string */
     protected $breadcrumb;
 
+    /** @var string */
+    protected $title;
+
     public function getId(): ?int
     {
         return $this->id;
@@ -144,4 +147,14 @@ public function setMetaDescription(?string $metaDescription): void
     {
         $this->metaDescription = $metaDescription;
     }
+
+    public function getTitle(): ?string
+    {
+        return $this->title;
+    }
+
+    public function setTitle(?string $title): void
+    {
+        $this->title = $title;
+    }
 }

src/Entity/PageTranslationInterface.php

@@ -52,4 +52,8 @@ public function setMetaDescription(?string $metaDescription): void;
     public function getImage(): ?PageImageInterface;
 
     public function setImage(?PageImageInterface $image): void;
+
+    public function getTitle(): ?string;
+
+    public function setTitle(?string $title): void;
 }

src/Form/Type/Translation/PageTranslationType.php

@@ -55,6 +55,10 @@ public function buildForm(FormBuilderInterface $builder, array $options): void
                 'required' => false,
             ])
             ->add('content', WysiwygType::class)
+            ->add('title', TextType::class, [
+                'label' => 'bitbag_sylius_cms_plugin.ui.title',
+                'required' => false,
+            ])
         ;
     }
 

src/Resources/config/doctrine/PageTranslation.orm.yml

@@ -43,6 +43,10 @@ BitBag\SyliusCmsPlugin\Entity\PageTranslation:
             column: content
             type: text
             nullable: true
+        title:
+            column: title
+            type: string
+            nullable: true
     oneToOne:
         image:
             targetEntity: BitBag\SyliusCmsPlugin\Entity\PageImageInterface

src/Resources/translations/messages.cs.yml

@@ -2,6 +2,7 @@ bitbag_sylius_cms_plugin:
     ui:
         blocks: Bloky
         pages: Stránky
+        title: Titul
     cms:
         content_management: Obsahový management
         cms: Obsahový management