Moodle
  1. Moodle
  2. MDL-32290

Extension and type of uploaded files

        Details

        • Type: Bug Bug
        • Status: Closed
        • Priority: Minor Minor
        • Resolution: Duplicate
        • Affects Version/s: 2.2.2
        • Fix Version/s: None
        • Component/s: Files API
        • Labels:
          None
        • Environment:
          Moodle 2.2.2, russian
        • Database:
          MySQL
        • Testing Instructions:
          Hide

          1) Choose a file with non-empty extension when you upload a file. I.e. 'file.doc'.
          2) Type a dot and the end of 'Save as' field's content. I.e. 'This is a file.'.
          3) Upload.
          The file's name will be 'This is a filedoc' and the file's type will be unknown (icon with a questyion symbol).

          Show
          1) Choose a file with non-empty extension when you upload a file. I.e. 'file.doc'. 2) Type a dot and the end of 'Save as' field's content. I.e. 'This is a file.'. 3) Upload. The file's name will be 'This is a filedoc' and the file's type will be unknown (icon with a questyion symbol).
        • Affected Branches:
          MOODLE_22_STABLE
        • Rank:
          39085

          Description

          Type '.' (dot) at the end of 'Save as' field uploading files. Then the name of 'file.ext' file with 'saveas.' save-as-field will be 'saveasext' and type of the file will be UNKNOWN.

          Ok if we type dot in a filename we have to set an extension. And if nothing comes after it extension must be empty. But:
          1) What does mean 'Save as' field? Name of FILE or NAME of file? It's difficult for my project's student & teachers...
          2) If it's a file name why 'ext' (WITHOUT DOT) is added to 'saveas' text?

            Issue Links

            is a clone of

            Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-32370 Extension and type of uploaded files are wrong

            • Minor - Minor loss of function where workaround is possible
            • Closed

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Michael de Raadt added a comment -

              I believe this is a duplicate, or very similar to, MDL-29748. Could you please contribute to that issue.

              Show
              Michael de Raadt added a comment - I believe this is a duplicate, or very similar to, MDL-29748 . Could you please contribute to that issue.
              Hide
              Permalink
              Alexander Kurakin added a comment -

              No that issue is about l12n. Didn't you mistake? This is not duplicate! Open please!

              Show
              Alexander Kurakin added a comment - No that issue is about l12n. Didn't you mistake? This is not duplicate! Open please!
              Hide
              Permalink
              Saswat Padhi added a comment -

              This is not a duplicate.

              Show
              Saswat Padhi added a comment - This is not a duplicate.
              Hide
              Permalink
              Dongsheng Cai added a comment -

              It's a duplicate, in repository/upload/lib.php L117, if no extension provided, the code will try to add uploaded file's extension to "save as" name, I guess the regular expression there doesn't work with the extra dot well.

              Alexande, please provide exact filename you uploaded and the save as name you entered, that will help fixing this issue. And please move the discussion to MDL-29749, it's different behaviour, but same cause.

              Regards,

              Show
              Dongsheng Cai added a comment - It's a duplicate, in repository/upload/lib.php L117, if no extension provided, the code will try to add uploaded file's extension to "save as" name, I guess the regular expression there doesn't work with the extra dot well. Alexande, please provide exact filename you uploaded and the save as name you entered, that will help fixing this issue. And please move the discussion to MDL-29749 , it's different behaviour, but same cause. Regards,
              Hide
              Permalink
              Saswat Padhi added a comment -

              @Dongsheng Cai,

              The problem is not at repository/upload/lib.php : 117.
              The problem is at lib/moodlelib.php : 887 and lib/moodlelib.php : 897.

              Thanks,

              Show
              Saswat Padhi added a comment - @Dongsheng Cai, The problem is not at repository/upload/lib.php : 117 . The problem is at lib/moodlelib.php : 887 and lib/moodlelib.php : 897 . Thanks,

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: