PHPとjqueryのAjaxで画面を更新しないと反映されない

Question

実装したAjaxで画面を更新しないと変更が反映されません。
検証で内容を確認したところ、
VM2129:1 Uncaught SyntaxError: Unexpected token < in JSON at position 0
at JSON.parse (<anonymous>)
at main.js:1059
と表紙されており、main.jsを確認したところ、

function addSetting(streamID, stickID) {
    var single = $(".single-video");
    var maxContents = 0;
  　maxContents = 3;
   　    if (stickID == 'pc') {
        if (single.length >= maxContents) {
            alert('登録は' + maxContents + '個までです');
            return false;
        }
    }
    maxContents = 20;
    if (single.length >= maxContents) {
        alert('登録は' + maxContents + '個までです');
        return false;
    }
	var condObj = new Object();
	condObj.func = 'add_setting';
	condObj.stream_id = streamID;
	condObj.stick_id = stickID;
	$.ajax({
		  context: this,
		  type: "POST",
		  url: "/api.php",
		  data: condObj
		})
		.done( function( data, textStatus, jqXHR ) {
          var jsonData = JSON.parse(data||"null");
          if (jsonData.result == 'fail') {
          	alert(jsonData.message);
          } else {
        	  var tag = '';
        	  var item = JSON.parse(jsonData.data.PlayData||"null");
        	  var thumbUrl = jsonData.data.ImageFileName;
        	  if( item.ThumbPath ) thumbUrl = item.ThumbPath;
              tag += '<div class="single-video" id="single-play-video' + jsonData.data.SettingNo + '">';
              tag += '<h5>' + jsonData.data.Title + ' <a class="delete" href="javascript:void(0);" onclick="deleteSetting(\'' + jsonData.data.SettingNo + '\',\'' + ID + '\');">削除</a></h5>';
              tag += '<a href="javascript:void(0);" class="forward-btn" onclick="forwardCategory(' + jsonData.data.SettingNo + ',\'' + ID + '\');"><img src="img/forward.jpg" alt=""></a>';
              tag += '<a href="javascript:void(0);" data-toggle="modal" data-target="#video-modal" onclick="showCategoryDetail(\'' + sID + '\',\'' + ID + '\',1);">' + '<img class="img-responsive" src="' + thumbUrl + '" alt="" onerror="this.src=\'img/video-pic.jpg\';">' + '</a>';
        	  tag += '<a href="javascript:void(0);" data-toggle="modal" data-target="#video-modal" onclick="showCategoryDetail(\'' + sID + '\',\'' + ID + '\',1);">' + item.Title + '</a>';
              tag += '<a href="javascript:void(0);" class="backward-btn"><img src="img/backward.jpg" alt="" onclick="backCategory(' + jsonData.data.SettingNo + ',\''+D+'\');"></a>';
              tag += '</div>';
        	  $("#play-video").append(tag);
        	  adjstSettingListButton();
        	  $("#video-list").show();
        	  $("#dummy-video-list").hide();
          }
		} )
		.fail( function( jqXHR, textStatus, errorThrown ) {
		})
		.always(function( jqXHR, textStatus, jqXHR ) {
		});
	return false;
}

の中の、

1059 var jsonData = JSON.parse(data||"null");

の部分でした。さらに検証画面でdataの中身を確認したところ
data = Non-static method Crypt_Blowfish::factory() should not be called statically in
と書かれていました。
下記が警告部分です。

$blowfish = Crypt_Blowfish::factory('cbc', $keyword, $iv);

バージョンの変化に伴って出た警告なので,data自体は問題ないと考えておりますが、
現在、画面自体を更新しないと変更が反映されない為ご意見を頂ければと思います。
よろしくお願い致します。

Answer 1

JavaScriptの返却値はJSONを期待している→PHPでエラーが出ている→JSON形式として正しくない
ということでしょう。

Non-static method Crypt_Blowfish::factory() should not be called statically in

Google翻訳まま：非静的メソッドCrypt_Blowfish :: factory（）は静的に呼び出すべきではありません

エラーの通りです。
Crypt_Blowfishクラスのfactoryメソッドはstaticではないという意味です。
PHP側のコードの全容が見えませんが、インスタンス生成すれば良いのでは。
クラスの基礎

Comments

[maki0128]

ご回答有難うございます。

function getEncryptValue($value)
{
if (!$value)
{
return null;
}
$keyword = 'キーワード';
$iv = left(getGuid(), 8);
$blowfish = Crypt_Blowfish::factory('cbc', $keyword, $iv);
$binary = $blowfish->encrypt($value);
$base64 = base64_encode($binary);
return $iv . $base64;
}

PHPの文は上記になります。

[m.ts10806]

回答読まれたのでしょうか・・・。

[maki0128]

ご丁寧に回答頂いたのに申し訳ありません。
js側では無く、phpの方を修正しなければいけないのは理解できているのですが、
修正方法が理解できておらず、phpの文章をclassで囲うということでしょうか？

[m.ts10806]

おそらく分かっていないものと思い、PHPマニュアルのリンクを添えています。クラスの基礎からおさえないと今回以外でも困る場面が増えますよ。

[maki0128]

classで囲い、unction getEncryptValueを呼び出していいた箇所でインスタンスを作成したら、解決しました。
正直解決はしましたが、理解仕切れていないので、引き続きphpマニュアルを読みたいと思います。
ご指導ありがとうございます。

[m.ts10806]

>classで囲い、
がどこから出てきたのかどう対応したのか分からないのですけど・・

[maki0128]

function getEncryptValue($value){
if (!$value){
return null;
}
$keyword = 'キーワード';
$iv = left(getGuid(), 8);
$blowfish = Crypt_Blowfish::factory('cbc', $keyword, $iv);
$binary = $blowfish->encrypt($value);
$base64 = base64_encode($binary);
return $iv . $base64;
}

を

class getEncryptValue{
function _construct($value){
if (!$value){
return null;
}
$keyword = '';
$iv = left(getGuid(), 8);
$blowfish = Crypt_Blowfish::factory('cbc', $keyword, $iv);
$binary = $blowfish->encrypt($value);
$base64 = base64_encode($binary);
return $iv . $base64;
}
}

に変更し
$ret[$key] = getEncryptValue($value);
を
$ret[$key] = new getEncryptValue($value);
に
書き換えました。

[m.ts10806]

Crypt_Blowfishの方じゃなかったんでしょうか(そのつもりで回答してるんですが)

[maki0128]

なるほど！再度試してみます。

Answer 2

エラーになっているようなら.failのコールバックで

console.log(textStatus)

を参照してください

Comments

[yambejp]

doneに流れているようなら、受け取ったデータが正しいjsonデータになっているか確認してください
ajax内のdataTypeをjsonにしてもいいかもしれません