netgen / NetgenAdminUIBundle

Netgen Admin UI implements an alternate administration UI for eZ Platform, based on eZ Publish Legacy administration interface
https://netgen.io
GNU General Public License v2.0
33 stars 12 forks source link

ConfigurationGenerator is not setting ezpublish.system.cache_service_name #20

Open SalvatorePollaci opened 6 years ago

SalvatorePollaci commented 6 years ago

ConfigurationGenerator(/vendor/netgen/admin-ui-bundle/bundle/Installer/Generator/ConfigurationGenerator.php) is not setting the 'cache_service_name' attribute to '%cache_pool%' in method generate.

Consequently if NetgenAdminUIBundle is used with Legacy Bridge:

So if a content is published from Legacy Admin UI, keys in pool 'cache.app' will be purged, leaving 'cache.redis' keys untouched and EZ REST API will return stale data.

Alternatively state in README.md that such attribute must be set to avoid the above problems.

emodric commented 6 years ago

I'd rather it being mentioned in README.md file for two reasons:

1) It's a configuration which does not match all usecases, therefore cannot be autogenerated 2) It is a bug in eZ Platform, so I'd rather avoid writing code that works around a bug (which was recently fixed too)

@SalvatorePollaci Do you want to create a PR? :)

SalvatorePollaci commented 6 years ago

Yes sound reasonable to me too. When I have a bit of free time I will create a PR on INSTALL.md