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

YouTube invalid JSON string for search terms that return a small number of videos

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.5.3, 2.6
    • Fix Version/s: 2.5.6, 2.6.3
    • Component/s: Repositories
    • Labels:
    • Testing Instructions:
      Hide
      1. Open a text editor and open the "Media manager"
      2. Browse the YouTube repository and make sure results are loaded as you scroll.
      3. Try searches with limited number of results so you could reach the end. Examples:
        • "lavaaaaaaaaaaa"
        • "garemoko lava"
      4. Make sure content load as you scroll, but stops when you reach the final results.
      5. Check your error log
      Show
      Open a text editor and open the "Media manager" Browse the YouTube repository and make sure results are loaded as you scroll. Try searches with limited number of results so you could reach the end. Examples: "lavaaaaaaaaaaa" "garemoko lava" Make sure content load as you scroll, but stops when you reach the final results. Check your error log
    • Affected Branches:
      MOODLE_25_STABLE, MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE, MOODLE_26_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-44345-master

      Description

      To replicate:
      Add a label resource.
      Use Moodle media to search for a YouTube video.
      Search for a video that will return a small number of results e.g. "Epic learning papermation" or "garemoko lava"

      What I expect:
      A list of videos is displayed

      What actually happens:
      The Moodle displays an animation that appears to indicate it's looking for more videos. Eventually it displays an error "In valid JSON string". You can replicate this on demo.moodle.net using the search terms I suggested above.

      I expect this effects more versions than those I've listed, but I've not checked.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              fred Frédéric Massart
              Reporter:
              andrewdo Andrew Downes
              Peer reviewer:
              Dan Poltawski
              Integrator:
              Sam Hemelryk
              Tester:
              David Monllaó
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                12/May/14