[lancewillett]

Bug fix

JavaScript change in getUserFor(), plus an updated QUnit test. Follow-up to #274.

The logged-out comment form template, inc/tpl/logged-out-create-comment.php, appends &s= to the avatar URL that getUserFor() builds. Since #274, a site with avatar_default set to gravatar_default gets an avatar URL with no ?, so the size lands in the URL path:

• Before: https://gravatar.com/avatar/&s=32. Gravatar returns its 80px default image, which the <img> scales down.
• After: https://gravatar.com/avatar/?&s=32. Gravatar returns the 32px image.

getUserFor() now always adds the ? and only leaves out d=. Other defaults are unchanged.

Rationale

Code may append parameters to this URL, and with no ? the first & lands in the path. Keeping the ? makes appended parameters work the same for every default.

Testing this change

• npx grunt qunit passes. The getUserFor test now checks the URL the template builds, and it fails on master.
• npx grunt lint passes.
• In a wp-env site with p2-breathe, o2 enabled and avatar_default set to gravatar_default, the logged-out comment form renders https://gravatar.com/avatar/?&s=32. On master it renders https://gravatar.com/avatar/&s=32.
• With avatar_default set to identicon, the form renders https://gravatar.com/avatar/?d=identicon&s=32.

Definition of done

• The logged-out form keeps s in the query string for gravatar_default
• Other defaults keep d
• The updated test fails without the fix
• CI green

🤖 Generated with Claude Code