How bad is it to have code like this for readability & maintainability

Readability.php

<?php
if($paramFetcher['owner'] != null && ($owner = $userMgr->getById($paramFetcher['owner']))) {
  $org->setOwner($owner);
}
?>

vs.

<?php
if($paramFetcher['owner'] != null) {
  if($owner = $userMgr->getById($paramFetcher['owner'])){
    $org->setOwner($owner);
  }
}
?>

FINAL:

<?php
$owner = $this->getUserManager()->getById($paramFetcher['owner']);
if ($owner) {
    $org->setOwner($owner);
}
?>

hmmm ... first thought was that the first way, imho, is best for both but then I started to doubt ... I'd go with #1 for only that, if you need to check more than that then go with #2

I agree with the above.

I would probably create an isOwner() function (containing two separate if blocks) and call it to do the checks, returning a bool.

Reusable, testable, self documenting, and less cyclomatic complexity.

Ended up changing it. I realized I was duplicating some code to get users and that if the parameter for getById() is null, it will return null. So there was no need to check $paramFetcher['owner'] individually.

Thank you all!