Move setPermissionsSync to util

31cc05ae46a307ffa144392e495d7d2e2fad568a

Cohee <18619528+Cohee1207@users.noreply.github.com>

src/endpoints/content-manager.js+1 -24 Viewed

 import sanitize from 'sanitize-filename';
 import { sync as writeFileAtomicSync } from  'write-file-atomic';
 import { getConfigValue, color, setPermissionsSync } from '../util.js';
 import { write } from '../character-card-parser.js';
 import { serverDirectory } from '../server-directory.js';

+        }
         fs.cpSync(contentPath, targetPath, { recursive: true, force: false });
 -        function setPermissionsSync(targetPath_) {
+-
 -            function appendWritablePermission(filepath, stats) {
 -                const currentMode = stats.mode;
 -                const newMode = currentMode | 0o200;
 -                if (newMode != currentMode) {
 -                    fs.chmodSync(filepath, newMode);
 -                }
 -            }
+-
 -            const stats = fs.statSync(targetPath_);
+-
 -            if (stats.isDirectory()) {
 -                appendWritablePermission(targetPath_, stats);
 -                const files = fs.readdirSync(targetPath_);
+-
 -                files.forEach((file) => {
 -                    setPermissionsSync(path.join(targetPath_, file));
 -                });
 -            } else {
 -                appendWritablePermission(targetPath_, stats);
 -            }
 -        }
         setPermissionsSync(targetPath);
         console.info(`Content file ${contentItem.filename} copied to ${contentTarget}`);
         anyContentAdded = true;

src/util.js+36 -0 Viewed

         return jsonString;
+    }
+}
++
 +/**
 + * Sets the permissions of a file or directory to be writable.
 + * @param {string} targetPath Path to the file or directory
 + */
 +export function setPermissionsSync(targetPath) {
 +    /**
 +     * Appends writable permission to the file mode.
 +     * @param {string} filePath Path to the file
 +     * @param {fs.Stats} stats File stats
 +     */
 +    function appendWritablePermission(filePath, stats) {
 +        const currentMode = stats.mode;
 +        const newMode = currentMode | 0o200;
 +        if (newMode != currentMode) {
 +            fs.chmodSync(filePath, newMode);
 +        }
 +    }
++
 +    try {
 +        const stats = fs.statSync(targetPath);
++
 +        if (stats.isDirectory()) {
 +            appendWritablePermission(targetPath, stats);
 +            const files = fs.readdirSync(targetPath);
++
 +            files.forEach((file) => {
 +                setPermissionsSync(path.join(targetPath, file));
 +            });
 +        } else {
 +            appendWritablePermission(targetPath, stats);
 +        }
 +    } catch (error) {
 +        console.error(`Error setting write permissions for ${targetPath}:`, error);
 +    }
 +}