Callback функция progress #90

Closed
VitaliySunny opened this Issue May 20, 2013 · 13 comments

2 participants

@VitaliySunny

При отправке нескольких файлов функция progress, а точнее параметр loaded который создается и отправляется в эту функцию, странно работает.

Код:

...
progress: function(e){
    var percent = Math.abs(Math.round(e.loaded/e.total*100));
    console.log("Loaded: " + e.loaded, "Total:" + e.total, " / " + percent + "%");
}
...

Консоль:

Loaded: 163439 Total:4951062  / 3%
Loaded: 330648 Total:4951062  / 7%
Loaded: 360021 Total:4951062  / 7%
Loaded: 382115 Total:4951062  / 8%
Loaded: 595178 Total:4951062  / 12%
Loaded: 878663 Total:4951062  / 18%
Loaded: 775500 Total:4951062  / 16%
Loaded: 533302 Total:4951062  / 11%
Loaded: 498510 Total:4951062  / 10%
Loaded: 555707 Total:4951062  / 11%
Loaded: 582177 Total:4951062  / 12%
Loaded: 1028573 Total:4951062  / 21%
Loaded: 976866 Total:4951062  / 20%
Loaded: 442591 Total:4951062  / 9%
Loaded: 487865 Total:4951062  / 10%
Loaded: 580525 Total:4951062  / 12%
Loaded: 511513 Total:4951062  / 10% 
@RubaXa

Да, есть такая проблема, фикс есть, просто ещё не перенесен в основную ветку, обязательно сделаю это сегодня.

@RubaXa RubaXa added a commit that referenced this issue May 21, 2013
@RubaXa RubaXa Resolve #90 (prgoress event) d0b5573
@RubaXa RubaXa added a commit that referenced this issue May 21, 2013
@RubaXa RubaXa Resolve #90 (progress event) fcd5331
@RubaXa

Поправил в dev ветке.

@VitaliySunny
Uncaught TypeError: Object #<File> has no method 'cloneNode' 
@RubaXa

А что за браузер? (сам проверял в IE9, Chrome, FF)

@VitaliySunny

Google Chrome, Версия 26.0.1410.64 m
Windows 7 64bit
jQuery 1.8.3

@RubaXa
@RubaXa

А можете включить debug режим и скопировать содержимое консоли?

var FileAPI = { debug: true }
<script src="FileAPI.min.js"></script>
@RubaXa

Спасибо, уже разобрался, с проверками переборщил.

@RubaXa

Обновил dev ветку.

@RubaXa

Кстати, как видно из консоли, вы пропустил класс js-fileapi-wrapper, он нужен для IE < 10 и Opera < 12.

@VitaliySunny

Понял. Большое спасибо, все работает.

@RubaXa

Спасибо вам.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment