Ok fixes 1 #192

Merged
merged 6 commits into from Feb 4, 2014

Conversation

Projects
None yet
2 participants
@mbezoyan

mbezoyan commented Feb 3, 2014

No description provided.

@@ -99,7 +99,7 @@
xhr = document.createElement('div');
xhr.innerHTML = '<form target="'+ uid +'" action="'+ url +'" method="POST" enctype="multipart/form-data" style="position: absolute; top: -1000px; overflow: hidden; width: 1px; height: 1px;">'
+ '<iframe name="'+ uid +'" src="javascript:false;"></iframe>'
- + (jsonp && (options.url.indexOf('=?') == -1) ? '<input value="'+ uid +'" name="'+jsonp+'" type="hidden"/>' : '')
+ + (jsonp && (options.url.indexOf('=?') < 0) ? '<input value="'+ uid +'" name="'+jsonp+'" type="hidden"/>' : '')

This comment has been minimized.

Show comment Hide comment
@RubaXa

RubaXa Feb 3, 2014

Collaborator

Михаил, ну я же просил, не нужно делать изменений, которые ничего не меняют. Это только засоряет историю.

@RubaXa

RubaXa Feb 3, 2014

Collaborator

Михаил, ну я же просил, не нужно делать изменений, которые ничего не меняют. Это только засоряет историю.

@@ -121,7 +121,7 @@
var
onPostMessage = function (evt){
- if( url.indexOf(evt.origin) != -1 ){
+ if( ~url.indexOf(evt.origin) ){

This comment has been minimized.

Show comment Hide comment
@RubaXa

RubaXa Feb 3, 2014

Collaborator

Опять изменение, которое ничего не меняет, а даже наоборот, понижает читаемость кода.

@RubaXa

RubaXa Feb 3, 2014

Collaborator

Опять изменение, которое ничего не меняет, а даже наоборот, понижает читаемость кода.

This comment has been minimized.

Show comment Hide comment
@mbezoyan

mbezoyan Feb 3, 2014

Переделано по образу и подобию соседнего кода

@mbezoyan

mbezoyan Feb 3, 2014

Переделано по образу и подобию соседнего кода

This comment has been minimized.

Show comment Hide comment
@RubaXa

RubaXa Feb 3, 2014

Collaborator

Да, ты прав, есть такое место, но если ты приводил к ежиному виду, то почему тут сделал по другому?

@RubaXa

RubaXa Feb 3, 2014

Collaborator

Да, ты прав, есть такое место, но если ты приводил к ежиному виду, то почему тут сделал по другому?

lib/FileAPI.core.js
@@ -107,11 +107,11 @@
_elEvents[uid] = {};
}
+ var isFileReader = FileReader !=null && el instanceof FileReader;

This comment has been minimized.

Show comment Hide comment
@RubaXa

RubaXa Feb 3, 2014

Collaborator

Правильно будет так:
var isFileReader = (FileReader && el) && (el instanceof FileReader);

т.к.
FileReader — может быть определен, а el — быть DOMNull объектом (IE), что приведет к ошибке.

@RubaXa

RubaXa Feb 3, 2014

Collaborator

Правильно будет так:
var isFileReader = (FileReader && el) && (el instanceof FileReader);

т.к.
FileReader — может быть определен, а el — быть DOMNull объектом (IE), что приведет к ошибке.

This comment has been minimized.

Show comment Hide comment
@mbezoyan

mbezoyan Feb 3, 2014

Разве instanceof в этом случае вернет true?

@mbezoyan

mbezoyan Feb 3, 2014

Разве instanceof в этом случае вернет true?

This comment has been minimized.

Show comment Hide comment
@RubaXa

RubaXa Feb 3, 2014

Collaborator

Если el будет DOMNull, то IE кидает ошибку, а FileReader может быть определен и не быть FileReader'ом.

@RubaXa

RubaXa Feb 3, 2014

Collaborator

Если el будет DOMNull, то IE кидает ошибку, а FileReader может быть определен и не быть FileReader'ом.

lib/FileAPI.core.js
@@ -134,8 +134,9 @@
if( el ){
var uid = api.uid(el), events = _elEvents[uid] || {};
+ var isFileReader = FileReader != null && el instanceof FileReader;

This comment has been minimized.

Show comment Hide comment
dist/FileAPI.js
@@ -3894,7 +3859,7 @@
return this.parent.apply(this, arguments);
}
else {
- api.log('FlashAPI.XHR._send: '+ flashId +' -> '+ fileId, files);
+ api.log('FlashAPI.XHR._send: '+ flashId +' -> '+ fileId, JSON.stringify(files));

This comment has been minimized.

Show comment Hide comment
@RubaXa

RubaXa Feb 3, 2014

Collaborator

JSON — есть только с IE8, ножно сделать метод:
api.stringify(obj){
return window.JSON && JSON.stringify(obj) : null;
}

@RubaXa

RubaXa Feb 3, 2014

Collaborator

JSON — есть только с IE8, ножно сделать метод:
api.stringify(obj){
return window.JSON && JSON.stringify(obj) : null;
}

@RubaXa

This comment has been minimized.

Show comment Hide comment
@RubaXa

RubaXa Feb 3, 2014

Collaborator

Посмотрел, по сути есть одна серьезная проблема — #192 (diff)

И минорная — #192 (diff)

P.S. https://travis-ci.org/mailru/FileAPI/builds/18119171 — тесты «упали»

Collaborator

RubaXa commented Feb 3, 2014

Посмотрел, по сути есть одна серьезная проблема — #192 (diff)

И минорная — #192 (diff)

P.S. https://travis-ci.org/mailru/FileAPI/builds/18119171 — тесты «упали»

Mikhail Bezoyan added some commits Feb 3, 2014

RubaXa added a commit that referenced this pull request Feb 4, 2014

@RubaXa RubaXa merged commit c353e36 into mailru:master Feb 4, 2014

1 check passed

default The Travis CI build passed
Details

RubaXa added a commit that referenced this pull request Feb 5, 2014

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