beta5 breaks filter

It was not easy to find regex to handle different scenarios. Pushed the fix in the release 8.x-2.0-beta7.

Still having issues with this using beta7, if I change the regexp to #([^"])?(https?://[^\s<]+)([^"])# it works. Made the first match capturing group optional.

We'd need to come up with a regex that works with ckeditor 4, ckeditor 5, and no editor. Will the suggested fix satisfy it?

Sorry for late reply. I have not tried it beyond my own use case. I will test and report my findings.

Yes, the pattern #([^"])?(https?://[^\s<]+)([^"])# works for all three cases.

Applied the fix.

This is almost identical to the pattern for `Replace in-line URLs` option, though. Now I wonder if I should get rid of the option, which is getting irrelevant.

This now fails for me when using no text editor and there are no characters following the url, see screenshot

This patch fixes it for me.

Peter Törnstrand

I've applied loze's patch and pushed to the latest 8.x-2.x-dev version.

Could you it works for you?

Thanks.
Ki

After further testing testing these latests commit, the regex still fails in a few places

in particular the url inside the tag gets converted to a video in the following example.
<a href="https://www.youtube.com/watch?v=CsGQOPLGImA">https://www.youtube.com/watch?v=CsGQOPLGImA</a>

Also some urls that do not have an enabled provider are mangled. for example:
the website <a href="https://drupal.org"><em>Drupal</em></a>
displays broken html, see screenshot

I think is regex should work
'#(<p>|\n|^)(https?://\S+)(</p>|\n|$)#'

This works with ckeditor and without in my testing when urls are on their own line.
It also works for Peter Törnstrand's use case when it is the only text in the textarea.

the regex in #16 still failed in some cases. Try this one, It seems to work.
'#(<p>|\n|^)?(https?://[^\s<]+)(</p>|\n|$)?#'

It seems one regex that works for one easily breaks another. I'll try to find time to do tests, but I'd appreciate those in the thread test their case and work together to come up with a regex that works for all cases. Or is that actually possible? Should we introduce a checkbox setting to handle ckeditor5 as an exceptional case? Since ckeditor5 merges all lines into one in the backend, it is essentially no different from the option setting "Replace in-line URLs"