WordPressでオリジナルアバターを作成したい

Question

###実現したいこと
オリジナルアバターを出力したいです。
下記について良い解決方法がありましたら教えて頂ければ幸いです。

###試したコード
こちらのコードについて、後述するエラーの解決方法が知りたいです。
ちなみmy_get_ImgPassは画像のフスパスを取得する関数で、正常に作動します。

function my_get_Avatar( $userId, $num=null ){
	global $wpdb;
	// ユーザーIDから、画像のURLを取得（フルパスです）
	$ImgPass = my_get_ImgPass('ImgPass' ,$userId);
	// 画像URLがあれば
	if ( ! empty($ImgPass) ){
		// 画像URLから、画像IDを取得
		$ImgId = $wpdb->get_col($wpdb->prepare("SELECT ID FROM $wpdb->posts WHERE guid='%s';", $ImgPass )); 
		// ※画像IDから、指定サイズ版を取得
		$ImgInfo = wp_get_attachment_image_src( $ImgId[0], 'img100' );	
		// 出力
		if( is_null($num) ){
			return '<img src="'.$ImgInfo[0].'" alt="">';
		}else{
			return '<img src="'.$ImgInfo[0].'" alt="">';
		} 
	}else{
		echo '画像なし';
	}
}

echo my_get_Avatar( 1, 100 );

エラー内容は次のものです。

Notice: Undefined offset: 0 in 
/export/user/c/zjp_95167c/live_77a5b6/var/wordpress/wp-content/themes/functions.php
on line 384

384行目はここです。

		// ※画像IDから、指定サイズ版を取得
		$ImgInfo = wp_get_attachment_image_src( $ImgId[0], 'img100' );

どう書けば解決できるか分かる方いませんか？

Comments

[退会済みユーザー]

「エラーになる」とは具体的にどういう状態でしょうか。表示されたメッセージなどを質問文に追記していただけますか？

[maimaime]

追記しました。すみませんでした。

Answer 1

現情報で考えられるのは、登録されていない画像URL情報を関数[my_get_ImgPass]の戻り値で返しているのではないかと思われます。

まず、SQLは間違っているように見えません。

その上で、関数[wp_get_attachment_image_src] の呼び出し時点でエラーになるということは、関数[my_get_ImgPass]の戻り値である画像URLの情報がワードプレスで利用しているDBの指定テーブル[wp_posts]に無いと思いますが、その点はチェックされていらっしゃいますか？

メソッド$wpdb->get_col()が データが無い場合 null を返すので 変数[ $ImgId]にはnullが代入されてしまいます。その結果nullの[0]を参照してしまっているのでエラーになっているのではないかと思います。

関数[my_get_ImgPass]の戻り値として取得した画像URL情報をDumpなど行い、MySQLで直接当該画像URL情報があるのかSQL叩いてみて取得できるかを調査なされてはいかがでしょうか。

Comments

[maimaime]

どうもありがとうございます。

画像データが空ならば、そもそも関数[wp_get_attachment_image_src] の呼び出しも起こらず、エラーにもならないのではないでしょうか？と言いますのは、

質問のコードには
if ( ! empty($ImgPass) ){}
という条件を書いており、これは
「$ImgPassが空じゃない場合だけ」
という意図のつもりです。

なので、もしこの条件に合致しない（つまり、画像URLの情報がワードプレスで利用しているDBの指定テーブル[wp_posts]に無い）のでしたら、上の条件の中は走らず、関数[wp_get_attachment_image_src] の呼び出しも起こらず、エラーにもならないのではないでしょうか？

[kanimaru]

まず、$wpdb->get_col()に関して、空配列を返すような仕様でした。失礼いたしました。

さて、回答いたしますが、やはり関数[my_get_ImgPass]が返すURL情報がワードプレスで利用しているDBのテーブル上に存在しないものと思います。存在しない為、メソッド[ $wpdb->get_col]が空配列を返し、次の行で空配列の[0]を参照し、追記されたエラーが出ています。

ワードプレスの準備関数ではない独自に作られた関数[my_get_ImgPass]の仕様は存じ上げません。故に関数[my_get_ImgPass]が何の画像URL情報を返しているか存じません。本当に関数[my_get_ImgPass]は、WPに登録されているユーザidに紐づく画像URL情報を返し、かつ、その画像URL情報はワードプレスDBのwp_postsに存在するのでしょうか？

ところで、指摘致しました $ImgPass に格納された画像URL情報がDB上にSQLを走らせて存在したことを確認してのご返答でしょうか。

[maimaime]

えっと、関数[my_get_ImgPass]が返すURL情報が存在しないのはその通りなんです。
ご指摘頂きました $ImgPass に格納された画像URL情報も存在しません。

で、どうして存在しないのに、

if ( ! empty($ImgPass) ){}

という「存在したら」の条件が走ってしまうのでしょうか？

存在しないんだから、エラーは表示されず、'画像なし'がechoされるのではないのでしょうか？？

[kanimaru]

追加情報をありがとうございます。

関数[my_get_ImgPass]がphpビルド関数emptyの仕様である以下URLのリンク先にある返り値 の項目の、いずれかに合致しない戻り値を返していると判断できます。関数[my_get_ImgPass]調査をなされるべきかと思います。

http://php.net/manual/ja/function.empty.php

Answer 2

$ImgInfo = wp_get_attachment_image_src( $ImgId[0], 'img100' );
Notice: Undefined offset: 0 in

このエラーの内容は配列のサイズを超えた要素にアクセスているというエラーです。
$ImgId配列に0番目の要素が無いということで、空っぽの配列の可能性が高いです。

$ImgId = $wpdb->get_col($wpdb->prepare("SELECT ID FROM $wpdb->posts WHERE guid='%s';", $ImgPass ));

こちらのSQLで想定されるデータが取得できていないのではないでしょうか。
想定されるguidが存在するか、$ImgPassが正しいかご確認されてみてはいかがでしょうか？

Comments

[maimaime]

ありがとうございます。

$ImgPassをvar_dumpすると、

string(0) ""

となります。

つまり、

if ( ! empty($ImgPass) ){

の条件に合致しないと思うのですが、

なぜこの条件の中のコードが走って、エラーになってしまうのでしょうか？

[tabuu]

my_get_ImgPass関数の戻り値はどのようになっていますか？

また、どうしても解決しないようであれば、以下のように条件を現実的な内容にするかですかね。
例えば$ImgPassの文字数が通常10文字以上になる場合
if ($ImgPass && strlen($ImgPass) >= 10)

あと、$ImgIdのチェックもしておいたほうが安全だと思います。
if (is_array($ImgId) && count($ImgId) > 0)

※ImgPassのif文の条件間違っていたので修正しました。

Answer 3

「Ideone.com」でコードを確認したところ
4行目に使われている関数 my_get_ImgPass() が未定義というエラーが見つかりました。

関数が登場する手前で function my_get_ImgPass() {} で括るとエラーが出なくなります。

Comments

[maimaime]

それはフルパスで画像URLを取得する関数で、別のとこで定義してます。正常に動きますので$ImgPassは問題ないです。