?
avatar ThomasK-Hub
ThomasK-Hub
5 Oct 2017

Steps to reproduce the issue

  1. Create a new article with text inside like: "This is a bug test IV/264/2017"
  2. Search for IV/264/2017 in frontend Joomla search

Expected result

The result page should display a search result for it with a link to the article.

Actual result

The result page has a search result but without any link.
Warning: preg_replace(): Unknown modifier '2' in \components\com_search\views\search\view.html.php on line 268

Additional comments

This fix doesn't help with my issue: https://issues.joomla.org/tracker/joomla-cms/18221
This is my first issue here, hope I didn't mess something up ;)

avatar ThomasK-Hub ThomasK-Hub - open - 5 Oct 2017
avatar joomla-cms-bot joomla-cms-bot - labeled - 5 Oct 2017
avatar ThomasK0lasa
ThomasK0lasa - comment - 5 Oct 2017

Tested, didn't help. I wrote about it in comment section.

5 paź 2017 14:18 "Franz Wohlkönig" notifications@github.com napisał(a):

can you please test #18221
#18221?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#18244 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQd_iFOTinhBTjJPiBP9VYi40tYOkD6fks5spMkFgaJpZM4Pu-8C
.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 5 Oct 2017

have read it to late.

Issue confirmed:
screenshot-2017-10-5 search

avatar franz-wohlkoenig franz-wohlkoenig - change - 5 Oct 2017
Status New Confirmed
avatar joomla-cms-bot joomla-cms-bot - edited - 5 Oct 2017
avatar infograf768
infograf768 - comment - 6 Oct 2017

Has'nt this be solved by #18221 ?

avatar brianteeman
brianteeman - comment - 6 Oct 2017

The op explicitly says not - at a guess it is because that PR was only tested against a single / but this example has two

avatar infograf768
infograf768 - comment - 6 Oct 2017

I tested here and it works fine.

avatar infograf768
infograf768 - comment - 6 Oct 2017

screen shot 2017-10-06 at 10 20 31

avatar brianteeman
brianteeman - comment - 6 Oct 2017

The original report wasnt very clear but I can repeat the problem when the string is a link

avatar infograf768
infograf768 - comment - 6 Oct 2017

I can't here. This is what I used

<p>some text</p>
<p> </p>
<p>This is a bug test <a href="https://github.com/">IV/264/2017</a></p>

As for str_replace:
"str_replace — Replace all occurrences of the search string with the replacement string"

avatar brianteeman
brianteeman - comment - 6 Oct 2017

OK I will test again soon

avatar infograf768
infograf768 - comment - 6 Oct 2017

I see the op did

 `This is a bug test <a href="/">IV/264/2017</a>`

TinyMCE changes this to
<a href="../">IV/264/2017</a> on save

And no problem either

avatar brianteeman
brianteeman - comment - 6 Oct 2017

My branch must have been out of date - i cant replicate it either - sorry for confusion

avatar Quy
Quy - comment - 6 Oct 2017

Unable to reproduce.
Line 268 is // Make sure there are no slashes in the needle with the merged of #18221.
@ThomasK0lasa Are you sure you applied #18221?
@franz-wohlkoenig Please test again and make sure #18221 is applied.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 6 Oct 2017

Can't confirm Issue on fresh installed 3.8.2-dev.

avatar infograf768
infograf768 - comment - 7 Oct 2017

I guess this can be closed.

avatar franz-wohlkoenig franz-wohlkoenig - change - 7 Oct 2017
Status Confirmed Closed
Closed_Date 0000-00-00 00:00:00 2017-10-07 05:29:33
Closed_By franz-wohlkoenig
avatar joomla-cms-bot joomla-cms-bot - change - 7 Oct 2017
Closed_By franz-wohlkoenig joomla-cms-bot
avatar joomla-cms-bot joomla-cms-bot - close - 7 Oct 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 7 Oct 2017

closed as stated above.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/18244.

avatar joomla-cms-bot
joomla-cms-bot - comment - 7 Oct 2017

Add a Comment

Login with GitHub to post a comment