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.

Sign up for GitHub

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

Jump to bottom

creating items uses default NS ACL, should use current namespace ACL #1760

Closed
wants to merge 2 commits into from
Closed

creating items uses default NS ACL, should use current namespace ACL #1760

wants to merge 2 commits into from
+15 −14

Conversation

RogerHaase
Copy link
Member

fixes #1705

@RogerHaase
creating items uses default NS ACL, should use current namespace ACL
92e8c36 
fixes moinwiki#1705
@RogerHaase
fix several namespace bugs where wrong item names were used. fixes mo…
3c127bc 
…inwiki#1705

use fqname not name when checking permission to modify an item

use fqname not name when redirecting after failing to obtain edit lock

use fqname.fullname not name when processing edit conflicts

use fqname not name when checking permission to update new item revision

use fqname.fullname not name on edit lock sql db

cleanup comments
@RogerHaase RogerHaase closed this Sep 22, 2024
@RogerHaase RogerHaase deleted the 1705-create branch September 22, 2024 16:14
@RogerHaase RogerHaase restored the 1705-create branch September 22, 2024 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

default namespace ACL must include create if other namespace allows create
1 participant
@RogerHaase