diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-24 04:29:57 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-24 04:29:57 -0500 |
| commit | b26b74cb447966c0283a46afba908866777bd6ae (patch) | |
| tree | fa9112ae3ed13be6f23d44ee4cebd512c1e9cc95 /tests/test-org-config-finalize-task.el | |
| parent | 35e4d70116c8a2a5b82eaf4b8c58889dc02cbe46 (diff) | |
| download | dotemacs-b26b74cb447966c0283a46afba908866777bd6ae.tar.gz dotemacs-b26b74cb447966c0283a46afba908866777bd6ae.zip | |
refactor(video-capture): scope capture URL to a dynamic binding
quick-video-capture passed the org-protocol URL through a global cj/video-download-current-url: the protocol handler setq the global, the capture handler read and cleared it. If a capture was aborted or errored between those steps, the stale URL survived into the next manual capture.
Renamed it to cj/--video-download-url and let-bind it around the org-capture call in the protocol handler instead of setq-ing a global. The binding lives only for the dynamic extent of the capture, so the handler still sees the URL while the capture runs, and an abort or error unwinds the binding automatically — no stale state, no manual clear. The handler still prompts when invoked manually with no URL bound. Tests cover the bound-URL download, the manual prompt, the empty-URL error, that the URL is visible during the capture, and that an aborted capture leaves nothing behind.
Diffstat (limited to 'tests/test-org-config-finalize-task.el')
0 files changed, 0 insertions, 0 deletions
