eniotna <eniotna.t@gmail.com> writes:

> Hello,
>
>> I'm sure, it's only a small error of mine.
>
> Replace `(mapcar #'list (org-entry-properties nil nil))` by
> `(org-entry-properties nil nil)` in your `org-read-entry-property-name`
> definition:
>
> ```
> (defun org-read-entry-property-name ()
>   "Read a property name from the current entry (basically a rewrite of
>    org-read-property-name which uses org-entry-properties instead to speed
> up)"
>   (let ((completion-ignore-case t)
>         (default-prop (or (and (org-at-property-p)
>                                (org-match-string-no-properties 2))
>                           org-last-set-property)))
>     (org-completing-read
>      (concat "Property"
>              (if default-prop (concat " [" default-prop "]") "")
>              ": ")
>      (org-entry-properties nil nil)
>      nil nil nil nil default-prop)))
> ```

Also, for information you could use format function.
It clarifies the prompt computation ^^:

```
(defun org-read-entry-property-name ()
  "Read a property name from the current entry."
  (let ((completion-ignore-case t)
        (default-prop (or (and (org-at-property-p)
                               (org-match-string-no-properties 2))
                          org-last-set-property)))
    (org-completing-read
     (format "Property [%s]: " (if default-prop default-prop ""))
     (org-entry-properties nil nil)
     nil nil nil nil default-prop)))
```

>
> Regards,
>
> tony / @ardumont
>
> On Tue, Apr 12, 2016 at 3:55 PM, Karl Voit <devnull@karl-voit.at> wrote:
>
>> * Thierry Banel <tbanelwebmin@free.fr> wrote:
>> > Nice function!
>> > Maybe a rewrite of (org-read-property-name) calling
>> > (org-entry-properties) could speed up things.
>> > (org-entry-properties) returns an assoc list of the local header
>> properties.
>>
>> OK, great idea.
>>
>> I tried by myself (Elisp noob) and failed: see code below.
>>
>> The code evaluates without complaining but when I use the function,
>> I get asked for a property which gets also correctly written to the
>> property. However, the completion of the property does not work.
>>
>> I'm sure, it's only a small error of mine.
>>
>> Thanks for your help here!
>>
>>
>> ;; version that tries to read only properties of current entry:
>> (defun org-read-entry-property-name ()
>>   "Read a property name from the current entry (basically a rewrite of
>>    org-read-property-name which uses org-entry-properties instead to speed
>> up)"
>>   (let ((completion-ignore-case t)
>>     (default-prop (or (and (org-at-property-p)
>>                    (org-match-string-no-properties 2))
>>               org-last-set-property)))
>>     (org-completing-read
>>      (concat "Property"
>>          (if default-prop (concat " [" default-prop "]") "")
>>          ": ")
>>      ;; ORIGINAL LINE: (mapcar #'list (org-buffer-property-keys nil t t))
>>      (mapcar #'list (org-entry-properties nil nil))
>>      nil nil nil nil default-prop)))
>>
>> (defun my-org-region-to-property (&optional property)
>>   (interactive)
>>   ;; if no region is defined, do nothing
>>   (if (use-region-p)
>>       ;; if a region string is found, ask for a property and set property
>> to
>>       ;; the string in the region
>>       (let ((val (replace-regexp-in-string
>>                   "\\`[ \t\n]*" ""
>>                   (replace-regexp-in-string "[ \t\n]*\\'" ""
>>                                             (substring (buffer-string)
>>                                                        (-
>> (region-beginning) 1)
>>                                                        (region-end))))
>>                  )
>>             ;; if none was stated by user, read property from user
>>             (prop (or property
>>                       (org-read-entry-property-name))))
>>         ;; set property
>>         (org-set-property prop val))))
>>
>>
>> --
>> mail|git|SVN|photos|postings|SMS|phonecalls|RSS|CSV|XML to Org-mode:
>>        > get Memacs from https://github.com/novoid/Memacs <
>>
>> https://github.com/novoid/extract_pdf_annotations_to_orgmode + more on
>> github
>>
>>
>>

Cheers,
--
tony / @ardumont

-----------------------------------------------------------------
gpg fingerprint BF00 203D 741A C9D5 46A8 BE07 52E2 E984 0D10 C3B8