Support PHP module

LOL!

You're right! Of course, you would have to enable this module in order to get the dependency inserted via that hook.

This is the greatest patch I have ever seen in my entire life.

In the true spirit of this module, I'm marking it reviewed and tested even though I neither reviewed it nor tested it.

+1lolz

this is no coincidence, greater powers are at work here

LMAO, awesome!

+1. will buy again. A truly positive addition to the module space.

Guys I dunno. What about the themers?

Nice.

If you like this sort of humor, join us at Drupal for Evil

Don't trump me Moshe.

redintegro est recipero

...

There's known workaround where module developers can directly edit the module to make this work. Good work team.

Now that the consensus is to not commit this, I've committed it.

http://drupal.org/commitlog/commit/14044/ed4c6ecd16876af0b6524225e27a381...

I believe we need a feature that rebuilds theme cache on every page load. I'm putting it in this issue since this is where the discussion is happening.

CAUTION, CAUTION.

IRONY ALERT: there are at least two projects that do that already.

Nick, I really think we need it rebuilt *twice* each page load - what if I make some changes while the page is loading? I don't want those not to propagate through...

+1

Oh, and I have something else to say. The name of this module should be changed to something else. I don't know what though.

Here's some stuff you can add to your drush make file:

Example stub:

projects[drupal][type] = core
projects[drupal][download][type] = git
projects[drupal][download][url] = http://git.drupal.org/project/drupal.git
projects[drupal][download][branch] = 6.x
projects[drupal][patch][] = "http://drupal.org/files/786208-bad_judgement_php-d6.patch"

I just learned core patches in drush make are regular diffs, not git ones.

core = 6.x
api = 2
projects[drupal][type] = core
projects[drupal][patch][] = "http://drupal.org/files/786208-bad_judgement_php-d6-diff.patch"

PHP module no longer exists in Drupal 8. You're welcome.

Discussion about having it use Bad Judgement: #2089647: Improve documentation about PHP security implication

c.f. #2106561: Missing dependency on bad judgement

#15:

Fixing here: #2455085: No Hooks?

Because adding hooks totally has to do with theming. And this thread.