Simplepie Error handling

[X-Tream]

Hello,

I'm in need of some sort of an error handling script...
My site fetches rss feeds from other sites (by using simplepie) and displays them like this: (http://netti-tv.net), but i'm having problems with one of the rss feeds, as some times the feed has no image -> my site crashes giving me this error code (http://netti-tv.net/katsomo):

Warning: Invalid argument supplied for foreach() in /home2/nettib22/public_html/katsomo.php on line 133

Fatal error: Call to a member function get_link() on a non-object in /home2/nettib22/public_html/katsomo.php on line 134 

Lines 133-138:

        foreach($item->get_enclosures() as $enclosure);
        $i['img'] = str_replace("_m.jpg","_s.jpg" , $enclosure->get_link());
        
        if(empty($i['img'])){
        $i['img'] = 'img/noimage.png';
        } 

I'm looking for a script that checks the rss feeds images and if there is no image it replaces it with "img/noimage.png".
The above script works other vice but it has no error handling...

Sorry for my bad English. I really hope somebody could help me as it's really annoying that my site crashes...I would really appreciate it.

Edit: For now there is no error in http://netti-tv.net/katsomo but i'm still looking for the script. =/

[traq]

# the first error is happening because foreach{} expects an array (or iterable object).
# you need to make sure that's what you give it.
# the second error is happening because of the first error.

# try the method first
$enclosures = $item->get_enclosures();

# see if it worked
# (I'm assuming this get_enclosures method returns FALSE if it doesn't return an array.
#  If that is not the case, you might need to use "if( is_array( $enclosures ) )" instead - 
#  more correct, more accurate; but, if it works, doing "if( $enclosures )" is faster.
if( $enclosures ){

// foreach($item->get_enclosures() as $enclosure);
# ALWAYS use brackets!
    # pass $enclosures to foreach{}
    foreach( $enclosures as $enclosure ){ //<-- important opening bracket

        # likewise, you might want to check if ::get_link() is returning a string as you expect
        $i['img'] = str_replace("_m.jpg","_s.jpg" , $enclosure->get_link());

    }//<-- important closing bracket

} // end "if( $enclosures )"
     
if(empty($i['img'])){
    $i['img'] = 'img/noimage.png';
} 

[X-Tream]

Thank you so much for your help! It's working like a dream.

[traq]

glad to hear it!

If your question has been answered, please mark your thread "resolved":

• On your original post (post #1), click [edit], then click [go advanced].
• In the "thread prefix" box, select "Resolved".
• Click [save changes].