Probably a controversial change because I refused of using Drupal\Component\Graph\Graph class included into Drupal core in sake of a custom method. However, it simplified the code significantly and made it more readable as for me.

The original functionality seems buggy for me - it doesn't include all the entity dependencies into the queue. This patch does.

CommentFileSizeAuthor
content_sync-dependency_resolver-0-D8.patch3.64 KBklimp

Comments

klimp created an issue. See original summary.

klimp’s picture

klimp
Location Montréal, QC
commented
Status: Active » Needs review
blanca.esqueda’s picture

blanca.esqueda commented

Hi @klimp,

I credited and committed your patch in the content_sync Github repository.
I'm still doing some testing about importing dependencies but so far so good, and I hope to push those commits to a new release.

Also, I worked on top of it for the partial entity dependencies import/export. I put it as part of:
https://www.drupal.org/project/content_sync/issues/2873074#comment-13019230

Any comment or improvement please let me know.
Thank you!

blanca.esqueda’s picture

blanca.esqueda commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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