Replace deprecated usage of Drupal::url in .install status report

Hi Vinayak Kothule,
Thanks for noticing this. I have added links to the old and new functions in the issue simmary.

Your patch needs a bit of work, though:

1. It fails with Fatal error: Class 'Url' not found because you forgot to add use \Drupal\Core\Url; at the top.
2. If you do add this, and the page /admin/reports/status actually returns a result, we do not actually get the required urls. This is because Url::fromRoute('entity.user.edit_form', ['user' => $user->id()]) returns an object, when previously \Drupal::url was returning the actual string.

Your patch passed because we do not actually have test coverage for the status report page yet. That is something that I might look to add in future. However, if you visit /admin/reports/status you can see the output manually.

Jonathan

I have added tests to cover the status report page, on #2712465-2: [meta] 8.x-1.x test coverage. Now re-queueing the patch above, which should show the failures.

No response from Vinayak, so here is a new patch. I added the necessary use \Drupal\Core\Url; and both of the calls to Url::fromRoute needed ->toString()

Committed. Thanks Vinayak for starting this issue.