Skip to content

Правка форматирования код-блоков: пробелы вместо табов, стиль скобок,…#116

Open
shoman4eg wants to merge 3 commits into
bitrix-tools:mainfrom
shoman4eg:patch-2
Open

Правка форматирования код-блоков: пробелы вместо табов, стиль скобок,…#116
shoman4eg wants to merge 3 commits into
bitrix-tools:mainfrom
shoman4eg:patch-2

Conversation

@shoman4eg

Copy link
Copy Markdown
Contributor
  • табы в код-блоках заменил на пробелы. В блоках кода едет форматирование
  • скобки if/foreach перенёс на строку с условием, привел всё к одному виду.
  • поправил метки языков и схемы папок, пару опечаток

@ech-bitrix-doc

Copy link
Copy Markdown
Collaborator

Добрый день!
Необходимо оформить по нашим правилам - фигурную скобку нужно переносить на новую строку. Ознакомиться можно на странице - [https://docs.1c-bitrix.ru/CONTRIBUTING.html#figurnye-skobki-v-upravlyayushih-konstrukciyahо .](https://docs.1c-bitrix.ru/CONTRIBUTING.html#figurnye-skobki-v-upravlyayushih-konstrukciyah%D0%BE)

@shoman4eg

Copy link
Copy Markdown
Contributor Author

Фигурные скобки обновил,
; сделал по стандарту.

@ech-bitrix-doc

ech-bitrix-doc commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

нужно

  1. исправить опечтаку pages/cms-basics/page-templates.md:196 (сейчас et вместо get)
    $TEMPLATE["announcement.php"] = ["name" => Loc::getMessage("announcement"), "sort" => 2];

  2. pages/framework/background-jobs.md:424 - лишний пробел в конце строки

  3. в списке файлов перенести use Bitrix\Main\Loader; выше для привычного вида (сначала use, после if...)
    Должно быть так:
    `<?php

use Bitrix\Main\Loader;

if (!defined("B_PROLOG_INCLUDED") || B_PROLOG_INCLUDED !== true) die();`

файлы:
pages/cms-basics/mail.md:581
pages/cms-basics/menu.md:154
pages/cms-basics/page-templates.md:194, 240, 470
pages/framework/components.md:623, 637, 638
pages/get-started/create-component.md:220
pages/get-started/create-controller.md:304, 469, 470

@shoman4eg

Copy link
Copy Markdown
Contributor Author

Обновил импорты, убрал лишний пробел.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants