search

deltanet / adapt_authoring

Source code for the Adapt Authoring tool
GNU General Public License v3.0
0 stars 1 forks source link

Course import only allows 2 leter ISO 639 codes resulting in broken asset links #362

Closed dancgray closed 4 years ago

dancgray commented 4 years ago

Expected Behaviour

Import should not be broken if unexpected asset language code paths are present in the content.

Actual Behaviour

Import only checks the content JSON for asset paths with two character language codes see here

Steps to Reproduce

  1. Import a course with a non standard language code e.g. prbr
  2. Change language to something else, e.g. "en"
  3. Assets in component.json keep the ptbr part of the path.
  4. Any subsequent build/export will have prbr in the asset path.

Should decide on what language codes are accepted e.g ISO 639_1

Versions

dancgray commented 4 years ago

Cannot reproduce in the latest release.