The codesnippetgeshi documentation is broken - colorize.php does not exist

[Piotrek Koszuliński]

My idea is to include its source in the documentation for the plugin. The example is just few lines of code and IMO, the file should be kept outside CKEditor directory, to not cause problems during updating CKEditor.

<?php

include_once 'geshi/geshi.php';

$json_string = file_get_contents( 'php://input' );
$json_object = json_decode( $json_string );

$geshi = new GeSHi( $json_object->html, $json_object->lang );

echo $geshi->parse_code();

[Wiktor Walc]

I'm fine with keeping the file inside the documentation.

If we ever gonna include colorize.php in the plugin folder by default and point codeSnippetGeshi_url into it, we should do two things:

1. provide a reasonable error message in case the PHP file was returned as plain text (PHP is popular, but not every website in the world is using it)
2. provide an error message (also an instruction), in case the geshi library was not detected

The 2nd point could look like this:

<?php

if (function_exists('stream_resolve_include_path') && stream_resolve_include_path('geshi/geshi.php') === FALSE) {
  die('<pre class="html">Please install the geshi library. Refer to plugins/codesnippetgeshi/README.md for more information.</pre>');
}

include_once 'geshi/geshi.php';

$json_string = file_get_contents( 'php://input' );
$json_object = json_decode( $json_string );

$geshi = new GeSHi( $json_object->html, $json_object->lang );

echo $geshi->parse_code();

die() it is not required if this code will land only in the documentation.

[Marek Lewandowski]

Docs pushed to ​t/11480@docs. They don't incldue JSON entry, because it will cause conflicts with ckeditor-sdk branch. It might be good idea for Anna to have a look on these docs.

Pushed small change to ​t/11781 at dev.

[Piotrek Koszuliński]

README.md in codesnippetgeshi plugin directory should contain only a link to docs and nothing more. DRY.

[Piotrek Koszuliński]

And second thing - created guides were not added to guides.json so they are not available in the menu.

[Piotrek Koszuliński]

Ok, I got an info from Ania that she'll update guides.json so ignore comment:6.

[Piotrek Koszuliński]

While using editor in [inline](#!/guide/dev_inline) or divarea mode - styles are not preserved.

Not preserved? This doesn't make sense. There's nothing to be preserved. They are not automatically loaded by CKEditor.

[Marek Lewandowski]

Added requested update and few typos to ​t/11480@docs.

Pushed to ​t/11781 at dev.

[Marek Lewandowski]

Extra changes pushed to ​docs and ​t/11781 at dev branch.

[Marek Lewandowski]

Fixed with ​git:ab2f93f049 (merged to major) at dev. Added without changelog entry, since these are only docs changes.