Skip to content

Merge main into dev/v1.x - #4

Merged
MaximHarder merged 4 commits into
mainfrom
dev/v1.x
Sep 21, 2026
Merged

MaximHarder merged 4 commits into
mainfrom
dev/v1.x

Conversation

@MaximHarder

@MaximHarder MaximHarder commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

Что меняется

Переход от динамической генерации схемы Cycle ORM к статическому определению схемы. Удалён класс EntityClassLocator, убран параметр entityClasses из метода fromConfig.

Зачем

Упрощение конфигурации ORM, снижение зависимости от Cycle Annotated, оптимизация производительности.

Как проверить

  1. Выполнить миграции базы данных
  2. Проверить работу CRUD-операций для AnimeEntity
  3. Убедиться в отсутствии ошибок при инициализации ORM

Затронутые пути

  • src/Persistence/CycleOrmFactory.php
  • src/Persistence/EntityClassLocator.php (удалён)

Коммиты

DevCraftBot and others added 3 commits September 21, 2026 15:59
Удаляет динамическую генерацию схемы через Compiler и
заменяет её на статическую конфигурацию. Это упрощает
инициализацию ORM и убирает зависимость от аннотаций.
Теперь схема явно определена в коде, что облегчает
отладку и тестирование.
@MaximHarder

Copy link
Copy Markdown
Contributor Author

@DevCraftBot summerize

@DevCraftBot

DevCraftBot commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

Описание запроса обновлено.

@DevCraftClub DevCraftClub deleted a comment from DevCraftBot Sep 21, 2026
Извлекает повторяющуюся логику валидации и преобразования
данных в отдельный класс ArrayUtil. Это делает код в fromArray
методах чище и избегает дублирования. Также уточняет тип
параметра в PHPDoc с string на array-key для корректности.
@MaximHarder

Copy link
Copy Markdown
Contributor Author

@DevCraftBot auto-merge

@DevCraftBot

Copy link
Copy Markdown
Contributor

Принял команду, обрабатываю.

@MaximHarder
MaximHarder merged commit 6058bf6 into main Sep 21, 2026
3 checks passed
@MaximHarder
MaximHarder deleted the dev/v1.x branch September 21, 2026 15:36
@MaximHarder
MaximHarder restored the dev/v1.x branch September 21, 2026 15:36
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