sticky_posts somehow ends up with duplicate IDs and unstickying only removes one at a time

[archon810]

Hi all,

Somehow we ran into this bug on production where the sticky_posts array ended up with duplicate IDs that were stickied, and unstickying only removed the same ID one at a time instead of all of them at once, so unstickying kept the post stickied until it was unstickied enough times to clear the array.

For example, this is how I found sticky_posts:

a:3:{i:1;i:744381;i:2;i:725382;i:3;i:725382;}

Note that 725382 is duplicated. Unstickying once did this:

a:2:{i:0;i:744381;i:1;i:725382;}

How we got to the duplicate ID state, I cannot say (perhaps that's a separate bug), but solution here is to not exit the loop when the ID is reached and removed and instead keep going till the end.

[SergeyBiryukov]

Thanks for the ticket! It seems like we should run array_unique() on the sticky_posts option at some point here.

[archon810]

This just happened again. The fix should be really simple, can we give this one a kick please?

This change tries to prevent duplicate post IDs in the sticky_posts option.

Trac ticket: https://core.trac.wordpress.org/ticket/52007

Hi @peterwilsoncc

Thank you for your review. I'll take a look.

@peterwilsoncc I've split up the assertions a little. :grimacing:

These tests look way more focused, thank you.

It might be good to use a data provider but I can push those as they are a little fiddly.

Code changes and tests look good to me.

@Rahmon I've pushed some minor changes to the tests in ​https://github.com/WordPress/wordpress-develop/pull/981/commits/64c4b7fa599a0da29ccee3187d88c6dc194be15f

• Used a data provider where possible.
• Added tests to make sure the database isn't updated unnecessarily
• Added docblocks too

@Rahmon Oh, meant to ask: what is your wordpress.org username so i can give props on the ticket?

[peterwilsoncc]

Reckon this is good to go.

@peterwilsoncc my username is rahmohn (​https://profiles.wordpress.org/rahmohn/).

[peterwilsoncc]

In 50380:

Posts/Post Types: Prevent duplicates in sticky posts option.

In unstick_post() if a post ID is duplicated in the sticky_posts option remove all instances.

In both stick_post() and unstick_post() check for duplicate IDs already stored in the sticky_post option and remove them if the option is updated.

Props rahmohn, archon810.
Fixes #52007.

Committed in https://core.trac.wordpress.org/changeset/50380

[SergeyBiryukov]

In 50384:

Tests: Rename stick_post() and unstick_post() tests for a bit more clarity.

Add missing DocBlock for test_unstick_post_updates_option().

Follow-up to [50380].

See #52007.