alphard/Comfyui-LayerForge
1
0
Fork 0
mirror of https://github.com/Azornes/Comfyui-LayerForge.git synced 2026-03-21 20:52:12 -03:00
Code Issues Packages Projects Releases Wiki Activity

Remove legacy delegating methods from Canvas.js

Browse Source 
Removed all remaining delegating methods from Canvas.js, including handleMouseMove and garbage collection helpers, as part of finalizing the refactor to a pure facade. Updated REFACTORING_GUIDE.md to reflect the completion of this refactor and clarify the new architecture.
This commit is contained in:
Dariusz L
2025-06-29 20:43:02 +02:00
parent 0b3bdaf769
commit 8efb9d91b0
2 changed files with 11 additions and 41 deletions
Download Patch File Download Diff File Expand all files Collapse all files

37
js/Canvas.js
View File

@@ -457,41 +457,4 @@ export class Canvas {
this.render();
}
// ==========================================
// METODY DELEGUJĄCE DLA KOMPATYBILNOŚCI
// ==========================================
/**
* Te metody są zachowane tymczasowo dla kompatybilności wstecznej.
* W nowych implementacjach należy używać bezpośrednio odpowiednich modułów:
* - this.canvasLayers dla operacji na warstwach
* - this.canvasInteractions dla obsługi interakcji
* - this.canvasIO dla operacji I/O
* - this.canvasState dla zarządzania stanem
*
* UWAGA: Metody delegujące do CanvasLayers i CanvasState zostały usunięte.
* Używaj: canvas.canvasLayers.copySelectedLayers(), canvas.canvasState.undo(), etc.
*/
// Delegacje do CanvasInteractions
handleMouseMove(e) { this.canvasInteractions.handleMouseMove(e); }
// Delegacje do ImageReferenceManager
async runGarbageCollection() {
if (this.imageReferenceManager) {
await this.imageReferenceManager.manualGarbageCollection();
}
}
getGarbageCollectionStats() {
if (this.imageReferenceManager) {
const stats = this.imageReferenceManager.getStats();
return {
...stats,
operationCount: this.imageReferenceManager.operationCount,
operationThreshold: this.imageReferenceManager.operationThreshold
};
}
return null;
}
}

15
js/REFACTORING_GUIDE.md
View File

@@ -181,17 +181,24 @@ canvas.imageReferenceManager.manualGarbageCollection()
- Wszystkie operacje na warstwach używają teraz bezpośrednio modułu `canvasLayers`
- Canvas.js zawiera teraz tylko główne operacje fasady i niezbędne metody pomocnicze
6. **Usunięcie metod delegujących do CanvasInteractions** ✅
- Usunięto ostatnią metodę delegującą `handleMouseMove()` z Canvas.js
- Metoda nie była używana w żadnym pliku, więc usunięcie było bezpieczne
- Wszystkie operacje interakcji używają teraz bezpośrednio modułu `canvasInteractions`
- Canvas.js jest teraz prawdziwą fasadą bez niepotrzebnych metod delegujących
## Uwagi dla deweloperów
- ✅ **Refaktoryzacja zakończona** - wszystkie pliki zostały zaktualizowane
- ✅ **Refaktoryzacja w pełni zakończona** - wszystkie pliki zostały zaktualizowane
- ✅ **Nowy kod** używa modułów bezpośrednio zgodnie z wzorcem fasady
- ✅ **Wszystkie delegacje** wskazują na istniejące metody w modułach
- ✅ **Wszystkie metody delegujące** do głównych modułów zostały usunięte
- ✅ **Czysta fasada** - Canvas.js zawiera tylko główne operacje wysokiego poziomu
- ✅ **Spójna architektura** - wszystkie moduły używają poprawnych referencji
- ⚠️ **Metody delegujące** są zachowane dla kompatybilności, ale oznaczone jako tymczasowe
- ✅ **Minimalne delegacje** - pozostały tylko metody do ImageReferenceManager
- 📚 **Dokumentacja** została zaktualizowana w tym przewodniku
- 🔄 **Kompatybilność** z istniejącym kodem jest zachowana
**Refaktoryzacja została zakończona pomyślnie!** System jest gotowy do dalszego rozwoju z lepszą architekturą opartą na wzorcu fasady.
**Refaktoryzacja została w pełni zakończona!** Canvas.js jest teraz prawdziwą fasadą bez niepotrzebnych metod delegujących. System jest gotowy do dalszego rozwoju z czystą architekturą opartą na wzorcu fasady.
### 📋 Mapowanie kompletnych funkcjonalności