Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
hook_theme() should at least return an empty array. Though really this unused function shouldn't be declared at all. But for now, here's a patch.
Comment | File | Size | Author |
---|---|---|---|
0001-Don-t-crash-drupal-with-an-empty-hook_theme-in-_auth.patch | 769 bytes | sfyn | |
Comments
Comment #1
johnbarclay CreditAttribution: johnbarclay commentedNot sure what project or version this bug/patch is against.
Comment #2
sfyn CreditAttribution: sfyn commentedWell I set the version of the issue to 6.x-1.x-dev - does that give you any indication?
Comment #3
johnbarclay CreditAttribution: johnbarclay commentedNo. I don't see a version 6.x-1.x of this module. And I don't see the code you referenced in ldap_integration project. So I'm at a loss.
Comment #4
sfyn CreditAttribution: sfyn commentedThe version exists in your git repos - I generated this patch and the one in #1864316: Parse error in ldap_authentication (6.x) from a checkout I made yesterday:
http://drupalcode.org/project/ldap.git/shortlog/refs/heads/6.x-1.x
This version is also an available tag in your issue queue.
If you use drush, you can
drush dl ldap
into a drupal 6 installation. Which is how I wound up with it.I guess "maintainer doesn't know it exists" is the very definition of unsupported.
All this patch and the above mentioned one does is to allow people to enable ldap_authentication on drupal 6 installs without crashing their install. I don't mean to be an irritant. I won't re-reopen this again should you choose to close it.
I hope you will excuse my earlier snippiness but I found it hard to credit you would deny the existence of the version entirely. I am closing this issue.