Closed (fixed)
Project:
D7 Media
Version:
7.x-2.x-dev
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
23 Feb 2011 at 20:50 UTC
Updated:
3 Jan 2014 at 02:58 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
damienmckennaA quick patch.
Comment #2
JacobSingh commentedI committed this, but also marking "needs work". This is our most common support request. What else can we do to ensure that peopel don't get bitten by it? should we just enable the filter on all input formats during install? Is that too invasive?
Comment #3
damienmckennaMaybe also add a message during installation informing the site developer that it must be enabled per text filter before it really works? Maybe a reminder on the Media settings pages, or a list of text filters & a status indicating which ones have the input filter enabled, though I'm not sure what page that would go on?
Comment #4
JacobSingh commentedYeah, I've long advocated that modules need "installer wizards" to go along with them for this kind of stuff... Hard to architect into the flow though. Do you think it's too invasive to just turn the filter on for everything except plain tesx? It won't really do any harm just being there I suppose.
Comment #5
agerson commentedI think something like this displayed after enabling the module would go a long way. Would that go in media_enable()? Happy to write a patch if you think its a good idea.
Comment #6
oxyc commentedI'm actually for enabling the feature by default. I'm sure some people will be against it but the general community will be thankful for media module taking the initiative.
Comment #7
CinemaSaville commented+1 for enabling it by default. Just had this issue, and I consider myself fairly savvy in this world now. God help the novices.
Comment #8
amateescu commentedThere's a related issue (with a patch) at #1238772-8: Disable the 'Media browser' WYSIWYG button until it has been enabled for the text format. Would that patch be enough to close this one too?
Comment #9
dave reidAgreed, let's mark this as a duplicate.
Comment #10
dave reidBetter status.