Ticket #301 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

Visual Editor broken in r955

Reported by: gwagenknecht Assigned to: somebody
Priority: highest Milestone: WPMU 1.0
Component: component1 Version: 1.0
Severity: major Keywords: has-patch
Cc: gwagenknecht

Description

I just tested [955] today and it seems that the visual editor is broken. It doesn't even load on FireFox? 2.0.0.3 and loads with errors on IE 7.

From the web server logs I could at least see some wrong image URLs: 

[error] ... File does not exist: /wwwhome/wp-includes/js/tinymce/themes/advanced/images/{$lang_b
old_img}, referer: .../wp-admin/post-new.php
[error] ... File does not exist: /wwwhome/wp-includes/js/tinymce/themes/advanced/images/{$lang_i
talic_img}, referer: .../wp-admin/post-new.php
[[error] ... File does not exist: /wwwhome/wp-includes/js/tinymce/themes/advanced/images/{$lang_u
nderline_img}, referer: .../wp-admin/post-new.php

The IE JavaScript? error is "formElement.value is Null or not an object" (line 2035, char 3).

Change History

04/05/07 13:48:19 changed by gwagenknecht

  • cc set to gwagenknecht.

04/05/07 13:52:37 changed by GIGALinux

  • priority changed from normal to highest.
  • status changed from new to closed.
  • resolution set to fixed.

http://trac.wordpress.org/changeset/5114

04/05/07 13:58:27 changed by gwagenknecht

Mhm, it's fixed in WordPress? but when will it be merged into WordPress? MU?

04/05/07 21:19:58 changed by drmike

  • keywords set to has-patch.
  • status changed from closed to reopened.
  • version set to 2.0.
  • resolution deleted.
  • milestone set to WPMU 2.0.

Reopened as it hasn't been murged into the WPMu code

04/13/07 18:12:04 changed by GIGALinux

  • version changed from 2.0 to 1.0.
  • severity changed from normal to blocker.
  • milestone changed from WPMU 2.0 to WPMU 1.0.

http://trac.wordpress.org/changeset/5114 Update the Code!

04/14/07 15:25:18 changed by drmike

Donncha's up to 5109 it appears.

04/17/07 16:03:46 changed by donncha

  • status changed from reopened to closed.
  • resolution set to fixed.

I merged 5114 but I can't do a simple WP merge because there is experimental tag code in there that isn't ready and may change. See wp-hackers list for discussion

(follow-up: ↓ 9 ) 04/23/07 09:52:44 changed by gwagenknecht

  • status changed from closed to reopened.
  • resolution deleted.
  • severity changed from blocker to major.

Still not working in r971.

The output from Firebug (a Firefox add-on) is: 

realTinyMCE is not defined
http://lilly.ausapolda.de/wp-includes/js/tinymce/tiny_mce_gzip.php?ver=20070326
Line 43
realTinyMCE is not defined
http://localhost/wp-includes/js/tinymce/tiny_mce_gzip.php?ver=20070326
Line 43

2 errors are reported.

(in reply to: ↑ 8 ) 04/23/07 09:54:43 changed by gwagenknecht

Replying to gwagenknecht:

http://lilly.ausapolda.de/... http://localhost/...

Sorry, the last one is not 'localhost' but also 'lilly.ausapolda.de'. I accidentally replaced it.

04/23/07 14:55:34 changed by gwagenknecht

  • status changed from reopened to closed.
  • resolution set to fixed.

OK, after a lot of debugging I found the issue in Firefox itself. Some Add-On is causing this.