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

Improve "Show unlisted content" preference in contentbank

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • 4.5
    • 4.5
    • Content bank

      There are some issues with the new feature from MDL-76242 that we can improve before it's released (as this is still brand new, we might as well improve it now rather than wait for random errors/bugs in the future)

      Javascript

      1. The whole AJAX request should be replaced with repository method to achieve the same;
      2. Undefined this.userId property;
      3. Incorrect (jQuery style) promise handling, it should use native then/catch
        • Catch is missing
      4. Copy/pasted docs (also line 27)

      Index page

      1. Unreachable code here [*];
      2. There are no tests

      [*] To see the problem with this code, try visiting WWWROOT/contentbank/index.php?displayunlisted=0 and using the toggle to "Show unlisted content" - it will turn the preference on, but then reload the page and turn the preference off again (but the checkbox shows the wrong value)

            pholden Paul Holden
            pholden Paul Holden
            Pedro Jordao Pedro Jordao
            Shamim Rezaie Shamim Rezaie
            CiBoT CiBoT
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved:

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 2 hours, 32 minutes
                2h 32m

                  Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.