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

Cannot delete postings [Discussion Board Module] #5

Closed
wehowski opened this issue Jul 4, 2024 · 12 comments
Closed

Cannot delete postings [Discussion Board Module] #5

wehowski opened this issue Jul 4, 2024 · 12 comments

Comments

@wehowski
Copy link

wehowski commented Jul 4, 2024

Hi,
I installed the last two updates. Now I have the following issue:

  • Admins and Authors cannot delete postings

Sorry, I cannot reproduce any error messages so far in the protocolls, error_log is empty, after clicking the delete-icon, the page reloads but the posting still exists.

'Humhub version' => '1.16.1',
The PHP is 8.3.8
MariaDB is 10.3.39

@wehowski wehowski changed the title Cannot delete postings Cannot delete postings [Discussion Board Module] Jul 4, 2024
@ArchBlood
Copy link
Contributor

This is probably due to the switch of permissions from ManageContent to ManageUsers, this is needed due to issues that were reported with the ManageContent permissions, so now to be able to manage actions in the Discussions module you must have permissions to manage users.

@ArchBlood
Copy link
Contributor

If it's needed in your community it's recommended to switch use humhub\modules\admin\permissions\ManageUsers; with use humhub\modules\content\permissions\ManageContent; and public $deleteAccess = [ManageUsers::class]; with public $deleteAccess = [ManageContent::class];

@wehowski
Copy link
Author

wehowski commented Jul 4, 2024

I changed $deleteAccess to humhub\modules\content\permissions\ManageContent now.
The problem still exists. Although I can delete answers to postings I cannot delete the opening posting/the thread.

@ArchBlood
Copy link
Contributor

I changed $deleteAccess to humhub\modules\content\permissions\ManageContent now. The problem still exists. Although I can delete answers to postings I cannot delete the opening posting/the thread.

I'm still not able to reproduce the issue on my end with either permission set. I'd need more information, if possible could you provide a testing account?

@wehowski
Copy link
Author

wehowski commented Jul 4, 2024

Hello,
I created a testuser and send the password to your email: kodo@greenmeteor.net

@ArchBlood
Copy link
Contributor

Hello, I created a testuser and send the password to your email: kodo@greenmeteor.net

Once I'm able to I'll jump on and take a look.

@ArchBlood
Copy link
Contributor

I'm not seeing any issues on my end

Test Post

Screenshot_1

Other Test Post

Screenshot_2

The reason why we made the switch is because there are no group permissions for ManageContent class as shown in the attached screenshot below;
Screenshot_3

If @luke- were to make this possible then I'd be more inclined to reintroduce the use of the ManageContent permission.

@ArchBlood
Copy link
Contributor

In the future GreenMeteor/discussions/pull/17 (i.e. ManageDiscussions) will replace both ManageUsers & ManageContent permissions.

@wehowski
Copy link
Author

wehowski commented Jul 6, 2024

Ok, thank you for testing, please excuse my late response.

Did you test to CLICK the delete button on the Test Post? If so, please ignore my answer.
I still cannot delete the initial opening posting of a disscussion board thread.

However, as it is still BETA, I can wait for next version.

@ArchBlood
Copy link
Contributor

Ok, thank you for testing, please excuse my late response.

Did you test to CLICK the delete button on the Test Post? If so, please ignore my answer.
I still cannot delete the initial opening posting of a disscussion board thread.

However, as it is still BETA, I can wait for next version.

I'll have to look into the delete issue.

@ArchBlood
Copy link
Contributor

Beta 7 has been pushed with the new permissions and we also completely replaced the broken delete function, currently by default administrators are allowed to edit and delete discussions, to allow other user groups to also do the same thing you must change the Manage Discussions permission within the user group's permissions area.

@wehowski
Copy link
Author

wehowski commented Jul 7, 2024

I just installed beta.7 and no CAN delete the postings without any error.

Thank you!!!

[can be closed]

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

No branches or pull requests

2 participants