Uploaded image for project: 'LAMS Development'
  1. LAMS Development
  2. LDEV-2450

Change language file folder format for combined tools to use a unique name instead of an id number

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: 2.4, 2.3.2
    • Component/s: General
    • Labels:
      None

      Description

      Combined tools have their language folders put inside a folder which is a number referring to it's id in the learning library table. Since this number is possibly different for each instance, to update the language files we have to run a script that checks the id in the database and finds the correct folder to copy the files - which is annoying.

      If we change it so the folder is a unique name, we can always assume that folder is there and update it by simply copying the files.

      The unique name should also be stored in the database and the langauge filter needs to be adjusted to search for the folder via this unique name instead of the id.

        Activity

        Hide
        ernieg Ernie Ghiglione added a comment -
        Great. Thanks Marcin
        Show
        ernieg Ernie Ghiglione added a comment - Great. Thanks Marcin
        Hide
        marcin Marcin Cieslak added a comment -
        Problem was supposed to be fixed in 2.3.5 version already, but due to previous missing commits, users using 2.3.5 still have a bug.
        The patch fixing it had to be used in upgrade to LAMS 2.4.
        Show
        marcin Marcin Cieslak added a comment - Problem was supposed to be fixed in 2.3.5 version already, but due to previous missing commits, users using 2.3.5 still have a bug. The patch fixing it had to be used in upgrade to LAMS 2.4.
        Hide
        marcin Marcin Cieslak added a comment -
        This JIRA is missing some fixes.
        Show
        marcin Marcin Cieslak added a comment - This JIRA is missing some fixes.
        Hide
        ernieg Ernie Ghiglione added a comment -
        Marcin, I'll reassign this to you, so you can decide how to fix this for 2.4. Thanks
        Show
        ernieg Ernie Ghiglione added a comment - Marcin, I'll reassign this to you, so you can decide how to fix this for 2.4. Thanks
        Hide
        lfoxton Luke Foxton added a comment -
        Done on 2.3 branch and head
        Show
        lfoxton Luke Foxton added a comment - Done on 2.3 branch and head

          People

          • Assignee:
            marcin Marcin Cieslak
            Reporter:
            lfoxton Luke Foxton
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development