Re: [O] Update tables 8.3.1

[Rasmus]

Hi Scott,

Scott Randby <address@hidden> writes:

> I'm beginning to wonder if upgrading to 8.3.1 is a mistake. I had no 
> trouble at all with 8.2.10, but many problems with 8.3.1. My latest 
> issue follows.

We appreciate your bug reports.  If you have got the time, please keep
testing Org 8.3 and let us know about errors you find.

> When I update a table (using C-u C-c *, or C-u C-c C-c, or M-x 
> org-table-recalculate), the table doesn't update and I get the following 
> error message:
>
>    Wrong type argument: number-or-marker-p, (4)

I see this as well.

> I can provide example tables if necessary.

The example below is sufficient. 

> I tried the example table given in section 3.5.10 of the documentation 
> and I get the same error.
>
> |---+---------+--------+--------+--------+-------+------|
> |   | Student | Prob 1 | Prob 2 | Prob 3 | Total | Note |
> |---+---------+--------+--------+--------+-------+------|
> | ! |         |     P1 |     P2 |     P3 |   Tot |      |
> | # | Maximum |     10 |     15 |     25 |    50 | 10.0 |
> | ^ |         |     m1 |     m2 |     m3 |    mt |      |
> |---+---------+--------+--------+--------+-------+------|
> | # | Peter   |     10 |      8 |     23 |    41 |  8.2 |
> | # | Sam     |      2 |      4 |      3 |     9 |  1.8 |
> |---+---------+--------+--------+--------+-------+------|
> |   | Average |        |        |        |  25.0 |      |
> | ^ |         |        |        |        |    at |      |
> | $ | max=50  |        |        |        |       |      |
> |---+---------+--------+--------+--------+-------+------|
>
> #+TBLFM: 
> $6=vsum($P1..$P3)::$7=10*$Tot/$max;%.1f::$at=vmean(@address@hidden);%.1f
>
> Error message: Wrong type argument: number-or-marker-p, (6)
>
> I will appreciate any help with this issue. If it can't be fixed, I will 
> have to downgrade to 8.2.10.

Can you try the attached patch?  It seems to work on the example from the
manual, but you likely have more straining examples at hand.

Thanks,
Rasmus

-- 
Enough with the bla bla!

>From 5fbcbffea86757e068a866643f3e7470dae1e10f Mon Sep 17 00:00:00 2001
From: Rasmus <address@hidden>
Date: Sat, 8 Aug 2015 23:11:39 +0200
Subject: [PATCH 1/5] org-table: Fix typo

* org-table.el (org-table-goto-field): Fix typo.
---
 lisp/org-table.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lisp/org-table.el b/lisp/org-table.el
index 62de402..ef6427c 100644
--- a/lisp/org-table.el
+++ b/lisp/org-table.el
@@ -2567,7 +2567,7 @@ This function assumes the table is already analyzed 
(i.e., using
                  (string-to-number (match-string 2 ref))))
           (t (user-error "Unknown field: %s" ref))))
         (line (car coordinates))
-        (column (cdr coordinates))
+        (column (cadr coordinates))
         (create-new-column (if (functionp create-column-p)
                                (funcall create-column-p column)
                              create-column-p)))
-- 
2.5.0