In my testing the following files are not implemented:
* js/plugins/media.fromurl.js
* js/plugins/media.upload.js

I'm not sure that I've identified all use cases where they might be used, however I've not seen them.

CommentFileSizeAuthor
#2 1883670_remove_files.diff2.18 KBarthurf

Comments

ParisLiakos’s picture

ParisLiakos commented 
paris@WL:~$ grep -r 'media.upload.js' /var/www/d7/sites/all/modules/media
Binary file /var/www/d7/sites/all/modules/media/.git/index matches
Binary file /var/www/d7/sites/all/modules/media/.git/objects/pack/pack-edd70c43a30c96e87787413e8c5611239305a0fc.pack matches
paris@WL:~$ grep -r 'media.fromurl.js' /var/www/d7/sites/all/modules/media
Binary file /var/www/d7/sites/all/modules/media/.git/index matches

So..they are not referenced anywhere...should be ok to remove them, but i think i ll check a bit git history to be sure

arthurf’s picture

arthurf commented
Status: Active » Needs review
StatusFileSize
new 1883670_remove_files.diff2.18 KB

Here's a patch that removes these files

rob c’s picture

rob c commented

All functionality still seems to work, tested adding/editing, browsing admin, selecting (popup still works on navigating to other page), editing steps seem to work, and i know from my development on media provider modules we did remove some related bits and pieces, i'll keep an eye on it, but looks ok.

ParisLiakos’s picture

ParisLiakos commented
Status: Needs review » Reviewed & tested by the community

well, if everything seems to work and they are nto referenced anywhere there is no point keeping em..thanks rob for checking

ParisLiakos’s picture

ParisLiakos commented
Status: Reviewed & tested by the community » Fixed

yay! two more files down:D
http://drupalcode.org/project/media.git/commit/86e8a47

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.