Problem/Motivation

While debugging this issue: #2509188: The directory public://pictures does not exist or is not writable., I noticed a slight inefficiency in the user_admin_settings() function. The variable_get('user_pictures', 0) code is called twice where it could only be called once.

Remaining tasks

Review patch.

CommentFileSizeAuthor
#1 drupal-7.x-2509214-remove_redundant_vget-1.patch1.51 KBdooug

Comments

dooug’s picture

dooug commented
Issue summary: View changes
StatusFileSize
new drupal-7.x-2509214-remove_redundant_vget-1.patch1.51 KB
dooug’s picture

dooug commented
Status: Active » Needs review

Status: Needs review » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.