Skip to content

[Backport 2025.01.xx] Fix #11404 Problem with user permissions still involving the "Save As" (#11408)#11410

Merged
tdipisa merged 1 commit intogeosolutions-it:2025.01.xxfrom
allyoucanmap:issue_11404_2025.01.xx
Aug 28, 2025
Merged

[Backport 2025.01.xx] Fix #11404 Problem with user permissions still involving the "Save As" (#11408)#11410
tdipisa merged 1 commit intogeosolutions-it:2025.01.xxfrom
allyoucanmap:issue_11404_2025.01.xx

Conversation

@allyoucanmap
Copy link
Copy Markdown
Contributor

Description

This PR adds logic to exclude the owner attribute while saving a resource.
Permission requests are failing if the owner attribute is different from the current editing user (see related issue #4634)

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Bugfix

Issue

What is the current behavior?

#11404

What is the new behavior?

Removed the owner attribute from cloned resources

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • No

Other useful information

…ng the "Save As" (geosolutions-it#11408)

* Fix geosolutions-it#11404 Problem with user permissions still involving the 'Save As'

* add comment
@allyoucanmap allyoucanmap added this to the 2025.01.02 milestone Aug 28, 2025
@allyoucanmap allyoucanmap self-assigned this Aug 28, 2025
@allyoucanmap allyoucanmap requested a review from tdipisa August 28, 2025 10:43
@allyoucanmap allyoucanmap linked an issue Aug 28, 2025 that may be closed by this pull request
1 task
@tdipisa tdipisa enabled auto-merge (squash) August 28, 2025 10:48
@tdipisa tdipisa merged commit ca30362 into geosolutions-it:2025.01.xx Aug 28, 2025
5 checks passed
@tdipisa
Copy link
Copy Markdown
Member

tdipisa commented Aug 28, 2025

@ElenaGallo please verify in QA once deployed and let me know.

@ElenaGallo
Copy link
Copy Markdown
Contributor

ElenaGallo commented Aug 28, 2025

@allyoucanmap On QA, for a user with edit permissions, the permission added in the permissions tab is not visible and cannot be edited.

save-as.mp4

@allyoucanmap
Copy link
Copy Markdown
Contributor Author

@ElenaGallo @tdipisa the specific map showed in the video has the owner attribute and for this reason the request to permission is failing showing no information, related to this #11404 (comment). We tested with a new map and it works as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Problem with user permissions still involving the "Save As"

4 participants