Re: [PATCH] Add %L (the link content *not* as a full link) to Capture expansions

[Kyle Meyer]

Samuel W. Flint writes:

>> Bastien  writes:
>
> B> Can you provide a patch to announce this in etc/ORG-NEWS?
>
> Please see the attached patch.

Thanks.  Applied (488076e0a).

Re: [PATCH] Add %L (the link content *not* as a full link) to Capture expansions

[Samuel W. Flint]

> Bastien  writes:

B> Hi Samuel, "Samuel W. Flint"  writes:

>> Marked as TINYCHANGE.

B> Applied as d06aa486d, thanks!

Thanks!

B> Can you provide a patch to announce this in etc/ORG-NEWS?

Please see the attached patch.

B> -- Bastien

Sam

-- 
Samuel W. Flint
4096R/FA13D704
  (F50D 862B 4F65 5943 A8C2  EF0E 86C9 3E7A FA13 D704)
λs.(s s) λs.(s s)
>From b1242c2af3d5ab16ddcec52d3b3f47bb60895d9a Mon Sep 17 00:00:00 2001
From: "Samuel W. Flint" 
Date: Thu, 24 Sep 2020 17:57:32 -0500
Subject: [PATCH] Describe new %L formatting for org-capture

* etc/ORG-NEWS (New formatting directive ~%L~ for org-capture):

Describe new %L formatting for org-capture

TINYCHANGE
---
 etc/ORG-NEWS | 6 ++
 1 file changed, 6 insertions(+)

diff --git a/etc/ORG-NEWS b/etc/ORG-NEWS
index eed3871d4..f036aa6a5 100644
--- a/etc/ORG-NEWS
+++ b/etc/ORG-NEWS
@@ -11,6 +11,12 @@ See the end of the file for license conditions.
 Please send Org bug reports to mailto:emacs-orgmode@gnu.org.
 
 * Version 9.5 (not yet released)
+** New options and settings
+*** New formatting directive ~%L~ for org-capture
+
+The new ~%L~ formatting directive contains the bare link target, and
+may be used to create links with programmatically generated
+descriptions.
 * Version 9.4
 ** Incompatible changes
 *** Possibly broken internal file links: please check and fix
-- 
2.18.1

Re: [PATCH] Add %L (the link content *not* as a full link) to Capture expansions

[Bastien]

Hi Samuel,

"Samuel W. Flint"  writes:

> Marked as TINYCHANGE.

Applied as d06aa486d, thanks!

Can you provide a patch to announce this in etc/ORG-NEWS?

-- 
 Bastien

Re: [PATCH] Add %L (the link content *not* as a full link) to Capture expansions

[Bastien]

Hi Samuel,

"Samuel W. Flint"  writes:

> Is there anything I can do to move this forward?

You just need to be patient.  Either I will have some free time
at some point to help with this, either someone else will.

But the contribution is not forgotten, don't worry.

-- 
 Bastien

Re: [PATCH] Add %L (the link content *not* as a full link) to Capture expansions

[Samuel W. Flint]

Is there anything I can do to move this forward?

Thanks,

Sam


> Samuel W Flint writes:

SWF> Marked as TINYCHANGE.  Sam


> Samuel W Flint writes:

> Bastien  writes:
B> Hi Samuel, swfl...@flintfam.org (Samuel W. Flint) writes:

 In an attempt at automating a part of my workflow, I found that
 org-capture's %l didn't quite fit what I needed.  Normally,
 org-store-link does the right thing, but for some capture
 tasks, I want to custom generate the description, so %l doesn't
 work.  I've attached a patch that adds %L, a version that
 doesn't add the link brackets.

B> It looks good to me for Org 9.5.  Can you add a Changelog entry
B> to your patch and also patch the tests and the manual, if needed?

SWF> Apologies for spamming you with multiple copies Bastien.

SWF> Certainly!  Hopefully this looks better.  I didn't see any
SWF> tests for %a, %A or %l already, and I don't have time atm to
SWF> write them, otherwise I'd have done so.

SWF> I can mark this as TINYCHANGE, if necessary, as I don't have a
SWF> copyright assignment on file, though I'm willing to do so.

SWF> Thanks,

SWF> Sam


B> See
B> 
B> for the format of the Changelog.

B> Thanks,

B> -- Bastien

SWF> -- Samuel W. Flint 4096R/FA13D704 (F50D 862B 4F65 5943 A8C2
SWF> EF0E 86C9 3E7A FA13 D704) λs.(s s) λs.(s s)



SWF> -- Samuel W. Flint 4096R/FA13D704 (F50D 862B 4F65 5943 A8C2
SWF> EF0E 86C9 3E7A FA13 D704) λs.(s s) λs.(s s)



-- 
Samuel W. Flint
4096R/FA13D704
  (F50D 862B 4F65 5943 A8C2  EF0E 86C9 3E7A FA13 D704)
λs.(s s) λs.(s s)

Re: [PATCH] Add %L (the link content *not* as a full link) to Capture expansions

[Samuel W. Flint]

Marked as TINYCHANGE.

Sam


> Samuel W Flint writes:

> Bastien  writes:
B> Hi Samuel, swfl...@flintfam.org (Samuel W. Flint) writes:

>>> In an attempt at automating a part of my workflow, I found that
>>> org-capture's %l didn't quite fit what I needed.  Normally,
>>> org-store-link does the right thing, but for some capture tasks,
>>> I want to custom generate the description, so %l doesn't work.
>>> I've attached a patch that adds %L, a version that doesn't add
>>> the link brackets.

B> It looks good to me for Org 9.5.  Can you add a Changelog entry
B> to your patch and also patch the tests and the manual, if needed?

SWF> Apologies for spamming you with multiple copies Bastien.

SWF> Certainly!  Hopefully this looks better.  I didn't see any
SWF> tests for %a, %A or %l already, and I don't have time atm to
SWF> write them, otherwise I'd have done so.

SWF> I can mark this as TINYCHANGE, if necessary, as I don't have a
SWF> copyright assignment on file, though I'm willing to do so.

SWF> Thanks,

SWF> Sam


B> See
B> 
B> for the format of the Changelog.

B> Thanks,

B> -- Bastien

SWF> -- Samuel W. Flint 4096R/FA13D704 (F50D 862B 4F65 5943 A8C2
SWF> EF0E 86C9 3E7A FA13 D704) λs.(s s) λs.(s s)



-- 
Samuel W. Flint
4096R/FA13D704
  (F50D 862B 4F65 5943 A8C2  EF0E 86C9 3E7A FA13 D704)
λs.(s s) λs.(s s)
>From 02de43c3457419f1031f4988d6c95fac535134e7 Mon Sep 17 00:00:00 2001
From: "Samuel W. Flint" 
Date: Mon, 14 Sep 2020 09:07:07 -0500
Subject: [PATCH] Add in support for filling in the bare link in org capture

* doc/org-manual.org: Document new %L capture template formatting
directive.
* lisp/org-capture.el: (org-capture-templates) Document new %L capture
template formatting directive.
* lisp/org-capture.el: (org-capture-fill-template) Add in support for
%L, bare link formatting, in org-capture-fill-template.

TINYCHANGE
---
 doc/org-manual.org  | 4 
 lisp/org-capture.el | 7 ++-
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/doc/org-manual.org b/doc/org-manual.org
index 7ab7d1c94..229575b13 100644
--- a/doc/org-manual.org
+++ b/doc/org-manual.org
@@ -7799,6 +7799,10 @@ here:
 
   Like =%a=, but only insert the literal link.
 
+- =%L= ::
+
+  Like =%l=, but without brackets (the link content itself).
+
 - =%c= ::
 
   Current kill ring head.
diff --git a/lisp/org-capture.el b/lisp/org-capture.el
index d73e927fc..2ef55cd5c 100644
--- a/lisp/org-capture.el
+++ b/lisp/org-capture.el
@@ -315,6 +315,7 @@ be replaced with content and expanded:
   %a  Annotation, normally the link created with `org-store-link'.
   %A  Like %a, but prompt for the description part.
   %l  Like %a, but only insert the literal link.
+  %L  Like %l, but without brackets (the link content itself).
   %c  Current kill ring head.
   %x  Content of the X clipboard.
   %k  Title of currently clocked task.
@@ -1592,6 +1593,9 @@ The template may still contain \"%?\" for cursor positioning."
 	 (v-l (if (and v-a (string-match l-re v-a))
 		  (replace-match "[[\\1]]" nil nil v-a)
 		v-a))
+	 (v-L (if (or v-a (string-match l-re v-a))
+		  (replace-match "\\1" nil nil v-a)
+		v-a))
 	 (v-n user-full-name)
 	 (v-k (if (marker-buffer org-clock-marker)
 		  (org-no-properties org-clock-heading)
@@ -1644,7 +1648,7 @@ The template may still contain \"%?\" for cursor positioning."
   ;; Mark %() embedded elisp for later evaluation.
   (org-capture-expand-embedded-elisp 'mark)
   ;; Expand non-interactive templates.
-  (let ((regexp "%\\(:[-A-Za-z]+\\|<\\([^>\n]+\\)>\\|[aAcfFikKlntTuUx]\\)"))
+  (let ((regexp "%\\(:[-A-Za-z]+\\|<\\([^>\n]+\\)>\\|[aAcfFikKlLntTuUx]\\)"))
 	(save-excursion
 	  (while (re-search-forward regexp nil t)
 	;; `org-capture-escaped-%' may modify buffer and cripple
@@ -1681,6 +1685,7 @@ The template may still contain \"%?\" for cursor positioning."
 			  (?k v-k)
 			  (?K v-K)
 			  (?l v-l)
+			  (?L v-L)
 			  (?n v-n)
 			  (?t v-t)
 			  (?T v-T)
-- 
2.18.1

Re: [PATCH] Add %L (the link content *not* as a full link) to Capture expansions

[Samuel W. Flint]

> Bastien  writes:

B> Hi Samuel, swfl...@flintfam.org (Samuel W. Flint) writes:

>> In an attempt at automating a part of my workflow, I found that
>> org-capture's %l didn't quite fit what I needed.  Normally,
>> org-store-link does the right thing, but for some capture tasks,
>> I want to custom generate the description, so %l doesn't work.
>> I've attached a patch that adds %L, a version that doesn't add
>> the link brackets.

B> It looks good to me for Org 9.5.  Can you add a Changelog entry
B> to your patch and also patch the tests and the manual, if needed?

Apologies for spamming you with multiple copies Bastien.

Certainly!  Hopefully this looks better.  I didn't see any tests for %a,
%A or %l already, and I don't have time atm to write them, otherwise I'd
have done so.

I can mark this as TINYCHANGE, if necessary, as I don't have a copyright
assignment on file, though I'm willing to do so.

Thanks,

Sam


B> See
B> 
B> for the format of the Changelog.

B> Thanks,

B> -- Bastien

-- 
Samuel W. Flint
4096R/FA13D704
  (F50D 862B 4F65 5943 A8C2  EF0E 86C9 3E7A FA13 D704)
λs.(s s) λs.(s s)
>From 02de43c3457419f1031f4988d6c95fac535134e7 Mon Sep 17 00:00:00 2001
From: "Samuel W. Flint" 
Date: Mon, 14 Sep 2020 09:07:07 -0500
Subject: [PATCH] Add in support for filling in the bare link in org capture

* doc/org-manual.org: Document new %L capture template formatting
directive.
* lisp/org-capture.el: (org-capture-templates) Document new %L capture
template formatting directive.
* lisp/org-capture.el: (org-capture-fill-template) Add in support for
%L, bare link formatting, in org-capture-fill-template.
---
 doc/org-manual.org  | 4 
 lisp/org-capture.el | 7 ++-
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/doc/org-manual.org b/doc/org-manual.org
index 7ab7d1c94..229575b13 100644
--- a/doc/org-manual.org
+++ b/doc/org-manual.org
@@ -7799,6 +7799,10 @@ here:
 
   Like =%a=, but only insert the literal link.
 
+- =%L= ::
+
+  Like =%l=, but without brackets (the link content itself).
+
 - =%c= ::
 
   Current kill ring head.
diff --git a/lisp/org-capture.el b/lisp/org-capture.el
index d73e927fc..2ef55cd5c 100644
--- a/lisp/org-capture.el
+++ b/lisp/org-capture.el
@@ -315,6 +315,7 @@ be replaced with content and expanded:
   %a  Annotation, normally the link created with `org-store-link'.
   %A  Like %a, but prompt for the description part.
   %l  Like %a, but only insert the literal link.
+  %L  Like %l, but without brackets (the link content itself).
   %c  Current kill ring head.
   %x  Content of the X clipboard.
   %k  Title of currently clocked task.
@@ -1592,6 +1593,9 @@ The template may still contain \"%?\" for cursor positioning."
 	 (v-l (if (and v-a (string-match l-re v-a))
 		  (replace-match "[[\\1]]" nil nil v-a)
 		v-a))
+	 (v-L (if (or v-a (string-match l-re v-a))
+		  (replace-match "\\1" nil nil v-a)
+		v-a))
 	 (v-n user-full-name)
 	 (v-k (if (marker-buffer org-clock-marker)
 		  (org-no-properties org-clock-heading)
@@ -1644,7 +1648,7 @@ The template may still contain \"%?\" for cursor positioning."
   ;; Mark %() embedded elisp for later evaluation.
   (org-capture-expand-embedded-elisp 'mark)
   ;; Expand non-interactive templates.
-  (let ((regexp "%\\(:[-A-Za-z]+\\|<\\([^>\n]+\\)>\\|[aAcfFikKlntTuUx]\\)"))
+  (let ((regexp "%\\(:[-A-Za-z]+\\|<\\([^>\n]+\\)>\\|[aAcfFikKlLntTuUx]\\)"))
 	(save-excursion
 	  (while (re-search-forward regexp nil t)
 	;; `org-capture-escaped-%' may modify buffer and cripple
@@ -1681,6 +1685,7 @@ The template may still contain \"%?\" for cursor positioning."
 			  (?k v-k)
 			  (?K v-K)
 			  (?l v-l)
+			  (?L v-L)
 			  (?n v-n)
 			  (?t v-t)
 			  (?T v-T)
-- 
2.18.1

Re: [PATCH] Add %L (the link content *not* as a full link) to Capture expansions

[Bastien]

Hi Samuel,

swfl...@flintfam.org (Samuel W. Flint) writes:

> In an attempt at automating a part of my workflow, I found that
> org-capture's %l didn't quite fit what I needed.  Normally,
> org-store-link does the right thing, but for some capture tasks, I want
> to custom generate the description, so %l doesn't work.  I've attached a
> patch that adds %L, a version that doesn't add the link brackets.

It looks good to me for Org 9.5.  Can you add a Changelog entry to
your patch and also patch the tests and the manual, if needed?

See  
for the format of the Changelog.

Thanks,

-- 
 Bastien

[PATCH] Add %L (the link content *not* as a full link) to Capture expansions

[Samuel W. Flint]

Hello,

In an attempt at automating a part of my workflow, I found that
org-capture's %l didn't quite fit what I needed.  Normally,
org-store-link does the right thing, but for some capture tasks, I want
to custom generate the description, so %l doesn't work.  I've attached a
patch that adds %L, a version that doesn't add the link brackets.

Thanks,

Sam Flint

-- 
Samuel W. Flint
4096R/FA13D704
  (F50D 862B 4F65 5943 A8C2  EF0E 86C9 3E7A FA13 D704)

diff --git i/lisp/org-capture.el w/lisp/org-capture.el
index 2cc1ce394..e638eabfa 100644
--- i/lisp/org-capture.el
+++ w/lisp/org-capture.el
@@ -308,6 +308,7 @@ be replaced with content and expanded:
   %a  Annotation, normally the link created with `org-store-link'.
   %A  Like %a, but prompt for the description part.
   %l  Like %a, but only insert the literal link.
+  %L  Like %l, but no brackets (the link content itself).
   %c  Current kill ring head.
   %x  Content of the X clipboard.
   %k  Title of currently clocked task.
@@ -1574,6 +1575,9 @@ The template may still contain \"%?\" for cursor positioning."
 	 (v-l (if (and v-a (string-match l-re v-a))
 		  (replace-match "[[\\1]]" nil nil v-a)
 		v-a))
+	 (v-L (if (or v-a (string-match l-re v-a))
+		  (replace-match "\\1" nil nil v-a)
+		v-a))
 	 (v-n user-full-name)
 	 (v-k (if (marker-buffer org-clock-marker)
 		  (org-no-properties org-clock-heading)
@@ -1626,7 +1630,7 @@ The template may still contain \"%?\" for cursor positioning."
   ;; Mark %() embedded elisp for later evaluation.
   (org-capture-expand-embedded-elisp 'mark)
   ;; Expand non-interactive templates.
-  (let ((regexp "%\\(:[-A-Za-z]+\\|<\\([^>\n]+\\)>\\|[aAcfFikKlntTuUx]\\)"))
+  (let ((regexp "%\\(:[-A-Za-z]+\\|<\\([^>\n]+\\)>\\|[aAcfFikKlLntTuUx]\\)"))
 	(save-excursion
 	  (while (re-search-forward regexp nil t)
 	;; `org-capture-escaped-%' may modify buffer and cripple
@@ -1663,6 +1667,7 @@ The template may still contain \"%?\" for cursor positioning."
 			  (?k v-k)
 			  (?K v-K)
 			  (?l v-l)
+			  (?L v-L)
 			  (?n v-n)
 			  (?t v-t)
 			  (?T v-T)