Bug #42342

Fatal error / no possibility to flush caches after package has been removed

Added by Robert Lemke over 6 years ago. Updated over 6 years ago.

Status:
Resolved
Priority:
Should have
Category:
Package
Start date:
2012-10-24
Due date:
% Done:

100%

PHP Version:
5.4
Has patch:
No
Complexity:
medium

Description

Steps to reproduce (production context!):

- create Acme.Demo package
- warm up caches
- remove the package manually by just deleting the package directory
- run ./flow or anyhting else

Actual result:

Fatal error: Uncaught exception 'TYPO3\Flow\Package\Exception\InvalidPackagePathException' with message 'Package path does not exist or is no directory.' in /Volumes/HDD/opt/local/apache2/htdocs/Sites/robertlemke.com/Packages/Framework/TYPO3.Flow/Classes/TYPO3/Flow/Package/Package.php on line 99

Expected result:

Handle this more gracefully – either just ignore it and update PackageStates.php or at least output an error message.
But I'd rather see 1) implemented …

Associated revisions

Revision f0ef1636 (diff)
Added by Christian Jul Jensen over 6 years ago

[BUGFIX] Handle missing - but registered - packages gracefully

In production a missing, but registered package leads to a fatal
error. Handle this situation gracefully.

Allow logging early on in the package manager.

Change-Id: Ia8e697516d6b4ed9953d52e6cd3b28cd0ceca47c
Fixes: #42342
Releases: 1.2

History

#1 Updated by Robert Lemke over 6 years ago

Robert Lemke wrote:

Steps to reproduce:

- create Acme.Demo package
- warm up caches
- remove the package manually by just deleting the package directory
- run ./flow or anyhting else

Remark: I tried this in Production context

#2 Updated by Gerrit Code Review over 6 years ago

  • Status changed from New to Under Review

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

#3 Updated by Gerrit Code Review over 6 years ago

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

#4 Updated by Gerrit Code Review over 6 years ago

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

#5 Updated by Gerrit Code Review over 6 years ago

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

#6 Updated by Gerrit Code Review over 6 years ago

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

#7 Updated by Gerrit Code Review over 6 years ago

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

#8 Updated by Gerrit Code Review over 6 years ago

Patch set 7 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/15931

#9 Updated by Gerrit Code Review over 6 years ago

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

#10 Updated by Karsten Dambekalns over 6 years ago

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

Also available in: Atom PDF