Back To Top module needs to be converted to use short array syntax as per new coding standard.

CommentFileSizeAuthor
#3 back to top patch applied.PNG6.76 KBpankajsachdeva
#2 short_array_syntax-2863663-2.patch5.65 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-2863663-2.patch5.65 KB
pankajsachdeva’s picture

pankajsachdeva CreditAttribution: pankajsachdeva as a volunteer and at ]init[ AG commented
FileSize
back to top patch applied.PNG6.76 KB

Hi Munavijayalakshmi,

I have manually reviewed your patch and also applied this patch.

pankajsachdeva’s picture

pankajsachdeva CreditAttribution: pankajsachdeva as a volunteer and at ]init[ AG commented
Status: Needs review » Reviewed & tested by the community
yogeshmpawar’s picture

yogeshmpawar
He/Him/His
Primary language English
Location 🇮🇳 Pune(MH)
CreditAttribution: yogeshmpawar at QED42 commented

Looks good to me, +1 for RTBC.

acke’s picture

acke CreditAttribution: acke at Happiness commented
Status: Reviewed & tested by the community » Fixed

Included in 8.x-1.0. Thanks for your help!

Status: Fixed » Closed (fixed)

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