Default the contents of the "Source site address" field to "http://".

How to test

  • Install a clean D8 website (optional)
  • Download and enable migrate_upgrade
  • Navigate to /upgrade

What you currently see

The settings webform has a textfield "Source site address". This textfield is currently empty.

What you should see

This textfield should be pre-populated with "http://" as its value.

Members fund testing for the Drupal project. Drupal Association Learn more

Comments

mikeryan’s picture

hussainweb’s picture

Version: » 8.x-1.x-dev
Assigned: Unassigned » hussainweb
Status: Active » Needs review
FileSize
637 bytes

Trying this...

hussainweb’s picture

Okay, apparently there are no tests to run. Just to confirm, this does indeed set the default as 'http://' when loading the upgrade form at /upgrade.

jp.stacey’s picture

Issue summary: View changes

It took me a while to work out what was going on here (especially as the patches is prefixed "migrate_update-", and I'm new to IMP!) so I've reworded the issue description with steps to repeat. Hope that's OK.

I reviewed the patch in #2; looks fine to me. Worth getting someone else to review before moving to RTBC? Seems a pretty straightforward change.

benjy’s picture

Assigned: hussainweb » Unassigned
Status: Needs review » Reviewed & tested by the community

Seems good to me.

mikeryan’s picture

Status: Reviewed & tested by the community » Fixed

Committed, thanks!

  • mikeryan committed e6b2f71 on 8.x-1.x authored by hussainweb
    Issue #2281775 by hussainweb: Default Source site address field to http...

Status: Fixed » Closed (fixed)

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