Task #40304
As a user, I'd like to be redirected to the last edited page after logging out
| Status: | Resolved | Start date: | ||
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | Karsten Dambekalns | % Done: | 100% |
|
| Category: | - | |||
| Target version: | - |
Related issues
Associated revisions
[TASK] Redirect to last edit page after logout
Uses the Neos_lastVisitedUri in a similar way to the login.
Change-Id: Iabf45c44edb1584c0d5c630b60e5ddf02f6fa6bb
Resolves: #40304
[TASK] Refactored backend login/logout code
This introduces a BackendRedirectionService which is responsible for
the redirections after a backend login or logout.
For redirecting an editor "back" to the frontend page where he came
from before logging in, a session key ``lastVisitedUri`` is maintained.
Using the browser's sessionStorage it stores the user's last visited URI
to the session (see I31d77af758cc938e68952344315546d9a7c9379b).
On logout, the content editor's last visited URI has the namespace
indicator (``@``) removed and the editor is redirected to that URI.
Resolves: #45361
Related: #40304
Change-Id: I964f0d1dc05b14e8bc6e0b866583513b91e08769
[FEATURE] Improved redirection upon login/logout
This improves the ``BackendRedirectionService`` by decoupling it from
hard-coded URLs.
Neos currently stores the absolute URL in the browsers session storage.
Upon login a redirect-URL is constructed by appending the user's
workspace to the stored path.
This breaks if ``FLOW_REWRITEURLS`` is not set or if one decides to use
custom Routes (for example skipping the ".html" suffix).
This patch changes this by storing the identifier of the last visited
document node instead and by only using the ``UriBuilder`` to construct
URLs.
This also moves the JavaScript snippet from NeosBackendHeaderData.html
to a separate template that is appended before the closing body tag of
a page via TypoScript for performance reasons and so that this
functionality can be disabled without having to adjust the templates.
Besides this introduces some minor tweaks:
- Disable default error FlashMessage for LoginController
- Only add "logout" FlashMessage when the login Form is displayed
afterwards (avoids outdated FlashMessages) - One DB lookup less in NodeController::showAction()
Change-Id: I652a6354cf70c7f06825c40d840303b2199fb7de
Related: #40304
Reviewed-on: https://review.typo3.org/25676
Tested-by: Bastian Waidelich
Reviewed-by: Aske Ertmann
Tested-by: Aske Ertmann
Reviewed-by: Bastian Waidelich
[TASK] Move lastVisitedNode functionality to external script
Replaces the inline script that stores the last visited document node
in the session storage to an external file so that it doesn't violate
the Content Security Policy specification and so that it can be cached
by the browser.
The current node identifier is passed to the external script via a
data-neos-node attribute on the script tag.
Change-Id: I7237f97499be1e56c3ea9fa0c4403c333149c220
Depends: Id3d887094e3a79e58ccfe4a89dff87ca0a96c291
Related: #40304
Reviewed-on: https://review.typo3.org/25688
Reviewed-by: Bastian Waidelich
Tested-by: Bastian Waidelich
[BUGFIX] Use FQ TypoScript paths for lastVisitedNode functionality
Change Idaa3e0c5c3393533158840dc0617676f2589c628 made the shorthand
TypoScript object names not being found in the prototype Page.ts2
for the lastVisitedNode inclusion. This fixes this by using fully
qualified paths.
Change-Id: I0b315f3fa5d0a2c5de263e714339dbdff9856d2b
Releases: master, 1.0
Related: #40304
Reviewed-on: https://review.typo3.org/26369
Reviewed-by: Christian Mueller
Reviewed-by: Adrian Föder
Tested-by: Adrian Föder
Reviewed-by: Dominique Feyer
Tested-by: Dominique Feyer
Reviewed-by: Christopher Hlubek
Tested-by: Christopher Hlubek
[BUGFIX] Use FQ TypoScript paths for lastVisitedNode functionality
Change Idaa3e0c5c3393533158840dc0617676f2589c628 made the shorthand
TypoScript object names not being found in the prototype Page.ts2
for the lastVisitedNode inclusion. This fixes this by using fully
qualified paths.
Change-Id: I0b315f3fa5d0a2c5de263e714339dbdff9856d2b
Releases: master, 1.0
Related: #40304
Reviewed-on: https://review.typo3.org/26388
Reviewed-by: Christopher Hlubek
Tested-by: Christopher Hlubek
History
#1 Updated by Karsten Dambekalns over 2 years ago
- Tracker changed from Story to Task
#2 Updated by Karsten Dambekalns over 2 years ago
- Project changed from Base Distribution to TYPO3.Neos
- Status changed from New to Accepted
- Assigned To set to Karsten Dambekalns
#3 Updated by Karsten Dambekalns over 2 years ago
- Status changed from Accepted to Under Review
#4 Updated by Karsten Dambekalns over 2 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 34eb9ae0928c4b6ecc28e55da72a9e8fc8b80657.