Просмотр исходного кода

Code improvement

https://youtrack.weseek.co.jp/issue/GW-7759
- Set isDefaultLogo true by default if brand logo not uploaded
- Set value of customize:uploadedLogoSrc and customize:uploadedLogoSrc when upload and delete brand logo
mudana 3 лет назад
Родитель
Сommit
1a6c4dd705

+ 1 - 1
packages/app/src/client/services/AdminCustomizeContainer.js

@@ -108,7 +108,7 @@ export default class AdminCustomizeContainer extends Container {
         currentCustomizeCss: customizeParams.customizeCss,
         currentCustomizeScript: customizeParams.customizeScript,
         brandLogoAttachmentId: customizeParams.brandLogoAttachmentId,
-        isDefaultLogo: customizeParams.isDefaultLogo,
+        isDefaultLogo: customizeParams.brandLogoAttachmentId === undefined ? true : customizeParams.isDefaultLogo,
         uploadedLogoSrc: customizeParams.uploadedLogoSrc,
       });
       // search style name from object for display

+ 3 - 0
packages/app/src/server/routes/attachment.js

@@ -737,6 +737,8 @@ module.exports = function(crowi, app) {
       attachment = await attachmentService.createAttachment(file, req.user, null, attachmentType);
       const attachmentConfigParams = {
         'customize:brandLogoAttachmentId': attachment.id,
+        'customize:isDefaultLogo': false,
+        'customize:uploadedLogoSrc': attachment.filePathProxied,
       };
       await crowi.configManager.updateConfigsInTheSameNamespace('crowi', attachmentConfigParams);
     }
@@ -767,6 +769,7 @@ module.exports = function(crowi, app) {
       const attachmentConfigParams = {
         'customize:brandLogoAttachmentId': null,
         'customize:isDefaultLogo': true,
+        'customize:uploadedLogoSrc': null,
       };
       await crowi.configManager.updateConfigsInTheSameNamespace('crowi', attachmentConfigParams);
     }