sagemath/sagenb — pull request #40 by jasongrout (2012-02-16 18:15:43+00:00)
Title: Fix the user history log

It turns out that the problem is two extra quote marks, accidentally added last May when making some massive changes:

https://github.com/sagemath/sagenb/commit/28c23623d8c64bb981a5cce9462e4bc0f1d9ad91#L4L617

It also exposed a problem with the queue list for cells (look in the communication with the server and you'll see an extra request for cell status, cells are queued up *twice* for checking, once when the evaluate cell request is sent, and once upon successful response of the evaluate request:

https://github.com/sagemath/sagenb/blob/master/sagenb/data/sage/js/notebook_lib.js#L3005

and

https://github.com/sagemath/sagenb/blob/master/sagenb/data/sage/js/notebook_lib.js#L3133

Which one of these should update the queue_id_list? @rkirov or @mhansen, do you remember if there is a problem for the cell to listed as running upon successful return of the eval request (i.e., delete the two lines in this pull request)?