Bug #34837

flow3:cache:flush should care about the .Shortcuts state

Added by Adrian Föder over 3 years ago. Updated over 3 years ago.

Status:Resolved Start date:2012-03-14
Priority:Should have Due date:
Assigned To:Christian Müller % Done:

100%

Category:Core
Target version:TYPO3 Flow Base Distribution - 1.1 beta 1
PHP Version: Complexity:
Has patch:No Affected Flow version:Git master

Description

Currently, after renaming a package path (name; from Foo to Acme.Foo) manually, flow3:cache:flush resulted in an error a class couldn't be found (naming the correct, new namespace).

So, for example, the old namespace was

Foo\ViewHelpers\BarViewHelper

while the new one, after renaming, is
Acme\Foo\ViewHelpers\BarViewHelper

(located in Packages/Application/Acme.Foo/Classes/....); and the corresponding error message was
Class Acme\Foo\ViewHelpers\BarViewHelper could not be found 

which means, he correctly looked for the new, actually intended one; but didn't have the file available.

The reason was that .Shortcuts didn't contain that (new) package path's representation.
IRC suggestion was to remove .Shortcuts completely, in order to get it rebuilt from scratch, when calling flow3:cache:flush.

Associated revisions

Revision 6d13ef27
Added by Christian Müller over 3 years ago

[BUGFIX] Clear .Shortcuts on cache flush command

In case some directories were renamed you are able to get
back to a defined state with the cache:flush command.

It will also clean the .Shortcuts directory to prevent
mismatches.

Change-Id: Ic5ca9f1d10a07f7b70c36fdca152e46e8e87ee7d
Fixes: #34837
Releases: 1.1

History

#1 Updated by Gerrit Code Review over 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/9648

#2 Updated by Christian Müller over 3 years ago

  • Assigned To set to Christian Müller

#3 Updated by Gerrit Code Review over 3 years ago

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

#4 Updated by Gerrit Code Review over 3 years ago

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

#5 Updated by Sebastian Kurfuerst over 3 years ago

  • Status changed from Under Review to Resolved
  • Target version set to 1.1 beta 1
  • % Done changed from 0 to 100

Also available in: Atom PDF