Project

General

Profile

Bug #4546

LP: detailed stats allow for other attempts

Added by Yannick Warnier over 8 years ago. Updated about 8 years ago.

Status:
Bug resolved
Priority:
Urgent
Assignee:
Category:
-
Target version:
Start date:
23/03/2012
Due date:
% Done:

100%

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

Description

This one is both urgent and critical: in Chamilo 1.8.8.4, when looking at the detailed tracking of a learning path (where you can see the learning paths attempts), you have the possibility to see the results of any exercise that was included in the learning path. Apparently clicking this link then gives you access to the exercises tool, in which you can then see the exercise, even if hidden. This can then be used to pass the exercise more than once and breaks the whole tracking system. Apparently, it is used by some students to pass over the limitation of number of attempts!!!

Associated revisions

Revision 75f4f42c (diff)
Added by Julio Montoya about 8 years ago

Fixing exercise result display when doing a exercise inside a LP see #4546

Revision 7e90dd22 (diff)
Added by Julio Montoya about 8 years ago

Hiding an exercise if we added it in a LP see #4546

Revision 2654a197 (diff)
Added by Julio Montoya about 8 years ago

Removing tag + adding description lang variable already added see #4546

History

#1

Updated by Julio Montoya over 8 years ago

  • Status changed from Assigned to Needs more info
  • Assignee changed from Julio Montoya to Yannick Warnier
  • % Done changed from 0 to 10

I think this is already fixed at least in Chamilo 1.9. Yes, I have access to the exercise tool but if an exercise is hidden I can't do it.

#2

Updated by Julio Montoya over 8 years ago

  • % Done changed from 10 to 80
#3

Updated by Yannick Warnier about 8 years ago

  • Status changed from Needs more info to Assigned
  • Assignee changed from Yannick Warnier to Julio Montoya
  • % Done changed from 80 to 10

This is still the case. The full procedure is (long):

  • create a test with a question
  • set it as one single attempt allowed
  • create a LP
  • insert a document
  • insert the one-single-attempt exercise
  • subcribe a student to the course
  • log as the student
  • enter the course, lp and test
  • click the stats icon
  • click the + icon next to the test
  • click the "correction" icon next to the test (extended list that just appeared)
  • click the breadcrumb (name of the test)
  • click "Start test"
  • there you go, you can take it again :-)
#4

Updated by Julio Montoya about 8 years ago

  • Status changed from Assigned to Needs more info
  • Assignee deleted (Julio Montoya)

I think this was already fixed if the exercise is hidden you will see a message and you can't pass the exam.

If the exercise is visible you can take the exam, a feature could be to make invisible an exercise when adding to a LP by default ... what do you think?

#5

Updated by Julio Montoya about 8 years ago

  • Assignee set to Yannick Warnier
#6

Updated by Yannick Warnier about 8 years ago

  • Status changed from Needs more info to Assigned
  • Assignee changed from Yannick Warnier to Julio Montoya
  • Target version changed from 1.9 Beta to 1.9 RC1

Julio Montoya wrote:

I think this was already fixed if the exercise is hidden you will see a message and you can't pass the exam.

If the exercise is visible you can take the exam, a feature could be to make invisible an exercise when adding to a LP by default ... what do you think?

I don't understand why a single-attempt exercise should be possible to take more than once. This is where the bug is.
I understand a multi-attempts exercise could be taken this way, but not a single-attempt one. The check on the single attempt is probably only chcking the track_e_attempts table while the attempt has been registered in an lp_* table, or something like that. The check should be on both tables.

And finally, when seeing the stats from the learning path report screen and sending the user to the exercises comment screen, there should be an orig=lp to allow us to remove the breadcrumb (or put another one) to avoid "loosing" the user in the exercises when he started in the LP.

Yeah, that's a bit complicated to read, but I'm sure you understand what I mean :-)

I'm moving this to RC1. Beta tasks should only be the ones that involve database changes, for now.

#7

Updated by Julio Montoya about 8 years ago

Yannick Warnier wrote:

Julio Montoya wrote:

I think this was already fixed if the exercise is hidden you will see a message and you can't pass the exam.

If the exercise is visible you can take the exam, a feature could be to make invisible an exercise when adding to a LP by default ... what do you think?

I don't understand why a single-attempt exercise should be possible to take more than once. This is where the bug is.
I understand a multi-attempts exercise could be taken this way, but not a single-attempt one. The check on the single attempt is probably only chcking the track_e_attempts table while the attempt has been registered in an lp_* table, or something like that. The check should be on both tables.

That's a feature that was never implemented.

And finally, when seeing the stats from the learning path report screen and sending the user to the exercises comment screen, there should be an orig=lp to allow us to remove the breadcrumb (or put another one) to avoid "loosing" the user in the exercises when he started in the LP.

oka

Yeah, that's a bit complicated to read, but I'm sure you understand what I mean :-)

I'm moving this to RC1. Beta tasks should only be the ones that involve database changes, for now.

#8

Updated by Yannick Warnier about 8 years ago

Julio Montoya wrote:

Yannick Warnier wrote:

I don't understand why a single-attempt exercise should be possible to take more than once. This is where the bug is.
I understand a multi-attempts exercise could be taken this way, but not a single-attempt one. The check on the single attempt is probably only chcking the track_e_attempts table while the attempt has been registered in an lp_* table, or something like that. The check should be on both tables.

That's a feature that was never implemented.

OK, but you do agree that allowing students to take an exam multiple time can be a problem, right? :-)
I'm OK with it being a new feature to be implemented. It's just that it's difficult to explain to a professional customer that we "ignored" this missing feature, so that the single-attempt exercise is in fact flawed :-/

#9

Updated by Julio Montoya about 8 years ago

Yannick Warnier wrote:

Julio Montoya wrote:

Yannick Warnier wrote:

I don't understand why a single-attempt exercise should be possible to take more than once. This is where the bug is.
I understand a multi-attempts exercise could be taken this way, but not a single-attempt one. The check on the single attempt is probably only chcking the track_e_attempts table while the attempt has been registered in an lp_* table, or something like that. The check should be on both tables.

That's a feature that was never implemented.

OK, but you do agree that allowing students to take an exam multiple time can be a problem, right? :-)

yes of course, I agree if you select 1 attempt it should be 1 attempt not 2 :)

I'm OK with it being a new feature to be implemented. It's just that it's difficult to explain to a professional customer that we "ignored" this missing feature, so that the single-attempt exercise is in fact flawed :-/

got it

#10

Updated by Julio Montoya about 8 years ago

  • Status changed from Assigned to Needs more info
  • Assignee changed from Julio Montoya to Yannick Warnier

Well this means merging the exercise results + exercise LP results. I could add a tag to say that one exercise was done inside a LP or not ...
this will generate extra work and confusion to the users ...

I think the most fast and easy fix will be to hide the exercise if you already added in a LP, this way users can't make 2 attempts if the exercise was set to 1.

I agree that 1 attempt should be 1 attempt so one exercise can be published in a LP or in a exercise not in both ... what do you think?

#11

Updated by Yannick Warnier about 8 years ago

The problem is that if it's not limited in number of attempts, it is likely that it would be used on both sides. This being said, if we can copy a whole exercise (inside the same course), the problem disappears if we use the method you suggest...

#12

Updated by Yannick Warnier about 8 years ago

Technically copying an entire exercises should really not be a big deal now, as there is already a procedure to copy questions...

#13

Updated by Julio Montoya about 8 years ago

You suggest to copy the entire exercise when adding to the LP?

If yes, where the teacher will edit the exercises? In the LP tool? In the exercise tool?
Is like creating a new tool so the teacher can go to the "exercise" and "exercise in LP" tool ?

#14

Updated by Yannick Warnier about 8 years ago

  • Status changed from Needs more info to Assigned
  • Assignee changed from Yannick Warnier to Julio Montoya

No no, I suggest, as you said before, that if an exercise is included in any learning path, it should not appear to the students through the exercises tool.
The teacher will see it appear as invisible with a comment saying "This exercise has been included in a learning path, so it cannot be accessed by students directly from here. If you want to put the same exercise available through the exercises tool, please make a copy of the current exercise using the copy icon."

Is that OK for you?

#15

Updated by Julio Montoya about 8 years ago

Yannick Warnier wrote:

No no, I suggest, as you said before, that if an exercise is included in any learning path, it should not appear to the students through the exercises tool.
The teacher will see it appear as invisible with a comment saying "This exercise has been included in a learning path, so it cannot be accessed by students directly from here. If you want to put the same exercise available through the exercises tool, please make a copy of the current exercise using the copy icon."

Is that OK for you?

yeap

#16

Updated by Julio Montoya about 8 years ago

  • Status changed from Assigned to Needs more info
  • Assignee deleted (Julio Montoya)
  • % Done changed from 10 to 80

The message will appear when you pass the mouse in the "eye" icon. Message to long to add it as a tag.

#17

Updated by Yannick Warnier about 8 years ago

Can the message not be shown as a "Description"? (light grey, below the exercise name, like in the learning paths when the date has passed)
I think it would be much clearer...

#18

Updated by Julio Montoya about 8 years ago

  • % Done changed from 80 to 90

added

#19

Updated by Yannick Warnier about 8 years ago

  • Status changed from Needs more info to Bug resolved
  • Assignee set to Julio Montoya
  • % Done changed from 90 to 100

Good for me. Worked as expected. Tried adding and removing from LP as well. The only "weird" effect was that the exercise remained invisible after removing it from the LP, but honestly I think it's best this way (than having the problem that if the exercise was invisible in the beginning it becomes visible afterwards).

Good job!

Also available in: Atom PDF