bugfix: rewind_posts when posts array index != 0

[diogovf]

When the posts array doesn't start with the index 0, it generates an error due to hard code 0.

Before:

<?php
public function rewind_posts() {
    $this->current_post = -1;
    if ( $this->post_count > 0 ) {
        $this->post = $this->posts[0];
    }
}

After:

<?php
public function rewind_posts() {
    $this->current_post = -1;
    if ( $this->post_count > 0 ) {
        $this->post = $this->posts[array_key_first($this->posts)];
    }
}

Hi team,
The rewind_posts crashes when we have an object with posts that don't start at position 0. I use an api with more than 5 thousand objects and sometimes the posts start with an index greater than 0.

Trac ticket: 58147

[Presskopp]

This won't work as is with PHP < v7.3.0, see ​https://www.php.net/manual/en/function.array-key-first.php

Until now WordPress still claims compatibility to PHP v5.6.20, see ​https://wordpress.org/about/requirements/

[petitphp]

Hi, welcome to the trac, and thanks for the ticket and the associated PR.

As far as I can tell, the rewind_posts method hasn't changed since it was introduced in WP 1.5. Can you share more info on how you end up with a posts array that doesn't start at index 0 for your query ?

As noted by @Presskopp WordPress is still supporting PHP 5.6, you could use ​reset to achieve the same result while keeping this compatibility.

[diogovf]

Hi,
I've forgot the support for the old php versions, yeah I'll update this with reset, thanks @Presskopp and @petitphp.

I'vee a wordpress linked to an external API and when I receive posts sorted by the API. The final array may start with a larger ID like 1572 or 3518 because the WP_post object is built and sorted by the external API. I noticed the problem when I joined the project and the team overcame the problem by writing a wrapper to work around the error.