Support » Plugin: Slideshow Reloaded » Error message

  • I changed for this plugin because of excessive error messages, but unfortunately this one does not appear to be flawless either:

    Only variables should be passed by reference in /var/www/domains/gerkoper.nl/subdomains/blog/wp-content/plugins/slideshow-reloaded/views/SlideshowReloadedSlideshowSlide/frontend_attachment.php on line 108

    Did I do something wrong?

    Thanks in advance for the help,

    Ger

    The page I need help with: [log in to see the link]

Viewing 3 replies - 1 through 3 (of 3 total)
  • While it’s definitely the Wrong Thing for the developer of this plugin to leave warning messages like that in a released plugin, you can switch off warnings and notices and you should stop seeing the messages anywhere except in your log files.

    Hello,

    i have the same issue.
    Notice: Only variables should be passed by reference in ./wp-content/plugins/slideshow-reloaded/views/SlideshowReloadedSlideshowSlide/frontend_attachment.php on line 108

    I made correction :
    replace line 112 in file:

    <?php if (in_array(end(explode('.', $imageSrc)), array('avi', 'flv', 'wmv', 'mov', 'mp4', 'webm', '3gp', 'ogg'))) { ?>

    with

    
    $arrayTmp = array('avi', 'flv', 'wmv', 'mov', 'mp4', 'webm', '3gp', 'ogg');
    $srcTAb = explode('.', $imageSrc);
    if (in_array(end($srcTAb), $arrayTmp)) {
    • This reply was modified 2 years, 7 months ago by abonnefoy.
    • This reply was modified 2 years, 7 months ago by abonnefoy.
    Thread Starter gkoper

    (@gkoper)

    Dear Abonnefoy

    That indeed worked well, albeit that the replacement required is

    <?php
                        $arrayTmp = array('avi', 'flv', 'wmv', 'mov', 'mp4', 'webm', '3gp', 'ogg');
                        $srcTAb = explode('.', $imageSrc);
                        if (in_array(end($srcTAb), $arrayTmp)) {
                            ?>
    

    Thank you for the suggestion.

    Ger

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘Error message’ is closed to new replies.