Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mdm-resources referencing CatalogueItemDomainType that isn't available in mdm-core for Classifier #35

Closed
GBishop-PHSA opened this issue Oct 7, 2021 · 4 comments
Assignees

Comments

@GBishop-PHSA
Copy link

I've made a change locally to enable User & Group Access editing for Classifications in the mdm-ui.

/src/app/classification/classification-details/classification-details.component.ts

toggleSecuritySection() {
-    this.messageService.toggleUserGroupAccess();
+    this.dialog.openSecurityAccess(this.classification, 'classifier');
}

After the above code change, I get a 400 error on requests to get the groupRoles and the securableGroupRoles when the User & Group Access modal is loading.

When I update the mdm-resources to set the Classifier domaintype from 'Classification' to 'classifier',
the requests return with 200. Should mdm-resources be updated to match the domainType in mdm-core?

mdm-core seems to set the Classifiers domainType here:
https://github.com/MauroDataMapper/mdm-core/blob/18e0a40a8c77206e307cdade6715f720d06e1ce5/mdm-core/grails-app/domain/uk/ac/ox/softeng/maurodatamapper/core/container/Classifier.groovy#L94

mdm-resources sets the Classifiers CatalogueItemDomainType here:

Classification = 'Classification',

@olliefreeman
Copy link
Contributor

im not a developer on mdm-resources but can i ask what the final URL is that mdm-resources calls?? out of curiosity

@GBishop-PHSA
Copy link
Author

The mdm-resources builds the 2 requests and sends:
/api/Classification/<classifier_uuid>/groupRoles/
/api/Classification/<classifier_uuid>/securableResourceGroupRoles/

but the mdm-core grails app expects these paths:
/api/classifiers/<classifier_uuid>/groupRoles/
/api/classifiers/<classifier_uuid>/securableResourceGroupRoles/

@GBishop-PHSA
Copy link
Author

Any update on this issue? gh-354 is blocked by this ticket.

gammonpeter pushed a commit that referenced this issue Oct 18, 2021
Corrected issue with Classification name
@gammonpeter
Copy link
Contributor

Updated Name

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants