Hi,

Recently noticed that user_pictures variable has wrong default value in UserController:attachLoad(). By default this variable is set to 0, but this method uses 1 instead.

Patch that fixes that bug attached.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch, drupal-wrong-default-value-for-user-pictures-variable.patch, failed testing.

Spleshka’s picture

Status: Needs work » Needs review
droplet’s picture

Status: Needs review » Reviewed & tested by the community

D8 no this issue. Above patch seems correct.

Spleshka’s picture

Any updates here? Patch seems to be obvious.

Spleshka’s picture

Still no progress here?

David_Rothstein’s picture

Status: Reviewed & tested by the community » Fixed
Spleshka’s picture

Great, thanks!

Status: Fixed » Closed (fixed)

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