While working on #2637336: Replace i.e. and e.g. with English words in /core/includes and /core/misc directory, we discovered that $lockBackend was used as a local variable instead of $lock_backend as defined in the coding standards. The purpose of this issue is to correct this inappropriately named local variable.

CommentFileSizeAuthor
#2 issue-2643570-2-8.patch6.03 KBgnuget
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Lars Toomre created an issue. See original summary.

gnuget’s picture

gnuget
Primary language Spanish
CreditAttribution: gnuget as a volunteer commented
Assigned: Lars Toomre » Unassigned
Status: Active » Needs review
FileSize
issue-2643570-2-8.patch6.03 KB

Patch attached.

Lars Toomre’s picture

Lars Toomre CreditAttribution: Lars Toomre as a volunteer commented
Status: Needs review » Reviewed & tested by the community

Thanks very much for the patch @gnuget. I have reviewed your work and can confirm that you have made good changes in all three of the appropriate files. This is RTBC.

  • alexpott committed 027c0d8 on 8.1.x 
    Issue #2643570 by gnuget: Correct $lockBackend local variable use

  • alexpott committed 937e29d on 
    Issue #2643570 by gnuget: Correct $lockBackend local variable use

(...
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Reviewed & tested by the community » Fixed

Committed 937e29d and pushed to 8.0.x and 8.1.x. Thanks!

Discussed with @catch we both feel this change is eligible for 8.0.x

Status: Fixed » Closed (fixed)

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