Merge pull request #4654 from FlowFuse/fix-sso-group-enable

hardillb · web-flow · commit 471194fa752f · 2024-10-16T16:39:38.000+01:00
Fix enabling sso group allocation in admin settings
diff --git a/frontend/src/pages/admin/Settings/SSO/createEditProvider.vue b/frontend/src/pages/admin/Settings/SSO/createEditProvider.vue
@@ -191,7 +191,7 @@ export default {
         },
         isGroupOptionsValid () {
             return !this.input.options.groupMapping || (
-                (this.input.options.type === 'saml' ? this.isGroupAssertionNameValid : this.isGroupsDNValid) &&
+                (this.input.type === 'saml' ? this.isGroupAssertionNameValid : this.isGroupsDNValid) &&
                   this.isGroupAdminNameValid
             )
         },

Original file line number	Diff line number	Diff line change
`@@ -191,7 +191,7 @@ export default {`
`191`	`191`	`},`
`192`	`192`	`isGroupOptionsValid () {`
`193`	`193`	`return !this.input.options.groupMapping \|\| (`
`194`		`- (this.input.options.type === 'saml' ? this.isGroupAssertionNameValid : this.isGroupsDNValid) &&`
	`194`	`+ (this.input.type === 'saml' ? this.isGroupAssertionNameValid : this.isGroupsDNValid) &&`
`195`	`195`	`this.isGroupAdminNameValid`
`196`	`196`	`)`
`197`	`197`	`},`