Bug #52779

Exception if uninstalled extension has "wrong key"

Added by Ernesto Baschny about 6 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Must have
Category:
Backend API
Target version:
Start date:
2013-10-14
Due date:
% Done:

100%

TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
easy
Is Regression:
No
Sprint Focus:

Description

I know of tons of installations where some deinstalled extension in typo3conf/ext/ gets renamed to "tt_news.old" or "news.bak" etc. This was no problem before.

With the new package management feature, this now throws an exception on first hit:

InvalidPackageKeyException('"' . $packageKey . '" is not a valid package key.', 1217959510)

Solution:

The loop that scans for packages should skip those who's key is not invalid, in order to avoid an exception later down the road.

Associated revisions

Revision d35e9645 (diff)
Added by Ernesto Baschny about 6 years ago

[BUGFIX] No exception if uninstalled extension has an invalid key

The loop that scans for packages should skip those whose key is not
valid, in order to avoid an exception later down the road.

Resolves: #52779
Releases: 6.2
Change-Id: Ic9903b7d69fd6c9b9581e85d2f6ea1222ebb8885
Reviewed-on: https://review.typo3.org/24728
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Thomas Maroschik
Tested-by: Thomas Maroschik

History

#1 Updated by Gerrit Code Review about 6 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24728

#2 Updated by Gerrit Code Review about 6 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24728

#3 Updated by Gerrit Code Review about 6 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24728

#4 Updated by Gerrit Code Review about 6 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24728

#5 Updated by Gerrit Code Review about 6 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24728

#6 Updated by Gerrit Code Review about 6 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24728

#7 Updated by Ernesto Baschny about 6 years ago

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

#8 Updated by Riccardo De Contardi about 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF