Telegram-Market/telegram-shop
1
1
Fork 0
Code Issues 40 Pull Requests Actions Packages Projects 1 Releases Wiki Activity

Разбить adminProductHandler.js на модули #51

New Issue
Closed
opened 2026-06-17 19:39:27 +00:00 by NW · 1 comment
NW commented 2026-06-17 19:39:27 +00:00
Owner
Copy Link

Проблема

adminProductHandler.js — 1054 строк. Нарушает правило 100 строк на файл. Смешивает роутинг, бизнес-логику и форматирование.

Решение

Разбить на модули:

  • src/handlers/adminHandlers/product/createHandler.js — создание товара
  • src/handlers/adminHandlers/product/updateHandler.js — обновление товара
  • src/handlers/adminHandlers/product/deleteHandler.js — удаление товара
  • src/handlers/adminHandlers/product/importHandler.js — импорт из ZIP/CSV
  • src/handlers/adminHandlers/product/categoryHandler.js — управление категориями
  • src/handlers/adminHandlers/product/locationHandler.js — управление локациями
  • src/handlers/adminHandlers/product/index.js — роутер, делегирующий к модулям

Правило

Каждый файл ≤100 строк, каждая функция ≤30 строк.

Приоритет: ВЫСОКИЙ

## Проблема `adminProductHandler.js` — 1054 строк. Нарушает правило 100 строк на файл. Смешивает роутинг, бизнес-логику и форматирование. ## Решение Разбить на модули: - [ ] `src/handlers/adminHandlers/product/createHandler.js` — создание товара - [ ] `src/handlers/adminHandlers/product/updateHandler.js` — обновление товара - [ ] `src/handlers/adminHandlers/product/deleteHandler.js` — удаление товара - [ ] `src/handlers/adminHandlers/product/importHandler.js` — импорт из ZIP/CSV - [ ] `src/handlers/adminHandlers/product/categoryHandler.js` — управление категориями - [ ] `src/handlers/adminHandlers/product/locationHandler.js` — управление локациями - [ ] `src/handlers/adminHandlers/product/index.js` — роутер, делегирующий к модулям ## Правило Каждый файл ≤100 строк, каждая функция ≤30 строк. ## Приоритет: ВЫСОКИЙ
NW added this to the Phase 2: Architecture & Refactoring milestone 2026-06-17 19:39:27 +00:00
NW added the priority::hightype::refactorphase::2 labels 2026-06-17 19:39:27 +00:00
NW commented 2026-06-17 21:41:19 +00:00
Author
Owner
Copy Link

✅ Реализовано

  • navigationHandler.js (86 строк) — вход в управление товарами + выбор страны
  • districtHandler.js (89 строк) — выбор города + района
  • categoryAddHandler.js (91 строка) — добавление категории
  • categoryEditHandler.js (82 строки) — редактирование категории
  • categorySelectionHandler.js (48 строк) — отображение категории
  • createHandler.js (56 строк) — промпт добавления товара
  • importHandler.js (75 строк) — импорт товара (JSON/файл)
  • editStartHandler.js (63 строки) — промпт редактирования
  • editImportHandler.js (94 строки) — обработка редактирования
  • deleteHandler.js (97 строк) — удаление + подтверждение
  • viewHandler.js (89 строк) — просмотр товара
  • listHandler.js (90 строк) — список товаров с пагинацией
  • productValidator.js (17 строк) — валидация
  • index.js (31 строка) — роутер, реэкспорт 17 методов

Удалено: дублирующий handleCategorySelection, методы с subcategories (таблица не существует)

Коммит: 4b7ed0c

## ✅ Реализовано - [x] `navigationHandler.js` (86 строк) — вход в управление товарами + выбор страны - [x] `districtHandler.js` (89 строк) — выбор города + района - [x] `categoryAddHandler.js` (91 строка) — добавление категории - [x] `categoryEditHandler.js` (82 строки) — редактирование категории - [x] `categorySelectionHandler.js` (48 строк) — отображение категории - [x] `createHandler.js` (56 строк) — промпт добавления товара - [x] `importHandler.js` (75 строк) — импорт товара (JSON/файл) - [x] `editStartHandler.js` (63 строки) — промпт редактирования - [x] `editImportHandler.js` (94 строки) — обработка редактирования - [x] `deleteHandler.js` (97 строк) — удаление + подтверждение - [x] `viewHandler.js` (89 строк) — просмотр товара - [x] `listHandler.js` (90 строк) — список товаров с пагинацией - [x] `productValidator.js` (17 строк) — валидация - [x] `index.js` (31 строка) — роутер, реэкспорт 17 методов **Удалено**: дублирующий handleCategorySelection, методы с subcategories (таблица не существует) **Коммит**: 4b7ed0c
NW closed this issue 2026-06-17 21:41:20 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
phase::1
Phase 1: Security
 phase::1
Phase 1
 phase::1
Phase 1
 phase::2
Phase 2: Architecture
 phase::2
Phase 2
 phase::3
Phase 3: Admin API
 phase::3
Phase 3
 phase::4
Phase 4: Web UI
 phase::4
Phase 4
 phase::5
Phase 5: Quality
 phase::5
Phase 5
 priority::critical
Critical priority
 priority::critical
Critical priority
 priority::high
High priority
 priority::high
High priority
 priority::medium
Medium priority
 priority::medium
Medium priority
 type::bug
Bug fix
 type::bug
Bug
 type::feature
New feature
 type::feature
Feature
 type::infra
Infrastructure
 type::infra
Infrastructure
 type::refactor
Code refactoring
 type::refactor
Refactoring
 type::security
Security issue
 type::security
Security
 type::testing
Testing
 type::testing
Testing
No Label phase::2 priority::high type::refactor
Milestone
No items
No Milestone Phase 2: Architecture & Refactoring
Projects
Clear projects
No project
Assignees
Clear assignees
NW
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Telegram-Market/telegram-shop#51
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?