Project

General

Profile

Actions

Bug #89515

closed

Backend users having old bookmark/shortcut with missing parameter query in returnUrl cannot access backend

Added by Anonymous about 5 years ago. Updated almost 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Authentication
Target version:
-
Start date:
2019-10-25
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
9
PHP Version:
7.2
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

TYPO3 9.5.9
1000+ BE Users
Can't login with one (found so far)

Error:


(1/1) TypeError
parse_str() expects parameter 1 to be string, null given

in /var/www/html/web/typo3/sysext/backend/Classes/Backend/Shortcut/ShortcutRepository.php line 741

Problem is when there are entrys in sys_be_shortcuts with an old style (Updated from TYPO3 7) like


/typo3/alt_doc.php?&returnUrl=/typo3/index.php?&M=web_layout&id=12046&edit[tt_content][189684]=edit&defVals=&overrideVals=&columnsOnly=&returnNewPageId=&editRegularContentFromId=&disHelp=&noView=&SET[showPalettes]=1

without the param "query" in returnUrl.

Checking the code line from the error, I solved it with the following change.


Files

ShortcutRepository.patch (947 Bytes) ShortcutRepository.patch Anonymous, 2019-10-25 15:34

Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #89533: parse_str() expects parameter 1 to be string, null given in typo3_src-9.5.10/typo3/sysext/backend/Classes/Backend/Shortcut/ShortcutRepository.php line 743Closed2019-10-29

Actions
Actions #1

Updated by Anonymous about 5 years ago

  • Description updated (diff)
Actions #2

Updated by Anonymous about 5 years ago

  • Description updated (diff)
Actions #3

Updated by Anonymous about 5 years ago

  • Subject changed from Exception on some BE Users "parse_str() expects parameter 1 to be string, null given" to Backend users having old bookmark/shortcut with missing parameter query in returnUrl cannot access backend
Actions #4

Updated by Gerrit Code Review about 5 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/c/Packages/TYPO3.CMS/+/62182

Actions #5

Updated by Oliver Bartsch about 5 years ago

  • Related to Bug #89533: parse_str() expects parameter 1 to be string, null given in typo3_src-9.5.10/typo3/sysext/backend/Classes/Backend/Shortcut/ShortcutRepository.php line 743 added
Actions #6

Updated by Anonymous almost 5 years ago

The patch for #89533 resolves this one. Can be closed. Thanks!

Actions #7

Updated by Susanne Moog almost 5 years ago

  • Status changed from Under Review to Closed

Closed, see previous comment.

Actions

Also available in: Atom PDF