Jump to content

The app is breaking some images in post


numismatique
  • Keyword Tooltip 3.0.10 Keyword Tooltip 3.0.9 Closed Bug

Step to reproduce : 

1- The keywords tooltips is activated on the forum
2- Create a topic and upload a image with a keyword in the photo name
3- Edit the topic and save

--> the image is broken (if you edit the topic, you can still see it)

bug1.jpg

bug2.jpg

 

 

 

 


User Feedback

Recommended Comments

something must've changed, i fixed this issue awhile back.

might have to end up just pulling all html tags out of of the content and then placing them back in after kw is done. i'll figure something out.

Link to comment
Share on other sites

Maybe it occurs in a very special case.

On one of the topics, there were an image with this name 

Quote

monnaie - jeton à identifier (4) - Copie.JPG

jeton is a keyword
"à" is a french special caracter which could cause some problems sometimes

Link to comment
Share on other sites

it is probably the spaces that are causing it, URL should be urlencoded, but it doesn't look like that one is. My regex might not be robust enough to be able to handle spaces.

Link to comment
Share on other sites

Here is the url of the image : ...../uploads/monthly_2016_12/58454dff21530_monnaie-jetonidentifier(4)-Copie.JPG.6b893c0e0024b192960142c696840fb3.JPG 

The spaces are replaced by -

But the original image name is keeped intact for the alt tag

Link to comment
Share on other sites

it will be 3.1, and i don't have an exact time frame for the release (many projects going atm :) ), but i'll try to see what i can do to patch this after dinner tonight :) 

Link to comment
Share on other sites

could you send me the link here or in pm to the troubled post so i can examine its html? i've tried with naming a image "monnaie - jeton à identifier (4) - Copie.JPG" and creating a keyword for every word in it and no dice... 

Link to comment
Share on other sites

fixed, will roll out a update for it tomorrow (about to hit the sack). had to completely replace the word to kw method (had to go with DOMDocumentor, over the regex i was using), as i couldn't loosen up the regex enough to prevent it from breaking the image on edit. which is a good thing, can drop it into 3.1 with a few changes. 

  • Like 1
Link to comment
Share on other sites


×
×
  • Create New...