Drupal 9 Deprecated Code Report [#3094386]

Comment	File	Size	Author
#10	Screenshot 2020-05-22 at 5.59.36 PM.png	46.55 KB	kkalashnikov
#4	field_redirection-n3094386-4.patch	2.63 KB	DamienMcKenna
8.x-2.x: PHP 7.3 & MySQL 5.7, D9.0 Patch Failed to Apply
#2	3094386-2.patch	3.04 KB	Deeksha B
8.x-2.x: PHP 7.1 & MySQL 5.5, D8.8 Build Successful
	Deprecated.png	112.6 KB	Deeksha B

Comment #1

14 November 2019 at 11:11

Deeksha B created an issue. See original summary.

Log in or register to post comments

Comment #2

Deeksha B CreditAttribution: Deeksha B at Specbee commented 14 November 2019 at 11:24

Status:

Active

» Needs review

File	Size
3094386-2.patch	3.04 KB
8.x-2.x: PHP 7.1 & MySQL 5.5, D8.8 Build Successful

Please review the patch

Log in or register to post comments

Comment #3

DamienMcKenna

NH, USA

CreditAttribution: DamienMcKenna at Mediacurrent commented 1 April 2020 at 00:41

Category:	Bug report	» Task
Status:	Needs review	» Needs work
Issue tags:		+Drupal 9 compatibility

This needs a reroll.

Log in or register to post comments

Comment #4

DamienMcKenna

NH, USA

CreditAttribution: DamienMcKenna at Mediacurrent commented 1 April 2020 at 00:42

Status:

Needs work

» Needs review

File	Size
field_redirection-n3094386-4.patch	2.63 KB
8.x-2.x: PHP 7.3 & MySQL 5.7, D9.0 Patch Failed to Apply

2 files were hidden/shown/deleted

File	Size
Deprecated.png	112.6 KB
3094386-2.patch	3.04 KB
8.x-2.x: PHP 7.1 & MySQL 5.5, D8.8 Build Successful

Rerolled.

Log in or register to post comments

Comment #5

1 April 2020 at 00:48

Status:

Needs review

» Needs work

The last submitted patch, 4: field_redirection-n3094386-4.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

Log in or register to post comments

Comment #6

shobhit_juyal CreditAttribution: shobhit_juyal as a volunteer and at Srijan | A Material+ Company commented 22 May 2020 at 05:55

Assigned:	Unassigned	» shobhit_juyal
Issue tags:	-Drupal 9 compatibility	+Drupal 9 compatibility #DIACWMay2020

Log in or register to post comments

Comment #7

vaibhavjain

he/him

Hindi

Delhi

CreditAttribution: vaibhavjain at Srijan | A Material+ Company for Drupal India Association commented 22 May 2020 at 06:28

Issue tags:

-Drupal 9 compatibility #DIACWMay2020

+Drupal 9 compatibility, +Drupal 9 porting weekend, +DIACWMay2020

Log in or register to post comments

Comment #8

shobhit_juyal CreditAttribution: shobhit_juyal as a volunteer and at Srijan | A Material+ Company commented 22 May 2020 at 07:50

Assigned:	shobhit_juyal	» Unassigned
Status:	Needs work	» Needs review

Deprecated code not found in 9.1.x. Moving to need review.

Log in or register to post comments

Comment #9

DamienMcKenna

NH, USA

CreditAttribution: DamienMcKenna at Mediacurrent commented 22 May 2020 at 11:00

Status:

Needs review

» Needs work

Log in or register to post comments

Comment #10

kkalashnikov CreditAttribution: kkalashnikov at Srijan | A Material+ Company for Drupal India Association commented 22 May 2020 at 12:32

File	Size
Screenshot 2020-05-22 at 5.59.36 PM.png	46.55 KB

This module is compatible with Drupal version 9.

Test with the drupal-check utility and found 0 errors. For me it's RTBC.

Log in or register to post comments

Comment #11

DamienMcKenna

NH, USA

CreditAttribution: DamienMcKenna at Mediacurrent commented 22 May 2020 at 18:16

Related issues:

+#3130917: Fix D9 tests in 8.x-1.x

There's more to making a module compatible with D9 than just running drupal-check, e.g. the fixes in #3130917: Fix D9 tests in 8.x-1.x.

Log in or register to post comments

Comment #12

DamienMcKenna

NH, USA

CreditAttribution: DamienMcKenna at Mediacurrent commented 22 May 2020 at 18:32

Status:

Needs work

» Fixed

Now that the other issue has been committed there are no remaining issues left. The changes from #2/#4 were made in other issues, so I'm marking this one done.