Please use short array syntax in code. Please check.

CommentFileSizeAuthor
#2 twitterlogin_short_array_syntax-2897428-2.patch30.51 KBjayesh_makwana
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jayesh_makwana created an issue. See original summary.

jayesh_makwana’s picture

jayesh_makwana CreditAttribution: jayesh_makwana at cmsMinds commented
FileSize
twitterlogin_short_array_syntax-2897428-2.patch30.51 KB

We created a patch for solving short array syntax for this module. Please check and apply it.

suresh.senthatti’s picture

suresh.senthatti CreditAttribution: suresh.senthatti commented

@jayesh_makwana, Let me check and update

suresh.senthatti’s picture

suresh.senthatti CreditAttribution: suresh.senthatti commented

I have applied your patch file and moved to dev version. Could you please let me know how your identified these error, do you have any tool or have your find these issues manually.

suresh.senthatti’s picture

suresh.senthatti CreditAttribution: suresh.senthatti commented
Status: Active » Fixed
jayesh_makwana’s picture

jayesh_makwana CreditAttribution: jayesh_makwana at cmsMinds commented

Hello suresh.senthatti,

Please check this url: https://pareview.sh/

Thank you.

Status: Fixed » Closed (fixed)

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