Project

General

Profile

Bug #2038

By removing lp, the items are not deleted

Added by Andre Boivin about 9 years ago. Updated almost 9 years ago.

Status:
Feature implemented
Priority:
Normal
Category:
-
Target version:
Start date:
21/09/2010
Due date:
% Done:

100%

Estimated time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

By removing lp, the items are not deleted. This prevents the editing of quizz that were in the lp.
I found the solution by changing the line 902 in learnpath.class.php
enable search == false

But I do not understand the reason for this condition

History

#1

Updated by Ivan Tcholakov almost 9 years ago

Current revision of the file learnpath.class.php: 13335:7644d7c675b6

I seems to me that the line 903:

        // TODO: Also delete items and item-views.

is related to this topic.

#2

Updated by Ivan Tcholakov almost 9 years ago

  • Status changed from New to Assigned
  • Assignee set to Ivan Tcholakov
#3

Updated by Ivan Tcholakov almost 9 years ago

  • Status changed from Assigned to Needs more info
  • % Done changed from 0 to 80

I tried to reproduce what has been described, it is not observed now.
On my testing system (current development code) when a LP is deleted its items are deleted too.
If a deleted LP-item is test-type, then the corresponded test is unblocked for modifications.

I looked at the code carefully - there are commands for deletion in TABLE_LP_ITEM and TABLE_LP_ITEM_VIEW.
There was an obsolete TODO which I removed.

13434:11972add706f Bug #2038 - Removing a misleading TODO that has been implemented in the past (learnpath.class.php, line 903).
http://code.google.com/p/chamilo/source/detail?r=11972add706f07d09a2fce1146614c194bff2299&repo=classic

There was a suggestion from Christophe, nickname:clefevre more than a year ago:
http://www.dokeos.com/forum/viewtopic.php?t=29673

Ok for closing this task?

#4

Updated by Andre Boivin almost 9 years ago

you're right about the version 1.8.7.1 the bug has been resolved

#5

Updated by Ivan Tcholakov almost 9 years ago

  • Status changed from Needs more info to Feature implemented
  • % Done changed from 80 to 100

Also available in: Atom PDF