From 296078b4221e33f66e050accd931f59870d49400 Mon Sep 17 00:00:00 2001 From: Scott Taylor Date: Tue, 10 Feb 2015 22:53:34 +0000 Subject: [PATCH] Use `Math.round()` instead of `Math.floor()` when determining page of attachments to query in `wp.media.model.Query.sync`. This was causing an infinite loop of AJAX requests - so.... good catch! Props Veritaserum. Fixes #31116. git-svn-id: https://develop.svn.wordpress.org/trunk@31406 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-includes/js/media/models.js | 2 +- src/wp-includes/js/media/models/query.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/wp-includes/js/media/models.js b/src/wp-includes/js/media/models.js index e475446a8f..6653ccba91 100644 --- a/src/wp-includes/js/media/models.js +++ b/src/wp-includes/js/media/models.js @@ -1230,7 +1230,7 @@ Query = Attachments.extend({ // Determine which page to query. if ( -1 !== args.posts_per_page ) { - args.paged = Math.floor( this.length / args.posts_per_page ) + 1; + args.paged = Math.round( this.length / args.posts_per_page ) + 1; } options.data.query = args; diff --git a/src/wp-includes/js/media/models/query.js b/src/wp-includes/js/media/models/query.js index 0ca84a788c..29d361a60c 100644 --- a/src/wp-includes/js/media/models/query.js +++ b/src/wp-includes/js/media/models/query.js @@ -138,7 +138,7 @@ Query = Attachments.extend({ // Determine which page to query. if ( -1 !== args.posts_per_page ) { - args.paged = Math.floor( this.length / args.posts_per_page ) + 1; + args.paged = Math.round( this.length / args.posts_per_page ) + 1; } options.data.query = args;