- Posts: 2
Joomdle 1.0 Installation
Warning: "continue" targeting switch is equivalent to "break".
- Patrick Gay
- Topic Author
- Offline
- New Member
-
Less
More
5 years 7 months ago #1
by Patrick Gay
Warning: "continue" targeting switch is equivalent to "break". was created by Patrick Gay
I installed Joomdle Auth in Moodle and now I get the following message at the top of every page I browse to in Moodle
Warning: "continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"? in /var/www/html/moodle/auth/joomdle/auth.php on line 2703
Here are my versions:
Joomla: 3.9.21
Joomdle: 1.2.7
Moodle: 3.9.1
PHP: 7.4.10
Apache: 2.4.6
Mariadb: 10.5.5
Warning: "continue" targeting switch is equivalent to "break". Did you mean to use "continue 2"? in /var/www/html/moodle/auth/joomdle/auth.php on line 2703
Here are my versions:
Joomla: 3.9.21
Joomdle: 1.2.7
Moodle: 3.9.1
PHP: 7.4.10
Apache: 2.4.6
Mariadb: 10.5.5
Please Log in or Create an account to join the conversation.
- Antonio Durán
-
- Offline
- Moderator
-
Less
More
- Posts: 7848
5 years 7 months ago #2
by Antonio Durán
Replied by Antonio Durán on topic Warning: "continue" targeting switch is equivalent to "break".
Hi Patrick,
Thanks for the bug report. We'll take a look at it and get back to you as soon as possible.
Thanks for the bug report. We'll take a look at it and get back to you as soon as possible.
Please Log in or Create an account to join the conversation.
- Antonio Durán
-
- Offline
- Moderator
-
Less
More
- Posts: 7848
5 years 7 months ago - 5 years 7 months ago #3
by Antonio Durán
Replied by Antonio Durán on topic Warning: "continue" targeting switch is equivalent to "break".
I checked this and made a fix for next release.
If you want to make the change, it is easy.
Edit file: moodle/auth/joomdle/auth.php
In get_course_students function change:
By:
If you want to make the change, it is easy.
Edit file: moodle/auth/joomdle/auth.php
In get_course_students function change:
Code:
if (!$user)
continue;
Code:
if (!$user) {
$include = false;
break;
}
Last edit: 5 years 7 months ago by Antonio Durán.
Please Log in or Create an account to join the conversation.