Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-26832

META - Several repository & file picker bugs

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.2
    • Fix Version/s: 2.3
    • Component/s: Repositories
    • Labels:
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE

      Description

      • When using the file picker and you have "View as List" activated, then paging doesn't work. It appears that the page number is not passed in the AJAX request, so the first page's content is always returned.
      • The default when viewing content in the repository file picker is the "View as Icons", but it doesn't initially render as "Selected" or "Highlighted". It's only after "View as List" has been selected and then "View as Icons" has been reselected.
      • When searching, should this dialog be modal? Currently you can mess around with the main file picker window when the search dialog is up.
      • The search dialog has a hard-coded title of "filepicker", this should be something more useful, like Search or Searching in [Repository Name].
      • When selecting a file in the file picker and then going through the UI to save the file, if you then remove the file name and click Save, you get an error: "File exists". It should probably spit out an error like "File name required" and maybe do that in the save as UI instead of on post.
      • The search dialog doesn't respond to the Enter or Return key being pressed. Those keys should submit the search dialog.
      • Script error displays when selecting a file while file are displayed in view as list mode in IE8. Error details:

        Message: Can't move focus to the control because it is invisible, not enabled, or of a type that does not accept the focus.
        Line: 2649
        Char: 25
        Code: 0
        URI: http://yui.yahooapis.com/combo?2.8.2/build/treeview/treeview-debug.js

      • (Note: might be related to above issue with pushing return/enter in search dialog) Error when enter/return key is pressed in the search dialog in IE8. Error details:

        Message: Invalid argument.
        Line: 482
        Char: 4
        Code: 0
        URI: http://yui.yahooapis.com/combo?2.8.2/build/yahoo-dom-event/yahoo-dom-event.js&2.8.2/build/connection/connection-debug.js&2.8.2/build/logger/logger-debug.js

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            dongsheng Dongsheng Cai added a comment -

            Adding this to stable backlog.

            Show
            dongsheng Dongsheng Cai added a comment - Adding this to stable backlog.
            Hide
            mblake Michael Blake added a comment -

            Raising priority on this issue: it's been reported by a MP and should greatly improve repository usability.

            Show
            mblake Michael Blake added a comment - Raising priority on this issue: it's been reported by a MP and should greatly improve repository usability.
            Hide
            bushido Mark Nielsen added a comment -

            Adding a few more discovered issues.

            Please feel free to ask about more details on any of the issues.

            Show
            bushido Mark Nielsen added a comment - Adding a few more discovered issues. Please feel free to ask about more details on any of the issues.
            Hide
            dougiamas Martin Dougiamas added a comment -

            Dongsheng can you please make some subtasks for these and tackle the low hanging fruit

            Show
            dougiamas Martin Dougiamas added a comment - Dongsheng can you please make some subtasks for these and tackle the low hanging fruit
            Hide
            dongsheng Dongsheng Cai added a comment -

            issue 5 is related to MDL-23989

            Show
            dongsheng Dongsheng Cai added a comment - issue 5 is related to MDL-23989
            Hide
            bushido Mark Nielsen added a comment -

            Thanks so much on the work on this issue thus far!!!

            I was wondering if this was going or has been addressed:

            When selecting a file in the file picker and then going through the UI to save the file, if you then remove the file name and click Save, you get an error: "File exists". It should probably spit out an error like "File name required" and maybe do that in the save as UI instead of on post.

            Show
            bushido Mark Nielsen added a comment - Thanks so much on the work on this issue thus far!!! I was wondering if this was going or has been addressed: When selecting a file in the file picker and then going through the UI to save the file, if you then remove the file name and click Save, you get an error: "File exists". It should probably spit out an error like "File name required" and maybe do that in the save as UI instead of on post.
            Hide
            dongsheng Dongsheng Cai added a comment -

            Hi Mark

            The 'file exists' issue will be fixed by MDL-26388, we already have a solution for this

            Show
            dongsheng Dongsheng Cai added a comment - Hi Mark The 'file exists' issue will be fixed by MDL-26388 , we already have a solution for this
            Hide
            mblake Michael Blake added a comment -

            MP has again flagged this issue as causing problems for their client. Please give it priority.

            Show
            mblake Michael Blake added a comment - MP has again flagged this issue as causing problems for their client. Please give it priority.
            Hide
            dongsheng Dongsheng Cai added a comment -

            I will focus on those issues for next sprint.

            Show
            dongsheng Dongsheng Cai added a comment - I will focus on those issues for next sprint.
            Hide
            armanis Silverio Armani added a comment -

            Hi Dongsheng,

            Please, could you add MDL-27025 as a sub-task?

            Show
            armanis Silverio Armani added a comment - Hi Dongsheng, Please, could you add MDL-27025 as a sub-task?
            Hide
            dongsheng Dongsheng Cai added a comment -

            A lot of issues here should be fixed by MDL-31907

            Show
            dongsheng Dongsheng Cai added a comment - A lot of issues here should be fixed by MDL-31907
            Hide
            poltawski Dan Poltawski added a comment -

            Hi,

            I'm closing this issue as most of the issues in subtasks have been addressed in 2.3

            I have moved the remaining 2 issues to bugs and i'm closing this issue. Thanks everyone.

            Show
            poltawski Dan Poltawski added a comment - Hi, I'm closing this issue as most of the issues in subtasks have been addressed in 2.3 I have moved the remaining 2 issues to bugs and i'm closing this issue. Thanks everyone.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  25/Jun/12