[code][elema@localhost ~]$ df -h
File system Dim. Usati Disp. Uso% Montato su
/dev/sda1 12G 4,0G 7,3G 36% /
/dev/sda6 902G 10G 892G 2% /home
/dev/sr1 13M 13M 0 100% /media/Alice MOBILE
[/code]

This piece of code broke geshifilter parsing, but bbcode module parse this text correctly just after geshifilter.

I think that the problem is located here: "[code][elema@localhost ~]"

Any hints ?

Thanks a lot and thank you so much for this great module.

Comments

paolomainardi’s picture

I made another test:

< code >[elema@localhost ~]$ df -h
File system Dim. Usati Disp. Uso% Montato su
/dev/sda1 12G 4,0G 7,3G 36% /
/dev/sda6 902G 10G 892G 2% /home
/dev/sr1 13M 13M 0 100% /media/Alice MOBILE
< /code >

The same text but with standard tag < code > works like a charm (i've just put spaces in order to not be parsed by editor)

soxofaan’s picture

can you give more info on the input format: which other filters are active and in which order?

paolomainardi’s picture

Dear soxofaan,

Of course:

HTML Filter
Quote
GeSHi filter
BB Code
Smileys filter
URL Filter
HTML Corrector

All activated and in this order.

soxofaan’s picture

Thanks,

does it work when you disable other filters like "Quote", "BBCode" and/or "Smiley filter"?

paolomainardi’s picture

Dear soxofan,

Sadly not, it is not parsed anyway, but only using "bbcode" like tagging.

soxofaan’s picture

StatusFileSize
new82.61 KB

Hi,

I just tried it and there is indeed a problem with [code][elema@localhost ~].
An easy workaround for the moment is to put a new line between [code] and [elem....

see screen shot in attachment

paolomainardi’s picture

Yes, I'm just fixing putting a space between the 2 square brackets.

Thanks a lot for the info.

paolomainardi’s picture

Dear soxofaan,

Do you have some news about this bug ? I didn't have much time to trying to fix the problem.

If you don't have time i'll try to make a patch within this days.

Thanks a lot for your precious work.

soxofaan’s picture

Status: Active » Needs work
StatusFileSize
new1.3 KB

Hi,

I started working on it, but only got time to write a test for it
see attachment

paolomainardi’s picture

That's wonderful.

Please tell me if you need support, in the meanwhile i'll try to code a patch.

soxofaan’s picture

StatusFileSize
new2.24 KB

Hi Paolo,

I hope you didn't burn a lot of effort on the patch already, because I guess the regular expressions involved here are not easy on the eyes :)

Anyway, I picked up my work in #9 a bit an I think a small tweak to a regular expression should do the trick.
See attachment.

paolomainardi’s picture

Wonderful, it works like a charm.

Thanks a lot for the patch :)

soxofaan’s picture

Version: 6.x-1.3 » 7.x-1.x-dev
Status: Needs work » Patch (to be ported)

committed for 6.x-1.x: http://drupal.org/cvs?commit=493796

to be ported to 7.x-1.x

soxofaan’s picture

Status: Patch (to be ported) » Fixed

Status: Fixed » Closed (fixed)

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

yukare’s picture

Issue summary: View changes
Status: Closed (fixed) » Active

I have this same issue on drupal 8, now this is the only failing test. Is possible to work arround it by using language, eg [code language="ini"], but since it works on drupal 6 and 7, must be something missing in upgrade.

yukare’s picture

Version: 7.x-1.x-dev » 8.x-1.x-dev
yukare’s picture

Status: Active » Closed (outdated)

It is working and the test pass, just forgot to close this.