Yabfog » e107

TinyMCE Error in Forums for e107

Dan — Wed, 06 Jul 2005 17:39:04 +0000

Here's a writeup of my fix for another bug in e107. Basically, the rich textarea editor (which uses TinyMCE) did not work in the forums.

e107 bugtracker

Error Page for e107

Dan — Fri, 17 Jun 2005 06:01:40 +0000

Formally submitted my three-part fix for the error page distribution with e107.

If you (1) setup the e107 error.php file as your ErrorDocument and (2) try to fetch a page NOT in your e107 installation root that causes an error, the display of the resulting error page is mangled because the server doesn't know where to get all the bits of your theme.

Here's the fix - in three parts.

1) class2.php defines a special constant if it is called by error.php

590a591,598
+ // This will set a constant to indicate that we are displaying the error page.
+ // This triggers a conditional in templates/header_default.php which will add
+ // a  header, which is necessary for the error page to display
+ // correctly if the error was triggered by a page located anywhere other than
+ // the root directory of the installation.
+ if (strstr($_SERVER['PHP_SELF'], 'error.php')) {
+ define("ISERRORPAGE", TRUE);
+ }

2) e107_themes/templates/header_default.php checks for that new constant and if present, adds a tag to the page header. This allows the server to find all the bits of your theme.

40a41,44
+ // This line will add a  header, which is necessary for the error
+ // page to display correctly if the error was triggered by a page located anywhere
+ // other than the root directory of the installation.
+ echo (defined("ISERRORPAGE") ? "" : "");

3) error.php had two errors. The first was a typo under the 404 case that displayed the same string (LAN_16) twice. The second was a repetition of LAN_2 in every case, which was completely silly (but not actually an error).

36c36
- 
".LAN_2."".LAN_20."";
---
+ 
".LAN_20."";
40c40
- 
".LAN_2."".LAN_20."";
---
+ 
".LAN_20."";
45,46c45
- 
".LAN_2."
- 
".LAN_16." {$errTo} ".LAN_16;
---
+ 
".LAN_16." {$errTo} ".LAN_17;
52c51
- 
".LAN_2."".LAN_20."";
---
+ 
".LAN_20."";
55c54
-; 
".LAN_2."".LAN_20."";
---
+ 
".LAN_20."";

Bugtracker

TrackBack Ping for e107

Dan — Wed, 15 Jun 2005 16:08:35 +0000

Fixed the TrackBack ping function in e107. The plugin doesn't create all the database fields, so manually add the field 'trackback_blogname' as the last field (I made it type VARCHAR(150)) in the 'e107_trackback' table of your database.

Here's the offending code in e107_plugins/trackback/plugin.php:
// List of sql requests to create tables ----------------------------------------------------------------------------- $eplug_tables = array( "CREATE TABLE ".MPREFIX."trackback ( trackback_id int(10) unsigned NOT NULL auto_increment, trackback_pid int(10) unsigned NOT NULL default '0', trackback_title varchar(200) NOT NULL default '', trackback_excerpt varchar(250) NOT NULL default '', trackback_url varchar(150) NOT NULL default '', PRIMARY KEY (trackback_id) ) TYPE=MyISAM;");

It should read as follows:
// List of sql requests to create tables ----------------------------------------------------------------------------- $eplug_tables = array( "CREATE TABLE ".MPREFIX."trackback ( trackback_id int(10) unsigned NOT NULL auto_increment, trackback_pid int(10) unsigned NOT NULL default '0', trackback_title varchar(200) NOT NULL default '', trackback_excerpt varchar(250) NOT NULL default '', trackback_url varchar(150) NOT NULL default '', trackback_blogname varchar(150) NOT NULL default '', PRIMARY KEY (trackback_id) ) TYPE=MyISAM;");