@reldens/cms 0.57.0 → 0.59.0

package/admin/reldens-admin-client.css

@@ -832,6 +832,8 @@
     .extra-content-container, .default-room-container {
         display: flex;
         flex-direction: column;
+        width: 100%;
+        box-sizing: border-box;
         padding: 1rem 0;
         margin-top: 1rem;
         text-align: left;

package/admin/reldens-admin-client.js

@@ -177,8 +177,7 @@ window.addEventListener('DOMContentLoaded', () => {
                         params.set('sortBy', columnName);
                         params.set('sortDirection', sortDirection);
                     }
-                    let newUrl = url.pathname+'?'+params;
-                    window.location.href = newUrl;
+                    window.location.href = url.pathname+'?'+params;
                     return false;
                 });
             }
@@ -190,44 +189,35 @@ window.addEventListener('DOMContentLoaded', () => {
     if(sortableHeaders){
         for(let header of sortableHeaders){
             header.addEventListener('click', () => {
+                let sortForm = header.querySelector('.sort-form');
+                if(!sortForm){
+                    return;
+                }
                 let columnName = header.getAttribute('data-column');
-                let currentSortDirection = header.classList.contains('sorted-asc') ? 'asc' : header.classList.contains('sorted-desc') ? 'desc' : '';
+                let currentSortDirection = header.classList.contains('sorted-asc')
+                    ? 'asc'
+                    : header.classList.contains('sorted-desc') ? 'desc' : '';
                 let newSortDirection = 'asc';
                 if('asc' === currentSortDirection){
                     newSortDirection = 'desc';
                 }
-                let sortForm = document.createElement('form');
-                sortForm.method = 'POST';
-                sortForm.action = window.location.pathname;
-                let sortByInput = document.createElement('input');
-                sortByInput.type = 'hidden';
-                sortByInput.name = 'sortBy';
+                let sortByInput = sortForm.querySelector('input[name="sortBy"]');
+                let sortDirectionInput = sortForm.querySelector('input[name="sortDirection"]');
                 sortByInput.value = columnName;
-                sortForm.appendChild(sortByInput);
-                let sortDirectionInput = document.createElement('input');
-                sortDirectionInput.type = 'hidden';
-                sortDirectionInput.name = 'sortDirection';
                 sortDirectionInput.value = newSortDirection;
-                sortForm.appendChild(sortDirectionInput);
                 let entitySearchInput = document.querySelector('#entityFilterTerm');
-                if(entitySearchInput && entitySearchInput.value){
-                    let filterTermInput = document.createElement('input');
-                    filterTermInput.type = 'hidden';
-                    filterTermInput.name = 'entityFilterTerm';
-                    filterTermInput.value = entitySearchInput.value;
-                    sortForm.appendChild(filterTermInput);
+                let entityFilterTermInput = sortForm.querySelector('input[name="entityFilterTerm"]');
+                if(entityFilterTermInput){
+                    entityFilterTermInput.value = entitySearchInput?.value || '';
                 }
                 let allFilters = document.querySelectorAll('.filters-toggle-content .filter input');
                 for(let filterInput of allFilters){
-                    if(filterInput.value){
-                        let filterFieldInput = document.createElement('input');
-                        filterFieldInput.type = 'hidden';
-                        filterFieldInput.name = filterInput.name;
-                        filterFieldInput.value = filterInput.value;
-                        sortForm.appendChild(filterFieldInput);
+                    let filterName = filterInput.name.replace(/^filters\[/, '').replace(/\]$/, '');
+                    let sortFormFilterInput = sortForm.querySelector('input[data-filter-key="'+filterName+'"]');
+                    if(sortFormFilterInput){
+                        sortFormFilterInput.value = filterInput.value;
                     }
                 }
-                document.body.appendChild(sortForm);
                 sortForm.submit();
             });
         }
@@ -349,26 +339,60 @@ window.addEventListener('DOMContentLoaded', () => {
             button.addEventListener('click', (event) => {
                 event.preventDefault();
                 let fieldName = button.getAttribute('data-field');
+                let fileName = button.getAttribute('data-filename');
                 let fileInput = document.getElementById(fieldName);
-                let currentFileDisplay = document.querySelector('.upload-current-file[data-field="'+fieldName+'"]');
+                let form = fileInput?.closest('form');
+                if(!fileInput){
+                    return;
+                }
+                if(!form){
+                    return;
+                }
+                let currentFileDisplay = button.closest('.upload-current-file');
+                let container = button.closest('.upload-files-container');
+                let isRequired = container && 'true' === container.dataset.required;
+                if(isRequired){
+                    let remainingFiles = container.querySelectorAll('.upload-current-file');
+                    if(2 === remainingFiles.length){
+                        let allRemoveButtons = container.querySelectorAll('.remove-upload-btn');
+                        for(let removeBtn of allRemoveButtons){
+                            removeBtn.remove();
+                        }
+                    }
+                }
                 if(currentFileDisplay){
-                    currentFileDisplay.style.display = 'none';
+                    currentFileDisplay.remove();
                 }
-                if(fileInput){
-                    fileInput.value = '';
-                    let form = fileInput.closest('form');
-                    if(form){
-                        let clearFieldName = 'clear_'+fieldName;
-                        let existingClearInput = form.querySelector('input[name="'+clearFieldName+'"]');
-                        if(!existingClearInput){
-                            let clearInput = document.createElement('input');
-                            clearInput.type = 'hidden';
-                            clearInput.name = clearFieldName;
-                            clearInput.value = '1';
-                            form.appendChild(clearInput);
+                if(fileName){
+                    let hiddenFieldName = 'removed_'+fieldName;
+                    let existingHiddenInput = form.querySelector('input[name="'+hiddenFieldName+'"]');
+                    if(existingHiddenInput){
+                        let currentValue = existingHiddenInput.value;
+                        let filesArray = currentValue ? currentValue.split(',') : [];
+                        if(-1 === filesArray.indexOf(fileName)){
+                            filesArray.push(fileName);
                         }
+                        existingHiddenInput.value = filesArray.join(',');
+                        return;
                     }
+                    let hiddenInput = document.createElement('input');
+                    hiddenInput.type = 'hidden';
+                    hiddenInput.name = hiddenFieldName;
+                    hiddenInput.value = fileName;
+                    form.appendChild(hiddenInput);
+                    return;
+                }
+                fileInput.value = '';
+                let clearFieldName = 'clear_'+fieldName;
+                let existingClearInput = form.querySelector('input[name="'+clearFieldName+'"]');
+                if(existingClearInput){
+                    return;
                 }
+                let clearInput = document.createElement('input');
+                clearInput.type = 'hidden';
+                clearInput.name = clearFieldName;
+                clearInput.value = '1';
+                form.appendChild(clearInput);
             });
         }
     }

package/admin/templates/fields/edit/file.html

@@ -1,2 +1,17 @@
-{{#fieldValue}}<p class="upload-current-file" data-field="{{&fieldName}}"><button type="button" class="remove-upload-btn" data-field="{{&fieldName}}" title="REMOVE">X</button> - {{&fieldValue}}</p>{{/fieldValue}}
+{{{renderedAlert}}}
+{{{renderedFiles}}}
+{{#fieldValue}}
+    {{^renderedFiles}}
+        <div class="upload-files-container" data-field="{{&fieldName}}" data-required="{{isRequired}}">
+            {{#filesArray}}
+                <p class="upload-current-file" data-field="{{&fieldName}}" data-filename="{{&filename}}">
+                    {{#isRemovable}}
+                        <button type="button" class="remove-upload-btn" data-field="{{&fieldName}}" data-filename="{{&filename}}" title="REMOVE">X</button> -
+                    {{/isRemovable}}
+                    {{&filename}}
+                </p>
+            {{/filesArray}}
+        </div>
+    {{/renderedFiles}}
+{{/fieldValue}}
 <input type="file" name="{{&fieldName}}" id="{{&fieldName}}"{{&required}}{{&fieldDisabled}}{{&multiple}}/>

package/admin/templates/list-content.html

@@ -27,6 +27,14 @@
                 </svg>
             </span>
         </div>
+        <form class="sort-form hidden" method="POST" action="{{&listRoute}}">
+            <input type="hidden" name="sortBy" value=""/>
+            <input type="hidden" name="sortDirection" value=""/>
+            <input type="hidden" name="entityFilterTerm" value=""/>
+            {{#filters}}
+            <input type="hidden" name="filters[{{&propertyKey}}]" value="" data-filter-key="{{&propertyKey}}"/>
+            {{/filters}}
+        </form>
     </th>
     {{/fieldsHeaders}}
     <th class="field field-edit"></th>

package/admin/templates/view.html

@@ -12,7 +12,7 @@
         </form>
     </div>
     <div class="extra-actions">
-        {{&extraContent}}
+        {{&extraContentTop}}
     </div>
     {{#fields}}
     <div class="view-field">
@@ -32,7 +32,7 @@
             </span>
         </form>
         <div class="extra-actions">
-            {{&extraContent}}
+            {{&extraContentBottom}}
         </div>
     </div>
 </div>

package/lib/admin-manager/contents-builder.js

@@ -185,7 +185,8 @@ class ContentsBuilder
                         id: '{{&id}}',
                         entityEditRoute: '{{&entityEditRoute}}',
                         entityNewRoute: '{{&entityNewRoute}}',
-                        extraContent: '{{&extraContentForView}}'+extraContentForView,
+                        extraContentTop: '{{&extraContentForViewTop}}',
+                        extraContentBottom: '{{&extraContentForViewBottom}}'+extraContentForView,
                         extraFormContent: '{{&extraFormContentForView}}'+extraFormContentForView
                     }
                 ),

package/lib/admin-manager/router-contents.js

@@ -197,6 +197,8 @@ class RouterContents
         await this.emitEvent('adminEntityExtraData', extraDataEvent);
         renderedViewProperties.entitySerializedData = JSON.stringify(extraDataEvent.entitySerializedData)
             .replace(/"/g, '"');
+        renderedViewProperties.extraContentForViewTop = '';
+        renderedViewProperties.extraContentForViewBottom = '';
         await this.emitEvent('reldens.adminViewPropertiesPopulation', {
             idProperty,
             req,
@@ -228,26 +230,53 @@ class RouterContents
             req,
             driverResource,
             renderedEditProperties,
-            loadedEntity
+            loadedEntity,
+            entityId: driverResource.id(),
+            entityData: loadedEntity
         });
         for(let propertyKey of Object.keys(driverResource.options.properties)){
             let property = driverResource.options.properties[propertyKey];
             let fieldDisabled = -1 === driverResource.options.editProperties.indexOf(propertyKey);
+            let fieldValue = await this.generatePropertyEditRenderedValue(
+                loadedEntity,
+                propertyKey,
+                property,
+                fieldDisabled
+            );
+            let templateData = {
+                fieldName: propertyKey,
+                fieldValue,
+                fieldDisabled: fieldDisabled ? ' disabled="disabled"' : '',
+                required: (!property.isUpload || !loadedEntity) && property.isRequired ? ' required="required"' : '',
+                multiple: property.isArray ? ' multiple="multiple"' : '',
+                inputType: this.getInputType(property, propertyKey, fieldDisabled),
+                isArray: property.isArray || '',
+                arraySeparator: property.isArray || '',
+                isRequired: property.isRequired || false
+            };
+            if(property.isUpload && fieldValue && '' !== fieldValue){
+                let filesArray = property.isArray ? fieldValue.split(property.isArray) : [fieldValue];
+                templateData.filesArray = filesArray.map((filename) => {
+                    return {
+                        filename,
+                        isRemovable: !property.isRequired || filesArray.length > 1
+                    };
+                });
+            }
+            await this.emitEvent('reldens.adminBeforeFieldRender', {
+                req,
+                driverResource,
+                propertyKey,
+                property,
+                templateData,
+                loadedEntity,
+                renderedEditProperties,
+                adminContentsRender: this.adminContentsRender.bind(this),
+                adminFilesContents: this.adminFilesContents
+            });
             renderedEditProperties[propertyKey] = await this.adminContentsRender(
                 this.adminFilesContents.fields.edit[this.propertyType(property, 'edit')],
-                {
-                    fieldName: propertyKey,
-                    fieldValue: await this.generatePropertyEditRenderedValue(
-                        loadedEntity,
-                        propertyKey,
-                        property,
-                        fieldDisabled
-                    ),
-                    fieldDisabled: fieldDisabled ? ' disabled="disabled"' : '',
-                    required: (!property.isUpload || !loadedEntity) && property.isRequired ? ' required="required"' : '',
-                    multiple: property.isArray ? ' multiple="multiple"' : '',
-                    inputType: this.getInputType(property, propertyKey, fieldDisabled)
-                }
+                templateData
             );
         }
         return await this.adminContentsRenderRoute(
@@ -282,6 +311,10 @@ class RouterContents
 
     async processSaveEntity(req, res, driverResource, entityPath)
     {
+        let idProperty = this.fetchEntityIdPropertyKey(driverResource);
+        let id = (req?.body[idProperty] || '');
+        let entityRepository = this.dataServer.getEntity(driverResource.entityKey);
+        let loadedEntity = id ? await this.loadEntityById(driverResource, id) : null;
         let uploadProperties = this.fetchUploadProperties(driverResource);
         if(0 < Object.keys(uploadProperties).length){
             if(this.uploaderFactory?.error?.message){
@@ -289,22 +322,20 @@ class RouterContents
                 return this.rootPath+'/'+entityPath+'?result=uploadError&error='
                     +encodeURIComponent(this.uploaderFactory.error.message);
             }
-            let uploadValidationResult = this.validateUploadedFiles(req, uploadProperties);
+            let uploadValidationResult = this.validateUploadedFiles(req, uploadProperties, loadedEntity);
             if(!uploadValidationResult.success){
                 Logger.error('Upload validation failed', uploadValidationResult.error);
                 return this.rootPath+'/'+entityPath+'?result=uploadValidationError&error='
                     +encodeURIComponent(uploadValidationResult.error.message);
             }
         }
-        let idProperty = this.fetchEntityIdPropertyKey(driverResource);
-        let id = (req?.body[idProperty] || '');
-        let entityRepository = this.dataServer.getEntity(driverResource.entityKey);
         let entityDataPatch = this.preparePatchData(
             driverResource,
             idProperty,
             req,
             driverResource.options.properties,
-            id
+            id,
+            loadedEntity
         );
         if(!entityDataPatch){
             Logger.error('Bad patch data.', entityDataPatch);
@@ -351,14 +382,21 @@ class RouterContents
         }
     }
 
-    validateUploadedFiles(req, uploadProperties)
+    validateUploadedFiles(req, uploadProperties, loadedEntity)
     {
         for(let uploadPropertyKey of Object.keys(uploadProperties)){
             let property = uploadProperties[uploadPropertyKey];
             let uploadedFiles = req.files?.[uploadPropertyKey];
             if(!uploadedFiles || 0 === uploadedFiles.length){
                 if(property.isRequired){
-                    return {success: false, error: {message: 'Required file upload missing: '+uploadPropertyKey}};
+                    let isExplicitClear = '1' === sc.get(req.body, 'clear_'+uploadPropertyKey, false);
+                    if(isExplicitClear){
+                        return {success: false, error: {message: 'Required file upload missing: '+uploadPropertyKey}};
+                    }
+                    let existingValue = loadedEntity ? sc.get(loadedEntity, uploadPropertyKey, '') : '';
+                    if(!existingValue || '' === existingValue){
+                        return {success: false, error: {message: 'Required file upload missing: '+uploadPropertyKey}};
+                    }
                 }
                 continue;
             }
@@ -484,7 +522,7 @@ class RouterContents
         }
     }
 
-    preparePatchData(driverResource, idProperty, req, resourceProperties, id)
+    preparePatchData(driverResource, idProperty, req, resourceProperties, id, loadedEntity)
     {
         let entityDataPatch = {};
         for(let i of Object.keys(resourceProperties)){
@@ -512,7 +550,8 @@ class RouterContents
                     propertyUpdateValue = null;
                 }
                 if(!isExplicitClear){
-                    propertyUpdateValue = this.prepareUploadPatchData(req, i, propertyUpdateValue, property);
+                    let currentValue = loadedEntity ? sc.get(loadedEntity, i, '') : '';
+                    propertyUpdateValue = this.prepareUploadPatchData(req, i, currentValue, property);
                 }
             }
             if('boolean' === propertyType){
@@ -539,7 +578,6 @@ class RouterContents
             }
             let isUploadCreate = property.isUpload && !id;
             if(property.isRequired && (isNull || isEmpty) && (!property.isUpload || isUploadCreate)){
-                // missing required fields would break the update:
                 Logger.critical(
                     'Missing property on prepare patch data.',
                     {propertyKey: i, config: property, propertyUpdateValue, entity: driverResource.entityKey}
@@ -562,14 +600,32 @@ class RouterContents
     prepareUploadPatchData(req, i, propertyUpdateValue, property)
     {
         let filesData = sc.get(req.files, i, null);
-        if(null === filesData){
+        let removedFiles = sc.get(req.body, 'removed_'+i, '');
+        let existingFiles = propertyUpdateValue || '';
+        if(property.isArray && removedFiles && '' !== removedFiles){
+            let removedFilesArray = removedFiles.split(',');
+            let existingFilesArray = existingFiles ? existingFiles.split(property.isArray) : [];
+            existingFilesArray = existingFilesArray.filter(file => {
+                return -1 === removedFilesArray.indexOf(file);
+            });
+            existingFiles = existingFilesArray.join(property.isArray);
+        }
+        if(!filesData || 0 === filesData.length){
+            if(removedFiles && '' !== removedFiles){
+                return existingFiles;
+            }
             return null;
         }
-        let fileNames = [];
+        let newFileNames = [];
         for(let file of filesData){
-            fileNames.push(file.filename);
+            newFileNames.push(file.filename);
+        }
+        if(property.isArray && existingFiles && '' !== existingFiles){
+            let existingFilesArray = existingFiles.split(property.isArray);
+            let allFiles = existingFilesArray.concat(newFileNames);
+            return allFiles.join(property.isArray);
         }
-        return fileNames.join(property.isArray);
+        return newFileNames.join(property.isArray);
     }
 
     async generatePropertyRenderedValue(fieldValue, fieldName, resourceProperty, templateType)
@@ -611,7 +667,7 @@ class RouterContents
             fieldValue = '' !== fieldValue ? sc.formatDate(new Date(fieldValue)) : '';
         }
         if('reference' === resourceProperty.type){
-            let relationEntity = entity[resourceProperty.alias || resourceProperty.reference];
+            let relationEntity = entity[resourceProperty.alias || 'related_'+resourceProperty.reference];
             if(relationEntity){
                 let relation = this.relations()[resourceProperty.reference];
                 if(relation){

package/lib/frontend/content-renderer.js

@@ -123,7 +123,7 @@ class ContentRenderer
             return layoutContent;
         }
         let routerKey = route?.router;
-        let categoryName = currentEntityData?.cms_categories?.name;
+        let categoryName = currentEntityData?.related_cms_categories?.name;
         let siteHandle = this.templateResolver.resolveDomainToSiteKey(domain)
             + (routerKey ? ' '+routerKey : '')
             + (categoryName ? ' cat-'+categoryName : '');

package/lib/sitemap-loader.js

@@ -65,14 +65,11 @@ class SitemapLoader
         try {
             let routesWithPages = await this.routesRepository.loadWithRelations(filters, 'cms_pages');
             let sitemapPages = routesWithPages.filter(route => {
-                if(!route.cms_pages || 0 === route.cms_pages.length){
+                if(!route.related_cms_pages || 0 === route.related_cms_pages.length){
                     return false;
                 }
-                let page = route.cms_pages[0];
-                if('noindex,nofollow' === page.meta_robots){
-                    return false;
-                }
-                return true;
+                return 'noindex,nofollow' !== route.related_cms_pages[0].meta_robots;
+
             });
             req.sitemapPages = sitemapPages;
             eventData.variables.sitemapPages = sitemapPages;

package/package.json

@@ -1,7 +1,7 @@
 {
     "name": "@reldens/cms",
     "scope": "@reldens",
-    "version": "0.57.0",
+    "version": "0.59.0",
     "description": "Reldens - CMS",
     "author": "Damian A. Pastorini",
     "license": "MIT",
@@ -35,9 +35,9 @@
     },
     "dependencies": {
         "@reldens/server-utils": "^0.46.0",
-        "@reldens/storage": "^0.90.0",
+        "@reldens/storage": "^0.91.0",
         "@reldens/utils": "^0.54.0",
-        "dotenv": "17.2.3",
+        "dotenv": "17.3.1",
         "mustache": "4.2.0"
     }
 }