Problem/Motivation

DCD uses $config['content_directory'] this is an unusual pattern for two reasons. The configuration does not contain a 'default_content_deploy' name space. Further this is this variable is not configurable via the interface and no initial value is available in /config/install.

Proposed resolution

The use of 'content_directory' is similar to the system public path. It is set in code and the audience is the developer/devops. I therefore propose to use the same pattern to set the content directory:

$settings['default_content_deploy_content_directory'] = '../content';

Remaining tasks

Perhaps add a deprecation warning for the admin.

Comments

Sutharsan created an issue. See original summary.

Sutharsan’s picture

Issue summary: View changes
martin_klima’s picture

Assigned: Unassigned » martin_klima

Thank you for the idea and explanation. I like to learn how to do things better. Work in progress.

martin_klima’s picture

Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.