?
avatar HK1966
HK1966
26 Jul 2017

Hello,

after every update i get an 0 after the Search field.
There is a solution, but i have to do this after every update.

Go to the mod_search folder and open mod_search.php

Comment out line 28 by changing:
Code: Select all
$button         = $params->get('button', 0);

To:
Code: Select all
// $button         = $params->get('button', 0);
It would be fine if it is getting part of the standard.

Horst

avatar HK1966 HK1966 - open - 26 Jul 2017
avatar joomla-cms-bot joomla-cms-bot - labeled - 26 Jul 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 26 Jul 2017

@HK1966 thanks for contributing, can you make a Pull Request? How to make a Pull Request.

avatar brianteeman
brianteeman - comment - 26 Jul 2017

I have seen this in the past and it was always because of an outdated mod_search override in the template

avatar HK1966
HK1966 - comment - 26 Jul 2017

Thank you!
I made an Pull Request.


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

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 26 Jul 2017

thanks @HK1966, have you checked @brianteeman comment above?

avatar brianteeman
brianteeman - comment - 26 Jul 2017

Just commenting out a line of code is not a fix

avatar franz-wohlkoenig franz-wohlkoenig - change - 26 Jul 2017
Status New Discussion
avatar joomla-cms-bot joomla-cms-bot - edited - 26 Jul 2017
avatar tonypartridge
tonypartridge - comment - 26 Jul 2017

Replace the 0 with '' if the value is being output as 0. We don't want to break code but change the default value from what I understand of the issue.


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

avatar tonypartridge
tonypartridge - comment - 27 Jul 2017

I've checked the code and you must have a template override as @brianteeman states. The code you are removing is a yes/no identifier for the original module template.


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 27 Jul 2017
Status Discussion Information Required
avatar tonypartridge
tonypartridge - comment - 29 Jul 2017

@franz-wohlkoenig can you close? Non-core issue

avatar brianteeman brianteeman - change - 29 Jul 2017
Status Information Required Closed
Closed_Date 0000-00-00 00:00:00 2017-07-29 22:16:27
Closed_By brianteeman
avatar brianteeman brianteeman - close - 29 Jul 2017

Add a Comment

Login with GitHub to post a comment