Project

General

Profile

Actions

Bug #72238

closed

After renaming a page only reload current page if it's the renamed one

Added by Martin Gebert almost 9 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Pagetree
Target version:
-
Start date:
2015-12-15
Due date:
% Done:

100%

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

Description

A page can be renamed by double clicking it in the pagetree. Once the renaming is done, the page currently loaded in the backends main frame is reloaded, no matter if it is the renamed page or not. This reloading should only happen if the page in the main frame is the renamed page. Otherwise the reloading should be skipped, because it makes no sense and might interrupt the users actions.

Steps to reproduce:
  • click page A in the pagetree to load the page in the main frame
  • double click page B in the pagetree
  • enter a new page name for page B (page A is still loaded in the main frame)
  • (do not press enter)
  • double click page A in the page tree

Current behaviour:
page A will be reloaded in the main frame.

Proposed behaviour:
page A will not be reloaded in the main frame.

The reload is caused by this line: https://github.com/TYPO3/TYPO3.CMS/blob/539644a7dbb5f807450185bd9e474a48d793ad97/typo3/sysext/backend/Resources/Public/JavaScript/extjs/components/pagetree/javascript/actions.js#L778

Actions #1

Updated by Gerrit Code Review almost 9 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/45391

Actions #2

Updated by Gerrit Code Review almost 9 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/45391

Actions #3

Updated by Gerrit Code Review almost 9 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/45391

Actions #4

Updated by Gerrit Code Review almost 9 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/45391

Actions #5

Updated by Gerrit Code Review almost 9 years ago

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

Actions #6

Updated by Gerrit Code Review almost 9 years ago

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

Actions #7

Updated by Gerrit Code Review almost 9 years ago

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

Actions #8

Updated by Gerrit Code Review almost 9 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/46354

Actions #9

Updated by Gianluigi Martino almost 9 years ago

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

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF