Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Notebook error page after deleting data file #6475

Closed
sagetrac-khorton mannequin opened this issue Jul 7, 2009 · 7 comments
Closed

Notebook error page after deleting data file #6475

sagetrac-khorton mannequin opened this issue Jul 7, 2009 · 7 comments

Comments

@sagetrac-khorton
Copy link
Mannequin

sagetrac-khorton mannequin commented Jul 7, 2009

If I delete a data file using the notebook (Data... pop up menu -> file name -> delete file name link), the file is deleted, but the browser then goes to a blank page titled "Error | Sage Notebook". The browser is Safari 4 on OS X 10.5, in case that makes a difference.

William Stein reported on Sage-Support:

I've seen this. This definitely didn't used to happen. Somebody introduced this bug a few months ago. Please report it to trac. I'll fix it in September/October when I work on the notebook, if nobody beats me to it.

CC: @williamstein @sagetrac-acleone @qed777

Component: notebook

Keywords: notebook delete file error

Author: Tim Dumol

Reviewer: Alex Leone

Merged: sagenb-0.7

Issue created by migration from https://trac.sagemath.org/ticket/6475

@sagetrac-khorton sagetrac-khorton mannequin added this to the sage-4.3.2 milestone Jul 7, 2009
@qed777
Copy link
Mannequin

qed777 mannequin commented Dec 14, 2009

comment:1

Is this still a problem, in 4.3.rc0?

@TimDumol
Copy link
Mannequin

TimDumol mannequin commented Jan 19, 2010

Changes the title of the successful delete page

@TimDumol
Copy link
Mannequin

TimDumol mannequin commented Jan 19, 2010

comment:2

Attachment: trac_6475-error-delete-data-file.patch.gz

@TimDumol
Copy link
Mannequin

TimDumol mannequin commented Jan 19, 2010

Author: Tim Dumol

@TimDumol TimDumol mannequin added the s: needs review label Jan 19, 2010
@sagetrac-acleone
Copy link
Mannequin

sagetrac-acleone mannequin commented Jan 19, 2010

Reviewer: Alex Leone

@sagetrac-acleone
Copy link
Mannequin

sagetrac-acleone mannequin commented Jan 19, 2010

comment:3

LGTM.

@qed777
Copy link
Mannequin

qed777 mannequin commented Jan 25, 2010

Merged: sagenb-0.7

@qed777 qed777 mannequin removed the s: positive review label Jan 25, 2010
@qed777 qed777 mannequin closed this as completed Jan 25, 2010
@qed777 qed777 mannequin mentioned this issue Feb 10, 2010
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant