I love the idea of the website being taken over by pirates. But, I want it to be a surprise when it happens. The input filter tip "Avast! This website be taken over by pirates on September 19th. Yarr!" kind of prevents that. I could disable the filter until then, but I don't really want to be in the admin panel at 12am adding the filter to all of my input formats. An example of how to do this is in the core HTML Filter.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sillygwailo’s picture

Version: 6.x-1.0 » 6.x-1.x-dev
Status: Active » Needs review
FileSize
1.61 KB

Interesting idea. Attached is a patch against 6.x-dev. I've made the default to display, since that's the expectation for filter modules. The text could use some "pirating" up.

meustrus’s picture

Ay, do pirates come by light of day or dark of night?

Maybe?

sillygwailo’s picture

Assigned: Unassigned » sillygwailo
sillygwailo’s picture

Status: Needs review » Fixed

Committed to 6.x-dev without the pirated-up language, since it's in the configuration only.

sillygwailo’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Fixed » Needs review
FileSize
2.36 KB

Attached is a patch for D7. I might have exposed a bug in core as a result: the way the patch for 7.x-dev works, the tips callback returns nothing if the setting is set to do that. Unfortunately, it will show an empty list item. That's because the tips callback will return a populated array with one of its keys having an empty value. I'll investigate the latter part and submit a separate issue if appropriate.

sillygwailo’s picture

sillygwailo’s picture

In the meantime, a better patch for an updated 7.x-dev.

sillygwailo’s picture

Status: Needs review » Fixed

With the patch to core committed, I've committed the feature to the 7.x-dev branch. Closing.

Status: Fixed » Closed (fixed)

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