Webcam snapshot
Maintained by
Luuk Verhoeven, Vincent Cornelis, hamza tamyachte
This block allows users to make a profile picture with their webcam.
Comments
Comments are no longer open for new posts. Existing comments remain available to read.
- More validation if the user is allowed to replace there avatar
- Change uploader to more moodley way
- Change VCS to public, was still private sorry about that
- Remove hurlant from the source and add another that is allowed to add to moodle.org
Hope this enough to be approved, adding to there private files would be nice for the next version. Also have some more ideas for that.
There are some small and easy to fix issues I spotted while reviewing the code.
* You do not seem to take $CFG->disableuserimages into account. The block should not change the user image if that flag is set (I admit the admin should not install the block in that case, but still).
* Also, you should check for the moodle/user:editownprofile capability before updating the image.
* The more "moodley" way would probably be to implement your upload.php file as AJAX_SCRIPT (grep for this constant), although I admit I do not know ActionScript enough to judge pros and cons of it in your situation.
* When trying to access your bug tracker, I am getting "Access denied" error (I would report these issues there otherwise).
* I am unable to find more information about the license of the com.hurlant.util package you use in your block. The notices in the source code refer to LICENSE.txt file that is not included. Is there a way to check and make sure that the package's license is compatible with GNU GPL v3 ?
I think that your solution might eventually turn into a more robust snapshoting tool in moodle that would save the files into user's private area files. But I really this block doing one thing, and doing it well.
I am going to mark this as needing more work for now to get the raised issues fixed. Thanks for your patience with the review and approval process.