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

URL with emoticon characters renders emoticon which breaks the url

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Open
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 2.6.3, 3.4.3, 3.5, 3.9
    • Fix Version/s: None
    • Component/s: Filters
    • Labels:
    • Testing Instructions:
      Hide
      1. Create a course
      2. Insert a label to the course, with the content '<p><a href="http://emedicine.medscape.com/article/178948-overview" target="_blank">http://emedicine.medscape.com/article/178948-overview</a></p>'
      3. View the result and cross-check the eight-dash-o has not been converted to a emoticon
      4. Insert another label to the course
      5. Upload an image with eight-dash-o in the filename to the label, which then gets the content '<p><img src="… … …/animal-cat-cute-126408-o.jpg" alt="" role="presentation" class="img-responsive atto_image_button_text-bottom" width="400" height="225"></p>'
      6. View the result and cross-check the image displays correctly
      7. Insert another label to the course
      8. Insert an image which has no special filename into the label and have the alt text having a : ) combination in it.
      9. View the result and cross-check the image displays correctly

      Show
      Create a course Insert a label to the course, with the content '<p><a href="http://emedicine.medscape.com/article/178948-overview" target="_blank"> http://emedicine.medscape.com/article/178948-overview </a></p>' View the result and cross-check the eight-dash-o has not been converted to a emoticon Insert another label to the course Upload an image with eight-dash-o in the filename to the label, which then gets the content '<p><img src="… … …/animal-cat-cute-126408-o.jpg" alt="" role="presentation" class="img-responsive atto_image_button_text-bottom" width="400" height="225"></p>' View the result and cross-check the image displays correctly Insert another label to the course Insert an image which has no special filename into the label and have the alt text having a : ) combination in it. View the result and cross-check the image displays correctly
    • Affected Branches:
      MOODLE_26_STABLE, MOODLE_34_STABLE, MOODLE_35_STABLE, MOODLE_39_STABLE

      Description

      Steps to reproduce the error:

      For a course, turn editing on :
      Add Activity or Resource
      Select Label
      Under label-text add http://emedicine.medscape.com/article/178948-overview as a hyperlink
      Save changes

      You shall now see that "8-o" resolves to an emoticon when rendered on the course page.

      Probably,we can limit emoticon rendering only to texts and not urls.

        Attachments

        1. animal-cat-cute-126408-o.jpg
          animal-cat-cute-126408-o.jpg
          290 kB
        2. MDL46851.patch
          1 kB
        3. MDL-46851-after.png
          MDL-46851-after.png
          727 kB
        4. MDL-46851-before.png
          MDL-46851-before.png
          104 kB
        5. screenshot-1.jpg
          screenshot-1.jpg
          10 kB
        6. screenshot-2.png
          screenshot-2.png
          59 kB

          Issue Links

            Activity

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              hitteshahuja Hittesh Ahuja
              Peer reviewer:
              Jun Pataleta Jun Pataleta
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona)
              Votes:
              8 Vote for this issue
              Watchers:
              9 Start watching this issue

                Dates

                Created:
                Updated:

                  Time Tracking

                  Estimated:
                  Original Estimate - 0 minutes
                  0m
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 2 minutes
                  2m