Bug: clocktable :inherit-props does not respect global property setting [9.4.6 (9.4.6-12-gdcc3a8-elpa @ /home/jet/.config/emacs/elpa/org-20210823/)]

Bug: clocktable :inherit-props does not respect global property setting [9.4.6 (9.4.6-12-gdcc3a8-elpa @ /home/jet/.config/emacs/elpa/org-20210823/)]

[Jeff Trull]

[-- Attachment #1: Type: text/plain, Size: 10049 bytes --]

:inherit-props seems to not consider global properties.

Testcase:

# Create a global property

#+PROPERTY: HOURLY_RATE 150

* Top 1
** Sub 1A
# here we should get the global property, but do not
   :LOGBOOK:
   CLOCK: [2021-09-26 Sun 12:25]--[2021-09-26 Sun 13:25] =>  1:00
   - I did some tasks
   :END:

** Sub 1B
* Top 2
  :PROPERTIES:
  :HOURLY_RATE: 110
  :END:
** Sub 2A
# successfully inherit from "Top 2"
   :LOGBOOK:
   CLOCK: [2021-09-26 Sun 21:26]--[2021-09-26 Sun 22:26] =>  1:00
   - I did some other tasks
   :END:

** Sub 2B
   :PROPERTIES:
   :HOURLY_RATE: 90
   :END:
# override Top 2
   :LOGBOOK:
   CLOCK: [2021-09-26 Sun 18:30]--[2021-09-26 Sun 19:00] =>  0:30
   - Attended a meeting
   :END:

#+BEGIN: clocktable :scope file :maxlevel 2 :properties ("HOURLY_RATE")
:inherit-props t
#+CAPTION: Clock summary at [2021-09-26 Sun 22:48]
| HOURLY_RATE | Headline     |   Time |      |
|-------------+--------------+--------+------|
|             | *Total time* | *2:30* |      |
|-------------+--------------+--------+------|
|             | Top 1        |   1:00 |      |
|             | \_  Sub 1A   |        | 1:00 |
|         110 | Top 2        |   1:30 |      |
|         110 | \_  Sub 2A   |        | 1:00 |
|          90 | \_  Sub 2B   |        | 0:30 |
#+END:

I expect that "Sub 1A" will show the global HOURLY_RATE of 150, but it is
empty.

Emacs  : GNU Emacs 27.1.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version
2.24.32, cairo version 1.16.0)
 of 2020-09-01
Package: Org mode version 9.4.6 (9.4.6-12-gdcc3a8-elpa @
/home/jet/.config/emacs/elpa/org-20210823/)

current state:
==============
(setq
 org-duration-format 'h:mm
 org-src-mode-hook '(org-src-babel-configure-edit-buffer
org-src-mode-configure-edit-buffer)
 org-latex-listings t
 org-link-shell-confirm-function 'yes-or-no-p
 org-metadown-hook '(org-babel-pop-to-session-maybe)
 org-export-global-macros '((LASTMONTH .
                             "(eval (format-time-string \"%B %Y\"
(encode-time (decoded-time-add (decode-time) (make-decoded-time :month
-1)))))")
                            (NET30 .
                             "(eval (format-time-string \"%m/%d/%Y\"
(encode-time (decoded-time-add (decode-time) (make-decoded-time :day
30)))))")
                            (LASTOFLASTMONTH .
                             "(eval (format-time-string \"%m/%d/%Y\"
(encode-time (let ((d (decode-time))) (setf (decoded-time-day d) 1)
(decoded-time-add d (make-decoded-time :day -1))))))")
                            (FIRSTOFLASTMONTH .
                             "(eval (format-time-string \"%m/%d/%Y\"
(encode-time (let ((d (decoded-time-add (decode-time) (make-decoded-time
:month -1)))) (setf (decoded-time-day d) 1) d))))")
                            )
 org-clock-out-hook '(org-clock-remove-empty-clock-drawer)
 org-html-format-inlinetask-function
'org-html-format-inlinetask-default-function
 org-log-note-clock-out t
 org-ascii-format-inlinetask-function 'org-ascii-format-inlinetask-default
 org-mode-hook '(#[0 "\301\211 \207" [imenu-create-index-function
org-imenu-get-tree] 2]
                 org-bullets-mode org-tempo-setup (lambda nil
(electric-indent-local-mode -1))
                 #[0 "\300\301\302\303\304$\207"
                   [add-hook change-major-mode-hook org-show-all append
local] 5]
                 #[0 "\300\301\302\303\304$\207"
                   [add-hook change-major-mode-hook
org-babel-show-result-all append local] 5]
                 org-babel-result-hide-spec org-babel-hide-all-hashes)
 org-archive-hook '(org-attach-archive-delete-maybe)
 org-confirm-elisp-link-function 'yes-or-no-p
 org-agenda-before-write-hook '(org-agenda-add-entry-text)
 org-metaup-hook '(org-babel-load-in-session-maybe)
 org-bibtex-headline-format-function #[257 "\300 \236A\207" [:title] 3
"\n\n(fn ENTRY)"]
 org-latex-format-drawer-function #[514 "\207" [] 3 "\n\n(fn _ CONTENTS)"]
 org-babel-pre-tangle-hook '(save-buffer)
 org-tab-first-hook '(org-babel-hide-result-toggle-maybe
org-babel-header-arg-expand)
 org-babel-load-languages '((emacs-lisp . t) (dot . t) (ditaa . t))
 org-export-backends '(ascii html icalendar latex md confluence re-reveal)
 org-ascii-format-drawer-function #[771 " \207" [] 4 "\n\n(fn NAME CONTENTS
WIDTH)"]
 org-agenda-loop-over-headlines-in-active-region nil
 org-occur-hook '(org-first-headline-recenter)
 org-ditaa-jar-path "/usr/share/ditaa/ditaa.jar"
 org-structure-template-alist '(("n" . "notes") ("a" . "export ascii") ("c"
. "center")
                                ("C" . "comment") ("e" . "example") ("E" .
"export")
                                ("h" . "export html") ("l" . "export
latex") ("q" . "quote")
                                ("s" . "src") ("v" . "verse"))
 org-cycle-hook '(org-cycle-hide-archived-subtrees org-cycle-hide-drawers
                  org-cycle-show-empty-lines
org-optimize-window-after-visibility-change)
 org-speed-command-hook '(org-speed-command-activate
org-babel-speed-command-activate)
 org-export-before-parsing-hook '(org-attach-expand-links)
 org-confirm-shell-link-function 'yes-or-no-p
 org-link-parameters '(("attachment" :follow org-attach-follow :complete
org-attach-complete-link)
                       ("id" :follow org-id-open)
                       ("eww" :follow org-eww-open :store
org-eww-store-link)
                       ("rmail" :follow org-rmail-open :store
org-rmail-store-link)
                       ("mhe" :follow org-mhe-open :store
org-mhe-store-link)
                       ("irc" :follow org-irc-visit :store
org-irc-store-link :export
                        org-irc-export)
                       ("info" :follow org-info-open :export
org-info-export :store
                        org-info-store-link)
                       ("gnus" :follow org-gnus-open :store
org-gnus-store-link)
                       ("docview" :follow org-docview-open :export
org-docview-export :store
                        org-docview-store-link)
                       ("bibtex" :follow org-bibtex-open :store
org-bibtex-store-link)
                       ("bbdb" :follow org-bbdb-open :export
org-bbdb-export :complete
                        org-bbdb-complete-link :store org-bbdb-store-link)
                       ("w3m" :store org-w3m-store-link)
                       ("treemacs" :store treemacs-store-org-link)
("file+sys") ("file+emacs")
                       ("shell" :follow org-link--open-shell)
                       ("news" :follow
                        #[514 "\301\300\302 Q \"\207" ["news" browse-url
":"] 6
                          "\n\n(fn URL ARG)"]
                        )
                       ("mailto" :follow
                        #[514 "\301\300\302 Q \"\207" ["mailto" browse-url
":"] 6
                          "\n\n(fn URL ARG)"]
                        )
                       ("https" :follow
                        #[514 "\301\300\302 Q \"\207" ["https" browse-url
":"] 6
                          "\n\n(fn URL ARG)"]
                        )
                       ("http" :follow
                        #[514 "\301\300\302 Q \"\207" ["http" browse-url
":"] 6
                          "\n\n(fn URL ARG)"]
                        )
                       ("ftp" :follow
                        #[514 "\301\300\302 Q \"\207" ["ftp" browse-url
":"] 6 "\n\n(fn URL ARG)"]
                        )
                       ("help" :follow org-link--open-help)
                       ("file" :complete org-link-complete-file)
                       ("elisp" :follow org-link--open-elisp) ("doi"
:follow org-link--open-doi))
 org-latex-format-headline-function
'org-latex-format-headline-default-function
 org-re-reveal-root "https://cdn.jsdelivr.net/npm/reveal.js@3.6.0"
 org-link-elisp-confirm-function 'yes-or-no-p
 org-latex-format-inlinetask-function
'org-latex-format-inlinetask-default-function
 org-tab-before-tab-emulation-hook '(org-tempo-complete-tag)
 org-html-format-drawer-function #[514 "\207" [] 3 "\n\n(fn NAME CONTENTS)"]
 org-latex-packages-alist '(("" "moresize") ("" "fancyvrb") (""
"inconsolata") ("" "tikz")
                            ("" "color") ("" "listings"))
 org-latex-classes '(("beamer"
                      "\\documentclass[11pt,aspectratio=169]{beamer}\n
             \\usenavigationsymbolstemplate{} % no navigation bar\n
         \\beamertemplateballitem         % items have little balls"
                      ("\\section{%s}" . "\\section*{%s}")
                      ("\\subsection{%s}" . "\\subsection*{%s}")
                      ("\\subsubsection{%s}" . "\\subsubsection*{%s}"))
                     ("article" "\\documentclass[11pt]{article}"
                      ("\\section{%s}" . "\\section*{%s}")
                      ("\\subsection{%s}" . "\\subsection*{%s}")
                      ("\\subsubsection{%s}" . "\\subsubsection*{%s}")
                      ("\\paragraph{%s}" . "\\paragraph*{%s}")
                      ("\\subparagraph{%s}" . "\\subparagraph*{%s}"))
                     ("report" "\\documentclass[11pt]{report}"
("\\part{%s}" . "\\part*{%s}")
                      ("\\chapter{%s}" . "\\chapter*{%s}") ("\\section{%s}"
. "\\section*{%s}")
                      ("\\subsection{%s}" . "\\subsection*{%s}")
                      ("\\subsubsection{%s}" . "\\subsubsection*{%s}"))
                     ("book" "\\documentclass[11pt]{book}" ("\\part{%s}" .
"\\part*{%s}")
                      ("\\chapter{%s}" . "\\chapter*{%s}") ("\\section{%s}"
. "\\section*{%s}")
                      ("\\subsection{%s}" . "\\subsection*{%s}")
                      ("\\subsubsection{%s}" . "\\subsubsection*{%s}"))
                     )
 org-html-format-headline-function
'org-html-format-headline-default-function
 org-ditaa-eps-jar-path
"/home/jet/.config/emacs/elpa/contrib/scripts/DitaaEps.jar"
 org-confirm-babel-evaluate nil
 org-latex-listings-options '(("basicstyle" "\\footnotesize\\ttfamily")
                              ("showstringspaces" "false") ("breaklines"
"true"))
 org-re-reveal-title-slide "<h2>%t</h2><h2>%a</h2><h2>%d</h2>"
 )

[-- Attachment #2: Type: text/html, Size: 13634 bytes --]

Re: Bug: clocktable :inherit-props does not respect global property setting [9.4.6 (9.4.6-12-gdcc3a8-elpa @ /home/jet/.config/emacs/elpa/org-20210823/)]

[Ihor Radchenko]

Jeff Trull <edaskel@att.net> writes:

> :inherit-props seems to not consider global properties.
>
> Testcase:

I am unable to reproduce on the latest main.
The inherited properties are displayed as expected on my side.

-- 
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>

Re: Bug: clocktable :inherit-props does not respect global property setting [9.4.6 (9.4.6-12-gdcc3a8-elpa @ /home/jet/.config/emacs/elpa/org-20210823/)]

[Jeff Trull]

[-- Attachment #1: Type: text/plain, Size: 1367 bytes --]

I just confirmed that it is still doing this (Emacs 28.2, org 9.5.5). The
result looks the same as before:

#+BEGIN: clocktable :scope file :maxlevel 2 :properties ("HOURLY_RATE")
:inherit-props t
#+CAPTION: Clock summary at [2022-10-17 Mon 16:50]
| HOURLY_RATE | Headline     |   Time |      |
|-------------+--------------+--------+------|
|             | *Total time* | *2:30* |      |
|-------------+--------------+--------+------|
|             | Top 1        |   1:00 |      |
|             | \_  Sub 1A   |        | 1:00 |
|         110 | Top 2        |   1:30 |      |
|         110 | \_  Sub 2A   |        | 1:00 |
|          90 | \_  Sub 2B   |        | 0:30 |
#+END:

Note that the first column next to "Sub 1A" is empty.

Can you send the result of evaluating the clocktable in your config?

Thanks,
Jeff


On Sun, Oct 16, 2022 at 4:00 AM Ihor Radchenko <yantar92@posteo.net> wrote:

> Jeff Trull <edaskel@att.net> writes:
>
> > :inherit-props seems to not consider global properties.
> >
> > Testcase:
>
> I am unable to reproduce on the latest main.
> The inherited properties are displayed as expected on my side.
>
> --
> Ihor Radchenko // yantar92,
> Org mode contributor,
> Learn more about Org mode at <https://orgmode.org/>.
> Support Org development at <https://liberapay.com/org-mode>,
> or support my work at <https://liberapay.com/yantar92>
>

[-- Attachment #2: Type: text/html, Size: 2254 bytes --]

Re: Bug: clocktable :inherit-props does not respect global property setting [9.4.6 (9.4.6-12-gdcc3a8-elpa @ /home/jet/.config/emacs/elpa/org-20210823/)]

[Ihor Radchenko]

Jeff Trull <edaskel@att.net> writes:

> I just confirmed that it is still doing this (Emacs 28.2, org 9.5.5). The
> result looks the same as before:
>
> #+BEGIN: clocktable :scope file :maxlevel 2 :properties ("HOURLY_RATE")
> :inherit-props t
> #+CAPTION: Clock summary at [2022-10-17 Mon 16:50]
> ...
> Note that the first column next to "Sub 1A" is empty.

Do note that :inherit-props t must be on the same line with #+BEGIN

> Can you send the result of evaluating the clocktable in your config?

#+BEGIN: clocktable :scope file :maxlevel 2 :properties ("HOURLY_RATE") :inherit-props t
#+CAPTION: Clock summary at [2022-10-18 Tue 12:47]
| HOURLY_RATE | Headline   | Time     |          |
|-------------+------------+----------+----------|
|             | *Total time* | *2h 30min* |          |
|-------------+------------+----------+----------|
|         150 | Top 1      | 1h 0min  |          |
|         150 | \_  Sub 1A |          | 1h 0min  |
|         110 | Top 2      | 1h 30min |          |
|         110 | \_  Sub 2A |          | 1h 0min  |
|          90 | \_  Sub 2B |          | 0h 30min |
#+END:


-- 
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>

Re: Bug: clocktable :inherit-props does not respect global property setting [9.4.6 (9.4.6-12-gdcc3a8-elpa @ /home/jet/.config/emacs/elpa/org-20210823/)]

[Jeff Trull]

[-- Attachment #1: Type: text/plain, Size: 2084 bytes --]

Yes, I can make it work now - but only after reloading it at least one time
from disk.

When I type it in for the first time, it doesn't work. I have to save, then
(revert-buffer).
In other words, in a fresh buffer, paste all the above, then hit C-c C-c on
the BEGIN line.
The global properties don't work. You can save the file, and it still
doesn't work. Only after
reading it from disk (either with revert-buffer or loading it for the first
time) does it work.

Also, if you change the value (say, from 150 to 210) the old value persists
until you
reload the file. Even saving the file doesn't help.

On Mon, Oct 17, 2022 at 9:48 PM Ihor Radchenko <yantar92@posteo.net> wrote:

> Jeff Trull <edaskel@att.net> writes:
>
> > I just confirmed that it is still doing this (Emacs 28.2, org 9.5.5). The
> > result looks the same as before:
> >
> > #+BEGIN: clocktable :scope file :maxlevel 2 :properties ("HOURLY_RATE")
> > :inherit-props t
> > #+CAPTION: Clock summary at [2022-10-17 Mon 16:50]
> > ...
> > Note that the first column next to "Sub 1A" is empty.
>
> Do note that :inherit-props t must be on the same line with #+BEGIN
>
> > Can you send the result of evaluating the clocktable in your config?
>
> #+BEGIN: clocktable :scope file :maxlevel 2 :properties ("HOURLY_RATE")
> :inherit-props t
> #+CAPTION: Clock summary at [2022-10-18 Tue 12:47]
> | HOURLY_RATE | Headline   | Time     |          |
> |-------------+------------+----------+----------|
> |             | *Total time* | *2h 30min* |          |
> |-------------+------------+----------+----------|
> |         150 | Top 1      | 1h 0min  |          |
> |         150 | \_  Sub 1A |          | 1h 0min  |
> |         110 | Top 2      | 1h 30min |          |
> |         110 | \_  Sub 2A |          | 1h 0min  |
> |          90 | \_  Sub 2B |          | 0h 30min |
> #+END:
>
>
> --
> Ihor Radchenko // yantar92,
> Org mode contributor,
> Learn more about Org mode at <https://orgmode.org/>.
> Support Org development at <https://liberapay.com/org-mode>,
> or support my work at <https://liberapay.com/yantar92>
>

[-- Attachment #2: Type: text/html, Size: 3017 bytes --]

Re: Bug: clocktable :inherit-props does not respect global property setting [9.4.6 (9.4.6-12-gdcc3a8-elpa @ /home/jet/.config/emacs/elpa/org-20210823/)]

[Ihor Radchenko]

Jeff Trull <edaskel@att.net> writes:

> Yes, I can make it work now - but only after reloading it at least one time
> from disk.
>
> When I type it in for the first time, it doesn't work. I have to save, then
> (revert-buffer).
> ...
> Also, if you change the value (say, from 150 to 210) the old value persists
> until you
> reload the file. Even saving the file doesn't help.

It is expected and documented.
See 17.8 Summary of In-Buffer Settings section of the manual.
Basically, you need to press C-c C-c on the property line to make Org
refresh the settings.

Alternatively, you can use document property drawer. It is updated
automatically on the latest development version of Org.

-- 
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>

Re: Bug: clocktable :inherit-props does not respect global property setting [9.4.6 (9.4.6-12-gdcc3a8-elpa @ /home/jet/.config/emacs/elpa/org-20210823/)]

[Jeff Trull]

[-- Attachment #1: Type: text/plain, Size: 1466 bytes --]

OK, got it now, thanks! I confirmed everything you said.

My takeaway is that instead of using global properties as a single property
at the top level,
I should create a top-level properties drawer, which will have immediate
usability.

In the case of my example that would be:

C-u C-c C-x d HOURLY_RATE <enter> 150 <enter>

and then HOURLY_RATE is immediately usable in my table, without even having
to save the file.

On Mon, Oct 17, 2022 at 11:03 PM Ihor Radchenko <yantar92@posteo.net> wrote:

> Jeff Trull <edaskel@att.net> writes:
>
> > Yes, I can make it work now - but only after reloading it at least one
> time
> > from disk.
> >
> > When I type it in for the first time, it doesn't work. I have to save,
> then
> > (revert-buffer).
> > ...
> > Also, if you change the value (say, from 150 to 210) the old value
> persists
> > until you
> > reload the file. Even saving the file doesn't help.
>
> It is expected and documented.
> See 17.8 Summary of In-Buffer Settings section of the manual.
> Basically, you need to press C-c C-c on the property line to make Org
> refresh the settings.
>
> Alternatively, you can use document property drawer. It is updated
> automatically on the latest development version of Org.
>
> --
> Ihor Radchenko // yantar92,
> Org mode contributor,
> Learn more about Org mode at <https://orgmode.org/>.
> Support Org development at <https://liberapay.com/org-mode>,
> or support my work at <https://liberapay.com/yantar92>
>

[-- Attachment #2: Type: text/html, Size: 2274 bytes --]