Code style fixes, isset() is faster then array_key_exists()

masquerade_user_update() should use multi-insert query

Added @TODO to session update hunk because we should update session only when session_id() been changed.

Commited patch

CommentFileSizeAuthor
masquerade-d7-cleanup.patch3.43 KBandypost
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

andypost’s picture

Status: Needs work » Needs review

Any reviews?

andypost’s picture

Status: Needs review » Fixed

This already fixed

Status: Fixed » Closed (fixed)

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

  • Commit bb61b7e on master, 8.x-2.x, 8.x-2.x-admin-menu, 8.x-1.x-1836516 by andypost:
    Issue #1103588 by andypost: Code performance cleanup
    
    

  • Commit bb61b7e on master, 8.x-2.x, 8.x-2.x-admin-menu, 8.x-1.x-1836516 by andypost:
    Issue #1103588 by andypost: Code performance cleanup