http://forge.typo3.org/http://forge.typo3.org/themes/typo3_forge/favicon/favicon.png?17058661692016-01-25T14:55:41ZTYPO3 ForgeTYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=2948362016-01-25T14:55:41ZBenno Florybenno.flory@gmx.ch
<ul></ul>Sorry, with better formatting: <br />Old lines 1589...:
<ul>
<li>@param boolean $showResultCount Show result count </li>
<li>@param string $addString String appended to "displaying results..." notice. </li>
<li>@param string $addPart String appended after section "displaying results... </li>
<li>@param string $freeIndexUid List of integers pointing to free indexing configurations to search. -1 represents no filtering, 0 represents TYPO3 pages only, any number above zero is a uid of an indexing configuration! </li>
<li>@return string HTML output </li>
<li>@todo Define visibility<br />*/<br />public function pi_list_browseresults($showResultCount = TRUE, $addString = '', $addPart = '', $freeIndexUid = -1) {</li>
</ul>
New lines 1589...:
<ul>
<li>@param integer $showResultCount Show result count </li>
<li>@param string $addString String appended to "displaying results..." notice. </li>
<li>@param array $addParts Array containing string appended after section "displaying results... </li>
<li>@param string $freeIndexUid List of integers pointing to free indexing configurations to search. -1 represents no filtering, 0 represents TYPO3 pages only, any number above zero is a uid of an indexing configuration! </li>
<li>@param boolean $strictStandardsDummy1 - See BUGFIXES above </li>
<li>@param boolean $strictStandardsDummy2 - See BUGFIXES above </li>
<li>@return string HTML output </li>
<li>@todo Define visibility<br />*/<br />public function pi_list_browseresults($showResultCount = 1, $addString = '', $addParts = array(), $freeIndexUid = '-1', $strictStandardsDummy1 = TRUE, $strictStandardsDummy2 = TRUE) {</li>
</ul>
<p>Additionally: <br />Old lines 680/681:<br />$browseBox1 = $this->pi_list_browseresults(1, $addString, $this->printResultSectionLinks(), $freeIndexUid);<br />$browseBox2 = $this->pi_list_browseresults(0, '', '', $freeIndexUid);</p>
<p>New lines 680/681:<br />$browseBox1 = $this->pi_list_browseresults(1, $addString, array($this->printResultSectionLinks()), $freeIndexUid);<br />$browseBox2 = $this->pi_list_browseresults(0, '', array(), $freeIndexUid);</p> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=2948482016-01-25T16:19:26ZRiccardo De Contardierredeco@gmail.com
<ul><li><strong>Parent task</strong> set to <i>#65814</i></li></ul> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=2951662016-01-28T14:56:55ZMorton Jonuschatm.jonuschat@mojocode.de
<ul><li><strong>Target version</strong> changed from <i>6.2.18</i> to <i>Candidate for patchlevel</i></li></ul> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=2973432016-02-19T11:25:41ZChristian Kuhnlolli@schwarzbu.ch
<ul></ul><p>confirmed.</p>
<p>we also need to fix this now with php 7 on core v7 and master, since e_strict was mapped to other error_types with php 7, so this is now a solid warning.</p> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=2973452016-02-19T11:28:51ZChristian Kuhnlolli@schwarzbu.ch
<ul></ul><p>default 6.2 configuration is to suppress e_strict and 6.2 is not compatible with php 7 anyway.</p>
<p>so: do NOT fix in 6.2, but do fix for 7.6 and master.</p> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=2973602016-02-19T11:51:23ZChristian Kuhnlolli@schwarzbu.ch
<ul><li><strong>TYPO3 Version</strong> changed from <i>6.2</i> to <i>7</i></li><li><strong>PHP Version</strong> changed from <i>5.4</i> to <i>7.0</i></li></ul> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=3001552016-03-21T21:13:13ZTymoteusz Motylewskit.motylewski@gmail.com
<ul><li><strong>Assignee</strong> set to <i>Tymoteusz Motylewski</i></li></ul><p>I'm wondering what is the best way to solve it.</p>
<p>1. Simplest is to add dummy parameters, to make the count match with parent method as Benno suggested.</p>
<p>However this means that the method signature will stay different from the logical point of view - indexed search params have different meaning then one from AbstractPlugin.</p>
<p>2. other solution would be to rename pi_list_browseresults to something else. Indexed Search is not calling parent::pi_list_browseresults anyway.</p>
<p>As both solutions are breaking I think we should go for the 2nd.</p> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=3001652016-03-21T22:18:33ZGerrit Code Review
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Under Review</i></li></ul><p>Patch set 1 for branch <strong>master</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/47349">https://review.typo3.org/47349</a></p> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=3001732016-03-21T22:56:21ZGerrit Code Review
<ul></ul><p>Patch set 2 for branch <strong>master</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/47349">https://review.typo3.org/47349</a></p> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=3001742016-03-21T22:59:30ZGerrit Code Review
<ul></ul><p>Patch set 3 for branch <strong>master</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/47349">https://review.typo3.org/47349</a></p> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=3002802016-03-23T17:21:18ZGerrit Code Review
<ul></ul><p>Patch set 4 for branch <strong>master</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/47349">https://review.typo3.org/47349</a></p> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=3005272016-03-30T09:29:33ZGerrit Code Review
<ul></ul><p>Patch set 5 for branch <strong>master</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/47349">https://review.typo3.org/47349</a></p> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=3005352016-03-30T10:11:11ZGerrit Code Review
<ul></ul><p>Patch set 6 for branch <strong>master</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/47349">https://review.typo3.org/47349</a></p> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=3005402016-03-30T10:49:41ZGerrit Code Review
<ul></ul><p>Patch set 1 for branch <strong>TYPO3_7-6</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="https://review.typo3.org/47416">https://review.typo3.org/47416</a></p> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=3005412016-03-30T10:50:06ZTymoteusz Motylewskit.motylewski@gmail.com
<ul><li><strong>Status</strong> changed from <i>Under Review</i> to <i>Resolved</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li></ul><p>Applied in changeset <a class="changeset" title="[!!!][BUGFIX] Make indexed search plugin PHP7 compatible Rename SearchFormController::pi_list_br..." href="http://forge.typo3.org/projects/typo3cms-core/repository/1749/revisions/c39ea264d7191c418e62c40609dbb7846e59541f">c39ea264d7191c418e62c40609dbb7846e59541f</a>.</p> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=3018352016-04-14T15:22:03ZSteffen Liebigsteffen.liebig@gmx.de
<ul></ul><p>Hi there,</p>
<p>in our installation, the mistake is still valid:</p>
<p>#1: PHP Warning: Declaration of TYPO3\CMS\IndexedSearch\Controller\SearchFormController::pi_list_browseresults($showResultCount = true, $addString = '', $addPart = '', $freeIndexUid = -1) should be compatible with TYPO3\CMS\Frontend\Plugin\AbstractPlugin::pi_list_browseresults($showResultCount = 1, $tableParams = '', $wrapArr = Array, $pointerName = 'pointer', $hscText = true, $forceOutput = false) in /home/www/typo3751/typo3_src-7.6.4/typo3/sysext/indexed_search/Classes/Controller/SearchFormController.php line 2504</p>
<p>I tried to apply the given changes, but this only transfers the error message from line 2504 to line 28 (where's only mentioned what the file is supposed to do). Curiosity show in it: I have to leave out the comment "former class xyz". If I don't, the error message occurs related to another (a 3rd) line. The explanation seems to be interpreted as an order - though the line is surely outcommented as an explanation for the following code.</p>
<p>Technical stuff:<br />Typo3 7.6.4<br />PHP 7.0.4<br />indexed_sarch from updated 7.5.1/7.6.2-installation<br />own xclasses: none</p>
<p>Macina_searchbox works fine - well, it's another module of course, but the output is on the same page as it should be with indexed_search. Doesn't matter for the error, does it ?</p>
<p>(Maybe) interesting info: <br />- Our 7.6.4 is an updated testing installation where the 7.5.1-installation was done clearly by our provider's installation routine while the rest worked properly after a core update in the installation tool. Additionally, I used a db-dump of the 6.2.14-site and ran the install toll doing necessary db updates. I still needed the old T3 cause I had to test the 7.x first. Therefore and cause I'm not really familar with consoles I didn't use a new core via changing the "never-found" symlinks. Too confusing and not working, a clean installation by routine is easier <strong>g</strong>. Maybe this affected indexed_search/T3 7.x ?!<br />- Our live-site works under 6.2.14/PHP 7 - the backend shows error messages, but I can handle the system. In this installation, no problem with indexed_search.</p>
<p>Under another 6.2.14(-testing) installation, the module shows up the search text filed, but doesn't find any results (tried a few TS-configuration hints, but it didn't work - I restored the old state).</p>
<p>Cu, Steffen</p> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=3018422016-04-14T15:55:36ZWouter Wolterstypo3@wouterwolters.nl
<ul></ul><p>Can you try a 7.6.5 installation and make sure you clear the opcode caches in the install tool if available?</p> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=3018502016-04-14T16:39:53ZTymoteusz Motylewskit.motylewski@gmail.com
<ul></ul><p>I have just tested indexed_search on 7.6.6-dev and it's working fine on PHP 7</p> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=3028712016-04-23T19:14:41ZSteffen Liebigsteffen.liebig@gmx.de
<ul></ul><p>Trying a core update, I got a general error which is not defined furthermore. Our provider has 7.6.5 so I will try to make clear about this error and update. Back asap...</p> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=3028732016-04-23T19:33:46ZSteffen Liebigsteffen.liebig@gmx.de
<ul></ul><p>Strange: i just found out that meanwhile the macina searchbox is leading to the same error (line 28), but don't know if that's a result from clearing caches. Via install tool, I cleared all caches and the log table, op code cache button not available (maybe it "returns" after core update ?!). Then, I run a database and ax extension check (as awaited - both are ok).<br />Even more strange: apache tells me about two errors - not existing files "ExtensionCompatibilityTesterErrors.json" and "robots.txt" (I know what a robots.txt is - maybe it belongs to the standard system and I just don't really use it...but what about this ...testererrors.json ?).</p> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=3029972016-04-25T16:51:52ZSteffen Liebigsteffen.liebig@gmx.de
<ul></ul><p>Update: our provider replaced the 7.6.4-core by a 7.6.5. At first (and second) glance i can report that search is working again :-).</p>
<p>This has not fixed the general error which still occurs when clicking core update in install tool, but this should be a topic to be posted somewhere else ?!</p>
<p>Would be glad about an answer to this question - best by mail I think. We can close here for the moment :-).</p> TYPO3 Core - Bug #72931: Indexed search with strict standardshttp://forge.typo3.org/issues/72931?journal_id=3801472018-10-02T11:19:32ZBenni Mackbenni@typo3.org
<ul><li><strong>Status</strong> changed from <i>Resolved</i> to <i>Closed</i></li></ul>