Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error when trying to bookmark an Org QL view #407

Open
karthink opened this issue Feb 8, 2024 · 1 comment
Open

Error when trying to bookmark an Org QL view #407

karthink opened this issue Feb 8, 2024 · 1 comment
Assignees
Milestone

Comments

@karthink
Copy link

karthink commented Feb 8, 2024

OS/platform

Linux

Emacs version and provenance

Emacs 29.1, distro packages

Emacs command

emacs

Org version and provenance

Org 9.7-pre, Org development branch.

org-ql package version and provenance

0.9-pre, commit 9606aaf, straight.el recipe, straight.el

Actions taken

  1. Run org-ql-search and pick org-agenda-files in the completing-read menu with the prompt "Buffers/Files: "
  2. Try to bookmark the view buffer that is generated with C-x r m, bookmark-set

Observed results

The bookmark is not set, I get the error: cl-etypecase failed: org-agenda-files, (string buffer list)

Expected results

The bookmark to be set.

Backtrace

Debugger entered--Lisp error: (error "cl-etypecase failed: org-agenda-files, (string buffer list)")
  error("cl-etypecase failed: %s, %s" org-agenda-files (string buffer list))
  org-ql-view-bookmark-make-record()
  bookmark-make-record()
  bookmark-set-internal("Set bookmark named" nil overwrite)
  bookmark-set(nil nil)
  funcall-interactively(bookmark-set nil nil)
  command-execute(bookmark-set)

Etc.

The error is connected to the value of the infix option "In buffers/files" in this transient:

2024-02-08-131605_959x211_scrot

If appears to be set to the symbol org-agenda-files. If I refresh it by pressing i again it is set to the value of this symbol instead:

2024-02-08-131831_992x209_scrot

then it passes the cl-etypecase test in org-ql-view-bookmark-make-record(since it's a list) and I can bookmark it without errors.

@alphapapa alphapapa self-assigned this Feb 8, 2024
@alphapapa alphapapa added the bug label Feb 8, 2024
@alphapapa alphapapa added this to the 0.9 milestone Feb 8, 2024
@alphapapa
Copy link
Owner

Thanks. The "expansion" and "contraction" of that argument to org-ql-search is due for some refactoring. There are some relevant PRs but the code is complicated and coming back to it after a while makes it difficult to review. I'll fix that eventually, but in the meantime, this can probably be fixed separately.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants