Ticket #2681 (closed defect (bug): fixed)
Error on Inline Uploading after deleting page with attached file
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 2.0.6 |
| Component: | Administration | Version: | 2.1 |
| Severity: | normal | Keywords: | upload, attachment links, bg|has-patch, bg|dev-feedback |
| Cc: |
Description
OK, here's how to replicate:
1) Create a page with test content 2) Publish the page 3) Edit the page 3) Upload an image in the inline iframe when on the edit screen for that page 4) Click Manage->Pages 5) Delete the page 6) Click Write->Write Page 7) In the inline uploading iframe, click Browse All 8) You should see this error:
Warning: strtotime() [function.strtotime]: Called with an empty time parameter. in wp-includes\template-functions-links.php on line 53 Warning: date() [function.date]: Windows does not support dates prior to midnight (00:00:00), January 1, 1970 in wp-includes\template-functions-links.php on line 65
The get_attachment_link function is trying to use a parent post that doesn't exist. (template_function_links.php - around line 117)
SOLUTION: We should intentionally be orphaning attachments (update post_parent to 0) when deleting pages/posts. Otherwise, we just have bad data.
I'll submit a patch in a minute...
Attachments
Change History
ryanscheuermann — 6 years ago
-
attachment
orphan_attachments.patch
added
ryanscheuermann — 6 years ago
-
attachment
orphan_attachments2.patch
added
Patch update: when deleting posts, parent_post should inherit attachments
comment:1
ryanscheuermann — 6 years ago
- Keywords links, bg|has-patch, bg|dev-feedback added; links removed
- Owner changed from anonymous to ryanscheuermann
- Status changed from new to assigned
comment:2
markjaquith — 5 years ago
Confirmed still an issue.
This solution seems right to me. 2nd opinion?
comment:3
markjaquith — 5 years ago
- Status changed from assigned to closed
- Resolution set to fixed

orphan attachments after deleting post (update post_parent = 0)