sergiotarxz
/
Wordpress
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

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
This commit is contained in:
Scott Taylor 2015-02-10 22:53:34 +00:00
parent 267f68c0d6
commit 296078b422
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/wp-includes/js/media/models.js
View File

@ -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;

2
src/wp-includes/js/media/models/query.js
View File

@ -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;