We have to convert the module to short array as per new coding standard and for configure link we should provide the route name

CommentFileSizeAuthor
#3 prlp_array_search_results.png50.33 KBdhruveshdtripathi
#2 2881602-2.patch2.37 KBPavan B S
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Pavan B S created an issue. See original summary.

Pavan B S’s picture

Pavan B S CreditAttribution: Pavan B S at Valuebound commented
Status: Active » Needs review
FileSize
2881602-2.patch2.37 KB

Applying the patch, please review

dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda commented
Status: Needs review » Reviewed & tested by the community
FileSize
prlp_array_search_results.png50.33 KB

Patch applied cleanly. All the arrays converted to short array syntax. Checked it manually. As you can see in the attached png file, earlier there were 3 array declarations, after applying patch there are 0.

Good work!

Jitesh Doshi’s picture

Jitesh Doshi CreditAttribution: Jitesh Doshi commented
Status: Reviewed & tested by the community » Closed (won't fix)

This is not a real issue.

I looked at the reporter's profile. He has reported such trivial issues and submitted patches for them just to get commit authoring credit. This is issue spam.

Please stop.