解決済$data_list_upgrade(インスタンス)が上書き?
タグ: 新しいデータリスト, 新規会員登録メール認証
-
投稿者投稿
-
2023年8月12日 8:46 AM #100542
kiyoshi
参加者——————————————-
Welcart のバージョン:(2.8.20)
——————————————–
いつも利用させていただき、ありがとうございます。バグでもなく不具合もありませんが、下記のコードが気になったので、念のため知らせておきます。
wp-content/plugins/usc-e-shop/functions/filters.php 1679行,1682行
require_once USCES_EXTENSIONS_DIR . ‘/DataListUpgrade/data_list_upgrade.php’;
$data_list_upgrade = new USCES_DATALIST_UPGRADE();require_once USCES_EXTENSIONS_DIR . ‘/VerifyMembersEmail/verify_members_email.php’;
$data_list_upgrade = new USCES_VERIFY_MEMBERS_EMAIL();↑どちらのインスタンスも$data_list_upgradeに代入しています。
USCES_DATALIST_UPGRADEのインスタンスが、USCES_VERIFY_MEMBERS_EMAILのインスタンスに上書きされていると思うのですが。welcartプラグインの全コード内において、$data_list_upgradeグローバル変数(インスタンス)のメソッドやプロパティを参照する記述はありません。
ですので、特に不具合が起こることはないと思いますが、気になったので知らせておきます。
もし、あえて上記のようにしているのでしたら余計なこと言ってスミマセン。
ちなみに自分は、新規会員登録メール認証システムが関わってくるカスタマイズをしていて、USCES_VERIFY_MEMBERS_EMAILのメソッドを使おうとして、今回知らせた部分に気づきました。
だから、カスタマイズでも「$data_list_upgrade->」として問題なく使えました。
が、USCES_DATALIST_UPGRADEのメソッドやプロパティを使うカスタマイズだと上手くいかなかったのでは、と考えています。以上、お知らせでした。
2023年8月17日 9:48 AM #100552
kitagawaキーマスターkiyoshi 様
ご報告いただき誠にありがとうございます。
該当のコードを確認いたしました。
仰るとおりです。
次期バージョンで修正いたします。よろしくお願いいたします。
2023年8月28日 2:34 PM #100632
kitagawaキーマスターkiyoshi 様
先日はご報告いただき誠にありがとうございました。
先程リリースしましたWelcart 2.8.21にて対応いたしました。> ちなみに自分は、新規会員登録メール認証システムが関わってくるカスタマイズをしていて、USCES_VERIFY_MEMBERS_EMAILのメソッドを使おうとして、今回知らせた部分に気づきました。
> だから、カスタマイズでも「$data_list_upgrade->」として問題なく使えました。
> が、USCES_DATALIST_UPGRADEのメソッドやプロパティを使うカスタマイズだと上手くいかなかったのでは、と考えています。USCES_VERIFY_MEMBERS_EMAILのメソッドご利用のために$data_list_upgradeをお使いとのことで、今回の修正の影響がございます。
USCES_VERIFY_MEMBERS_EMAILに対応した変数$verify_members_emailをご用意いたしましたのでこちらをご利用いただきたく存じます。お手数ですがご確認・ご対応いただければ幸いです。
よろしくお願いいたします。2023年8月29日 11:33 AM #100644kiyoshi
参加者kitagawaさま
わざわざ丁寧にお知らせありがとうございます。
バージョンアップでの修正、確認しました。このスレッドは解決済みにしていただければと思います。
ご対応ありがとうございました。
2023年8月29日 11:37 AM #100645
kitagawaキーマスターkiyoshi 様
> バージョンアップでの修正、確認しました。
ご確認・ご対応いただきありがとうございます。
このトピックは「解決済み」とさせていただきますので、ご了承ください。
今後ともどうぞよろしくお願いします。 -
投稿者投稿
- このトピックに返信するにはログインが必要です。
PAGE TOP