mirror of
https://github.com/storytold/lightcraft.git
synced 2026-10-28 03:57:57 -06:00
Several commands read their photo parameters leniently. An `ids` that wasn't a list of whole numbers (`"ids": 3`, `["3"]`) fell back to the selection or the active photo, and an `id` that wasn't a whole number (`"id": "5"`) fell back to the active photo, so a call meant for one photo acted on another: - ids: album.addPhotos / removePhotos / toggleTarget, library.select, develop.set, denoise.queue, faces.index, mask.applyEffect (mask ids) and app.export; - id: photo.relink (relinked the active photo to the file), photo.removeRegion / setRegion, faces.setName, stack.setTop / moveUp / moveDown / split, library.findSimilar, the photo queries, denoise.cancel (cancelled every photo) / status and ui.render. They now refuse such values with an error, as Session::targets already does for the other commands (shared cmd::photo_ids / photo_id). A missing or null value still means the selection or the active photo.