News:

Bored?  Looking to kill some time?  Want to chat with other SMF users?  Join us in IRC chat or Discord

Main Menu

SMF 2.x url and iurl tags bug

Started by asmith, January 18, 2016, 03:35:42 AM

Previous topic - Next topic

asmith

Hello,

I don't know if this is intended. On both 2.0.x and 2.1 versions.

Both of url and iurl tags, when the address is inside quotes, the output link breaks. (both single quotes and double quotes)

example:
simplemachines.org

Moreover, on SMF 2.1 Beta 2, if the link is added via the sceditor button (WYSIWYG mode). It inserts iurl tag (instead of url) and if the given address has space in it, it adds double quotes and there we have a broken link.

Suki

Hi, not sure if I understood you right.  So you are trying to do this:

[url]"http://www.simplemachines.org/community/index.php"[/url]

If thats the case then yes it will fail as either single or double quotes cannot be used in an url, SMF encodes the single quotes but the result is an invalid url. Perhaps what you want to achieve is this:

"http://www.simplemachines.org/community/index.php"

[url=http://www.simplemachines.org/community/index.php]"http://www.simplemachines.org/community/index.php"[/url]
Disclaimer: unless otherwise stated, all my posts are personal and does not represent any views or opinions held by Simple Machines.

asmith

No, that's not what I meant.
This is the example I used:

[url="http//www.simplemachines.org"]simplemachines.org[/url]

Some urls have space in them. (bad practice but they exist)

[url="http//www.example.com/my file.mp3"]My song[/url]

and as I mentioned in the first post, when in WYSIWYG mode (SMF 2.1) and trying to add a link via the link button on top, if the URL has space in it, sceditor automatically wraps it up in double quotes. (also inserts iurl instead of url)

Suki

I cannot reproduce this on SMF 2.1

Using your example:

http//www.example.com/my file.mp3

Using WYSIWYG mode I get a clean url, the same with using BBC tags. What 2.1 version are you using? or what are the steps you do to reproduce this behavior?
Disclaimer: unless otherwise stated, all my posts are personal and does not represent any views or opinions held by Simple Machines.

asmith

I just downloaded the latest from Github and the WYSIWYG bug is not happening there anymore. :)

But this double quote use:

[url="http//www.example.com/my file.mp3"]My song[/url]

Includes double quotes in the url. Is that intended?

Kindred

Слaва
Украинi

Please do not PM, IM or Email me with support questions.  You will get better and faster responses in the support boards.  Thank you.

"Loki is not evil, although he is certainly not a force for good. Loki is... complicated."

Gwenwyfar

The tag is not made to use quotes, so this isn't a bug. You can fill in a url that contains spaces without the quotes and it will work fine.

The iurl problem in WYSIWYG has been fixed.
"It is impossible to communicate with one that does not wish to communicate"

asmith

Not an issue, but only for the sake of the argument, if quotes are not supported, the URL tag (or similar ones) cannot accept optional parameters. Let's say if a mod wanted to.

Assuming spaces in url tag already work, how about this one:
[url=http//www.example.com/my file.mp3 author=artist]My song[/url]

Kindred

That's because spaces are not valid in a url, unless the are encoded....

%20 is the encoding for a space.
Слaва
Украинi

Please do not PM, IM or Email me with support questions.  You will get better and faster responses in the support boards.  Thank you.

"Loki is not evil, although he is certainly not a force for good. Loki is... complicated."

Sesquipedalian

It is worth noting that SMF 2.1 does in fact support [url="http://www example.com"]example[/url] syntax. See https://github.com/SimpleMachines/SMF2.1/pull/3565.
I promise you nothing.

Sesqu... Sesqui... what?
Sesquipedalian, the best word in the English language.

Bigguy

Quote from: Gwenwyfar on March 03, 2018, 03:22:04 PM
The tag is not made to use quotes, so this isn't a bug. You can fill in a url that contains spaces without the quotes and it will work fine.

The iurl problem in WYSIWYG has been fixed.

Updated mine not to long ago because the iurl was fixed. After installing I find it is not fixed. In my version anyway. As I said I updated my site after it said on GH that the iurl problem was fixed. I still have to change from iurl to url when using the editor.

Illori

you created a bug on github about it right? if it is still open add a comment to it, if the issue is closed create a new one so we are sure the devs know the issue may not actually be resolved, but you should upgrade to the latest code first.

Bigguy

I just did a few days ago. I guess when I get home from work I can do it again. If it is still there after that then I will open an issue.

Bigguy

Ok so I just completed an upgrade. I ran the upgrade.php not just overwriting the files. Two issues I see right now.

1.) iurl is still present in the editor. I have to switch to source view and edit the tags to make it work right.
2.) Pressing spellcheck saves a draft. It does not open a popup window. confirmed on two site.

This upgrade is from the very latest on GH. Cache (browser) has been cleared and all maintenance ran on the forum. To the best of my knowledge, with my server rebooted and SMF reporting that spell check does in fact work...all it does when pressing it is save a draft.


Bigguy

Please move this out of fixed or bogus bugs.

Bigguy

This is concerning spellcheck. I just installed SMF Articles mod on my site. I was playin around a bit. I tested the article and I saw it had a spell check button, so I pushed it. A popup window showed up and checked my spelling. So it is working on the server and in this mod but not when posting a message on the forum.

Mod I installed:

https://custom.simplemachines.org/mods/index.php?mod=1354

Bigguy

This is the only error I can find for the spell check. When pushing the button it stays down and generates this:

TypeError: sceditor.instance(...).storeLastState is not a function


Found this in editor.js

// Start up the spellchecker!
smc_Editor.prototype.spellCheckStart = function()
{
   if (!spellCheck)
      return false;

   var e = $('#' + this.sUniqueId).get(0);

   sceditor.instance(e).storeLastState();

Bigguy

Spellcheck is a bug. Just updating the thread.

Bigguy

Ok, I asked on GH but this is probably the better place to get feedback so they can get work done over there:

If you have the editor on and you click on the link icon and fill in both fields...the url and the description and post that. The link opens as it should (In a new tab without iurl). If you have the editor on and you highlight some text in your post then click the link icon and just fill in the url and not the description this is when it inserts the iurl. Is this behavior supposed to be like this.

When I posted this bug all I was doing was highlighting text and pressing the link icon to insert a url to that text. I never tried entering a description and a url in the link icon. So I have to ask are both these actions intended and this is not a bug for iurl. ??

Advertisement: