フォーラム | XOOPS Cube日本サイト

User_RegisterEditFormのactkey, user_regdateは不要では？

投稿者: suin | 投稿日時: 2008-11-4 22:32 | 閲覧: 7990回

Userモジュールをいじっていて疑問に思ったので書きます。

承認キーとユーザ登録日についてプロパティーをフォームに登録してますが、これは何を意図しているんでしょうか？

29~30行目

		$this->mFormProperties['actkey']=new XCube_StringProperty('actkey');
		$this->mFormProperties['user_regdate']=new XCube_IntProperty('user_regdate');

updateメソッドを見ても、actkeyとuser_regdateのプロパティーが使われている形跡がありません。

	function update(&$obj)
	{
		$obj->set('uname', $this->get('uname'));
		$obj->set('email', $this->get('email'));
		$obj->set('user_viewemail', $this->get('user_viewemail'));
		$obj->set('url', $this->get('url'));
		$obj->set('user_avatar','blank.gif',true);
		$obj->set('timezone_offset', $this->get('timezone_offset'));
		$obj->set('pass', md5($this->get('pass')));
		$obj->set('user_mailok', $this->get('user_mailok'));
		$obj->set('agree', $this->get('agree'));

		$actkey=substr(md5(uniqid(mt_rand(),1)),0,8);
        $obj->set('actkey',$actkey,true);
        $obj->set('user_regdate',time(),true);
	}

User_UserRegister_confirmActionに有っても良さそうですが…。


	function execute(&$controller, &$xoopsUser)
	{
		$memberHandler =& xoops_gethandler('member');
		$this->mNewUser =& $memberHandler->createUser();
		$this->mNewUser->vars['uname']['maxlength'] = 255;
		$this->mRegistForm->update($this->mNewUser);
		$this->mNewUser->set('uorder', $controller->mRoot->mContext->getXoopsConfig('com_order'), true);
		$this->mNewUser->set('umode', $controller->mRoot->mContext->getXoopsConfig('com_mode'), true);
		if ($this->mConfig['activation_type'] == 1) {
			$this->mNewUser->set('level', 1, true);
		}
		$actkey=substr(md5(uniqid(mt_rand(),1)),0,8);
		$this->mNewUser->set('actkey',$actkey,true);
		$this->mNewUser->set('user_regdate',time(),true);
		...
	}

タグ: user

9人中0人のユーザがこのトピックに投票しています: 投票 | 返信する

コメント(2)

新しいものから | 古いものから | ネスト表示 |

Re: User_RegisterEditFormのactkey, user_regdateは不要では？

投稿者: onokazu | 投稿日時: 2008-11-7 15:01

特別な意味はないと思いますが、確かに29~30行目は不要だと思います。
この箇所のコードって自動生成とかしているんでしたっけ？その場合は単に消し忘れではないでしょうか？

User_UserRegister_confirmAction::execute()の下記部分でUser_RegisterEditForm::update()を呼んで、そこでactkey/user_regdateを更新しているので、suinさんの書かれた、User_UserRegister_confirmAction::execute()への追加も特に必要ないのではと思います。


function execute(&$controller, &$xoopsUser)
{
    $memberHandler =& xoops_gethandler('member');
    $this->mNewUser =& $memberHandler->createUser();
    $this->mRegistForm->update($this->mNewUser);
    ...

#19266 | 返信する |

Re: User_RegisterEditFormのactkey, user_regdateは不要では？

投稿者: minahito | 投稿日時: 2008-11-8 1:20

onokazu さんフォローありがとうございます。

バグトラッカーに登録しておきました

自動生成コードの削り忘れに間違いないと思いますが、まず本日のところはこれ(登録)のみで……

# もしよければバグトラのアイテムを Monitoring してくださいませ

#19275 | 返信する |

投票(0)

新しいものから | 古いものから |