Returning feedData from beforeProcessFeed() and counting the number of items after, instead of before

[kristiansp]

The proposed changes adds the feed data returned from EVENT_BEFORE_PROCESS_FEED to the return array, and counts the number of items after beforeProcessFeed() is called, instead of before. This gives the ability to actually modify the feed data via the EVENT_BEFORE_PROCESS_FEED event.

The change in #462 give the EVENT_BEFORE_PROCESS_FEED the ability to change the $feedData in the event. However, as nothing else is happening in the beforeProcessFeed() function, and the feed data is not part of the $return array returned from the function, any changes made to the actual feed data is effectively lost.

Furthermore, as the items in the feed data are counted before beforeProcessFeed() is called, any changes to the number of items in the feed (e.g. removing duplicates), will give the wrong number of iterations in the queue.

[brandonkelly]

Agree this is better. Thanks!

[brandonkelly]

Feed Me 4.2.0 is out now with this change.