Moodle
  1. Moodle
  2. MDL-32164

'encoding' string missing in database activity import form

    Details

    • Testing Instructions:
      Hide
      • Create a new Database activity
      • Define basic fields and templates
      • From Settings choose Database activity administration -> Import entries
      • Confirm:
        • The 'File Encoding' string is present and no errors are displayed
      Show
      Create a new Database activity Define basic fields and templates From Settings choose Database activity administration -> Import entries Confirm: The 'File Encoding' string is present and no errors are displayed
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-32164-master-2
    • Rank:
      38891

      Description

      The string for 'encoding' is missing on the Database Activity's import form

        Issue Links

          Activity

          Hide
          Andrew Nicols added a comment -

          The encoding string was moved when user upload strings were moved out of admin.

          It may be that it's preferential to move this encoding string to mod/data rather than keeping it in lang/en/admin.php

          Show
          Andrew Nicols added a comment - The encoding string was moved when user upload strings were moved out of admin. It may be that it's preferential to move this encoding string to mod/data rather than keeping it in lang/en/admin.php
          Hide
          Dan Poltawski added a comment -

          David, adding you as peer reviewer here because i'm never really certain what is the best solution here for translators.

          Should the string be self contained in mod/data and then use amos scripts to CPY the existing string into new string?

          Would be interested in knowing what our best practice should be

          Show
          Dan Poltawski added a comment - David, adding you as peer reviewer here because i'm never really certain what is the best solution here for translators. Should the string be self contained in mod/data and then use amos scripts to CPY the existing string into new string? Would be interested in knowing what our best practice should be
          Hide
          David Mudrak added a comment -

          Well, it really depends case to case. I found the same string in grades.php for example, too.

          en/grades.php:$string['encoding'] = 'Encoding';
          

          Here, I would use the original string (now living in tool_uploaduser) and CPY it into mod_data.

          Show
          David Mudrak added a comment - Well, it really depends case to case. I found the same string in grades.php for example, too. en/grades.php:$string['encoding'] = 'Encoding'; Here, I would use the original string (now living in tool_uploaduser) and CPY it into mod_data.
          Hide
          Andrew Nicols added a comment -

          I've made the following changes based on David's comments:

          • moved the encoding to mod/data
          • renamed it as fileencoding
          • added AMOS commands to copy over from the location it was stolen from

          I've provided branches for master and 2.2.
          2.1 is unaffected

          Show
          Andrew Nicols added a comment - I've made the following changes based on David's comments: moved the encoding to mod/data renamed it as fileencoding added AMOS commands to copy over from the location it was stolen from I've provided branches for master and 2.2. 2.1 is unaffected
          Hide
          Andrew Nicols added a comment -

          Hi David,

          Any chance that you could glance your eye over this and check that it's correct - particularly the AMOS commands?

          Cheers,

          Andrew

          Show
          Andrew Nicols added a comment - Hi David, Any chance that you could glance your eye over this and check that it's correct - particularly the AMOS commands? Cheers, Andrew
          Hide
          David Mudrak added a comment -

          +1 both patch and the AMOScript looking good, thanks Andrew!

          Show
          David Mudrak added a comment - +1 both patch and the AMOScript looking good, thanks Andrew!
          Hide
          Andrew Nicols added a comment -

          Should rebase cleanly still

          Show
          Andrew Nicols added a comment - Should rebase cleanly still
          Hide
          Sam Hemelryk added a comment -

          Thanks Andrew, this has been integrated now

          Show
          Sam Hemelryk added a comment - Thanks Andrew, this has been integrated now
          Hide
          Ankit Agarwal added a comment -

          works as expected.
          passing
          Thanks

          Show
          Ankit Agarwal added a comment - works as expected. passing Thanks
          Hide
          Eloy Lafuente (stronk7) added a comment -

          This is now part of Moodle and a few millions people around the globe will be using it soon. Isn't that awesome?

          Many, many thanks and don't forget http://youtu.be/4N7dPaP5Z8U

          Closing, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - This is now part of Moodle and a few millions people around the globe will be using it soon. Isn't that awesome? Many, many thanks and don't forget http://youtu.be/4N7dPaP5Z8U Closing, ciao

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: