Allow org-ql-refile to handle values for org-refile-targets where the symbol in a given cons cell has a variable value#488
Open
apc wants to merge 2 commits intoalphapapa:masterfrom
Open
Allow org-ql-refile to handle values for org-refile-targets where the symbol in a given cons cell has a variable value#488apc wants to merge 2 commits intoalphapapa:masterfrom
org-ql-refile to handle values for org-refile-targets where the symbol in a given cons cell has a variable value#488apc wants to merge 2 commits intoalphapapa:masterfrom
Conversation
…efile-targets` As per the documentation of `org-refile-targets`, each specification of the files in a given cons cell can be a list of files "or a symbol whose function or variable value will be used..." This ensures `org-refile-targets` can handle the case where the specification is a symbol with a variable value.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This fixes #487. We simply add a test to make sure
files-spechas a function value before applyingfuncallto it, and handle the case wherefiles-spechas a variable value separately.