Project

General

Profile

Actions
Copy link

Feature #96868

closed

Also load "Configuration/Services.yml" for dependency injection, or at least emit a warning if it exists

Added by Jost Baron almost 3 years ago. Updated almost 3 years ago.

Status:
Rejected
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2022-02-13
Due date:
% Done:

0%

Estimated time:
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

Currently, only "Configuration/Services.yaml" is loaded, but if the file "Configuration/Services.yml" exists, this is almost always a bug (which has cost me much time).

To handle this better, two solutions come to mind:

  1. Also load the .yml file (After or before the .yaml file?)
  2. Emit a warning if the .yml file exists.

Both options should not have much of a performance impact due to caching.

Actions
Copy link
 #1

Updated by Georg Ringer almost 3 years ago

  • Status changed from New to Rejected

Thanks for creating the issue. I understand the problem but adding this one additional check adds one variant again and there are more variants like Configuration/Service.yaml or Configuration/service.yaml and every combination with yml.

We try to remove variants (same with .typoscript, .txt) to make it easier for everyone, therefore I reject the issue.

Actions
Copy link

Also available in: Atom PDF