- Issue created by @rajeshreeputra
- Merge request !33Resolve #3500658 "Fix media asset_id handling not honored during migration." → (Merged) created by rajeshreeputra
- 🇮🇳India rajeshreeputra Pune
With this fix the media listing also gets fixed at
/admin/content/dam-media. - 🇮🇳India rajeshreeputra Pune
external_id related work will be completed in related issue.
- 🇺🇸United States japerry KVUO
This might have something to do with an error I see when deleting migrated content as well.
"Warning: Attempt to read property "asset_id" on null in acquia_dam_media_delete() (line 696 of /var/www/html/upstreams/acquia_dam/acquia_dam.module). acquia_dam_media_delete(Object)"Won't make a new ticket until we validate against this one.
- 🇮🇳India chandu7929 Pune
Verified the migration with latest changes, and I can confirm that its working as expected. Also I like the idead of using existing queue worker to update asset with external_id becuase we already making api call to get asset details.
Hence RTBC
"Warning: Attempt to read property "asset_id" on null in acquia_dam_media_delete() (line 696 of /var/www/html/upstreams/acquia_dam/acquia_dam.module).
acquia_dam_media_delete(Object)"
Not able to replicate this issue.
-
japerry →
committed 069a6ebe on acms3570-migration-path authored by
rajeshreeputra →
Issue #3500658: Media asset_id, version_id, external_id handling not...
-
japerry →
committed 069a6ebe on acms3570-migration-path authored by
rajeshreeputra →
- 🇺🇸United States japerry KVUO
lgtm as well! The addition to the queue worker is perfect. Committed.
Automatically closed - issue fixed for 2 weeks with no activity.