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

Устранить дублирование getBaseWalletType() — унифицировать #56

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

Проблема

getBaseWalletType() дублируется в 3 местах с РАЗНОЙ логикой:

  • walletUtils.js:78-91 — обрабатывает ERC-20 и _timestamp
  • userWalletsHandler.js:725-728 — только ERC-20
  • models/Wallet.js:7-10 — только ERC-20

Решение

  • Оставить только версию из walletUtils.js (самую полную)
  • Заменить все остальные вызовы на WalletUtils.getBaseWalletType()
  • Добавить тесты для edge cases: USDT-ERC-20, BTC_timestamp и т.д.

Приоритет: СРЕДНИЙ

## Проблема `getBaseWalletType()` дублируется в 3 местах с РАЗНОЙ логикой: - `walletUtils.js:78-91` — обрабатывает ERC-20 и _timestamp - `userWalletsHandler.js:725-728` — только ERC-20 - `models/Wallet.js:7-10` — только ERC-20 ## Решение - [ ] Оставить только версию из `walletUtils.js` (самую полную) - [ ] Заменить все остальные вызовы на `WalletUtils.getBaseWalletType()` - [ ] Добавить тесты для edge cases: USDT-ERC-20, BTC_timestamp и т.д. ## Приоритет: СРЕДНИЙ
NW added this to the Phase 2: Architecture & Refactoring milestone 2026-06-17 19:39:36 +00:00
NW added the priority::mediumtype::refactorphase::2 labels 2026-06-17 19:39:36 +00:00
NW closed this issue 2026-06-17 21:10:06 +00:00
NW commented 2026-06-17 21:22:52 +00:00
Author
Owner
Copy Link

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

  • Оставить только версию из walletUtils.js (самую полную — обрабатывает ERC-20 и _timestamp)
  • Заменить все остальные вызовы на WalletUtils.getBaseWalletType()
  • Удалить дублирующие методы из Wallet.js и userWalletsHandler.js

Файлы: src/utils/walletUtils.js, src/models/Wallet.js, src/handlers/userHandlers/userWalletsHandler.js

## ✅ Реализовано - [x] Оставить только версию из `walletUtils.js` (самую полную — обрабатывает ERC-20 и _timestamp) - [x] Заменить все остальные вызовы на `WalletUtils.getBaseWalletType()` - [x] Удалить дублирующие методы из Wallet.js и userWalletsHandler.js **Файлы**: `src/utils/walletUtils.js`, `src/models/Wallet.js`, `src/handlers/userHandlers/userWalletsHandler.js`
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::medium 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#56
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?