Bug #42342
Fatal error / no possibility to flush caches after package has been removed
| Status: | Resolved | Start date: | 2012-10-24 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | Christian Jul Jensen | % Done: | 100% |
|
| Category: | Package | |||
| Target version: | TYPO3 Flow Base Distribution - 2.0 beta 1 | |||
| PHP Version: | 5.4 | Complexity: | medium | |
| Has patch: | No | Affected Flow version: | Git 1.2 (master) |
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
[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 almost 3 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 almost 3 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 almost 3 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 almost 3 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 almost 3 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 almost 3 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 2 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 2 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 2 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 2 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100