CoreCommunity ExtensionsIncubatorDistributionsTYPO3 4.5 ProjectsTYPO3 4.6 ProjectsTYPO3 4.7 ProjectsTYPO3 6.0 ProjectsTYPO3 6.1 ProjectsTYPO3 6.2 Projects (+)

Bug #31723

Issues with frequentSearches on fresh installations

Added by Peter Kraume over 1 year ago. Updated about 1 year ago.

Status:Closed Start date:2011-11-10
Priority:Should have Due date:
Assignee:Ingo Renner % Done:

100%

Category:Frontend Plugins
Target version:2.5-dkd
TYPO3 Version: Has patch:
PHP Version: Tags:
Votes: 0

Description

There are two issues in the function getSearchTermMarkerProperties() in /solr/pi_results/class.tx_solr_pi_results_frequentsearchescommand.php:

When tx_solr_statistics is empty, e.g. on a fresh installation, max() and min() throw exceptions.

Furthermore in some cases, $spread = $maximumHits - $minimumHits; can be zero which leads to a division by zero error.

A patch is attached to this bug report.

frequentSearches.diff (2 kB) Peter Kraume, 2011-11-10 16:12

frequentSearches_v2.diff (2.1 kB) Peter Kraume, 2011-11-10 17:32

History

Updated by Peter Kraume over 1 year ago

I found and fixed a third issue: there is a problem with German umlauts.

The term used with &tx_solr[q] still has umlauts converted to entities. So clicking on the link leads to a broken search.

Updated by Olivier Dobberkau over 1 year ago

  • Status changed from New to Accepted
  • Assignee set to Ingo Renner

Updated by Ingo Renner over 1 year ago

  • Assignee deleted (Ingo Renner)
  • Priority changed from Must have to Should have

Updated by Peter Kraume over 1 year ago

Well, there is a patch attached. Why don't you just apply it?

Updated by Ingo Renner over 1 year ago

Peter Kraume wrote:

Well, there is a patch attached. Why don't you just apply it?

don't panic ;)

Updated by Ingo Renner over 1 year ago

  • Subject changed from Issues with frequentSearches to Issues with frequentSearches on fresh installations
  • Status changed from Accepted to Resolved
  • Assignee set to Ingo Renner
  • % Done changed from 0 to 100

Fixed in dkd-EAP r 89749. Thanks Peter!

Updated by Ingo Renner about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF