TIP: Use Markdown or, <pre> for multi line code blocks / <code> for inline code.
These forums are read-only and for archival purposes only!
Please join our new forums at discourse.kohanaframework.org
  • $userexist = ORM::factory('user')->where('user_profileurl', '=', $userdata->identity)->find();
    if (isset($userexist->user_id)) {
    $last_id = $userexist->user_id;
    } else {
    $user = ORM::factory('user');
    $user->user_nick = $name;
    $user->user_avatar = $userdata->photo;
    $user->user_profileurl = $userdata->identity;
    $user->user_date = date('Y-m-d H:i:s');
    $user->save();
    $last_id = $user->user_id;
    }

    Ща организовано вот так.
    Наверняка можно изящнее?!
  • 1. Я так понимаю, что user_id это первичный ключ? Тогда лучше использовать if ($userexist->loaded()) и $userexist->pk().
    2. В ветке else можно воспользоваться методом values(array('user_nick' => $name, ...))
  • ну то есть просто немного изменится внешний вид кода.
    я думал может как то check() можно приспособить.
  • check() нужен несколько для других целей, и кстати, его все равно желательно "приспособить" :)

Howdy, Stranger!

It looks like you're new here. If you want to get involved, click one of these buttons!

In this Discussion