Re: [O] Patch org-capture-set-target-location

[Roberto Huelga]

Sorry for make you lose your time.

Using the package version of 20141124 of org, the bug is present, but in
the git master, it's solved in a nicer way. By in the
org-capture-place-entry function getting out of the cond the test of
:exact-position

Package version:

(cond
 ((org-capture-get :exact-position)
  (goto-char (org-capture-get :exact-position)))
 ((not target-entry-p)

Git Master version:

(and (org-capture-get :exact-position)
 (goto-char (org-capture-get :exact-position)))
(cond
 ((not target-entry-p)

Thank you very much.


On Sun, Nov 23, 2014 at 7:15 PM, Nicolas Goaziou 
wrote:

> Hello,
>
> Roberto Huelga  writes:
>
> > Patch proposal for org-capture-set-target-location
>
> Thanks for the patch.
>
> > The idea is that when somebody use the function or file+function targets
> > you can make the template subheading from the heading where the
> > custom-function set the point.
> > org-capture-set-target-loction force the org-capture-plist
> :exact-position
> > to (point).
> > So the template is inserted in the same level not as subheading
> >
> > With my patch the custom-function can (org-capture-put :exact-position
> > nil), and now if point its placed in a heading the template is created as
> > his subheading.
>
> AFAIU, :exact-position is an internal property (it isn't documented in
> (info "(org) Template elements"). You are not expected to set it
> directly.
>
> After a cursory look into "org-capture.el", if (point) is on a headline,
> the template should create a subheading (see `org-capture-place-entry').
> Can't you use that instead?
>
>
> Regards,
>
> --
> Nicolas Goaziou
>

[O] Patch org-capture-set-target-location

[Roberto Huelga]

Patch proposal for org-capture-set-target-location

The idea is that when somebody use the function or file+function targets
you can make the template subheading from the heading where the
custom-function set the point.
org-capture-set-target-loction force the org-capture-plist :exact-position
to (point).
So the template is inserted in the same level not as subheading

With my patch the custom-function can (org-capture-put :exact-position
nil), and now if point its placed in a heading the template is created as
his subheading.

For a use case can look at the following emacs.stackexchange question

http://emacs.stackexchange.com/questions/3750/org-capture-files-entry-under-wrong-datetree/3765#3765

Thanks.

Roberto Huelga.


0001-org-capture.el-set-exact-position-only-if-not-set.patch
Description: Binary data

Re: [O] Patch org-capure-fill-template

[Roberto Huelga]

I founded a section in the org-contribute.html with the names of the
tiny change contributors,
Do you know how could I get my name there? This is my second
tinychange patch. And
get my name in the list will make my day.

Thanks

2014-11-02 10:44 GMT+01:00 Nicolas Goaziou :
> Roberto Huelga  writes:
>
>> Suppose you want to write the parent directory name in your capture so write 
>> a
>> template like
>>
>> "* %(file-name-nondirectory (directory-file-name (file-name-directory 
>> \"%F\")))"
>>
>> Without the patch, file-name-directory get the useless string "%F"
>> instead of a string
>> with the full path name of the file.
>>
>> The orgmode manual template expansion page describe
>>
>>  %(sexp) Evaluate Elisp sexp and replace with the result.
>>For convenience, %:keyword (see below) placeholders
>>within the expression will be expanded prior to this.
>>The sexp must return a string.
>>
>> But without the patch is not true, first is expanded %(sexp) and after
>> that the %OneLetter and later the %:keyword
>
> Understood. Applied, thank you.
>
> Regards,

Re: [O] Patch org-capure-fill-template

[Roberto Huelga]

Suppose you want to write the parent directory name in your capture so write a
template like

"* %(file-name-nondirectory (directory-file-name (file-name-directory \"%F\")))"

Without the patch, file-name-directory get the useless string "%F"
instead of a string
with the full path name of the file.

The orgmode manual template expansion page describe

 %(sexp) Evaluate Elisp sexp and replace with the result.
   For convenience, %:keyword (see below) placeholders
   within the expression will be expanded prior to this.
   The sexp must return a string.

But without the patch is not true, first is expanded %(sexp) and after
that the %OneLetter and later the %:keyword

Thanks for your work at the awesome orgmode.


2014-10-30 16:48 GMT+01:00 Nicolas Goaziou :
> Hello,
>
> Roberto Huelga  writes:
>
>> Patch proposal fro org-capture-fill-template
>
> Thanks for your patch. However, I'd like to know your motivation for it
> as I fail to see how it is superior to the current design.
>
>
> Regards,
>
> --
> Nicolas Goaziou

[O] Patch org-capure-fill-template

[Roberto Huelga]

Patch proposal fro org-capture-fill-template

Hope it's useful.
Thanks.


0001-org-capture.el-Fix-expand-template-order.patch
Description: Binary data

[O] Patch org-timer-set-timer

[Roberto Huelga]

Patch proposal for org-timer-set-timer

Thanks for your great work, and I hope be sending all in the correct format.


0001-Timer-play-a-sound-as-configured-by-org-clock-sound-.patch
Description: Binary data

[O] org-program-exists not working on osx

[Roberto Huelga]

Here is a simple patch for the org-clock.el file to make the function
org-program-exists work on osx


org-clock.el.diff
Description: Binary data