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

Package broken on Melpa #23

Closed
adam52 opened this issue Mar 28, 2018 · 3 comments
Closed

Package broken on Melpa #23

adam52 opened this issue Mar 28, 2018 · 3 comments
Assignees

Comments

@adam52
Copy link

adam52 commented Mar 28, 2018

I did an Upgrade recently, of all my packages on Melpa. Now when I run helm-org-rifle I am prompted for a "PATTERN:" (which I key in) and I can then select a result in the indirect buffer.

However, when I hit [enter] to complete the selection, I receive an error message:

Wrong number of arguments: (1 . 1), 0

Any ideas?

Thanks,
-Adam

@adam52
Copy link
Author

adam52 commented Mar 28, 2018

Thierry Volpiatto, who maintains the helm package, gave me this fix. In helm-org-rifle.el find this code:

  (helm-cleanup-hook (lambda ()
                       ;; Close new buffers if enabled
                       (when helm-org-rifle-close-unopened-file-buffers
                         (if (= 0 helm-exit-status)
                             ;; Candidate selected; close other new buffers
                             (let ((candidate-source (helm-attr 'name (helm-get-current-source))))
                               (dolist (source (helm-get-sources))
                                 (unless (or (equal (helm-attr 'name source)
                                                    candidate-source)
                                             (not (helm-attr 'new-buffer source)))
                                   (kill-buffer (helm-attr 'buffer source)))))
                           ;; No candidates; close all new buffers
                           (dolist (source (helm-get-sources))
                             (when (helm-attr 'new-buffer source)
                               (kill-buffer (helm-attr 'buffer source))))))))))

and replace both lines that say (dolist (source (helm-get-sources)) with (dolist (source helm-sources).

Here's the thread over on the helm page: issue 2007.

@alphapapa alphapapa self-assigned this Apr 15, 2018
alphapapa added a commit that referenced this issue Apr 16, 2018
@alphapapa
Copy link
Owner

Thanks for reporting and finding the fix!

@adam52
Copy link
Author

adam52 commented Apr 16, 2018 via email

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

No branches or pull requests

2 participants