Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
forum.module
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
25 Feb 2010 at 05:55 UTC
Updated:
3 Jan 2014 at 01:08 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
scor commentedI suppose this qualify as critical since it's technically not possible to rearrange the forums (and their containers).
Comment #2
scor commentedComment #3
gregglesDefinitely seems critical to me. Here's a test for the form so that we can catch this in the future.
Comment #4
gregglesAlso, 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.
Comment #5
gregglesOk, 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.
Comment #6
catchComment #7
scor commentedlooks 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.
Comment #8
scor commentedx-post
Comment #10
scor commentedThese 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.
Comment #11
scor commentedback to RTBC
Comment #12
gregglesWoohoo http://drupal.org/cvs?commit=339624
We could do this to 6.x as well just for consistency, but it seems unnecessary to me.