I have ported it to D7. Would be interesting to have it commited?

Comments

penyaskito’s picture

StatusFileSize
new1.2 KB

Attached a patch. If you consider it necessary, I will be proud of being added as a maintainer of the D7 branch.

aren cambre’s picture

OK, I just gave you commit permissions.

wulff’s picture

StatusFileSize
new7.05 KB

As mentioned over in #1229894: WYSIWYG module integration, I have created a patch which updates the bridge module for Drupal 7 and makes it a little more flexible.

Instead of the few hardcoded languages provided by the current version, the updated version makes it possible to use all enabled geshi filter languages in CKEditor. First, enable the GeSHi filters you want to use, then edit the wysiwyg profile to enable the related buttons.

This version of the module makes the wysiwyg_geshi_ckeditor.css file obsolete, since the necessary css is generated on-the-fly.

Snugug’s picture

I'm attempting to apply this patch to the WYSIWYG-Geshi 6.x-1.x Release using the standard patch -p1 < {path/to/patch.patch} while inside the folder, and this is what I'm getting:

can't find file to patch at input line 5
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|diff --git a/js/plugin.js b/js/plugin.js
|index 01c8149..12b60f7 100644
|--- a/js/plugin.js
|+++ b/js/plugin.js
--------------------------
File to patch:

Help? Or is a D7 release going to be pushed out soon just using this patch?

wulff’s picture

StatusFileSize
new8.09 KB

Hmm. Seems that patch was a little half-baked. I have attached an updated version.

penyaskito’s picture

Status: Active » Fixed

Created the 7.x-1.x branch and commited patch #1351974-5: Port to D7? by wulf, that was much better than mine. Thanks!

Status: Fixed » Closed (fixed)

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