Promise.all(promises).then...で更新したいが処理が走らない

Question

###前提・実現したいこと
html javascript phpで画像や動画を削除する機能を作っています。
チェックボックスにチェックをつける→削除ボタンを押す→チェックされた画像のソース（相対パス）をajaxでphpに送る→unlink()で削除
全てajaxでphpにソース（相対パス）を送り終えたら
Promise.allで更新をするというのをしたいのですが

    Promise.all(promises).then(function (results) {
        console.log('通りましたよ');
        window.location.reload(true);
    });

この処理が走りません、console.log()で試したら何も表示されませんでした

でもブラウザのF12でちゃんとdelete.phpにpostで適正な値が渡されていました

###該当しているソースコード

$(function(){
    //フォームデータを一時保存する配列
    var formDataArray = []
    //クリックした時にデータ送信
    $('#delete').on('click',function(){
        //チェックボックス
        var im = document.forms.fm.Checkbox;
        var cnt = 0;
        // Array オブジェクトを作成する
        var img_id = new Array();
        //チェックボックスの数だけ繰り返す
        for(var i = 0; i < im.length; i++){
            //チェックボックスにチェックがついてるか
            console.log(im[i]);
            console.log(im[i].checked);
            if(im[i].checked){
                //チェックボックスのvalue（imgのidが入っている）をimg_idにプッシュ
                console.log(im[i].value);
                img_id.push(im[i].value);
                //チェックついてる数を数える
                cnt++;
            }
        }
        //チェックついてる数だけ繰り返す
        console.log(cnt);
        for(var i = 0; i < cnt; i++){
            var formData = new FormData();
            console.log(img_id[i]);
            var id = img_id[i];
            console.log(document.getElementById(img_id[i]));
            //チェックのついている画像のソースをformDataにappend
            formData.append('path',document.getElementById(img_id[i]).getAttribute("src"));
            //formDataをArrayにpush
            formDataArray.push(formData);
        }
        //入ってる配列を繰り返す
        var promises = [];
        formDataArray.forEach(function(formData){
            //画像のsrcが入っているか確認
            console.log(formData.get("path"));
            //ajaxで送る
            //deleteFiles(resolve,formData);
            promises.push(deleteFilesPromise(formData));
        });
        Promise.all(promises).then(function (results) {
            console.log('aaaa');
            window.location.reload(true);
        });    
    });
    function deleteFilesPromise(formData) {
    return new Promise(function (resolve, reject) {
    deleteFiles(formData, function (err, result) {
        if (err != null) {
            reject(err);
            return;
        }
        resolve(result);
    });
    });
    }
    var deleteFiles = function(formData,aaa){
        //ajaxで送信
        $.ajax({
            //POSTで送信
            type: 'POST',
            //ajaxが適切なcontentTypeに自動変換するのを防ぐ
            contentType: false,
            //データを文字列に自動変換するのを防ぐ
            processData: false,
            //送信先
            url: './delete.php',
            //送るデータ
            data: formData,
            //通信成功時に呼ばれる
            //data 接続先PHPファイルが出力したデータ
            success: function(data,dataType) {
                console.log("成功" +data);
                //res(formData);
            },
            //通信失敗時に呼ばれる
            //XMLHttpRequest.status HTTPステータス
            //textStatus            timeout、error、parsererror等の文字列
            //errorThrown           例外情報
            error: function(XMLHttpRequest, textStatus, errorThrown){
                console.log(XMLHttpRequest + " : " + textStatus + " : " + errorThrown);
            }
        });  
    }
});

Answer 1

ブラウザのF12をおしてdelete.phpにpostで適正な値が渡されているか確認してください

テスト

テストサンプルです。
iは連番で渡しているのでdelete.phpで間違っても削除を走らせないで下さい
以下で処理すれば、10個のpostデータをおくってすべてがもどってきたら
thenが走りますよね？

<script src="https://www.promisejs.org/polyfills/promise-6.1.0.min.js"></script>
<script>
var prm=[];
console.log("start");
for(var i=0;i<10;i++){
  (function(i){
    prm[i]= new Promise(function(resolver){
      var fd=new FormData();
      fd.append('del_id',i);
      var xhr = new XMLHttpRequest();
      xhr.open('POST','delete.php');
      xhr.addEventListener("load" ,function() {
        console.log(this.response)
        resolver(this);
      });
      xhr.send(fd);
    });
  })(i);
}
Promise.all( prm ).then(function(){
  console.log("end");
});

</script>

Comments

[rms398]

旧：適正な値が表示されました

テストサンプルはちゃんと走りました

[yambejp]

そうなるとpromiseがresolverをきちんと返してないのかもしれませんね
var deleteFiles = function(formData,aaa){
の第二引数のaaaは参照されていないようですが
successの中から利用してresolverを返さないとまずいんじゃないですか？

[rms398]

aaa にあたる引数をreturn new Promise(function (resolve, reject)のresolveにしたらちゃんと動きました！