Problem/Motivation

Make d7_user dependency optional and use lookup for getting media entity owner

Steps to reproduce

Proposed resolution

Remaining tasks

API changes

CommentFileSizeAuthor
#9 optional_d7_user_dependency-3224620-9.patch12.38 KBnarendrar
#2 php_logicexception-3224620-2.patch1.24 KBnarendrar

Issue fork media_migration-3224620

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

narendraR created an issue. See original summary.

narendrar’s picture

narendrar
He/Him
Primary language English
Location Jaipur, India
commented
StatusFileSize
new php_logicexception-3224620-2.patch1.24 KB

Patch file

narendrar’s picture

narendrar
He/Him
Primary language English
Location Jaipur, India
commented
Assigned: narendrar » Unassigned
Issue summary: View changes
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: php_logicexception-3224620-2.patch, failed testing. View results

huzooka made their first commit to this issue’s fork.

narendrar’s picture

narendrar
He/Him
Primary language English
Location Jaipur, India
commented
Title: Getting "Uncaught PHP Exception LogicException" » Make d7_user dependency optional and use lookup for getting media entity owner
Category: Bug report » Task
Issue summary: View changes
huzooka’s picture

huzooka
Primary language Hungarian
Location Hungary 🇭🇺🇪🇺
commented

FYI the failing source plugin tests were fixed just now in #3224625: Make Media Migration's source test base class compatible with Drupal 9.2.0+.

narendrar’s picture

narendrar
He/Him
Primary language English
Location Jaipur, India
commented
Version: 8.x-1.0-alpha11 » 8.x-1.x-dev
Assigned: Unassigned » narendrar
narendrar’s picture

narendrar
He/Him
Primary language English
Location Jaipur, India
commented
StatusFileSize
new optional_d7_user_dependency-3224620-9.patch12.38 KB

Updated patch along with test file changes

narendrar’s picture

narendrar
He/Him
Primary language English
Location Jaipur, India
commented
Status: Needs work » Needs review
huzooka’s picture

huzooka
Primary language Hungarian
Location Hungary 🇭🇺🇪🇺
commented
Assigned: narendrar » Unassigned
Status: Needs review » Reviewed & tested by the community

Thank you, @narendraR!

  • huzooka committed 0907940 on 8.x-1.x authored by narendraR 
    Issue #3224620 by narendraR, huzooka: Make d7_user dependency optional...
huzooka’s picture

huzooka
Primary language Hungarian
Location Hungary 🇭🇺🇪🇺
commented
Status: Reviewed & tested by the community » Fixed

Thank you, @narendraR!

Status: Fixed » Closed (fixed)

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