HugoFara / lwt

Learn languages by reading! A language learning app stemmed from Learning with Texts (LWT).
https://hugofara.github.io/lwt/
The Unlicense
164 stars 19 forks source link

Test ALL terms inside terms fails with traceback #192

Closed txwikinger closed 5 months ago

txwikinger commented 5 months ago

Describe the bug When using test all terms inside the terms page, an error message and tracback is shown

To Reproduce Steps to reproduce the behavior:

  1. Go to 'terms' inside the Multiaction section
  2. Click on the multi-selection box titled 'All n Terms' -> Test ALL terms
  3. See error

Fatal Error: do_test_test.php called with wrong parameters

Backtrace:

0 /var/www/html/lwt/inc/session_utility.php(782): my_die('...')

1 /var/www/html/lwt/inc/session_utility.php(802): do_test_test_get_projection('...', Array)

2 /var/www/html/lwt/do_test.php(62): do_test_test_from_selection(2, '...')

3 /var/www/html/lwt/do_test.php(187): get_l2_language_name()

4 /var/www/html/lwt/do_test.php(260): do_test_desktop_page()

5 /var/www/html/lwt/do_test.php(277): do_test_page('...')

6 /var/www/html/lwt/do_test.php(285): try_start_test('...')

Signal this issue on GitHub or Discord.

Expected behavior I can to the testing pages for the terms

Screenshots If applicable, add screenshots to help explain your problem.

Server (please complete) You can copy/paste data from http://[.. your LWT installation]/server_data.php.

<?php

/**

namespace Lwt\Interface\Server\Data;

require_once 'inc/session_utility.php';

/**

/**

display_content();

Desktop (if necessary):

Smartphone (if necessary):

Additional context Add any other context about the problem here.

HugoFara commented 5 months ago

Hi @txwikinger ! Thank you for the bug report, I fixed it, it was one line of code acting up (missing 's' character at inc/session_utility#L747).

Have a good day!