Your plugin doesn’t work too. It just make the keywords appear italics. That’s all.

Glad to hear that the plugin works more or less with the new build of TXP. I haven’t got around to updating yet but I guess I should soon. I’m guessing that they haven’t fixed the notextile not actually meaning notextile problem if the hacks are still necessary?

Thanks for posting your updated block function,

Cheers,

New Block Function for classTextile.php (txt)

Very amazing work Kelvin, and the hacks still apply well to the latest build.

It could possibly have some bugs, but it will give you an idea of how to get around the notextile. I ‘think’ the problem was that content from plugins DO NOT get passed through textile in the first place.

view the source here

Cheers

Thanks for the feedback…

Putting the ini_set in the function is probably a good idea although you could maybe make it more robust by doing:

ini_set(“include_path”, ”/path/to/geshi/:”.ini_get(“include_path”));

(untested) or something similar? Just so it doesn’t break anyone’s non-standard include paths…

Re. removing the height, this makes good sense.

And re. the notextile tag – yes, using the txp file upload capability would make sense. I’m actually a week into a 3 month trip around South America at the moment so unfortunately won’t be doing any changes myself in this time but you should feel free to and to post a link to the extended code here. Otherwise (and if the notextile tag still isn’t fixed) I’ll look into doing it when I get home…

Cheers :)

I also had to hack the plugin a little bit to change the php include path since my installation on Dreamhost uses PHP as a cgi module (and it seems safer leave it that way), it can’t make use of .htaccess file to change the include path. Basically, I just put this at the start of the plugin function:

ini_set(“include_path”, ”/path/to/geshi/:.:/usr/local/lib/php” );

Also note that the instructions you give for modifying geshi.php are a little out of date (v1.06 is the latest version and the constructor has changed slightly)

I’ve also messed around a little bit to provide an option to remove the height restriction on the code box (i.e., only constrain the width of the div and let the code blocik flow down the page).

My next thought: since the notextile tag is a pain in the perverbial, it would make more sense to use the file-based highlighting. I wonder if that could be integrated with txp’s new file upload facility to make things easier? i.e., upload the code file using txp and then refer to it using the txp name/id rather than the full path. This would avoid ftp’ing files around and remember paths etc. hmmm…

It seems like you figured it out :)

On another note, glad to hear that it is working with RC3 – I am still using an a hacked RC1 and hadn’t tested with anything newer…

Hope you find the plugin useful,

Kelvin :)