YouTube Field needs to be converted to use short array syntax as per new coding standard.

CommentFileSizeAuthor
#2 short_array_syntax-2862774-2.patch22.16 KBMunavijayalakshmi
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Munavijayalakshmi created an issue. See original summary.

Munavijayalakshmi’s picture

Munavijayalakshmi CreditAttribution: Munavijayalakshmi at Valuebound commented
Assigned: Munavijayalakshmi » Unassigned
Status: Active » Needs review
FileSize
short_array_syntax-2862774-2.patch22.16 KB
dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda for OpenSense Labs commented
Status: Needs review » Reviewed & tested by the community

Applied patch manually and it works well. It's OK for me.

imyaro’s picture

imyaro CreditAttribution: imyaro at DrupalJedi commented
Assigned: Unassigned » imyaro

imyaro’s picture

imyaro CreditAttribution: imyaro at DrupalJedi commented

Pushed into the DEV, thanks.

imyaro’s picture

imyaro CreditAttribution: imyaro at DrupalJedi commented
Status: Reviewed & tested by the community » Fixed
imyaro’s picture

imyaro CreditAttribution: imyaro at DrupalJedi commented
Parent issue: » #2919172: Module code cleanup

  • zvse committed e4e1f1d on 8.x-1.x 
    Issue #2862774 by Munavijayalakshmi, zvse: Convert module to use short...
dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda commented

Status: Fixed » Closed (fixed)

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