Project

General

Profile

Actions

Bug #56532

closed

PackageState.php gets rewritten on every ExtensionManager call (InvalidPackageKeyException)

Added by Alexander Opitz about 10 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Should have
Category:
Extension Manager
Target version:
Start date:
2014-03-04
Due date:
% Done:

100%

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

Description

See comment from Ernesto in https://review.typo3.org/#/c/27024/ Mar 4 2:02 PM

From chat:

laut debugger läuft er da rein weil er in "registerPackagesFromConfiguration" auf das hier stößt:
    } catch (\TYPO3\Flow\Package\Exception\InvalidPackageKeyException $exception) {
    $this->unregisterPackageByPackageKey($packageKey);
    continue;
   }

Translation:

according to debugger it runs into this, because it hits the following in "registerPackagesFromConfiguration":
    } catch (\TYPO3\Flow\Package\Exception\InvalidPackageKeyException $exception) {
    $this->unregisterPackageByPackageKey($packageKey);
    continue;
   }


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #55252: PHP 5.5 and require LocalConfigurationClosedAlexander Opitz2014-01-22

Actions
Actions #1

Updated by Daniel Siepmann about 10 years ago

  • Category set to Extension Manager
  • Assignee set to Daniel Siepmann
Actions #2

Updated by Daniel Siepmann about 10 years ago

The problem is that all packageKeys are added during testing if new are available. But invalid keys are skipped on write.

So the test for validate package keys should be done even while getting all package keys from file system.

Actions #3

Updated by Gerrit Code Review about 10 years ago

  • Status changed from Accepted 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/28286

Actions #4

Updated by Gerrit Code Review about 10 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/28286

Actions #5

Updated by Daniel Siepmann about 10 years ago

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

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF