Hello, Aaron Ecay <aarone...@gmail.com> writes:
> The docstring of org-store-link-functions says: “Each function will be > called in turn until one returns a non-nil value.” However, the code of > org-store-link actually calls all the functions, and if more than one > returns a non-nil value raises a prompt asking the user to choose (and > then calls that function a second time). > > Should we change the code to match the doc, or vice versa? The former, IMO. > Aaron > > PS I vote for keeping the semantics described in the doc, because the > current implementation’s double calling of the selected function > means that store-link functions that have side effects will behave > unpredictably. Imagine a store-link function for image buffers that > uploads the buffer to imgur or a similar hosting site, and stores the > http url of the uploaded file. The current implementation would > upload the image twice. I agree. Regards, -- Nicolas Goaziou P.S: next release is AFAIK "8.4", not "9.0", you may want to update your recent changes advertising the latter. Thank you.