Project

General

Profile

Actions

Bug #77664

closed

Search in translation in list view

Added by Ruud Silvrants over 7 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2016-08-29
Due date:
% Done:

100%

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

Description

When searching in the list view and "Localization view" is selected, translated items aren't found.

When you disable "Localization view" items are found.


Files

nimbus-image-1472474354270.png (39.2 KB) nimbus-image-1472474354270.png Ruud Silvrants, 2016-08-29 14:42
SC_NEW.JPG (45.4 KB) SC_NEW.JPG Lukas Taferner, 2016-10-12 13:10
SC_OLD.JPG (69.2 KB) SC_OLD.JPG Lukas Taferner, 2016-10-12 13:10
oldSearch.PNG (37.4 KB) oldSearch.PNG Jörg Velletti, 2016-10-20 18:16
actualSearch.PNG (11.3 KB) actualSearch.PNG Jörg Velletti, 2016-10-20 18:16
searchForGermanWord.PNG (16.3 KB) searchForGermanWord.PNG Jörg Velletti, 2016-10-20 18:16
Actions #1

Updated by Frans Saris over 7 years ago

  • Description updated (diff)
Actions #2

Updated by Frans Saris over 7 years ago

This is also broken for 6.2

Actions #3

Updated by Gerrit Code Review over 7 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/49668

Actions #4

Updated by Gerrit Code Review over 7 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/49668

Actions #5

Updated by Gerrit Code Review over 7 years ago

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

Actions #6

Updated by Gerrit Code Review over 7 years ago

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

Actions #7

Updated by Frans Saris over 7 years ago

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

Updated by Gerrit Code Review over 7 years ago

  • Status changed from Resolved to Under Review

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

Actions #9

Updated by Frans Saris over 7 years ago

  • Status changed from Under Review to Resolved
Actions #10

Updated by Lukas Taferner over 7 years ago

Unfortunately this patch completely changes the behaviour of the search in the list-view (please check the attached screenshots). Translations of Records with 'l10n_display' = 'defaultAsReadonly' are not shown any more.

Actions #11

Updated by Lukas Taferner over 7 years ago

Actions #12

Updated by Frans Saris over 7 years ago

Hi Lukas,

this indeed looks like a change in behavior. But the search is now more predictable for the editors than before.
In your use-case the translations are shown because the record of the default language matches your search and not because the translated record matches your search.

Groet Frans

Actions #13

Updated by Jörg Velletti over 7 years ago

Frans Saris wrote:

Hi Lukas,

this indeed looks like a change in behavior. But the search is now more predictable for the editors than before.
In your use-case the translations are shown because the record of the default language matches your search and not because the translated record matches your search.

Groet Frans

can someone explain me why line 771 in DatabaseRecordList.php was Changed during this BugFix ?

before this patch:
if ($this->localizationView && $l10nEnabled ) {

after this patch:
if ($this->localizationView && $l10nEnabled && $this->searchString === '') {

All other changes in that patch looks logical to me, but this change, changes the behavior:

in my Case:
a backend editor wants to change the image of a specific translated entry, in the past he could search for a word in the default language and had, in localisation View, all Translated Records of this search hit attached.

So he does not need to know the translation of his search word in Russian (what he does not understand) but is able to change the image in all Translations of this. Now he hast do do (with 8 translations) do 8 Searches or needs to know the ID

Reverting just this line, i get back the old behavior and also this bug #77664 is fixed a nd i cant see a negative effect.

Updated by Jörg Velletti over 7 years ago

PS: there is no diffence between activation of option "Localisation View" or not
See images:
actualSearch ,

and when reverting that line:
i get "oldSearch" Result for englisch words and can also Search for translated Records "searchForGermanWord"

Actions #15

Updated by Frans Saris over 7 years ago

That check is there to prevent that a record is shown 2 times.
1st time because the search matched, 2th because the translated record is added there

Actions #16

Updated by Jörg Velletti over 7 years ago

Thanks for the confirmation.
Yesterday i though that this situation will not happen, but i build a scenario and you are right.

I explained it to the backend editor and he accepted the new situation ...

Actions #17

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF