0

I'm trying to use onerror with source tag but the function isn't being called.

<picture>
                <source srcset="<?=home(1).str_replace(array('.jpg', '.png', '.gif'),".webp",$movie->thumb3) ?>" type="image/webp" onerror="fallback()">
                <source srcset="<?=home().$movie->thumb3 ?>" type="image/jpeg">
                <img src="<?=home().$movie->thumb3 ?>" alt="<?=$movie_title?>">
            </picture>

javascript

function fallback() {
      var src = $(this).attr("srcset");
        src = src.replace("https://webp","https://images");
        $(this).attr("srcset",src);
    }

The fallback image isn't being used? How do i solve?

user892134
  • 3,078
  • 16
  • 62
  • 128

1 Answers1

1

That's because $(this) is returning the window object rather than the video element. You also wouldn't need jQuery for it.

You should target the event with onerror. This gives you the correct object for which you would like to change the src instead.

Example:

<picture>
  <img srcset="https://www.example.com/200/300" onerror="fallback(this)" />
</picture>
<script>
function fallback(image) {
  /* prevent endless loop */
  image.onerror = null;
  /* an error occured, change the source */
  var src = image.srcset;
  var newSrc = src.replace("https://www.example","https://www.fillmurray");
  image.srcset = newSrc;
}
</script>

Update:

Although w3c still lists the onerror event as usable, onerror is not recommended by mozilla and has some browser compatibility issues.

Another reason it's not advised to use anymore: It turns out that in a webpage where users are able to post (or otherwise add) their own content the onerror event can be used in a very exploitative way to hijack the webpage.

Sources:

One alternative to using onerror is to manually check if it exists and if not provide an alternative path: Check if image exists on server using JavaScript?

Remi
  • 4,663
  • 11
  • 49
  • 84