search

chamilo / chamilo-lms

Chamilo is a learning management system focused on ease of use and accessibility
https://chamilo.org
GNU General Public License v3.0
798 stars 480 forks source link

Hotspot duplicate location error #2813

Closed yourKatharsis closed 7 months ago

yourKatharsis commented 5 years ago

Describe When solving a hotspot-test and klick different numbers on the same area, the previous numbers move to the left top corner.

To Reproduce Steps to reproduce the behavior:

  1. Create a hotspot-test with at least 2 areas
  2. try to solve and click two times on the same location
  3. submit
  4. on the results page you see that the first number is now in the left top corner (see screenshot)

Expected behavior It should be possible to set more than one number to a location.

Screenshots image

Desktop (please complete the following information):

**Server

Additional context

Apache Error Log:

[Thu Feb 21 10:35:07.367896 2019] [:error] [pid 7496] [client 000.00.00.00:0000] PHP Notice:  Undefined variable: correctAnswerId in /var/www/chamilo/www/main/exercise/exercise.class.php on line 4749, referer: http://xxx/main/exercise/exercise_submit.php?cidReq=TESTKURS&id_session=0&gidReq=0&gradebook=0&origin=learnpath&exerciseId=1&learnpath_id=3&learnpath_item_id=17&learnpath_item_view_id=84
[Thu Feb 21 10:35:10.709654 2019] [:error] [pid 7492] [client 000.00.00.00:0000] PHP Notice:  Undefined variable: correctAnswerId in /var/www/chamilo/www/main/exercise/exercise.class.php on line 4749, referer: http://xxx/main/lp/lp_controller.php?cidReq=TESTKURS&id_session=0&gidReq=0&gradebook=0&origin=learnpath&action=view&lp_id=3&lp_item_id=17&exeId=261&fb_type=0
ywarnier commented 5 years ago

Thanks for reporting @yourKatharsis I'm moving this issue to 2.0 because of its relatively limited impact. Hopefully this doesn't impact you too much, but if it does remember you can support the project and get it done faster by asking an official provider https://chamilo.org/providers

stale[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

ywarnier commented 2 years ago

Issue #4315 should be fixed before this

ywarnier commented 1 year ago

4315 is fixed now

ywarnier commented 1 year ago

https://github.com/chamilo/chamilo-lms/pull/4493 should have fixed the issue for upcoming version 2.0

ywarnier commented 7 months ago

This is fixed in 2.0. There are still issues with the visualization of hostpot prompts, but this is for another issue.