In recent testing without the d.o testing install profile (which defines the active_compatibility_tids stuff automatically), I noticed that if you have terms in the API compatibility vocabulary, but don't define any thing for the active_compatibility_tids setting, you get PHP notices. I have a patch, stay tuned.

CommentFileSizeAuthor
#1 643528-1.active_compatibility_tids_php_warning.patch2.48 KBdww

Comments

dww’s picture

dww
we/he/they
commented
Title: PHP notices on active_compatibility_tids when they're not defined » PHP warnings on active_compatibility_tids when they're not defined
Status: Active » Needs review
StatusFileSize
new 643528-1.active_compatibility_tids_php_warning.patch2.48 KB

Actually, the bug is a PHP warning, since we use '' for the default variable, but we treat it as an array() everywhere. So, we just need to invoke variable_get() with array() as the default and we're good.

dww’s picture

dww
we/he/they
commented
Status: Needs review » Fixed

Committed to HEAD and backported to DRUPAL-5.

Status: Fixed » Closed (fixed)

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