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.

Sign up for GitHub

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

Jump to bottom

Data file deletion in notebook throws errors #8234

Closed
rbeezer mannequin opened this issue Feb 10, 2010 · 11 comments
Closed

Data file deletion in notebook throws errors #8234

rbeezer mannequin opened this issue Feb 10, 2010 · 11 comments
Assignees
@williamstein
Labels
c: user interface p: blocker / 1 t: bug
Milestone
sage-4.5

Comments

@rbeezer
Copy link
Mannequin

rbeezer mannequin commented Feb 10, 2010

Delete a file in the data directory of the notebook and you get a page back about an "Internal Server Error" (by selecting the file in the drop-down box, and clicking on the "delete" hyperlink). File is actually deleted, though.

This is a follow-up to #6475.

The patch is at #6069.

CC: @robert-marik

Component: notebook

Author: Tim Dumol, Mitesh Patel

Reviewer: Mitesh Patel, Tim Dumol

Merged: sagenb-0.8

Issue created by migration from https://trac.sagemath.org/ticket/8234
@rbeezer rbeezer mannequin added this to the sage-4.4 milestone Feb 10, 2010
@rbeezer rbeezer mannequin assigned williamstein Feb 10, 2010
@qed777
Copy link
Mannequin

qed777 mannequin commented Feb 14, 2010

comment:1

#6069 should fix this. If it doesn't, please let me know.

@qed777
Copy link
Mannequin

qed777 mannequin commented Feb 14, 2010

comment:2

But we still need to fix the page title ("Error -- Sage"), which is wrong.

@qed777
Copy link
Mannequin

qed777 mannequin commented Feb 14, 2010

comment:3

Replying to @qed777:

But we still need to fix the page title ("Error -- Sage"), which is wrong.

See V3 at #6069.

@qed777 qed777 mannequin added p: blocker / 1 and removed p: major / 3 labels Feb 14, 2010
@qed777

This comment has been minimized.

Sign in to view
@qed777

This comment has been minimized.

Sign in to view
@qed777 qed777 mannequin added the s: needs review label Feb 14, 2010
@qed777
Copy link
Mannequin

qed777 mannequin commented Mar 12, 2010

comment:7

When #6069 is merged, we should close this ticket.

@TimDumol
Copy link
Mannequin

TimDumol mannequin commented Mar 15, 2010

Author: Tim Dumol, Mitesh Patel

@TimDumol
Copy link
Mannequin

TimDumol mannequin commented Mar 15, 2010

Reviewer: Mitesh Patel, Tim Dumol

@TimDumol
Copy link
Mannequin

TimDumol mannequin commented Mar 15, 2010

comment:8

As #6069 is marked as positive review, I'll do the same for this.

@jhpalmieri
Copy link
Member

comment:9

(I'm not changing any notebook code in Sage 4.4.)

@jhpalmieri jhpalmieri modified the milestones: sage-4.4, sage-5.0 Apr 23, 2010
@TimDumol
Copy link
Mannequin

TimDumol mannequin commented May 4, 2010

Merged: sagenb-0.8

@TimDumol TimDumol mannequin removed the s: positive review label May 4, 2010
@TimDumol TimDumol mannequin closed this as completed May 4, 2010
@sagetrac-mvngu sagetrac-mvngu mannequin modified the milestones: sage-5.0, sage-4.5 Jun 25, 2010
@qed777 qed777 mannequin mentioned this issue Feb 14, 2010
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@williamstein williamstein

Labels
c: user interface p: blocker / 1 t: bug
Projects
None yet
Milestone
sage-4.5
Development

No branches or pull requests
2 participants
@williamstein @jhpalmieri