Changeset 7152 for trunk/src/includes/common/functions.php

Timestamp:

11/05/2020 06:56:24 PM (4 years ago)

Author:

johnjamesjacoby

Message:

Emails: correct logic in bbp_get_email_addresses_from_user_ids().

This commit makes sure that the query offset (for looping through user IDs) does not unintentionally skip over any IDs in the allowed ranges.

In trunk, for 2.7. Fixes #3405.

Props muratgu.

File:

• trunk/src/includes/common/functions.php (modified) (1 diff)

trunk/src/includes/common/functions.php

@@ -1385 +1385 @@
 
             // Initial loop has no offset
-            $offset = ( 1 === $loop )
-                ? 0
-                : $limit * $loop;
+            $offset = $limit * ( $loop - 1 );
 
             // Calculate user IDs to include