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

Videos in messaging and chat give an error

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.7, 3.1.3, 3.2
    • Fix Version/s: 3.1.4
    • Component/s: JavaScript, Messages
    • Labels:
    • Testing Instructions:
      Hide
      1. Ensure you have 'Convert URLs into links and images' and 'Multimedia plugins' filters turned on with the URL one listed first.
      2. Visit the messaging page (on master) or chat window (on stables and master)
      3. Send the URL http://techslides.com/demos/sample-videos/small.mp4 to another user.
      4. Send a youtube video and a vimeo video as well
      5. Also send an mp3 file
      6. Check that you do not get errors on the screen or in error.log
      Show
      Ensure you have 'Convert URLs into links and images' and 'Multimedia plugins' filters turned on with the URL one listed first. Visit the messaging page (on master) or chat window (on stables and master) Send the URL http://techslides.com/demos/sample-videos/small.mp4 to another user. Send a youtube video and a vimeo video as well Also send an mp3 file Check that you do not get errors on the screen or in error.log
    • Affected Branches:
      MOODLE_30_STABLE, MOODLE_31_STABLE, MOODLE_32_STABLE
    • Fixed Branches:
      MOODLE_31_STABLE
    • Pull Master Branch:
      wip-MDL-56899-master

      Description

      Reproduction steps

      1. Enable video JS at Site administration / ▶︎ Plugins / ▶︎ Media players / ▶︎ Manage media players
      2. Go to the new messaging interface
      3. Send a video to someone
      4. Reload the page
      5. You will get this error

        Warning: DOMDocument::loadHTML(): Tag video invalid in Entity, line: 1 in /home/vagrant/moodles/integration_master/moodle/lib/weblib.php on line 1322
        

      6. The video will be tiny (the size of the play button)

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              marina Marina Glancy
              Reporter:
              johno John Okely
              Peer reviewer:
              John Okely
              Integrator:
              David Monllaó
              Tester:
              Rajesh Taneja
              Participants:
              Component watchers:
              Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                9/Jan/17