Project

General

Profile

Actions

Bug #95123

closed

Automatically added Redirects do not spawn a Notification any more

Added by Jonas Eberle over 2 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Backend User Interface
Target version:
Start date:
2021-09-06
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
11
PHP Version:
Tags:
Complexity:
medium
Is Regression:
Yes
Sprint Focus:

Description

There have been notifications when redirects where added for https://docs.typo3.org/c/typo3/cms-core/master/en-us/Changelog/10.1/Feature-89115-Auto-createRedirectsOnSlugChanges.html until commit bd0b2929027ed053c77f288a40c9e4f228cb37c7


Files

clipboard-202109061657-mu4ql.png (14.3 KB) clipboard-202109061657-mu4ql.png Jonas Eberle, 2021-09-06 14:57

Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Task #94114: EditDocumentController should use a POST request for persisting to DataHandlerClosedBenni Mack2021-05-11

Actions
Related to TYPO3 Core - Feature #89115: Auto-create Redirects on Slug ChangesClosedFrank Nägler2019-09-09

Actions
Actions #1

Updated by Jonas Eberle over 2 years ago

  • Related to Task #94114: EditDocumentController should use a POST request for persisting to DataHandler added
Actions #2

Updated by Oliver Hader over 2 years ago

  • Status changed from New to Accepted

Confirmed & caused by https://review.typo3.org/c/Packages/TYPO3.CMS/+/68714/9/typo3/sysext/backend/Classes/Controller/EditDocumentController.php#416

Previously the response did contain HTML and corresponding markup to trigger notifications.
With having an HTTP GET redirect in place, this does not work anymore and needs a different concept/approach.

Notifications are triggered by \TYPO3\CMS\Redirects\Service\SlugService::sendNotification using

        GeneralUtility::makeInstance(PageRenderer::class)->loadRequireJsModule(
            'TYPO3/CMS/Backend/BroadcastService',
            sprintf('function(service) { service.post(%s); }', json_encode($data))
        );

Actions #3

Updated by Oliver Hader over 2 years ago

  • Target version set to 11 LTS
Actions #4

Updated by Oliver Hader over 2 years ago

  • Related to Feature #89115: Auto-create Redirects on Slug Changes added
Actions #5

Updated by Oliver Hader over 2 years ago

  • Complexity set to medium
Actions #6

Updated by Gerrit Code Review over 2 years ago

  • Status changed from Accepted 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/c/Packages/TYPO3.CMS/+/71410

Actions #7

Updated by Gerrit Code Review over 2 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/c/Packages/TYPO3.CMS/+/71410

Actions #8

Updated by Gerrit Code Review over 2 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/c/Packages/TYPO3.CMS/+/71410

Actions #9

Updated by Benjamin Franzke over 2 years ago

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

Updated by Benni Mack over 2 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF