Спрашивай конкретно что непонятно - откомментирую.
$ctrl = new GuestbookController(new User(), new Guestbook());$user = $this->modelUser->getObjectById($request->id_user);$user = modelUser::getObjectById($request->id_user);
// далее работаем с этим юзером, имея всего один объект модели - $user
if (!$user->getId())
{
$this->error = 'Хакер? Хе-хе....';
}
echo 'Тебя зовут '.$user->user_name; // Наполняем view данным для подстановки в шаблон
$this->view->error = $this->error;
foreach ($this->bufer as $key => $value)
{
$this->view->$key = $value;
}а я разве не говорил что нужно сделать findById() ?
$this->modelUser->findById($id);верно!
я отпечатался, я говорил о моделях, да.
$this->modelUser->findById($id);Ну я просто подумал, что findById должен возвращать объект.. по аналогии со статическим методом ActiveRecord. Я просто в контексте ООП только так пока делал....
// заполняем модель $this->modelUser данными пользователя из БД
$this->modelUser->findById($request->id_user);
if (!$this->modelUser->getId())
{
$this->view->error = 'BAD_USER';
}class Controler.....
{
public function edit()
{
$this->view->menu_data = $this->menu();
$this->view->setErrorKeys($this->user_mapper->getModelAttributes());
$this->view->setErrorKeys(array('user_password_1', 'user_password_2'));
$params = array
(
'what' => '`id_country`, `country_name'.(LANG == 'en' ? '_en' : '').'` as `country_name`',
'order' => array('country_name' => 'ASC'),
);
$this->view->countries = Country::findByParams($params)->getData()->getDataAsArray();
$this->view->user_password_1 = $this->view->user_password_2 = '';
$this->view->user_phones = array();
$obj_phone_types = new Phone_Type();
$this->view->form_phones = $obj_phone_types->findAll()->getData()->getDataAsArray();
$group_mapper = new Group_Mapper();
// выбираем все группы, за исключением группы "гости".
$groups_list = $group_mapper->getObjectList_(array('where' => 'id <> 3'));
// Массив для формирования статической формы.
$this->view->setGroupsList($groups_list, 'form_groups');
$phone_mapper = new Phone_Mapper();
if (empty($this->user))
{
$this->user = $this->user_mapper->createNew();
foreach ($this->view->form_phones as $phone_type)
{
$this->view->user_phones[$phone_type->id] = $phone_mapper->createNew()->getData()->getDataAsArray();
}
}
$params = array
(
'where'=> array('id_user = ?' => array((int)$this->user->id))
);
$user_phones = $phone_mapper->getObjectList_($params);
foreach ($user_phones->getData() as $phone)
{
$this->view->user_phones[$phone->id_type] = $phone->getData();
}
// Неизменяемые данные формы.
$this->view->html_header = $this->user->user_name.' '.$this->user->user_last_name;
$this->view->id = $this->user->id;
$this->view->user_ip = $this->user->user_ip ? int2ip($this->user->user_ip) : 0;
$this->view->user_regdate = $this->user->user_regdate;
$this->view->user_visitdate = $this->user->user_visitdate;
$this->view->return_on_page = $this->request->get('return_on_page', 'int');
if (Request::isPost())
{
$this->user = $this->user_mapper->createFromArray($this->request->getData());
$validator = new User_Validator('User');
$validator->setDomainObject($this->user);
$validator->checkCorrectLogin('user_login');
$validator->checkExistLogin('user_login');
if (!$this->user->id OR
$this->user->id && (!empty($this->user->user_password_1) || !empty($this->user->user_password_2) ))
{
$validator->checkCorrectPassword('user_password_1');
$validator->checkCorrectPassword('user_password_2');
}
$validator->checkPasswords('user_password', 'user_password_1', 'user_password_2');
$validator->checkCorrectMail('user_mail');
$validator->checkExistMail('user_mail');
$validator->checkName('user_name');
$validator->checkAge('user_age_day', 'user_age_month', 'user_age_year');
$validator->checkSex('user_sex');
$validator->checkPlace('user_city', 'user_region', 'user_country');
//$validator->isEmpty('zip_code');
$validator->checkIcq('user_icq');
$validator->checkUrl('user_url');
// создаем массив объектов типа Phone на основе полученных данных
$user_phones = new Object_List();
foreach ($this->request->phones as $phone_data)
{
$user_phones->add($phone_mapper->createFromArray($phone_data), $phone_data['id_type']);
}
if ($err = $validator->getErrors())
{
$this->view->setErrorMessage($err);
$this->view->user_phones = $user_phones->getData()->getDataAsArray();
}
else
{
$this->user->user_ip = $_SERVER['REMOTE_ADDR'];
$this->user_mapper->save($this->user);
foreach ($user_phones as $phone)
{
$phone->id_user = $this->user->id;
$phone_mapper->save($phone);
}
$redirect = new Redirect();
$redirect->setMessage('user_edit_ok');
$redirect->addParam('user_name', Format::hsc($this->user->user_name.
($this->user->user_last_name ? ' '.$this->user->user_last_name : '')
)
);
$redirect->addParam('id_user', $this->user->id);
$redirect->setRedirectUrl($this->request->get('return_on_page', 'int')
? '/admin/user/edit/?id='.$this->user->id
: ($this->request->referer ? $this->request->referer : '/admin/user/')
);
$redirect->run();
}
}
$this->view->id_group = $this->user->id_group;
$this->view->user_active = $this->user->user_active;
$this->view->user_login = $this->user->user_login;
$this->view->user_mail = $this->user->user_mail;
$this->view->user_name = $this->user->user_name;
$this->view->user_last_name = $this->user->user_last_name;
$this->view->user_password_1 = $this->user->user_password_1;
$this->view->user_password_2 = $this->user->user_password_2;
$this->view->user_age_day = $this->user->user_age_day;
$this->view->user_age_month = $this->user->user_age_month;
$this->view->user_age_year = $this->user->user_age_year;
$this->view->user_sex = $this->user->user_sex;
$this->view->user_country = $this->user->user_country;
$this->view->user_region = $this->user->user_region;
$this->view->user_city = $this->user->user_city;
$this->view->zip_code = $this->user->zip_code;
$this->view->user_icq = $this->user->user_icq;
$this->view->user_url = $this->user->user_url;
$this->view->user_about = $this->user->user_about;
}$this->view->menu_data = $this->menu();
да, сейчас всё нормально, это один из перывых кодов...
то, что оно в контроллере маячит. То, что контроллер наполняется не только своей основной функциональностью, а ещё и всякими getMenu().