Project

General

Profile

Actions

Bug #72797

closed

EditDocumentController / languageSwitch / localized records without a default language record relation

Added by Wolfgang Klinger almost 9 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Localization
Start date:
2016-01-18
Due date:
% Done:

100%

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

Description

The 'Language' menu on top of the edit document view makes no sense and results in strange behaviour if you edit a record that has no default language (0) record relation.
The languageSwitch method of class EditDocumentController relies on an existing default language (0) record.
Generated links look like:

<option value="tce_db.php?&amp;cmd[tt_content][][localize]=8&amp;redirect=alt_doc.php…">Italian [NEW]</option>

('localize' with an empty id key)
and in the end redirects to an edit view of a randomly selected record(?)

I would suggest to remove the menu if the open record has no default language record relation.


Files

Schermata 2016-05-14 alle 16.13.15.png (53.2 KB) Schermata 2016-05-14 alle 16.13.15.png Riccardo De Contardi, 2016-05-14 16:17

Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #78955: “language” menu on top of the content element is missing the possibility to create [NEW] translations out of every language selectionClosed2016-12-12

Actions
Actions #1

Updated by Riccardo De Contardi over 8 years ago

This still occurs on 8.2-dev

steps to reproduce
1) have 2 languages, e.g. default (Italian) and english (ID=1)
2) have a page with 2 languages, go on it, select "languages" view
3) create a new content element in the alternative language (english)
in doing that you'll notice that the first dropdown reads the default language (wrong) but the language of the element is correct (see attached screenshot)
after saving, the first will become correct.
4) save and close it
5) edit again the element and, using the dropdown, "translate" it into "Italian Default [NEW]"
6) RESULT: you will be redirected to a random record.

Actions #2

Updated by Gerrit Code Review about 8 years ago

  • Status changed from New 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/50186

Actions #3

Updated by Gerrit Code Review about 8 years ago

Patch set 2 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/50186

Actions #4

Updated by Gerrit Code Review about 8 years ago

Patch set 3 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/50186

Actions #5

Updated by Gerrit Code Review about 8 years ago

Patch set 4 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/50186

Actions #6

Updated by Gerrit Code Review about 8 years ago

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/50298

Actions #7

Updated by Gerrit Code Review about 8 years ago

Patch set 5 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/50186

Actions #8

Updated by Tymoteusz Motylewski about 8 years ago

  • Category changed from Backend User Interface to Localization
  • TYPO3 Version changed from 6.1 to 7
Actions #9

Updated by Christina Hauk about 8 years ago

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

Updated by Gerrit Code Review almost 8 years ago

  • Status changed from Resolved 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/51092

Actions #11

Updated by Gerrit Code Review almost 8 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/51092

Actions #12

Updated by Gerrit Code Review almost 8 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/51092

Actions #13

Updated by Gerrit Code Review almost 8 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/51092

Actions #14

Updated by Patrick Gaiser almost 8 years ago

We've got the same problem (BUG #78955) and tested your latest patch which works successfully.
Thanks for this solution.

Actions #15

Updated by Gerrit Code Review over 7 years ago

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/51805

Actions #16

Updated by Sascha Nowak over 7 years ago

  • Status changed from Under Review to Resolved
Actions #17

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF