@solidstarters/solid-core 1.2.176 → 1.2.178

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@solidstarters/solid-core",
-  "version": "1.2.176",
+  "version": "1.2.178",
   "description": "This module is a NestJS module containing all the required core providers required by a Solid application",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/controllers/authentication.controller.ts

@@ -97,6 +97,7 @@ export class AuthenticationController {
 
     @ApiBearerAuth("jwt")
     @Post('logout')
+    @Public()
     @HttpCode(HttpStatus.OK)
     async logout() {
         return this.authService.logout();

package/src/controllers/setting.controller.ts

@@ -1,12 +1,12 @@
-import { Controller, Post, Body, Param, UploadedFiles, UseInterceptors, Put, Get, Query, Delete, Patch, ParseArrayPipe, BadRequestException } from '@nestjs/common';
+import { BadRequestException, Body, Controller, Delete, Get, Param, Patch, Post, Put, Query, UploadedFiles, UseInterceptors } from '@nestjs/common';
 import { AnyFilesInterceptor } from "@nestjs/platform-express";
 import { ApiBearerAuth, ApiQuery, ApiTags } from '@nestjs/swagger';
-import { SettingService } from '../services/setting.service';
-import { CreateSettingDto } from '../dtos/create-setting.dto';
-import { UpdateSettingDto } from '../dtos/update-setting.dto';
+import { Public } from 'src/decorators/public.decorator';
 import { SolidRequestContextDecorator } from 'src/decorators/solid-request-context.decorator';
 import { SolidRequestContextDto } from 'src/dtos/solid-request-context.dto';
-import { UpdateSettingsDto } from 'src/dtos/update-settings.dto';
+import { CreateSettingDto } from '../dtos/create-setting.dto';
+import { UpdateSettingDto } from '../dtos/update-setting.dto';
+import { SettingService } from '../services/setting.service';
 
 @ApiTags('Solid Core') 
 @Controller('setting') //FIXME: Change this to the model plural name 
@@ -44,6 +44,7 @@ export class SettingController {
 
   @ApiBearerAuth("jwt")
   @Get('/wrapped')
+  @Public()
   async wrapSettings() {
       return this.service.wrapSettings();
   }

package/src/dtos/create-chatter-message.dto.ts

@@ -11,7 +11,7 @@ export class CreateChatterMessageDto {
     @IsString()
     @ApiProperty()
     messageSubType: string;
-    @IsNotEmpty()
+    @IsOptional()
     @IsString()
     @ApiProperty()
     messageBody: string;

package/src/entities/chatter-message.entity.ts

@@ -10,7 +10,7 @@ export class ChatterMessage extends CommonEntity {
     messageType: string; // audit | custom 
     @Column({ type: "varchar" })
     messageSubType: string; // audit_update | audit_insert | audit_delete | custom
-    @Column({ type: "text" })
+    @Column({ type: "text", nullable: true })
     messageBody: string;
     @Index()
     @Column({ type: "integer" })

package/src/seeders/module-metadata-seeder.service.ts

@@ -241,21 +241,27 @@ export class ModuleMetadataSeederService {
         // 1. Give all permissions to the Admin role.
         this.logger.log(`About to add all permissions to the Admin role`);
         await this.roleService.addAllPermissionsToRole("Admin");
-        // 2. Give wrapSettings permissions to the Public role.
+        // 2. Give  permissions to the Internal / Public role.
         const internalRolePermission = [
-            'UserController.findMany',
+            // User permissions
+            'UserController.findMany', //Why do we need this?
             'UserController.checkIfPermissionExists',
             'UserController.findOne',
+            // Menu permissions
             'MenuItemMetadataController.findMany',
             'MenuItemMetadataController.findUserMenus',
             'MenuItemMetadataController.findOne',
+            // View metadata permissions
             'ViewMetadataController.getLayout',
             'ViewMetadataController.findMany',
             'ViewMetadataController.findOne',
+            // IAM permissions
             'AuthenticationController.changePassword',
+            // Field Metadata permissions
             'FieldMetadataController.getSelectionDynamicValues',
             'FieldMetadataController.getSelectionDynamicValue',
             'FieldMetadataController.findFieldDefaultMetaData',
+            // Saved Filters permissions
             'SavedFiltersController.delete',
             'SavedFiltersController.deleteMany',
             'SavedFiltersController.findOne',
@@ -266,7 +272,27 @@ export class ModuleMetadataSeederService {
             'SavedFiltersController.update',
             'SavedFiltersController.insertMany',
             'SavedFiltersController.create',
-            'AuthenticationController.logout'
+            // Logout permissions
+            'AuthenticationController.logout',
+            // Other permissions can be added here as required.
+            // Chatter permissions
+            'ChatterMessageController.create',
+            'ChatterMessageController.getChatterMessages',
+            // 'ChatterMessageController.postMessage', // Does not seem to be used from ui
+            // Import
+            'ImportTransactionController.getImportTemplate',
+            'ImportTransactionController.getImportInstructions',
+            'ImportTransactionController.getImportMappingInfo',
+            'ImportTransactionController.startImportSync',
+            'ImportTransactionController.startImportAsync',
+            'ImportTransactionController.exportFailedImportedImports',
+            // Export permissions
+            'ExportTemplateController.startExportSync',
+            'ExportTemplateController.startExportAsync',
+            // List of values
+            'ListOfValuesController.findMany',
+            'ListOfValuesController.findOne',
+            // Media // [Not required], since media is always populated as part of a model
         ]
         await this.roleService.addPermissionToRole('Internal User', internalRolePermission);
         await this.roleService.addPermissionToRole('Public', ['SettingController.wrapSettings', 'AuthenticationController.logout']);

package/src/seeders/seed-data/solid-core-metadata.json

@@ -3757,7 +3757,7 @@
             "displayName": "Message Body",
             "type": "longText",
             "ormType": "text",
-            "required": true,
+            "required": false,
             "unique": false,
             "index": false,
             "private": false,

package/src/services/chatter-message.service.ts

@@ -148,7 +148,7 @@ export class ChatterMessageService extends CRUDService<ChatterMessage> {
                 messageDetail.oldValue = null;
                 messageDetail.oldValueDisplay = null;
                 messageDetail.newValue = this.formatFieldValue(field, fieldValue);
-                messageDetail.newValueDisplay = this.formatFieldValueDisplay(field, fieldValue);
+                messageDetail.newValueDisplay = await this.formatFieldValueDisplay(field, fieldValue);
                 await this.chatterMessageDetailsRepo.save(messageDetail);
             }
         }
@@ -263,8 +263,8 @@ export class ChatterMessageService extends CRUDService<ChatterMessage> {
             messageDetail.fieldDisplayName = field.displayName;
             messageDetail.oldValue = this.formatFieldValue(field, oldValue);
             messageDetail.newValue = this.formatFieldValue(field, newValue);
-            messageDetail.oldValueDisplay = this.formatFieldValueDisplay(field, oldValue);
-            messageDetail.newValueDisplay = this.formatFieldValueDisplay(field, newValue);
+            messageDetail.oldValueDisplay = await this.formatFieldValueDisplay(field, oldValue);
+            messageDetail.newValueDisplay = await this.formatFieldValueDisplay(field, newValue);
             await this.chatterMessageDetailsRepo.save(messageDetail);
         }
     }
@@ -328,7 +328,7 @@ export class ChatterMessageService extends CRUDService<ChatterMessage> {
         return value.toString();
     }
 
-    private formatFieldValueDisplay(field: any, value: any): string {
+    private async formatFieldValueDisplay(field: any, value: any): Promise<string> {
         if (value === null || value === undefined) {
             return '';
         }
@@ -339,8 +339,30 @@ export class ChatterMessageService extends CRUDService<ChatterMessage> {
 
         if (field.type === 'relation') {
             if (field.relationType === "many-to-one") {
+            if (value.name) {
                 return value.name;
             }
+            
+            try {
+                const relatedModel = await this.modelMetadataRepo.findOne({
+                where: { singularName: field.relationCoModelSingularName || field.relation },
+                relations: { userKeyField: true }
+                });
+                
+                if (relatedModel && relatedModel.userKeyField) {
+                const userKeyFieldName = relatedModel.userKeyField.name;
+                return value[userKeyFieldName] ? value[userKeyFieldName].toString() : '';
+                }
+                
+                if (value.id) {
+                return value.id.toString();
+                }
+            } catch (error) {
+                console.error('Error fetching related model metadata:', error);
+                return value.id ? value.id.toString() : '';
+            }
+            }
+
             if (field.relationType === 'many-to-many') {
                 return value.map(item => item.name).join(', ');
             }
@@ -492,7 +514,7 @@ export class ChatterMessageService extends CRUDService<ChatterMessage> {
         entityName: string,
         query: any
     ) {
-        const { limit = 25, offset = 0, sort, populate = [] } = query;
+        const { limit = 25, offset = 0, sort, populate = [], populateMedia = [] } = query;
 
         const model = await this.modelMetadataRepo.findOne({
             where: {
@@ -571,6 +593,11 @@ export class ChatterMessageService extends CRUDService<ChatterMessage> {
 
         const [entities, count] = await qb.getManyAndCount();
 
+        if (populateMedia && populateMedia.length > 0) {
+            const normalizedPopulateMedia = this.crudHelperService.normalize(populateMedia);
+            await this['handlePopulateMedia'](normalizedPopulateMedia, entities);
+        }
+
         const currentPage = Math.floor(offset / limit) + 1;
         const totalPages = Math.ceil(count / limit);
         const nextPage = currentPage < totalPages ? currentPage + 1 : null;

package/src/solid-core.module.ts

@@ -693,7 +693,8 @@ import { ViewMetadataSubsciber } from './subscribers/view-metadata.subscriber';
     ChatterMessageService,
     ChatterMessageRepository,
     ChatterMessageDetailsService,
-    ChatterMessageDetailsRepository
+    ChatterMessageDetailsRepository,
+    TypeOrmModule
     // ThrottlerModule,
   ],
 })