Bug #28213

Sites Configuration should overrule package Configuration

Added by Bastian Waidelich about 4 years ago. Updated about 3 years ago.

Status:Resolved Start date:2011-07-14
Priority:Should have Due date:
Assigned To:- % Done:

100%
Category:Configuration
Target version:TYPO3 Flow Base Distribution - 1.0.5
PHP Version: Complexity:
Has patch: Affected Flow version:FLOW3 1.0.0

Description

This is not really a bug, more a missing feature, but I think, this is rather important:
It's currently not possible to override configuration from within a Sites Package.

For example:
If you want to change the default authentication redirect URI, you could add following to the Settings.yaml of your site package:

TYPO3:

   # redirect to login page when trying to access a protected page:
  FLOW3:
    security:
      authentication:
        providers:
          DefaultProvider:
            entryPoint:
              WebRedirect:
                uri: login.html

But this will be overriden by the respective configuration in the Settings.yaml of the TYPO3 package.

Of course, you can copy those lines to the global Settings.yaml, but our goal should be to be able to provide all required settings with your site package..

It should be possible to somehow configure the order of how package settings are merged..

Note: I'm setting the target version to 1.0 beta 1 because this would probably be a breaking change afterwards..

Related issues

related to TYPO3.Flow - Bug #36965: Packages are ordered reverse in configuration manager mak... Resolved 2012-05-08
related to TYPO3.Flow - Bug #28672: Inconsistent configuration overrides for packages Resolved 2011-08-02
related to TYPO3.Flow - Feature #30258: Support optional package dependencies New 2011-09-24

Associated revisions

Revision 2fba45eb
Added by Karsten Dambekalns about 3 years ago

[TASK] Add Doctrine.ORM to the dependencies

Change-Id: I50bb3c71ae59edd619155f318a0f6dff4487e37e
Related: #28213
Releases: 1.1

Revision df6b2a21
Added by Stephan Schuler about 3 years ago

[FEATURE] Order package loading by dependencies

This change simply reorders the internal packages arrays
of the PackageManager in a way that if PackageA depends
on PackageB, PackageA comes later then PackageB when
iterating over those arrays.

This change does not introduce a sophisticated dependency
management, however it makes sure to be be able to load
configuration settings in the correct order.

Change-Id: I5c9531ceaa7c08c8b48d9e572ff9a316345b1b74
Resolves: #28213
Releases: 1.1

History

#1 Updated by Stephan Schuler about 4 years ago

I agree that there has to be a certain ordre of packages. But I don't want to have all that stuff in the sites package.

As some projects will consist of many different parts, it should be allowed to split up such projects into different packages.
Especially, I want to be able to create a package "MyPackage" containing the functionality. This one may become public available. And I want to create another package "MyPackageCustom" that contains project specific changes like custom layout and settings.

So it should be possible to configure package dependencies in some way.
I would prefer to add package dependencies to the Package.php. That would simply allow the PackageManager to "usort" its internal array of packageInterfaces.

#3 Updated by Sebastian Kurfuerst almost 5 years ago

  • Target version changed from 1.0 beta 1 to 1.0 beta 2

moving to beta2, as it is an additional feature as behavior right now is undefined.

#4 Updated by Mr. Hudson almost 4 years ago

  • Status changed from New to Under Review

Patch set 5 of change I5c9531ceaa7c08c8b48d9e572ff9a316345b1b74 has been pushed to the review server.
It is available at http://review.typo3.org/3449

#5 Updated by Mr. Hudson almost 4 years ago

Patch set 6 of change I5c9531ceaa7c08c8b48d9e572ff9a316345b1b74 has been pushed to the review server.
It is available at http://review.typo3.org/3449

#6 Updated by Robert Lemke almost 4 years ago

  • Target version changed from 1.0 beta 2 to 1.0.0

#7 Updated by Karsten Dambekalns almost 4 years ago

  • Target version changed from 1.0.0 to 1.0.1

#8 Updated by Karsten Dambekalns almost 4 years ago

  • Affected Flow version set to FLOW3 1.0.0

#9 Updated by Karsten Dambekalns over 3 years ago

  • Target version changed from 1.0.1 to 1.0.2

#10 Updated by Karsten Dambekalns over 3 years ago

  • Target version changed from 1.0.2 to 1.0.3

#11 Updated by Karsten Dambekalns over 3 years ago

  • Target version changed from 1.0.3 to 1.0.4

#12 Updated by Karsten Dambekalns over 3 years ago

  • Target version changed from 1.0.4 to 1.0.5

#13 Updated by Gerrit Code Review over 3 years ago

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

#14 Updated by Gerrit Code Review over 3 years ago

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

#15 Updated by Gerrit Code Review about 3 years ago

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

#16 Updated by Gerrit Code Review about 3 years ago

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

#17 Updated by Gerrit Code Review about 3 years ago

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

#18 Updated by Gerrit Code Review about 3 years ago

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

#19 Updated by Gerrit Code Review about 3 years ago

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

#20 Updated by Gerrit Code Review about 3 years ago

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

#21 Updated by Gerrit Code Review about 3 years ago

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

#22 Updated by Gerrit Code Review about 3 years ago

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

#23 Updated by Gerrit Code Review about 3 years ago

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

#24 Updated by Stephan Schuler about 3 years ago

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

Also available in: Atom PDF