String with plus character in key doesn't display in AMOS

String with plus character in key doesn't display in AMOS

Martignoni Nicolas
Number of replies: 7
Language pack maintainers 사진

Hi,

I notice the following behavior in AMOS:

Is this a bug in AMOS or is it intended (a feature 미소)?

In reply to Martignoni Nicolas

Re: String with plus character in key doesn't display in AMOS

Mudrák David
Language pack maintainers 사진

It's a tricky one. Moodle core itself should throw at least a debugging warning because such an identifier does not fit into PARAM_STRINGID format that is defined as a regular expression '|^[a-zA-Z][a-zA-Z0-9\.:/_-]*$|'

I am wondering how many strings like that are there 슬픈

In reply to Mudrák David

Re: String with plus character in key doesn't display in AMOS

Martignoni Nicolas
Language pack maintainers 사진
Thanks. I forgot about PARAM_STRINGID 슬픈 Will change in my code.
In reply to Martignoni Nicolas

Re: String with plus character in key doesn't display in AMOS

Mudrák David
Language pack maintainers 사진

Actually, how that it worked for you? I just tried and when I use a string with a plus sign in it, I get an exception:

diff --git a/index.php b/index.php
index e95b0bb0d1..b86fac7ad7 100644
--- a/index.php
+++ b/index.php
@@ -328,4 +328,7 @@ foreach (explode(',', $frontpagelayout) as $v) {
 if ($editing && has_capability('moodle/course:create', context_system::instance())) {
     echo $courserenderer->add_new_course_button();
 }
+
+echo get_string('foo+bar');
+
 echo $OUTPUT->footer();
diff --git a/lang/en/moodle.php b/lang/en/moodle.php
index 4032ffb344..4485d63c0d 100644
--- a/lang/en/moodle.php
+++ b/lang/en/moodle.php
@@ -2176,3 +2176,5 @@ $string['sectionusedefaultname'] = 'Use default section name';
 $string['publish'] = 'Publish';
 $string['extendenrol'] = 'Extend enrolment (individual)';
 $string['groupextendenrol'] = 'Extend enrolment (common)';
+
+$string['foo+bar'] = 'Unsupported';

leads to

Coding error detected, it must be fixed by a programmer: Invalid string identifier.
The identifier cannot be empty. Please fix your get_string() call.

Error code: codingerror

Stack trace:
line 7051 of /lib/moodlelib.php: coding_exception thrown
line 332 of /index.php: call to get_string()
In reply to Mudrák David

Re: String with plus character in key doesn't display in AMOS

Martignoni Nicolas
Language pack maintainers 사진

Indeed. In fact I forgot then to test my plugin with debug mode on 얼굴을 붉힌. With debug off, Moodle doesn't complain at all. This made me think that it was something with AMOS, but it was just me 미소

With debug on, I got an error too. Can't remember well, but I thing that it was "Invalid string identifier" too.

PS. Maybe Moodle should throw an error even if the debug mode is off?

In reply to Martignoni Nicolas

Re: String with plus character in key doesn't display in AMOS

Mudrák David
Language pack maintainers 사진

Ah I see, thanks. I think we can leave the message to appear under developer debug mode only.

In reply to Mudrák David

Re: String with plus character in key doesn't display in AMOS

Martignoni Nicolas
Language pack maintainers 사진

> I think we can leave the message to appear under developer debug mode only.

Yes, agreed, so as not to bother users with strange debug messages.