Closed (fixed)
Project:
Range
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
6 May 2020 at 16:06 UTC
Updated:
11 Jun 2020 at 21:09 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
stevenlafl commentedHere's a patch for 8.x-1.1
Comment #3
stevenlafl commentedHere's a patch for 1.x-dev
Comment #4
taran2lhi @stevenlafl, thanks for your contribution.
A fatal error has been fixed already in 8.x-1.x-dev; so no action here.
The other part is pretty new, deprecated in 9.0.0:
Provided patch fixes only 3 occurrences (and I'm not sure if it can be committed taking into account D8 is still here)
Comment #5
taran2lComment #6
narendra.rajwar27Comment #7
narendra.rajwar27Updated patch for comment #4 and alos add changes regarding drupal-check for deprecations.
Comment #9
narendra.rajwar27Comment #10
narendra.rajwar27Comment #11
taran2lhi @narendra.rajwar27,
Thanks for your contribution. Let's fix it step by step, and not doing everything in one issue.
This one is about setUp() declaration. Other things should be fixed in the follow-up(s).
Comment #12
narendra.rajwar27Hi @Taran2L,
actually i was trying to upload patch for failed test case. So updating only that one but it seems some issue with upload. Surely we can fix further in follow-ups.
Comment #13
narendra.rajwar27Comment #14
narendra.rajwar27Comment #15
taran2lAs I said before, let's handle setUp here and then other things in a separate issue.
Comment #17
taran2l