1
mondarse
Possible bug in textsanitizer in xoops 2.5.5 (maybe in previous versions too)
  • 2012/8/23 6:50

  • mondarse

  • Just popping in

  • Posts: 96

  • Since: 2003/2/3 1


I have just upgraded my localhost site from 2.5.0 to 2.5.5 and I have possibly found a bug in textsanitizer with nested xoopcodes. I tried the same here in this forum and the result is the same:

if I nest a "img" xcode inside a "url" xcode the url breaks.

Here is my example:
[url=http://www.iecscyl.com/textos/Tr%EDptico%20II%20Curso%20de%20Ecograf%EDa%20Avanzada%20de%20Tiroides.pdf]
[img]http://www.iecscyl.com/uploads/img437c3b1ae7a8c.gif[/img]  
Folleto Informativo.[/url]


My online site still uses 2.5.0 and the previous code works fine, here is a link:
https://www.iecscyl.com/modules/news/article.php?storyid=358

Thank you for your help!!

2
irmtfan
Re: Possible bug in textsanitizer in xoops 2.5.5 (maybe in previous versions too)
  • 2012/8/23 7:18

  • irmtfan

  • Module Developer

  • Posts: 3419

  • Since: 2003/12/7


very nice catch!
thank you.
Also i test in 2.6.0 alpha and it is a bug in that version too.
please send it to the bug tracker.
https://sourceforge.net/tracker/?group_id=41586&atid=430840

then mention that it is a bug in 2.5.5 and 2.6.0 alpha 1

Edit:
I found it is because of CaricFoto js enable in file class/textsanitizer/image/config.php
// Click to open an image in a new window in full size using CaricFoto
    
'clickable' => 1,


set it to 0 and it will correctly show the link.

1- these kind of configs should be moved to database in the 2.6.0
2- with CaricFoto enable it still should show the link? is this a bug?
3- what is the benefit of CaricFoto?


3
mondarse
Re: Possible bug in textsanitizer in xoops 2.5.5 (maybe in previous versions too)
  • 2012/8/23 8:00

  • mondarse

  • Just popping in

  • Posts: 96

  • Since: 2003/2/3 1


Thank you very much for your quick solution. I have set 'clickable' => 0, and worked fine.

After your "edit" I don't understand if I should submit a bug in tracker or you have already sent by yourself, or if it can't be considered a bug.

MonDarSE

4
mondarse
Re: Possible bug in textsanitizer in xoops 2.5.5 (maybe in previous versions too)
  • 2012/8/24 8:04

  • mondarse

  • Just popping in

  • Posts: 96

  • Since: 2003/2/3 1


As I had no response if I should report the bug to SF, I have searched bug tracker in SF and as I found no report I have done it myself.

Thank you irmtfan again for your quick and useful fix.

5
irmtfan
Re: Possible bug in textsanitizer in xoops 2.5.5 (maybe in previous versions too)
  • 2012/8/24 8:38

  • irmtfan

  • Module Developer

  • Posts: 3419

  • Since: 2003/12/7


mondarse:
I didnt respond because generally my connection to the internet in thursdays and fridays are limited.
sending the bug to the tracker is good.
but more importantly i need to know if we need that CaricFoto in 2.6.0?
IMO core team can remove this feature totally

6
bumciach
Re: Possible bug in textsanitizer in xoops 2.5.5 (maybe in previous versions too)
  • 2012/8/24 12:17

  • bumciach

  • Not too shy to talk

  • Posts: 153

  • Since: 2007/6/25


Quote:

but more importantly i need to know if we need that CaricFoto in 2.6.0?
IMO core team can remove this feature totally


I agree. IMO CaricaFoto brings a little outdated way to creating pop-up windows.
http://translate.googleusercontent.co ... 4VkZ-ru1hPIPULEzr2mOs3EWA
There are more unobstructive methods to do such things nowadays.

7
Anonymous
Re: Possible bug in textsanitizer in xoops 2.5.5 (maybe in previous versions too)
  • 2013/1/13 20:53

  • Anonymous

  • Posts: 0

  • Since:


After a core update I today ran into this problem, like several times before. Had to edit the config file again... I agree, please remove this annoying CaricFoto script asap!

Login

Who's Online

67 user(s) are online (38 user(s) are browsing Support Forums)


Members: 0


Guests: 67


more...

Donat-O-Meter

Stats
Goal: $100.00
Due Date: Jul 31
Gross Amount: $0.00
Net Balance: $0.00
Left to go: $100.00
Make donations with PayPal!

Latest GitHub Commits