omar/panopainter
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Extract downloaded project reconcile helper

Browse Source
This commit is contained in:
omigamedev
2026-06-15 21:51:17 +02:00
parent 143d21b433
commit 69603ed6c9
3 changed files with 20 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
docs/modernization/debt.md
View File

@@ -18,6 +18,13 @@ agent or engineer to remove them without reconstructing context from chat.
## Recent Reductions ## Recent Reductions
- 2026-06-15: `DEBT-0038` was narrowed again. The retained cloud downloaded-
project open, layer refresh, and action-history reset in
`execute_legacy_downloaded_project_open()` now route through a focused
helper in `src/legacy_document_open_services.cpp` instead of living inline
in the document-open bridge; the remaining cloud bridge debt stays
concentrated in retained prompt/progress lifetime, OpenGL context guarding,
and the still-retained transfer-thread execution model.
- 2026-06-15: `DEBT-0038` was narrowed again. The retained cloud publish - 2026-06-15: `DEBT-0038` was narrowed again. The retained cloud publish
prompt setup in `show_cloud_publish_prompt()` now routes through a focused prompt setup in `show_cloud_publish_prompt()` now routes through a focused
helper in `src/legacy_cloud_services.cpp` instead of living inline in the helper in `src/legacy_cloud_services.cpp` instead of living inline in the

5
docs/modernization/roadmap.md
View File

@@ -104,6 +104,11 @@ Retained save-before-upload execution now also routes through
directly, so the remaining cloud debt is further concentrated on directly, so the remaining cloud debt is further concentrated on
prompt/progress lifetime, OpenGL context guarding, downloaded-project open, prompt/progress lifetime, OpenGL context guarding, downloaded-project open,
layer refresh, and action-history reset. layer refresh, and action-history reset.
Retained cloud downloaded-project open, layer refresh, and action-history
reset in `execute_legacy_downloaded_project_open()` now also route through a
focused helper in `src/legacy_document_open_services.*`, so the remaining
cloud bridge debt is further concentrated on prompt/progress lifetime, OpenGL
context guarding, and the still-retained transfer-thread execution model.
Retained cloud downloaded-project post-open reconciliation now also routes Retained cloud downloaded-project post-open reconciliation now also routes
through `src/legacy_document_open_services.*` instead of living inline in through `src/legacy_document_open_services.*` instead of living inline in
`src/legacy_cloud_services.cpp`, so the remaining cloud bridge debt is further `src/legacy_cloud_services.cpp`, so the remaining cloud bridge debt is further

11
src/legacy_document_open_services.cpp
View File

@@ -71,6 +71,13 @@ void open_legacy_project(App& app, const pp::app::DocumentOpenRoute& route)
} }
} }
void reconcile_downloaded_project_open(App& app)
{
for (auto& layer : app.canvas->m_canvas->m_layers)
app.layers->add_layer(layer->m_name.c_str(), false);
ActionManager::clear();
}
class LegacyDocumentOpenServices final : public pp::app::DocumentOpenServices { class LegacyDocumentOpenServices final : public pp::app::DocumentOpenServices {
public: public:
explicit LegacyDocumentOpenServices(App& app) noexcept explicit LegacyDocumentOpenServices(App& app) noexcept
@@ -155,9 +162,7 @@ void execute_legacy_downloaded_project_open(
app.doc_name = std::string(name); app.doc_name = std::string(name);
app.title_update(); app.title_update();
for (auto& layer : app.canvas->m_canvas->m_layers) reconcile_downloaded_project_open(app);
app.layers->add_layer(layer->m_name.c_str(), false);
ActionManager::clear();
} }
} // namespace pp::panopainter } // namespace pp::panopainter