search

Syxton / moodle-block_massaction

THE OFFICIAL Mass Actions block for Moodle 3.9 and beyond
9 stars 14 forks source link

Do not display block's content if a user doesn't have block/massaction:use #87

Closed dmitriim closed 10 months ago

dmitriim commented 1 year ago

There is a capability block/massaction:use that let users to use the block, however it's only checked in one spot https://github.com/Syxton/moodle-block_massaction/blob/master/action.php#L42

Would be much better UX if we check this capability before even rendering content so users without require permissions don't event see block.

TomoTsuyuki commented 1 year ago

I made PR #93 for this issue. Please review and merge if it's ok.

Syxton commented 10 months ago

merged in both branches. Thank you for your work on this!