FeedsProcessor's methods signatures have changed somewhat in the last year. I'm updating FeedsECKprocessor to match them. Also, some code cleaning and the patch from #2407899: strict warning FeedsECKprocessor->getMappingTargets() are included.

CommentFileSizeAuthor
eck-feeds-code-update.patch2.9 KBpeterpoe

Comments

peterpoe’s picture

peterpoe commented
Title: FeedsECKprocessor need to catch up with FeedsProcessor » FeedsECKprocessor needs to catch up with FeedsProcessor
lexbritvin’s picture

lexbritvin commented
Status: Needs review » Fixed

Hello, peterpoe!

FeedsECKprocessor has been updated to satisfy parent class methods. Redundant methods were removed. Other changes to form callback and code style were made.

I didn't see your patch, sorry for not crediting it.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.