qBittorrent icon indicating copy to clipboard operation
qBittorrent copied to clipboard

Published 20 hours ago • verified publisher icon qbittorrent

WebUI: avoid redundant re-initialization

Open Chocobo1 opened this issue 1 year ago • 1 comments

ps. Will apply formatting after conflicting PRs are merged: #20951, #21000, #21007.

Chocobo1 avatar Jun 30 '24 07:06 Chocobo1

The question is rather rhetorical... Isn't there a way in "web programming" to have a more understandable/logical/manageable "path" of code execution instead of producing such guards?

glassez avatar Jun 30 '24 08:06 glassez

Isn't there a way in "web programming" to have a more understandable/logical/manageable "path" of code execution instead of producing such guards?

Well, yeah. It's a matter of app architecture.

The WebUI has a fairly old legacy codebase. Web development changed dramatically over the years.

HanabishiRecca avatar Jul 04 '24 17:07 HanabishiRecca

Resolved merge conflicts. This PR is ready.

Chocobo1 avatar Jul 09 '24 05:07 Chocobo1

PR updated to resolve merge conflicts.

Chocobo1 avatar Jul 12 '24 06:07 Chocobo1

Resolved merge conflicts. This PR is ready.

I just restored the previous approval.

Thanks! I early merged this PR since it might get in the way of later WebUI PRs.

Chocobo1 avatar Jul 12 '24 07:07 Chocobo1