Skip to content

Conversation

@infograf768
Copy link
Member

See #5103
To test in en-GB, add var_dump ($upper_limit); after line 33 $upper_limit = $lang->getUpperLimitSearchWord(); in
ROOT/modules/mod_search/mod_search.php

You should get int 200

@RCheesley
Copy link

Trying to test this but it's not clear what you mean by "you should get 200" - I added the line to the file as described but see no output in the front end Joomla 3.3.6 with sample data having module in a position or embedded in an article, or when adding the string to view.html.php on com_search either (assumed this would also be impacted by the change so tried to look there too.

Unable to see anything before or after applying patch therefore unable to test properly.

Further clarification would be helpful.

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

@dgrammatiko
Copy link
Contributor

@infograf768 change this from 20 to 200 to make travis happy

@infograf768
Copy link
Member Author

@RCheesley
Please use staging instead of 3.3.6
Then patch with this PR.
Then make the modification manually as indicated above.

screen shot 2014-11-16 at 19 38 57

You will get:
screen shot 2014-11-16 at 19 36 48

@dgrammatiko
Copy link
Contributor

@test works
screen shot 2014-11-17 at 9 57 30

@infograf768
Copy link
Member Author

Simple patch. Merging. Thanks for testing.

infograf768 added a commit that referenced this pull request Nov 17, 2014
[imp] Changing minimum upper limit search to 200 characters
@infograf768 infograf768 merged commit 88a6f5c into joomla:staging Nov 17, 2014
@Bakual Bakual added this to the Joomla! 3.4.0 milestone Nov 21, 2014
@infograf768 infograf768 deleted the countsearch branch February 11, 2019 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants