Moodle

Can't delete an attachment from a forum post

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Won't Fix
  • Affects Version/s: 1.6
  • Fix Version/s: 2.0
  • Component/s: Forum
  • Labels:
    None
  • Environment:
    Linux
  • Affected Branches:
    MOODLE_16_STABLE
  • Fixed Branches:
    MOODLE_20_STABLE

Description

When one attaches a file to a forum post, there is a way to replace (by attaching another file), but there is no way to delete it (unless one goes into the moodle data directory - ordinary users can't do this of course).

We are currently doing an some activities where learners need to attach files to forum posts, but sometimes they do it on the wrong post. It would be great is a feature could that added that would allow the file to be deleted.

Issue Links

This issue will be resolved by:
MDL-8728 can't remove attachment from a glossary item Minor Closed
This issue duplicates:
MDL-16597 Formslib element: filemanager (JS) Minor Closed

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Martin Dougiamas added a comment -

From Darren Smith (dagza72 at gmail.com) Tuesday, 31 May 2005, 06:58 PM:

I would prefer the detetion period to be the same as the 'cooling off' editing time

From Timothy Allen (scop at hannam.ac.kr) Tuesday, 31 May 2005, 09:26 PM:

I agree that there should be a time limit during which the attachment can be deleted and after which it cannot, as this is consistent with the rule for editing posts.

Show
Martin Dougiamas added a comment - From Darren Smith (dagza72 at gmail.com) Tuesday, 31 May 2005, 06:58 PM: I would prefer the detetion period to be the same as the 'cooling off' editing time From Timothy Allen (scop at hannam.ac.kr) Tuesday, 31 May 2005, 09:26 PM: I agree that there should be a time limit during which the attachment can be deleted and after which it cannot, as this is consistent with the rule for editing posts.
Hide
Permalink
Martin Dougiamas added a comment -

Assigning to me temporarily because Vy-Shane no longer works for Moodle HQ.

Show
Martin Dougiamas added a comment - Assigning to me temporarily because Vy-Shane no longer works for Moodle HQ.
Hide
Permalink
Teresa Gibbison added a comment -

Is there any progress on this request? Is it possible for this to be an option for the teacher to set (allow attachment deletion)... I also think this should only be doable within the 30 minutes editing time and as long as someone hasn't replied.

Show
Teresa Gibbison added a comment - Is there any progress on this request? Is it possible for this to be an option for the teacher to set (allow attachment deletion)... I also think this should only be doable within the 30 minutes editing time and as long as someone hasn't replied.
Hide
Permalink
Teresa Gibbison added a comment -

Matt from Catalyst managed to enable us to delete attachments from a Dialogue (updated for 1.9) post via the edit link (I'm not sure if this will help here or not).

Show
Teresa Gibbison added a comment - Matt from Catalyst managed to enable us to delete attachments from a Dialogue (updated for 1.9) post via the edit link (I'm not sure if this will help here or not).
Hide
Permalink
Ann Adamcik added a comment -

This is still an issue in 1.8.x and 1.9.x.

Show
Ann Adamcik added a comment - This is still an issue in 1.8.x and 1.9.x.
Hide
Permalink
Mark Drechsler added a comment -

Just got off a help desk call about this problem - at least the info in the tracker meant I could say with confidence that it is a known issue

Show
Mark Drechsler added a comment - Just got off a help desk call about this problem - at least the info in the tracker meant I could say with confidence that it is a known issue
Hide
Permalink
Martin Dougiamas added a comment -

In Moodle 2.0 this is already fixed. Basically forum posts etc will now:

  • allow any number of attachments to be added
  • allow any of these to be deleted (during post time or on re-edit).
Show
Martin Dougiamas added a comment - In Moodle 2.0 this is already fixed. Basically forum posts etc will now:
  • allow any number of attachments to be added
  • allow any of these to be deleted (during post time or on re-edit).
Hide
Permalink
Dongsheng Cai added a comment -

will be resolved in 2.0

Show
Dongsheng Cai added a comment - will be resolved in 2.0
Hide
Permalink
Matteo Scaramuccia added a comment -

Rough patch proposal for 1.9(.7):

mod/forum/lib.php:

--- lib.php.orig        Mon Feb 08 15:22:38 2010
+++ lib.php     Mon Feb 08 15:25:32 2010
@@ -3965,6 +3965,10 @@
         unset($post->attachment);
     }

+    if (empty($post->attachment) && !empty($_POST['deleteattachment'])) {
+        forum_delete_old_attachments($post);
+    }
+
     if (forum_tp_can_track_forums($forum) && forum_tp_is_tracked($forum)) {
         forum_tp_mark_post_read($post->userid, $post, $post->forum);
     }

mod/forum/post_form.php:

--- post_form.php.orig  Sat Sep 26 17:25:48 2009
+++ post_form.php       Mon Feb 08 16:01:14 2010
@@ -61,7 +61,13 @@
         if ($forum->maxbytes != 1 && has_capability('mod/forum:createattachment', $modcontext))  {  //  1 = No attachments at all
             $mform->addElement('file', 'attachment', get_string('attachment', 'forum'));
             $mform->setHelpButton('attachment', array('attachment', get_string('attachment', 'forum'), 'forum'));
-
+            // Note: on update some properties are not evaluated e.g. attachment
+            if ($post->attachment) {
+                $mform->addElement('checkbox', 'deleteattachment', get_string('attachment', 'forum') . ' ⇒ ' . get_string('delete', 'forum'));
+                // Enhance the user experience
+                $mform->disabledIf('deleteattachment', 'attachment', 'ne', '');
+                $mform->disabledIf('attachment', 'deleteattachment', 'checked');
+            }
         }

         if (empty($post->id) && has_capability('moodle/course:manageactivities', $coursecontext)) { // hack alert
Show
Matteo Scaramuccia added a comment - Rough patch proposal for 1.9(.7): mod/forum/lib.php: 
--- lib.php.orig        Mon Feb 08 15:22:38 2010
+++ lib.php     Mon Feb 08 15:25:32 2010
@@ -3965,6 +3965,10 @@
         unset($post->attachment);
     }

+    if (empty($post->attachment) && !empty($_POST['deleteattachment'])) {
+        forum_delete_old_attachments($post);
+    }
+
     if (forum_tp_can_track_forums($forum) && forum_tp_is_tracked($forum)) {
         forum_tp_mark_post_read($post->userid, $post, $post->forum);
     }
mod/forum/post_form.php: 
--- post_form.php.orig  Sat Sep 26 17:25:48 2009
+++ post_form.php       Mon Feb 08 16:01:14 2010
@@ -61,7 +61,13 @@
         if ($forum->maxbytes != 1 && has_capability('mod/forum:createattachment', $modcontext))  {  //  1 = No attachments at all
             $mform->addElement('file', 'attachment', get_string('attachment', 'forum'));
             $mform->setHelpButton('attachment', array('attachment', get_string('attachment', 'forum'), 'forum'));
-
+            // Note: on update some properties are not evaluated e.g. attachment
+            if ($post->attachment) {
+                $mform->addElement('checkbox', 'deleteattachment', get_string('attachment', 'forum') . ' ⇒ ' . get_string('delete', 'forum'));
+                // Enhance the user experience
+                $mform->disabledIf('deleteattachment', 'attachment', 'ne', '');
+                $mform->disabledIf('attachment', 'deleteattachment', 'checked');
+            }
         }

         if (empty($post->id) && has_capability('moodle/course:manageactivities', $coursecontext)) { // hack alert

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.5#665-sha1:422aead) | Report a problem
Powered by a free Atlassian JIRA open source license for Moodle. Try JIRA - bug tracking software for your team.