Creation of dynamic property Drupal\feeds\Feeds\Item\SyndicationItem::$parent:fid is deprecated

'parent:fid' is actually a dynamic property. It is generated in \Drupal\feeds\Feeds\Source\BasicFieldSource. Besides 'parent:fid', you also have 'parent:uid', 'parent:created', 'parent:title' and many others.

So simply declaring $parent_fid as a variable here is not a sustainable solution.

So I think that BaseItem needs to check in methods `get()` and `set()` if the property to set exists. And if not, write to `$data` property like DynamicItem does. I don't know however if there any drawbacks in that approach, but I think the risk for that is low.

Almost. BaseItem should check first if the requested field exists as property in the class. And if not, fallback to the $data array.

@MegaChriz Tried to fix the review. Please check and do let me know, if anymore changes are needed.

• I think that property_exists() should be used in both BaseItem::get() and BaseItem::set().
• BaseItem::get() should check if $this->data[$field] exists before returning that.
• BaseItem::toArray() should merge the object vars with $data array and then remove the extra data array from it.
• In Syndicati‎onItem.php‎ there is a blank line removed that should not have been removed.
• Note that the code is currently failing tests.

Reverted the blank line code on Syndicati‎onItem.php for failing tests. I will working on the remaining.

Reviewed the MR!174 and suggested few changes

cc: @MegaChriz