Skip to content
This repository was archived by the owner on Nov 9, 2022. It is now read-only.

Commit 585ea92

Browse files
committed
fixed mistake in validate-roles/users
providing the entire role/user element instead of just the name
1 parent b1f85f6 commit 585ea92

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

deploy/lib/xquery/setup.xqy

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4916,7 +4916,7 @@ declare function setup:validate-roles(
49164916
$import-config as element(configuration))
49174917
{
49184918
(: get the existing role names from the default security DB :)
4919-
let $existing-role-names := setup:get-existing-role-names()
4919+
let $existing-roles := setup:get-roles(())
49204920
return
49214921

49224922
for $role in $import-config/sec:roles/sec:role
@@ -4929,7 +4929,7 @@ declare function setup:validate-roles(
49294929
let $privileges as element(sec:privilege)* := $role/sec:privileges/sec:privilege
49304930
let $amps as element(sec:amp)* := $role/sec:amps/*
49314931
let $external-names as xs:string* := $role/sec:external-names/sec:external-name
4932-
let $match := $existing-role-names[. = $role-name]
4932+
let $match := $existing-roles/sec:role[sec:role-name = $role-name]
49334933
return
49344934
if ($match) then
49354935
if ($match/sec:role-name != $role-name or
@@ -5101,8 +5101,8 @@ declare function setup:create-users($import-config as element(configuration))
51015101

51025102
declare function setup:validate-users($import-config as element(configuration))
51035103
{
5104-
(: get the existing user names from the default security DB :)
5105-
let $existing-user-names := setup:get-existing-user-names()
5104+
(: get the existing users from the default security DB :)
5105+
let $existing-users := setup:get-users(())
51065106
return
51075107

51085108
for $user in $import-config/sec:users/sec:user
@@ -5113,7 +5113,7 @@ declare function setup:validate-users($import-config as element(configuration))
51135113
let $permissions as element(sec:permission)* := $user/sec:permissions/*
51145114
let $collections as xs:string* := $user/sec:collections/*
51155115
let $external-names as xs:string* := $user/sec:external-names/sec:external-name
5116-
let $match := $existing-user-names[. = $user-name]
5116+
let $match := $existing-users/sec:user[sec:user-name = $user-name]
51175117
return
51185118
if ($match) then
51195119
if ($match/sec:description != $description or

0 commit comments

Comments
 (0)