Just installed this on 4.6.3. When I go to administer>>settings>>taxonomy_dafaults, I get the following error message:

warning: Invalid argument supplied for foreach() in /home/mydomain/public_html/beta2/includes/tablesort.inc on line 160.

Looks like a very useful module. Would be great if this could be fixed.

CommentFileSizeAuthor
#4 taxonomy_default_table.patch839 bytesheine

Comments

codexmas’s picture

codexmas commented
Assigned: Unassigned » codexmas
Status: Active » Fixed

Fix for 4.6.5:

Line 68 of taxonomy_defaults.module reads:
$output .= theme('table', '', $subtable);

Change it to:
$output .= theme('table', NULL, $subtable);

The problem is due to the theme function checking the $header argument with the count function.
Count will return 1 when passed an empty string. The handling of this may be different in 4.7.x

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)
cog.rusty’s picture

cog.rusty commented
Version: » master
Status: Closed (fixed) » Active

Could you please commit a patch for this?

By the way, this is a very useful module and it would be nice to tag it for the 4.6 branch. I wouldn't know about it if it had not been mentioned in the handbook.

Any chance for a 4.7 version?

Thanks

heine’s picture

heine commented
Status: Active » Reviewed & tested by the community
StatusFileSize
new taxonomy_default_table.patch839 bytes

Trivial patch

heine’s picture

heine commented
Status: Reviewed & tested by the community » Fixed

applied

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)