-
-
Notifications
You must be signed in to change notification settings - Fork 50.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Lodash can clone FileProxy #29718
Conversation
Size Change: +787 B (0%) Total Size: 838 kB
ℹ️ View Unchanged
|
蛋疼,还有这么一个 issue 需要是 File: 死循环了,如果是 File |
所以说现在只有onchange里返回的file和fileList是proxy对象吗??beforeupload还是file对象?确定了吗?我这次从4.5.0升上来遇到了原先onchange事件里file对象用不了的情况了 |
你说的 file 对象用不了是什么问题? |
Codecov Report
@@ Coverage Diff @@
## master #29718 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 394 394
Lines 7522 7533 +11
Branches 2114 2115 +1
=========================================
+ Hits 7522 7533 +11
Continue to review full report at Codecov.
|
你的 codesandbox 里 |
我不知道为什么在codesandbox里切换成4.5.0版本取到的info.file是一个object对象,但是我本地项目里的确是一个File对象,这个我们整个项目的人都验证过了,一直都是这样用的,这次升级导致上传文件的代码都挂了 |
是不是有 |
是的,我们beforeUpload都是直接return false,这有什么影响吗? |
哎,历史债务。 |
Chrome 里 FileReader 不支持读取 Proxy(File) 感觉走进了死胡同。要不然搞全模拟成老的行为,Proxy 方案看起来也不太可行。 cc @afc163 |
那请问有好的解决方法吗?... |
见上,和 @afc163 讨论一下 |
这个不一致是不是也被提过 bug。 |
看上去只能先保留 bug 作为 feature,留 5.0 去处理了。 |
emmmm那想问一下4.x后续的版本还会是用proxy对象吗?还是说会改为原先的模式? |
改成原先的模式,我会兼容一下给 File 添加一个 |
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
resolve #29646
💡 Background and solution
📝 Changelog
onChange
paramsfile
can notcloneDeep
by lodash.onChange
参数file
不能被 lodashcloneDeep
的问题。☑️ Self Check before Merge