* Bug: Table update fails for named fields [9.1.13 (release_9.1.13)]
@ 2018-05-20 14:39 Piyush Srivastava
0 siblings, 0 replies; 2+ messages in thread
From: Piyush Srivastava @ 2018-05-20 14:39 UTC (permalink / raw)
To: emacs-orgmode
In the current org mode version (release_9.1.13), when a named cell
formula is stored using C-u C-c =, the #+TBLFM line stores the name of
the cell without a prefix '$' sign, as in the following table (notice
that is the #+TBLFM row, there is no '$' sign before the 'sum'
variable):
| | c |
|---+-----|
| | 3 |
| | 10 |
|---+-----|
| # | 14 |
| ^ | sum |
|---+-----|
#+TBLFM: sum=vsum(@-II$0..@-I$0)
With this storage, pressing TAB in the row after changing some entries
does not update the filed referred to by 'sum', contrary to the
description in the manual.
However, if I manually prefix a '$; to the variable 'sum' as in the
table below, then TAB update work as described in the manual (notice the
$sum variable in the #+TBLFM row):
| | c |
|---+-----|
| | 3 |
| | 12 |
|---+-----|
| # | 15 |
| ^ | sum |
|---+-----|
#+TBLFM: $sum=vsum(@-II$0..@-I$0)
A fix is to ensure that if the LHS of a field formula being inserted
is a field name rather than a numerical reference, then a '$' is
prefixed to it. I have the following patch with implements this:
diff --git a/lisp/org-table.el b/lisp/org-table.el
--- a/lisp/org-table.el
+++ b/lisp/org-table.el
@@ -2301,7 +2301,13 @@ LOCATION instead."
(org-indent-line)
(insert (or (match-string 2) "#+TBLFM:")))
(insert " "
- (mapconcat (lambda (x) (concat (car x) "=" (cdr x)))
+ (mapconcat (lambda (x)
+ (let* ((name (car x))
+ (first (substring-no-properties name 0 1))
+ (test (or (string= first "@") (string= first "$")))
+ )
+ (concat (if test name (concat "$" name))
+ "=" (cdr x))))
(sort alist #'org-table-formula-less-p)
"::")
"\n"))))
What this patch does is the following: if the reference being inserted
does not begin with an '@' or a '$', then it is assumed to be a named
reference, and a '$' is prefixed to it. This will work *provided* no
named references themselves start with a '$'.
I am happy to assign copyright for the patch, if that is needed and if
the patch if found suitable, for it to be included in org-mode code.
Thanks,
-- Piyush.
Emacs : GNU Emacs 25.3.1 (x86_64-pc-linux-gnu, GTK+ Version 3.22.26)
of 2018-02-09
Package: Org mode version 9.1.13 (release_9.1.13)
^ permalink raw reply [flat|nested] 2+ messages in thread
* Bug: Table update fails for named fields [9.1.13 (release_9.1.13)]
@ 2018-05-20 14:40 Piyush Srivastava
0 siblings, 0 replies; 2+ messages in thread
From: Piyush Srivastava @ 2018-05-20 14:40 UTC (permalink / raw)
To: emacs-orgmode
In the current org mode version (release_9.1.13), when a named cell
formula is stored using C-u C-c =, the #+TBLFM line stores the name of
the cell without a prefix '$' sign, as in the following table (notice
that is the #+TBLFM row, there is no '$' sign before the 'sum'
variable):
| | c |
|---+-----|
| | 3 |
| | 10 |
|---+-----|
| # | 14 |
| ^ | sum |
|---+-----|
#+TBLFM: sum=vsum(@-II$0..@-I$0)
With this storage, pressing TAB in the row after changing some entries
does not update the filed referred to by 'sum', contrary to the
description in the manual.
However, if I manually prefix a '$; to the variable 'sum' as in the
table below, then TAB update work as described in the manual (notice the
$sum variable in the #+TBLFM row):
| | c |
|---+-----|
| | 3 |
| | 12 |
|---+-----|
| # | 15 |
| ^ | sum |
|---+-----|
#+TBLFM: $sum=vsum(@-II$0..@-I$0)
A fix is to ensure that if the LHS of a field formula being inserted
is a field name rather than a numerical reference, then a '$' is
prefixed to it. I have the following patch with implements this:
diff --git a/lisp/org-table.el b/lisp/org-table.el
--- a/lisp/org-table.el
+++ b/lisp/org-table.el
@@ -2301,7 +2301,13 @@ LOCATION instead."
(org-indent-line)
(insert (or (match-string 2) "#+TBLFM:")))
(insert " "
- (mapconcat (lambda (x) (concat (car x) "=" (cdr x)))
+ (mapconcat (lambda (x)
+ (let* ((name (car x))
+ (first (substring-no-properties name 0 1))
+ (test (or (string= first "@") (string= first "$")))
+ )
+ (concat (if test name (concat "$" name))
+ "=" (cdr x))))
(sort alist #'org-table-formula-less-p)
"::")
"\n"))))
What this patch does is the following: if the reference being inserted
does not begin with an '@' or a '$', then it is assumed to be a named
reference, and a '$' is prefixed to it. This will work *provided* no
named references themselves start with a '$'.
I am happy to assign copyright for the patch, if that is needed and if
the patch if found suitable, for it to be included in org-mode code.
Thanks,
-- Piyush.
Emacs : GNU Emacs 25.3.1 (x86_64-pc-linux-gnu, GTK+ Version 3.22.26)
of 2018-02-09
Package: Org mode version 9.1.13 (release_9.1.13)
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-05-20 14:40 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-05-20 14:39 Bug: Table update fails for named fields [9.1.13 (release_9.1.13)] Piyush Srivastava
-- strict thread matches above, loose matches on Subject: below --
2018-05-20 14:40 Piyush Srivastava
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.