5260: url changes

CHANGELOG.md

@@ -8,6 +8,9 @@ See [keep a changelog] for information about writing changes to this log.
 
 ## [Unreleased]
 
+[PR-38](https://github.com/itk-dev/itqr/pull/38)
+    - Limit number of URLs to 1
+    - Increase URL data type from varchar to text
 [PR-37](https://github.com/itk-dev/itqr/pull/37)
     - Rename "theme" to "design" in general
     - Rename "delete" to "archive" in general

assets/app.js

@@ -1,17 +1,21 @@
 import './styles/app.css';
 
 const uploadBasePath = 'uploads/qr_codes/';
+
 document.addEventListener('DOMContentLoaded', () => {
+    // Enforce only one URL per QR code.
+    handleQrUrlCollection();
+
     const qrCodeContainer = document.getElementById('qrCodeContainer');
     const tabsContainer = document.getElementById('qrCodeTabs'); // Navigation tabs
     const tabContentContainer = document.getElementById('qrCodeTabContent'); // Tab content
     const form = document.querySelector('.form-wrapper form');
-    const formName = form.getAttribute('name');
+    const formName = form ? form.getAttribute('name') : '';
     const selectedQrCodes = document.getElementById('selectedQrCodes');
 
     // Ensure all containers and elements exist
     if (!qrCodeContainer || !tabsContainer || !tabContentContainer || !form) {
-        console.error('Required elements not found!');
+        console.log('Required elements not found');
         return;
     }
 
@@ -200,6 +204,37 @@ function handleBatchDisableConfirm() {
     });
 }
 
+function handleQrUrlCollection() {
+    const qrUrlCollectionParent = document.querySelector('.qr-urls-collection');
+    const qrUrlCollectionAddButton = document.querySelector('.qr-urls-collection .field-collection-add-button');
+    const urlCollectionCount = qrUrlCollectionParent ? parseInt(qrUrlCollectionParent.getAttribute('data-num-items')) : null;
+
+    // If the button exists, and no URL is added - add it!
+    if (qrUrlCollectionAddButton) {
+        setTimeout(() => {
+            if (urlCollectionCount !== null && urlCollectionCount === 0) {
+                qrUrlCollectionAddButton.click();
+            }
+        }, 1)
+
+        // Hide the add-button if there is already an URL added.
+        if (parseInt(urlCollectionCount) === 1) {
+            qrUrlCollectionAddButton.classList.add('d-none');
+        }
+
+        // Handle the click event and hide add-button if an URL was added and the number of URLs is 1.
+        qrUrlCollectionAddButton.addEventListener('click', () => {
+            setTimeout(() => {
+                const urlCollectionCount = qrUrlCollectionParent.getAttribute('data-num-items');
+
+                if (parseInt(urlCollectionCount) === 1) {
+                    qrUrlCollectionAddButton.classList.add('d-none');
+                }
+            }, 1)
+        });
+    }
+}
+
 document.addEventListener('readystatechange', function(event) {
     if ('complete' === document.readyState) {
         handleBatchDisableConfirm();

assets/styles/app.css

@@ -101,3 +101,8 @@ form[name="batch_download"] .accordion-button {
     background-color: #fff;
     border-bottom: 1px solid #e9e9e9;
 }
+
+
+.qr-urls-collection .accordion-button {
+    display: none;
+}

migrations/Version20250826113021.php

@@ -0,0 +1,31 @@
+<?php
+
+declare(strict_types=1);
+
+namespace DoctrineMigrations;
+
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+/**
+ * Auto-generated Migration: Please modify to your needs!
+ */
+final class Version20250826113021 extends AbstractMigration
+{
+    public function getDescription(): string
+    {
+        return '';
+    }
+
+    public function up(Schema $schema): void
+    {
+        // this up() migration is auto-generated, please modify it to your needs
+        $this->addSql('ALTER TABLE url CHANGE url url TEXT NOT NULL');
+    }
+
+    public function down(Schema $schema): void
+    {
+        // this down() migration is auto-generated, please modify it to your needs
+        $this->addSql('ALTER TABLE url CHANGE url url VARCHAR(255) NOT NULL');
+    }
+}

src/Controller/Admin/QrCrudController.php

@@ -4,6 +4,7 @@
 
 use App\Controller\Admin\Embed\UrlCrudController;
 use App\Entity\Tenant\Qr;
+use App\Entity\Tenant\Url;
 use App\Helper\DownloadHelper;
 use App\Repository\QrHitTrackerRepository;
 use EasyCorp\Bundle\EasyAdminBundle\Config\Action;
@@ -97,9 +98,10 @@ public function configureFields(string $pageName): iterable
                 TextEditorField::new('description', new TranslatableMessage('qr.description')),
                 CollectionField::new('urls', new TranslatableMessage('qr.urls'))
                     ->allowAdd()
-                    ->allowDelete()
-                    ->renderExpanded()
-                    ->useEntryCrudForm(UrlCrudController::class),
+                    ->allowDelete(false)
+                    ->renderExpanded(true)
+                    ->useEntryCrudForm(UrlCrudController::class)
+                    ->addCssClass('qr-urls-collection'),
             ];
         }
 
@@ -115,9 +117,10 @@ public function configureFields(string $pageName): iterable
                 TextEditorField::new('description', new TranslatableMessage('qr.description')),
                 CollectionField::new('urls', new TranslatableMessage('qr.urls'))
                     ->allowAdd()
-                    ->allowDelete()
+                    ->allowDelete(false)
+                    ->useEntryCrudForm(UrlCrudController::class)
                     ->renderExpanded()
-                    ->useEntryCrudForm(UrlCrudController::class),
+                    ->addCssClass('qr-urls-collection'),
             ];
         }
 

src/Entity/Tenant/Url.php

@@ -13,7 +13,7 @@ class Url extends AbstractTenantScopedEntity
 {
     #[Assert\NotBlank(message: new TranslatableMessage('The URL field cannot be empty.'))]
     #[Assert\Url(message: new TranslatableMessage('The value "{{ value }}" is not a valid URL.'))]
-    #[ORM\Column(length: 255)]
+    #[ORM\Column(type: 'text', length: 65535)]
     private string $url = '';
 
     #[ORM\ManyToOne(targetEntity: Qr::class, inversedBy: 'urls')]