Прекратите использовать Else в ваших программах

def has_access(user) -> bool:
    if not user.is_authenticated():
        return False

    # пока что все понятно, функция проверяет не надо ли пользователю доступ
    # закрыть, и если все проверки провалятся, то она, видимо, вернет True

    if user.is_superuser():
        return True

    # а нет, всё плохо. Чтобы понять что функия вернет, мне надо
    # вчитываться в каждое условие и что при этом возвращается

    if user.has_children():
        return False

    # окей, видимо, проверка на суперпользователя была исключением,
    # сейчас все пойдет как надо

    if not user.has_active_subscription():
        return True

    if user.is_baba_tanya():
        return False

    # ААА! Я запутался.

    return True

    # вроде False будет если не залогинен или
    # не суперпользователь или с подпиской? Надо перечитать

$flag = proverka1();
if(!$flag){ /* если не выполнилась проверка 1 */}
$flag2 = false;
if($flag) { /* делаем шаг 1 */ $flag2= proverk2();}
if($flag1 && !$flag2){ /* если должны были сделать шаг 1 , но проверка 2 не прошла */}
if($flag2){ /* делаем шаг 2, потому что проверка 2 успешная и потому что сделали шаг 1 */}

    public function publishOne($identity)
    {
        $output = ['success' => false];

        $response = CIBlockElement::GetByID($identity);
        $isReadSuccess = BitrixOrm::isRequestSuccess($response);
        if (!$isReadSuccess) {
            $output['message'] = 'Fail request construction';
        }
        $construct = false;
        if ($isReadSuccess) {
            $construct = $response->Fetch();
        }
        $isConstructFound = BitrixOrm::isFetchSuccess($construct);
        if ($isReadSuccess && !$isConstructFound) {
            $output['message'] = 'Construction not found';
        }
        $isExistsChild = false;
        if ($isConstructFound) {
            $response = CIBlockElement::GetList([], $filter,
                false, false, $select);
            $isExistsChild = BitrixOrm::isRequestSuccess($response);
        }
        $child = false;
        if ($isExistsChild) {
            $child = $response->Fetch();
        }
        $gotChild = BitrixOrm::isFetchSuccess($child);
        $childId = 0;
        $childPermit = 0;
        $gotPublicPermit = false;
        if ($gotChild) {
            $childId = $child['ID'];
            $childPermit = $child['PROPERTY_PERMIT_OF_AD_VALUE'];
            $gotPublicPermit = !empty($childPermit);
        }
        $isSuccessDelete = false;
        if ($gotPublicPermit) {
            $isSuccessDelete = CIBlockElement::Delete($childPermit);
        }
        if ($gotPublicPermit && !$isSuccessDelete) {
            $output['message'] = 'Fail delete published permit;';
        }
        if ($gotPublicPermit && $isSuccessDelete) {
            $output['message'] = 'Success delete published permit;';
        }
        if ($gotChild) {
            $isSuccessDelete = CIBlockElement::Delete($childId);
        }
        if ($gotChild && !$isSuccessDelete) {
            $output['message'] = 'Fail delete published permit;';
        }
        if ($gotChild && $isSuccessDelete) {
            $output['message'] =  'Success delete published permit;';
        }
        $source = [];
        $published = 0;
        if ($isConstructFound) {
            $source = $construct;
            $published = (new CIBlockElement())->Add($construct);
        }
        $hasCopy = !empty($published);
        if ($isConstructFound && !$hasCopy) {
            $output['message'] =  ' Fail copying of construction;';
        }
        $values = [];
        if ($hasCopy) {
            $output['success'] = true;
            $output['message'] = ' Success copying of construction;';

            CIBlockElement::GetPropertyValuesArray($values,
                $source['IBLOCK_ID'], $filter, [],
                ['GET_RAW_DATA' => 'Y']);
        }
        $gotValues = !empty($values);
        if ($hasCopy && !$gotValues) {
            $output['message'] = ' Fail read construction properties';
        }
        $properties = [];
        if ($gotValues) {
            $values = current($values);
            foreach ($values as $key => $value) {
                if (!empty($value['VALUE'])) {
                    $properties[$key] = $value['VALUE'];
                }
            }
            $properties['original'] = $identity;
        }
        $answer = null;
        $hasPermit = !empty($properties)
            && !empty($properties['permit_of_ad']);
        if ($hasPermit) {
            $permitId = (int)$properties['permit_of_ad'];
            $answer = CIBlockElement::GetByID($permitId);
        }
        $hasAnswer = !empty($answer) && $answer->result !== false;
        if ($hasPermit && !$hasAnswer) {
            $output['message'] =  'Fail read permit';
        }
        $permit = [];
        if ($hasAnswer) {
            $permit = $answer->Fetch();
        }
        $publishedPermit = 0;
        $gotPermit = !empty($permit) && $permit !== false;
        if ($gotPermit) {
            $publishedPermit = static::copyElement($permit,
                $this->pubPermits);
        }
        if ($gotPermit && empty($publishedPermit)) {
            $output['success'] = false;
            $output['message'] = 'Fail copying of permit';
        }
        if ($gotPermit && !empty($publishedPermit)) {
            $output['message'] = ' Success copying of permit';
        }
        if ($gotPermit && !empty($publishedPermit) 
            && !empty($properties)) {
            $properties['permit_of_ad'] = $publishedPermit;
        }
        if (!empty($properties)) {
            CIBlockElement::SetPropertyValuesEx($published,
                $this->pubConstructs->getBlock(),
                $properties, ['NewElement' => true]);
        }
        return $output;
    }