Bug #71331

Epic #65814: Make Indexed search extbase plugin shine

Search results PID

Added by Rafal Brzeski almost 4 years ago. Updated 12 months ago.

Status:
Closed
Priority:
Should have
Category:
Indexed Search
Target version:
-
Start date:
2015-11-04
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
5.6
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

There is no posibility to configure search results target PID, by default action paramter always get current PID,
this behavior prevents using this extension at all.

This issue concerns the fluid based version.

Associated revisions

Revision 7aa79998 (diff)
Added by Tymoteusz Motylewski over 3 years ago

[FEATURE] Make indexed search extbase plugin form target Pid configurable

Allows to change extbase plugin search form target page configurable
from TypoScript.
The new TS option is introduced
plugin.tx_indexedsearch.settings.targetPid

when it is not set, the current pid will be used.

Resolves: #71331
Releases: master

Change-Id: I3e529d8e2a9f2008f04552e7da0a1f41b3a295e0
Reviewed-on: https://review.typo3.org/44716
Reviewed-by: Georg Ringer <>
Tested-by: Georg Ringer <>
Reviewed-by: Markus Sommer <>
Tested-by: Markus Sommer <>
Reviewed-by: Morton Jonuschat <>
Tested-by: Morton Jonuschat <>

History

#1 Updated by Tymoteusz Motylewski almost 4 years ago

  • Parent task set to #65814

#2 Updated by Tymoteusz Motylewski almost 4 years ago

Hi Rafał,
Can you add the configuration you're using and what is expected?

#3 Updated by Rafal Brzeski almost 4 years ago

I use default configuration from plugin directory -> https://github.com/TYPO3/TYPO3.CMS/tree/master/typo3/sysext/indexed_search/Configuration/TypoScript

in old version of indexed_search was option like:

search {
    targetPid = 123
   }

I tried to find the same one in extbase version but without luck, documentation say nothing about it
https://docs.typo3.org/typo3cms/extensions/indexed_search/latest/Configuration/TypoScript/Index.html

This setting is very important in case when search box is located on every subpage of website (for example on the top of website),
so results page ID should be configurable

#4 Updated by Gerrit Code Review almost 4 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/44716

#5 Updated by Markus Sommer almost 4 years ago

  • Assignee set to Tymoteusz Motylewski

#6 Updated by Rafal Brzeski almost 4 years ago

pageUid in fluid form do the job

#7 Updated by Tymoteusz Motylewski almost 4 years ago

Hi Rafał,
Can you test and vote the patch from gerrit?
Thanks
Tymoteusz

#8 Updated by Rafal Brzeski almost 4 years ago

I could, if I knew how :)

#9 Updated by Gerrit Code Review over 3 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/44716

#10 Updated by Tymoteusz Motylewski over 3 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#11 Updated by Benni Mack 12 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF