Hi,
When I set my questionnaire name to a hebrew word (I guess any non-latin language), I can't change any setting on the 'advanced settings' tab.
The error message is "Could not create a new survey!".
Apart from that, the questionnaire is functioning as usual.
I think it's related to the questionnaire "title" setting on the 'advanced settings' tab.
Can anyone check that?
Thanks!
Yaniv.
// new survey
if(empty($this->survey->id)) {
if (isset($sdata->name)) {
/* $sdata->name = eregi_replace(
"[^A-Z0-9]+", "_", trim($sdata->name) );
$sdata->name = ereg_replace('_$',"",$sdata->name);*/
}
// create a new survey in the database
$fields = array('name','realm','title','subtitle','email','theme','thanks_page','thank_head','thank_body','info');
$record = new Object();
$record->id = 0;
$record->owner = $sdata->owner;
foreach($fields as $f) {
if(isset($sdata->$f)) {
$record->$f = $sdata->$f;
}
}
$this->survey->id = insert_record('questionnaire_survey', $record);
$this->add_survey($this->survey->id);
if(!$this->survey->id) {
$tab = "general";
$errstr = get_string('errnewname', 'questionnaire') .' [ : ]';
return(false);
}
} else {
if (isset($sdata->name)) {
/* $sdata->name = eregi_replace(
"[^A-Z0-9]+", "_", trim($sdata->name) );
$sdata->name = ereg_replace('_$',"",$sdata->name);*/
}
That solved the problem, thanks!
Are you sure you don't want to include the IF line like this:
/* if (isset($sdata->name)) {
$sdata->name = eregi_replace(
"[^A-Z0-9]+", "_", trim($sdata->name) );
$sdata->name = ereg_replace('_$',"",$sdata->name); } */
One more thing.
I think the "Settings saved" confirmation is unnecessary and disturbes questionnaire navigation.
Maybe it should appear on the top of the page, like the confirmation on the moodle administration settings pages.
redirect ($CFG->wwwroot.'/mod/questionnaire/view.php?id='.$questionnaire->cm->id,
get_string('settingssaved', 'questionnaire'),$delay=0);
This will very briefly display the Settings saved message; I do not see how it can disturb navigation and I do not know how to do it otherwise.