@solidstarters/solid-core 1.2.24 → 1.2.25

package/src/services/mediaStorageProviders/file-s3-storage-provider.ts

@@ -51,6 +51,9 @@ export class FileS3StorageProvider<T> implements MediaStorageProvider<T> {
                 entityId: entity.id,
                 modelMetadataId: mediaFieldMetadata.model.id,
                 relativeUri: awsFileUrl,
+                mimeType: file.mimetype,
+                fileSize: file.size,
+                originalFileName: file.originalname,
                 mediaStorageProviderMetadataId: mediaFieldMetadata.mediaStorageProvider.id,
                 fieldMetadataId: mediaFieldMetadata.id
             }) as unknown as Media;
@@ -64,7 +67,7 @@ export class FileS3StorageProvider<T> implements MediaStorageProvider<T> {
         if (!(entity instanceof CommonEntity)) {
             throw new Error("Entity must be an instance of CommonEntity"); //FIXME This needs to be handled through generics. e.g T extends CommonEntity
         }
-        const existingMedia = await this.mediaService.findByEntityIdAndFieldIdAndModelMetadataId(entity.id, mediaFieldMetadata.id, mediaFieldMetadata.model.id);
+        const existingMedia = await this.mediaService.findByEntityIdAndFieldIdAndModelMetadataId(entity.id, mediaFieldMetadata.id, mediaFieldMetadata.model.id,['mediaStorageProviderMetadata']);
         this.mediaService.deleteByEntityIdAndFieldIdAndModelMetadataId(entity.id, mediaFieldMetadata.id, mediaFieldMetadata.model.id);
         existingMedia.forEach(media => {
             this.fileService.deleteFromS3(media.relativeUri, mediaFieldMetadata.mediaStorageProvider.bucketName); //TODO

package/src/services/mediaStorageProviders/file-storage-provider.ts

@@ -25,7 +25,7 @@ export class FileStorageProvider<T> implements MediaStorageProvider<T> {
         const media = await this.mediaService.findByEntityIdAndFieldIdAndModelMetadataId(entity.id, mediaFieldMetadata.id, mediaFieldMetadata.model.id, ['mediaStorageProviderMetadata']);
         // Add the full URL to the media
         media.forEach(m => {
-            m['_full_url'] = this.getFullFilePath(m.relativeUri);
+            m['_full_url'] = `${process.env.BASE_URL}/${this.getFullFilePath(m.relativeUri)}`;
         });
         return media;
     }
@@ -46,6 +46,9 @@ export class FileStorageProvider<T> implements MediaStorageProvider<T> {
                 entityId: entity.id,
                 modelMetadataId: mediaFieldMetadata.model.id,
                 relativeUri: this.getFileName(file),
+                mimeType: file.mimetype,
+                fileSize: file.size,
+                originalFileName: file.originalname,
                 mediaStorageProviderMetadataId: mediaFieldMetadata.mediaStorageProvider.id,
                 fieldMetadataId: mediaFieldMetadata.id
             }) as unknown as Media;
@@ -59,7 +62,7 @@ export class FileStorageProvider<T> implements MediaStorageProvider<T> {
         if (!(entity instanceof CommonEntity)) {
             throw new Error("Entity must be an instance of CommonEntity"); //FIXME This needs to be handled through generics. e.g T extends CommonEntity
         }
-        const existingMedia = await this.mediaService.findByEntityIdAndFieldIdAndModelMetadataId(entity.id, mediaFieldMetadata.id, mediaFieldMetadata.model.id);
+        const existingMedia = await this.mediaService.findByEntityIdAndFieldIdAndModelMetadataId(entity.id, mediaFieldMetadata.id, mediaFieldMetadata.model.id,['mediaStorageProviderMetadata']);
         this.mediaService.deleteByEntityIdAndFieldIdAndModelMetadataId(entity.id, mediaFieldMetadata.id, mediaFieldMetadata.model.id);
         existingMedia.forEach(media => {
             this.fileService.deleteFile(this.getFullFilePath(media.relativeUri));
@@ -67,7 +70,7 @@ export class FileStorageProvider<T> implements MediaStorageProvider<T> {
     }
 
     private getFullFilePath(fileName: string): string {
-        return `${process.env.BASE_URL}/${this.configService.get('app-builder.fileStorageDir')}/${fileName}`;
+        return `${this.configService.get('app-builder.fileStorageDir')}/${fileName}`;
     }
 
     private getFileName(file: Express.Multer.File): string {

package/src/services/menu-item-metadata.service.ts

@@ -148,9 +148,15 @@ export class MenuItemMetadataService extends CRUDService<MenuItemMetadata> {
   // Recursive function to build the tree
   private buildMenuTree(rootItems: MenuItemMetadata[], allMenuItems: MenuItemMetadata[], activeUser: ActiveUserData): any[] {
     const menuItemsData = rootItems.map(rootItem => {
-
+      const allowedMenuItems =allMenuItems.filter(i => {
+        if(!i.parentMenuItem){
+          return true
+        }else{
+          const hasPermission = hasReadPermissionOnModel(activeUser, i.action.model.singularName);
+          return  hasReadPermissionOnModel(activeUser, i.action.model.singularName)
+        }}); 
       // Get immediate children of the current loop variable menuItem.
-      const children = allMenuItems.filter(item => item.parentMenuItem && item.parentMenuItem.id === rootItem.id);
+      const children = allowedMenuItems.filter(item => item.parentMenuItem && item.parentMenuItem.id === rootItem.id);
 
       // TODO: We should specify path only if there are no more children present. 
       // For now adding path everywhere. 

package/src/services/model-metadata.service.ts

@@ -902,6 +902,7 @@ export class ModelMetadataService {
         dataSource: model.dataSource,
         table: model.tableName,
         fields: fieldsForRefresh,
+        modelEnableSoftDelete: model.enableSoftDelete
       },
       dryRun
     );

package/src/services/role-metadata.service.ts

@@ -99,8 +99,8 @@ export class RoleMetadataService extends CRUDService<RoleMetadata> {
     return await this._addPermissionsToRole(roleName, permissionNames);
   }
 
-  async addPermissionToRole(roleName: string, permissionName: string) {
-    return await this._addPermissionsToRole(roleName, [permissionName]);
+  async addPermissionToRole(roleName: string, permissionName: string[]) {
+    return await this._addPermissionsToRole(roleName, permissionName);
   }
 
   private async _addPermissionsToRole(roleName: string, permissionNames: string[]): Promise<RoleMetadata> {

package/src/services/user.service.ts

@@ -75,6 +75,21 @@ export class UserService extends CRUDService<User> {
     // return entity;
   }
 
+  async updateUser(id: any, updateDto, files) {
+    const user = await this.repo.findOne({
+      where: { id: id },
+      relations: {
+        roles: true
+      }
+    });
+    if (!user) {
+      throw new Error(`User not found.`);
+    }
+    const roles  = updateDto.roles ? updateDto.roles :[]; 
+    await this.addRolesToUser(user.username, roles);
+    await this.update(id, updateDto, files, true);
+  }
+
   async addRoleToUser(username: string, roleName: string): Promise<User> {
     // Find the role, find the user and populate the many 2 many table.
     const user = await this.repo.findOne({

package/src/services/view-metadata.service.ts

@@ -86,11 +86,11 @@ export class ViewMetadataService extends CRUDService<ViewMetadata> {
       const field = fields[i];
 
       // For fields of type relation & relationType many-to-one
-      // We fetch metadata regarding the relationModelSingularName
+      // We fetch metadata regarding the relationCoModelSingularName
       if (field.type === 'relation') {
         const relationModel = await this.modelMetadataRepo.find({
           where: {
-            singularName: field.relationModelSingularName
+            singularName: field.relationCoModelSingularName
           },
           relations: {
             userKeyField: true