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

Added new field type file upload #20

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

dasu2013
Copy link

Thank you so much for this valuable plugin. We are using this plugin in course module level and we added new field type, file upload. I thought it is worth to share it.

Copy link
Contributor

@mchurchward mchurchward left a comment

Choose a reason for hiding this comment

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

Got an error when adding a field:
Notice: Undefined property: stdClass::$defaultdata in /Users/mikechurchward/www/moodlehq.git/local/metadata/definelib.php on line 364

When I accessed the module metadata form, got these errors:
Warning: Use of undefined constant FILE_INTERNAL - assumed 'FILE_INTERNAL' (this will throw an Error in a future version of PHP) in /Users/mikechurchward/www/moodlehq.git/local/metadata/fieldtype/fileupload/classes/metadata.php on line 44



Warning: Use of undefined constant FILE_EXTERNAL - assumed 'FILE_EXTERNAL' (this will throw an Error in a future version of PHP) in /Users/mikechurchward/www/moodlehq.git/local/metadata/fieldtype/fileupload/classes/metadata.php on line 44

These ones were fixed with a "require_once($CFG->dirroot . '/repository/lib.php');

Copy link
Contributor

@mchurchward mchurchward left a comment

Choose a reason for hiding this comment

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

I think this is useful, but I can't see how to use it well.
I was able to attach a file to an activity module, but other than seeing in an editor as an attached file, I am unable to see the file.
Have you had this working?
This is for 3.10 by the way.

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.

3 participants