Коля SbWereWolf

А чем svelte:window on:load лучше просто window.onload?

Как он сработает, если до этого

на кой там вообще нужен %любой_новый_модный_фреймворк%?).

Из любви к особому пути я решил осваивать не проверенный миллионами разработчиков Angular/React/Vue, я решил попробовать Svelte.

нельзя просто Jquery вместо $ использовать?

<svelte:window/>

    public function jsonSerialize() 
    {
        $fields = get_object_vars($this);
        // что-то делаем ...
        return $fields;
    }

Про маштабирование на другие города не понятно в чём трудности. По идее надо распарсить гео данные из OSM или что то еще? Или описания достопримечательностей необходимы и забиваются в ручную?
Тогда понятно. Но можно в самом приложении дать возможность пользователям заводить свои тексты.
Дать людчм карту и пусть сами контент добавят, я думаю это выход.
Вопрос только в наполнеии базы на сервере. Но пара гигов под каждый город это наверное не очень дорого ?

Автор пиши ещё !

За что я люблю PHP.
Во первых я его не люблю. Я на нём пишу.
Чем он хорош? Хорош обилием готовых решений и готовыми ответами на любые вопросы.
Чем ещё хорош? Хорош тем что можно писать хоть что и хоть как.
Хочу пишу спагетти код, хочу блюду SOLID и проверку типов.
Можно накидать прототип, отработать идею, всё выкинуть и после этого написать с ноля и написать хорошо. И всё это на одном языке.
PHP даёт свободу творчества. И эта свобода как и любая другая накладывает ответственность.
Нет плохих инструментов, есть неподходящие. PHP это только инструмент, а ответственность на том кто этим инструментом работает.

Хороший пример как можно писать без ветвлений.

Всё просто, это спагетти код.

приходит новый человек

Откуда он узнает что какие-то переменные используются только в ограниченном месте

и какие из них можно переиспользовать и не будет ли конфликтов?

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

$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;
    }