Make the yaml importer load multiple languages.

[veluca93]

No description provided.

[codecov]

Codecov Report

Attention: Patch coverage is 17.64706% with 14 lines in your changes missing coverage. Please review.

Project coverage is 69.41%. Comparing base (7db9aa1) to head (02112a0).
Report is 6 commits behind head on master.

Files	Patch %	Lines
cmscontrib/loaders/italy_yaml.py	6.66%	14 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1253      +/-   ##
==========================================
- Coverage   69.45%   69.41%   -0.04%     
==========================================
  Files         328      328              
  Lines       26196    26208      +12     
==========================================
- Hits        18195    18193       -2     
- Misses       8001     8015      +14     

Flag	Coverage Δ
functionaltests	46.99% <0.00%> (-0.22%)	⬇️
unittests	56.79% <17.64%> (+0.07%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[wil93]

IIUC this would mean that when no primary_language is specified in the task.yaml and the following files are present in the statement folder:

• english.pdf
• french.pdf
• italian.pdf
• spanish.pdf

We would actually only import 3 files (skipping italian.pdf). Is it correct?

[wil93]

Oh, no actually it would fail with "Couldn't find any task statement, aborting".

Still, maybe the logic can be changed so that if statement.pdf / testo.pdf are both missing, we don't immediately fail if there are other usable statement files?

[veluca93]

Feel free to make that change.