#146 new
trapmuzik

Possible Bug in ProcessFeed function

Reported by trapmuzik | November 2nd, 2008 @ 10:39 PM

In the function ProcessFeed in the wpomatic.php file I noticed that there was a break statement issued if the if statement for class function isDuplicate returns true. Should this not be a continue statement? If you break then any other posts after the duplicate is ignored. After I changed this to a continue statement and manually fetched a feed that was showing a duplicate post and no post after that it came back with 32 posts after the duplicate post was encountered.

if($this->isDuplicate($campaign, $feed, $item))

  {
    $this->log('Filtering duplicate post "' . $item->get_title() . '"' );
    break;
  }

Comments and changes to this ticket

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

WP-o-Matic project.

People watching this ticket

Pages