alex/yomichan
1
Fork 0
Code Releases Activity

Fix not awaiting the AudioSystem.arrayBufferDigest promise (#524)

Browse Source
This commit is contained in:
toasted-nutbread 2020-05-09 12:28:48 -04:00 committed by GitHub
parent 69c783f861
commit 9900291b07
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
ext/mixed/js/audio-system.js
View File

@ -173,9 +173,9 @@ class AudioSystem {
return new Promise((resolve, reject) => {
const xhr = new XMLHttpRequest();
xhr.responseType = 'arraybuffer';
xhr.addEventListener('load', () => {
xhr.addEventListener('load', async () => {
const arrayBuffer = xhr.response;
if (!this._isAudioBinaryValid(arrayBuffer)) {
if (!await this._isAudioBinaryValid(arrayBuffer)) {
reject(new Error('Could not retrieve audio'));
} else {
resolve(arrayBuffer);
@ -195,8 +195,8 @@ class AudioSystem {
);
}
_isAudioBinaryValid(arrayBuffer) {
const digest = AudioSystem.arrayBufferDigest(arrayBuffer);
async _isAudioBinaryValid(arrayBuffer) {
const digest = await AudioSystem.arrayBufferDigest(arrayBuffer);
switch (digest) {
case 'ae6398b5a27bc8c0a771df6c907ade794be15518174773c58c7c7ddd17098906': // jpod101 invalid audio
return false;