Skip to content

System Admin: implement the file handler class during upload for tiny mce and developed a file manager system#2106

Open
ali-ichk wants to merge 9 commits into
GibbonEdu:v31.0.00from
ali-ichk:System---Implement-the-fileHandler-class-during-upload-for-TinyMCE
Open

System Admin: implement the file handler class during upload for tiny mce and developed a file manager system#2106
ali-ichk wants to merge 9 commits into
GibbonEdu:v31.0.00from
ali-ichk:System---Implement-the-fileHandler-class-during-upload-for-TinyMCE

Conversation

@ali-ichk

@ali-ichk ali-ichk commented May 7, 2026

Copy link
Copy Markdown
Contributor

Description
A system Admin can now:

  • Update file status of uploaded files
  • View uploaded file by user
  • View All uploaded files by user and delete files not in use anymore

@SKuipers SKuipers left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Ali, thanks for putting this together, it's looking good. One small change, could we update the files to match a similar pattern in Gibbon, such as uploadedFiles_view.php, uploadedFiles_search.php, uploadedFiles_manage.php and uploadedFiles_manageProcess.php. This pattern helps keep similar functionality together in the file list.

@ali-ichk

Copy link
Copy Markdown
Contributor Author

Hi @SKuipers, thanks for checking and the comments. The filenames have been updated to maintain consistency through the codebase. Thanks.

@SKuipers SKuipers left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Ali for the changes. I've noticed one more area to update, to ensure all the new files are each covered by the new permission. Based on this update to CHANGEDB, can you check and update the _manage and _search files so that the isActionAccessible call references their own filename, rather than uploadedFiles_view.php

Comment thread CHANGEDB.php Outdated
ali-ichk and others added 2 commits May 21, 2026 15:24
Co-authored-by: Sandra Kuipers <sandra@skuipers.com>
@ali-ichk ali-ichk requested a review from SKuipers May 28, 2026 01:14
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

Successfully merging this pull request may close these issues.

2 participants