Moodle

Course Reset for Forum Posts

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.8
  • Fix Version/s: 1.9, 2.0
  • Component/s: Course
  • Labels:
    None
  • Affected Branches:
    MOODLE_18_STABLE
  • Fixed Branches:
    MOODLE_19_STABLE, MOODLE_20_STABLE

Description

Just tested out the course reset in 1.8 and everything seems to be working good except I found 1 bug:

Forum posts are deleted, however if you look at a forum listing it still says that there are discussion in that forum.

See images.

  1. forum changes.txt
    15/May/07 4:12 AM
    4 kB
    Lori Bakken
  1. after_reset.jpg
    338 kB
    24/Apr/07 1:53 AM
  2. after_reset2.jpg
    140 kB
    24/Apr/07 1:53 AM

Issue Links

Activity

Hide
Lori Bakken added a comment -

Added Comment:

The problem lies inn the course reset. It's deleting the posts related to the discussion, but not the actual discussions relate to the forum are not. Therefor screwing up the count number.

I can take a look at the course reset area and see if I can fix it myself, but I'd rather someone who coded this fix it and update in the cvs.

Show
Lori Bakken added a comment - Added Comment: The problem lies inn the course reset. It's deleting the posts related to the discussion, but not the actual discussions relate to the forum are not. Therefor screwing up the count number. I can take a look at the course reset area and see if I can fix it myself, but I'd rather someone who coded this fix it and update in the cvs.
Hide
Lori Bakken added a comment -

1.8 FIX

I have fixed this issue. See attached file for what I did.

It basically adds a reset for q and a forum type and removes posts as well as discussions so that the above post has the right count after reset.

My code may not be pretty – but it works.

Show
Lori Bakken added a comment - 1.8 FIX I have fixed this issue. See attached file for what I did. It basically adds a reset for q and a forum type and removes posts as well as discussions so that the above post has the right count after reset. My code may not be pretty – but it works.
Hide
Lori Bakken added a comment -

Forum 1.8 Course Reset Fix

Show
Lori Bakken added a comment - Forum 1.8 Course Reset Fix
Hide
Shamim Rezaie added a comment -

This issue and many other issues (for example numbers 6228, 9527, 6045, ...) are highly co-related.
I attached my implementation, which is the most complete/bug-free implementation till now, to issue #6228

Show
Shamim Rezaie added a comment - This issue and many other issues (for example numbers 6228, 9527, 6045, ...) are highly co-related. I attached my implementation, which is the most complete/bug-free implementation till now, to issue #6228
Hide
Petr Škoda (skodak) added a comment -

fixed in cvs - see MDL-7028
thanks for the report

Show
Petr Škoda (skodak) added a comment - fixed in cvs - see MDL-7028 thanks for the report

People

Vote (1)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: