From mboxrd@z Thu Jan 1 00:00:00 1970 From: Piyush Srivastava Subject: Bug: Bug in handling of named fields in the org spreadsheet [9.1.13 (release_9.1.13)] Date: Sun, 20 May 2018 00:24:47 +0530 Message-ID: <87y3gf4gso.fsf@tifr.res.in> Mime-Version: 1.0 Content-Type: text/plain Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:48928) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fK719-0001IF-Kk for emacs-orgmode@gnu.org; Sat, 19 May 2018 14:55:48 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fK716-0001rI-J7 for emacs-orgmode@gnu.org; Sat, 19 May 2018 14:55:47 -0400 Received: from mailhost.tifr.res.in ([2406:f00:1:1::6c]:33752) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1fK716-0001ZY-7j for emacs-orgmode@gnu.org; Sat, 19 May 2018 14:55:44 -0400 Received: from localhost (localhost.localdomain [127.0.0.1]) by mailhost.tifr.res.in (Postfix) with ESMTP id DF9784086 for ; Sun, 20 May 2018 00:24:55 +0530 (IST) Received: from mailhost.tifr.res.in ([127.0.0.1]) by localhost (mailhost.tifr.res.in [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id FJXpkps1xWV1 for ; Sun, 20 May 2018 00:24:50 +0530 (IST) Received: from vyom (unknown [158.144.106.131]) by mailhost.tifr.res.in (Postfix) with ESMTPSA id 8E9F3404B for ; Sun, 20 May 2018 00:24:50 +0530 (IST) List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org Sender: "Emacs-orgmode" To: emacs-orgmode@gnu.org 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 in 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 convention, pressing TAB in the row after making a change in the table does not update the field referred to by $sum, contrary to what is said in the manual. However, if I manually add a prefix $ to the variable sum in the #+TBLFM row (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 '$' should be prefixed to it. I have the following patch that 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)