fix(model-server): getConfig returns the correct value for legacyNameBasedRoles (was inverted)

abstraktor · abstraktor · commit a334116d1c2d · 2025-11-27T11:38:52.000+01:00
diff --git a/model-server/src/main/kotlin/org/modelix/model/server/handlers/RepositoriesManager.kt b/model-server/src/main/kotlin/org/modelix/model/server/handlers/RepositoriesManager.kt
@@ -638,7 +638,7 @@ class RepositoriesManager(val stores: StoreManager) : IRepositoriesManager {
         }
         val treeData = version.obj.data.getTree(TreeType.MAIN).resolveNow().data
         return RepositoryConfig(
-            legacyNameBasedRoles = treeData.usesRoleIds,
+            legacyNameBasedRoles = !treeData.usesRoleIds,
             legacyGlobalStorage = isIsolated(repositoryId) == false,
             nodeIdType = if (treeData.trieWithNodeRefIds != null) RepositoryConfig.NodeIdType.STRING else RepositoryConfig.NodeIdType.INT64,
             primaryTreeType = if (treeData.trieWithNodeRefIds != null) RepositoryConfig.TreeType.PATRICIA_TRIE else RepositoryConfig.TreeType.HASH_ARRAY_MAPPED_TRIE,

Original file line number	Diff line number	Diff line change
`@@ -638,7 +638,7 @@ class RepositoriesManager(val stores: StoreManager) : IRepositoriesManager {`
`638`	`638`	`}`
`639`	`639`	`val treeData = version.obj.data.getTree(TreeType.MAIN).resolveNow().data`
`640`	`640`	`return RepositoryConfig(`
`641`		`- legacyNameBasedRoles = treeData.usesRoleIds,`
	`641`	`+ legacyNameBasedRoles = !treeData.usesRoleIds,`
`642`	`642`	`legacyGlobalStorage = isIsolated(repositoryId) == false,`
`643`	`643`	`nodeIdType = if (treeData.trieWithNodeRefIds != null) RepositoryConfig.NodeIdType.STRING else RepositoryConfig.NodeIdType.INT64,`
`644`	`644`	`primaryTreeType = if (treeData.trieWithNodeRefIds != null) RepositoryConfig.TreeType.PATRICIA_TRIE else RepositoryConfig.TreeType.HASH_ARRAY_MAPPED_TRIE,`