Skip to content

Commit 6c52731

Browse files
committed
Fix: return to prevent double setting of permissions
1 parent cc954be commit 6c52731

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

src/shell/commands/perms.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,12 +111,14 @@ export default class PermsCommand extends SolidCommand {
111111
for (let permission of parsedPermissions) {
112112
try {
113113
await acl_perms.changePermissions(url, [permission], options)
114+
return;
114115
} catch (e) {
115116
if (options.verbose) writeErrorString(`Could not set permissions for ${permission.id} using WAC`, e, options)
116117
}
117118
try {
118119
if (options.group || options.default) throw new Error("Cannot set WAC-specific options such as group and default for non-WAC environments ")
119120
await setPermission(url, [permission], options)
121+
return;
120122
} catch (e) {
121123
if (options.verbose) writeErrorString(`Could not set permissions for ${permission.id} using ACP`, e, options)
122124
}

0 commit comments

Comments
 (0)