Closed (fixed)
Project:
D7 Media
Version:
7.x-2.x-dev
Component:
Code
Priority:
Minor
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
25 Jan 2013 at 03:00 UTC
Updated:
10 Feb 2013 at 00:00 UTC
The following patch removes two functions that I'm not seeing called anywhere from the media module or elsewhere. I've tested and haven't seen any issues.
Removed:
* media_preview_ajax()
* theme_media_admin_thumbnail()
| Comment | File | Size | Author |
|---|---|---|---|
| function_cleanup.diff | 2.74 KB | arthurf |
Comments
Comment #1
dave reidDoes media_preview_ajax() have a related menu hook item that should also be removed?
Comment #2
ParisLiakos commentedsee here #1085136: die() usage should be replaced by exit()
we removed half of it then:p
Comment #3
ParisLiakos commentedComment #4
ParisLiakos commentedcommitted! thanks:)
http://drupalcode.org/project/media.git/commit/99f2762