Few modules of Kaltura requires taxonomy to be enabled to work properly or else people get an abrupt error which is a serious usability issue and might even pose an security threat. The dependency is not mentioned in the documentation nor the code takes care of the enabling the required dependency as it does for others.

Error message looks like

Fatal error: Call to undefined function taxonomy_get_vocabularies() in /var/www/vhosts/[server]/httpdocs/sites/default/modules/kaltura/plugins/node_kaltura_entry/node_kaltura_entry.module on line 59

Comments

xurizaemon’s picture

Status: Active » Needs review
StatusFileSize
new1.29 KB

I'm not (yet) in a position to test this, but maybe you can. This small patch should make Kaltura Media Node skip taxonomy related actions if the taxonomy module is disabled.

(Of course, we could add the dependency, if KMN *requires* taxonomy module.)

raviepic3’s picture

Thanks for putting it up quickly, how ever people felt kaltura is too advanced for them, so moving back to square one :P

il test this when i implement it again for my self updation.

osman’s picture

Issue still exists in 6.x-2.0-beta2.

Media Node Settings (admin/settings/kaltura/kaltura_entry_settings) page was returning a 500 server error but nothing in dblog. I had to hunt it down in the Apache error.log to find out the problem:

PHP Fatal error:  Call to undefined function taxonomy_get_vocabularies() in /var/www/sites/all/modules/con
trib/kaltura/plugins/node_kaltura_entry/node_kaltura_entry.module on line 74, referer: http://localhost/admin/settings/kaltura

Enabling Taxonomy module alone seems to be enough for a quick solution.

But, in my opinion, either the dependencies must be defined in module INFO file, or the Kaltura module should check if the function exists before executing it, just like in the patch. Thanks grobot for the patch. Even though it seems like the code has significantly updated since v1.4 it provides a good idea for an updated patch.

I'll try to post an updated patch later tonight.

Cheers,
Osman

xurizaemon’s picture

Version: 6.x-1.4 » 6.x-2.x-dev
StatusFileSize
new1.06 KB

Rerolling for 6.x-2.x.

xurizaemon’s picture

Title: Dependencies not implicitly taken care of » Fatal error when Taxonomy module not enabled

NB: D7 version doesn't include any taxonomy module calls.