itizawa 6 лет назад
Родитель
Сommit
1fe4d6cce7
2 измененных файлов с 3 добавлено и 4 удалено
  1. 1 1
      src/server/models/user.js
  2. 2 3
      src/server/routes/page.js

+ 1 - 1
src/server/models/user.js

@@ -201,7 +201,7 @@ module.exports = function(crowi) {
 
 
   userSchema.methods.canDeleteCompletely = function(creatorId) {
   userSchema.methods.canDeleteCompletely = function(creatorId) {
     const pageCompleteDeletionAuthority = crowi.configManager.getConfig('crowi', 'security:pageCompleteDeletionAuthority');
     const pageCompleteDeletionAuthority = crowi.configManager.getConfig('crowi', 'security:pageCompleteDeletionAuthority');
-    if (pageCompleteDeletionAuthority != null || this.admin) {
+    if (pageCompleteDeletionAuthority == null || this.admin) {
       return true;
       return true;
     }
     }
     if (pageCompleteDeletionAuthority === 'adminAndAuthor') {
     if (pageCompleteDeletionAuthority === 'adminAndAuthor') {

+ 2 - 3
src/server/routes/page.js

@@ -939,9 +939,8 @@ module.exports = function(crowi, app) {
 
 
     try {
     try {
       if (isCompletely) {
       if (isCompletely) {
-        // get useradmin flag
-        const isDeniedCompletelyDelete = crowi.configManager.getConfig('crowi', 'security:isEnabledDeleteCompletely');
-        if (!req.user.admin && isDeniedCompletelyDelete) {
+        const pageCompleteDeletionAuthority = crowi.configManager.getConfig('crowi', 'security:pageCompleteDeletionAuthority');
+        if (!req.user.admin && pageCompleteDeletionAuthority === 'adminOnly') {
           return res.json(ApiResponse.error('You can not delete completely', 'user_not_admin'));
           return res.json(ApiResponse.error('You can not delete completely', 'user_not_admin'));
         }
         }
         if (isRecursively) {
         if (isRecursively) {