Bug #55623

Framework Packages aren't sorted properly

Added by Thomas Maroschik almost 6 years ago. Updated over 1 year ago.

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

100%

TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Is Regression:
Yes
Sprint Focus:

Description

In the package dependency resolver, the method buildDependencyGraph builds the dependency graph first for the framework packages and afterwards for all left packages.

The filtering if a package is a framework package happens by comparing the package path with PATH_typo3 and a sysext folder contant. But as PATH_typo3 is absolute and the package path from the package states configuration is relative to PATH_site, this filter does not work.


Related issues

Related to TYPO3 Core - Bug #55646: Framework Packages aren't sorted properly if package is overriten in typo3conf/ext Closed 2014-02-04

Associated revisions

Revision 5e3b9dc4 (diff)
Added by Thomas Maroschik almost 6 years ago

[BUGFIX] Framework Packages aren't sorted properly

In the package dependency resolver, the method
buildDependencyGraph builds the dependency graph
first for the framework packages and afterwards
for all left packages.

The filtering if a package is a framework package
happens by comparing the package path with PATH_typo3
and a sysext folder contant. But as PATH_typo3 is
absolute and the package path from the package states
configuration is relative to PATH_site, this filter
does not work.

The patch changes the path to a relativ one.

Fixes: #55623
Releases: 6.2
Change-Id: I2f57dee433054463ebfd17aab5f76b19986c2747
Reviewed-on: https://review.typo3.org/27300
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Stefan Froemken
Tested-by: Stefan Froemken
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring

History

#1 Updated by Gerrit Code Review almost 6 years ago

  • Status changed from New 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/27300

#2 Updated by Thomas Maroschik almost 6 years ago

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

#3 Updated by Benni Mack over 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF