Menu === Dashboard » Structure
Create new container, and forum
then press save button
show this message.....
Fatal error: Call to undefined function taxonomy_overview_terms_validate() in D:\Xampp\htdocs\drupal\includes\form.inc on line 1026

Comments

scor’s picture

Version: 7.0-alpha2 » 7.x-dev
Priority: Normal » Critical

I suppose this qualify as critical since it's technically not possible to rearrange the forums (and their containers).

scor’s picture

Title: Error while save forum structure in "list" tab » Fatal error when saving forum administration form
greggles’s picture

StatusFileSize
new922 bytes

Definitely seems critical to me. Here's a test for the form so that we can catch this in the future.

greggles’s picture

Also, I looked into that function a bit and couldn't find where it went or how this form ever worked. I probably wasn't looking in enough places.

greggles’s picture

Status: Active » Needs review
StatusFileSize
new1.88 KB

Ok, here we go.

That validate function was added in #193333-24: Add drag and drop to taxonomy. In Drupal 6 a $form['#validate'] function that doesn't exist will be silently ignored. In Drupal 7 it's a fatal error.

Since this function has never existed I think it's safe to simply remove it.

catch’s picture

Status: Needs review » Reviewed & tested by the community
scor’s picture

Status: Reviewed & tested by the community » Needs review

looks like an easy fix. there was a similar issue with locale in D6 where a validate/submit function never existed and was only discovered recently via Drupal 7.

scor’s picture

Status: Needs review » Reviewed & tested by the community

x-post

Status: Reviewed & tested by the community » Needs work
Issue tags: -Forum

The last submitted patch, 725148_5_forum_admin_function_not_found_with_tests.patch, failed testing.

scor’s picture

Status: Needs work » Needs review
Issue tags: +Forum

These errors with JavaScript file created: not found on locale.test happen from time to time.

#5: 725148_5_forum_admin_function_not_found_with_tests.patch queued for re-testing.

scor’s picture

Status: Needs review » Reviewed & tested by the community

back to RTBC

greggles’s picture

Status: Reviewed & tested by the community » Fixed

Woohoo http://drupal.org/cvs?commit=339624

We could do this to 6.x as well just for consistency, but it seems unnecessary to me.

Status: Fixed » Closed (fixed)
Issue tags: -Forum

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