$ drupal-check -d plupload/
7/7 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

[OK] No errors

CommentFileSizeAuthor
#6 interdiff_2-6.txt314 bytessuresh prabhu parkala
#6 3130920-6.patch718 bytessuresh prabhu parkala
#2 3130920-2.patch312 byteskapilv

Comments

kapilkumar0324 created an issue. See original summary.

kapilv’s picture

kapilv commented
StatusFileSize
new 3130920-2.patch312 bytes
kapilv’s picture

kapilv commented
Status: Needs review » Reviewed & tested by the community
vensires’s picture

vensires
Location Greece
commented
Status: Reviewed & tested by the community » Needs work

Patch plays fine with the main module but not with the plupload_test module resulting in the following complaint:

 The 'core_version_requirement' constraint (9.0.2) requires the 'core' key not be set in modules/contrib/plupload/plupload_test/plupload_test.info.yml
suresh prabhu parkala’s picture

suresh prabhu parkala commented
Assigned: Unassigned » suresh prabhu parkala
suresh prabhu parkala’s picture

suresh prabhu parkala commented
Status: Needs work » Needs review
StatusFileSize
new 3130920-6.patch718 bytes
new interdiff_2-6.txt314 bytes

Please review!

suresh prabhu parkala’s picture

suresh prabhu parkala commented
Assigned: suresh prabhu parkala » Unassigned
vensires’s picture

vensires
Location Greece
commented
Status: Needs review » Reviewed & tested by the community

Thanks for the help @suresh-prabhu-parkala!
Setting as RTBC.

douggreen’s picture

douggreen commented
Status: Reviewed & tested by the community » Needs work

This cannot be merged before #3093844: Deprecated Code Report. Please test that issue and mark it RTBC first :)

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented

As #3093844: Deprecated Code Report was fixed and it includes the core_requirement_key, I will close this issue.
Thanks!

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Status: Needs work » Fixed
paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Status: Fixed » Closed (fixed)