--- loncom/homework/lonsimpleproblemedit.pm 2010/01/29 15:44:41 1.20.2.4
+++ loncom/homework/lonsimpleproblemedit.pm 2012/11/09 03:28:01 1.34
@@ -1,7 +1,7 @@
# The LearningOnline Network
# Simple Problem Parameter Setting "Editor"
#
-# $Id: lonsimpleproblemedit.pm,v 1.20.2.4 2010/01/29 15:44:41 raeburn Exp $
+# $Id: lonsimpleproblemedit.pm,v 1.34 2012/11/09 03:28:01 raeburn Exp $
#
# Copyright Michigan State University Board of Trustees
#
@@ -71,7 +71,7 @@ sub questiontext {
return (<<ENDQUESTION);
<table bgcolor="#dddd22" cellspacing="4" cellpadding="2" style="width:100%">
<tr><td><b>$qt</b><br />
-<textarea style="width:100%" name="questiontext" cols="80" rows="8">$text</textarea>
+<textarea style="width:99%" name="questiontext" cols="80" rows="8">$text</textarea>
<br />$spell_link
</td></tr>
</table>
@@ -85,9 +85,9 @@ sub hint {
my $spell_link=
&Apache::lonhtmlcommon::spelllink('simpleedit','hinttext');
return (<<ENDHINT);
-<table bgcolor="#accacc" cellspacing="4" cellpadding="2">
+<table bgcolor="#accacc" cellspacing="4" cellpadding="2" style="width:100%">
<tr><td><b>$ht</b><br />
-<textarea style="width:100%" name="hinttext" cols="80" rows="4">$text</textarea>
+<textarea style="width:99%" name="hinttext" cols="80" rows="4">$text</textarea>
<br />$spell_link
</td></tr>
</table>
@@ -99,9 +99,9 @@ sub script {
my $text=$qparms{$prefix.'numericalscript'};
my $ht=&mt('Scripting (optional)');
return (<<ENDSCRIPT);
-<table bgcolor="#ccccaa" cellspacing="4" cellpadding="2">
+<table bgcolor="#ccccaa" cellspacing="4" cellpadding="2" style="width:100%">
<tr><td><b>$ht</b><br />
-<textarea style="width:100%" name="numericalscript" cols="80" rows="4">$text</textarea>
+<textarea style="width:99%" name="numericalscript" cols="80" rows="4">$text</textarea>
</td></tr>
</table>
<br />
@@ -137,11 +137,11 @@ sub foil {
my $selectvalue=&Apache::loncommon::select_form(
$value,
'value'.$number,
- &Apache::lonlocal::texthash(%values));
+ \%values);
my $selectposition=&Apache::loncommon::select_form(
$position,
'position'.$number,
- &Apache::lonlocal::texthash(%positions));
+ {&Apache::lonlocal::texthash(%positions)});
my $text=$qparms{$prefix.'text'.$number};
my %lt=&Apache::lonlocal::texthash('foil' => 'Foil',
'value' => 'Value',
@@ -151,11 +151,11 @@ sub foil {
my $spell_link=
&Apache::lonhtmlcommon::spelllink('simpleedit',"text$number");
return (<<ENDFOIL);
-<table bgcolor="#dd55ff" cellspacing="4" cellpadding="2">
+<table bgcolor="#E8D8EE" cellspacing="2" cellpadding="1" style="width:100%">
<tr><td colspan="2"><b>$lt{'foil'}</b></td></tr>
<tr><td>$lt{'value'}: $selectvalue</td><td>$lt{'pos'}: $selectposition</td></tr>
<tr><td colspan="2">$lt{'text'}:<br />
-<textarea style="width:100%" name="text$number" cols="80" rows="4">$text</textarea>
+<textarea style="width:99%" name="text$number" cols="80" rows="4">$text</textarea>
<br />$spell_link
</td></tr>
</table>
@@ -201,10 +201,11 @@ sub handler {
# ------------------------------------------------ Prefix for everything stored
$prefix=$env{'request.course.id'}.'.'.$symb.'.0.';
-#----------------------------------- Prefix for storing weight of Problem Parts
+#------------------------------------------------- Prefix for storing weight of Problem Parts
my $weightprefix=$env{'request.course.id'}.'.'.$symb.'.';
+
# ---------------------------------------------------------- Anything to store?
-
+ my $storeresult;
if (($symb) && (defined($env{'form.questiontype'}))) {
my %storecontent=();
undef %storecontent;
@@ -230,22 +231,33 @@ sub handler {
}
# ---------------------------------------------------- Set weights of hidden parts to zero
# ------------------------------------------------------ and used part to 1
- $storecontent{$weightprefix.'essay.weight'}=0;
- $storecontent{$weightprefix.'numerical.weight'}=0;
- $storecontent{$weightprefix.'option.weight'}=0;
- $storecontent{$weightprefix.'radio.weight'}=0;
- $storecontent{$weightprefix.'string.weight'}=0;
- $storecontent{$weightprefix.$env{'form.questiontype'}.'.weight'}=1;
+ $storecontent{$weightprefix.'essay.weight'}=0;
+ $storecontent{$weightprefix.'numerical.weight'}=0;
+ $storecontent{$weightprefix.'option.weight'}=0;
+ $storecontent{$weightprefix.'radio.weight'}=0;
+ $storecontent{$weightprefix.'string.weight'}=0;
+ $storecontent{$weightprefix.$env{'form.questiontype'}.'.weight'}=1;
+
my $reply=&Apache::lonnet::cput
('resourcedata',\%storecontent,
$env{'course.'.$env{'request.course.id'}.'.domain'},
$env{'course.'.$env{'request.course.id'}.'.num'});
+
+ &Apache::lonnet::devalidatecourseresdata(
+ $env{'course.'.$env{'request.course.id'}.'.num'},
+ $env{'course.'.$env{'request.course.id'}.'.domain'});
+ if ($reply eq 'ok') {
+ if ($env{'form.forceview'}) {
+ my $dest = &get_parent_uri($symb);
+ if ($dest) {
+ $r->internal_redirect($dest);
+ }
+ }
+ } else {
+ $storeresult = $reply;
+ }
}
-
- &Apache::lonnet::devalidatecourseresdata(
- $env{'course.'.$env{'request.course.id'}.'.num'},
- $env{'course.'.$env{'request.course.id'}.'.domain'});
# ------------------------------------------------------------------- Read Data
%qparms=&Apache::lonnet::dump('resourcedata',
@@ -253,26 +265,57 @@ sub handler {
$env{'course.'.$env{'request.course.id'}.'.num'},
$env{'request.course.id'}.'.'.$symb);
+ my $js = <<"ENDJS";
+
+<script type="text/javascript">
+// <![CDATA[
+
+function setForceView() {
+ if (document.getElementById('spview')) {
+ document.getElementById('spview').value = 1;
+ }
+ return true;
+}
+// END LON-CAPA Internal -->
+// ]]>
+</script>
+
+ENDJS
+
# ------------------------------------------------------------ Print the screen
my $spell_header=&Apache::lonhtmlcommon::spellheader();
$r->print(&Apache::loncommon::start_page('Simple Problem Editor',
- $spell_header));
+ $spell_header.$js));
if ($symb) {
- $r->print('<h1>'.&Apache::lonnet::gettitle($symb).'</h1>');
- $r->print('<table width="100%" bgcolor="#FFFFAA" border="2"><tr><td>');
- my $displaylink = &get_parent_uri($symb);
- if ($displaylink ne '') {
- $r->print('<a href="'.$displaylink.'">'.&mt('Student View').'</a> - ');
+ my $title='<h1>'.&Apache::lonnet::gettitle($symb).'</h1>';
+ if (&get_parent_uri($symb)) {
+ $r->print($title);
+ if ($storeresult) {
+ $r->print('<p class="LC_error">'.&mt('An error: [_1] occurred saving your changes',$storeresult).'</p>');
+ }
} else {
- $r->print('<span class="LC_error">'.&mt('An error occurred retrieving the link to this problem').'<br />'.&mt('You may need to [_1]re-select the course[_2] and then return to this resource to view it.','<a href="/adm/roles">','</a>').'</span><br />');
+ $r->print($title
+ .'<p class="LC_error">'
+ .&mt('An error occurred retrieving the link to this problem.')
+ .'<br />'
+ .&mt('You may need to [_1]re-select the course[_2] and then return to this resource to view it.'
+ ,'<a href="/adm/roles">','</a>')
+ .'</p>'
+ );
}
- $r->print(&mt('Note: it can take up to 10 minutes for changes to take effect for all users.').
- &Apache::loncommon::help_open_topic('Caching').'</td></tr></table>');
- $r->print('<table border="2" bgcolor="#FFFFFF" width="100%"><tr><td>'.
- &rawrendering($symb).
- '</td></tr></table><br />');
+ $r->print('<p class="LC_warning">'
+ .&mt('Note: it can take up to 10 minutes for changes to take effect for all users.')
+ .&Apache::loncommon::help_open_topic('Caching')
+ .'</p>'
+ );
+
+ $r->print(
+ '<div class="LC_Box">'
+ .&rawrendering($symb)
+ .'</div>'
+ );
$r->print('<form name="simpleedit" method="post">');
-# Question Type
+# Question Type
my %questiontypes=(
'radio' => '1 out of N multiple choice (radio button)',
'option' => 'Option Response',
@@ -282,13 +325,23 @@ sub handler {
$qtype=$qparms{$prefix.'questiontype'};
unless (defined($qtype)) { $qtype='radio'; }
unless ($questiontypes{$qtype}) { $qtype='radio'; }
- $r->print('<b>'.&mt('Question Type').': '
+ $r->print(
+ '<fieldset style="width:400px;">'
+ .'<legend>'.&mt('Question Type').'</legend>'
.&Apache::loncommon::select_form(
$qtype,
'questiontype',
- &Apache::lonlocal::texthash(%questiontypes)).
- '</b><br /><input type="submit" value="'.&mt('Save and Edit').
- '" /><p> </p>');
+ {&Apache::lonlocal::texthash(%questiontypes)})
+ .'</fieldset>'
+ );
+ $r->print(
+ '<p>'
+ .'<input type="hidden" name="forceview" value="" id="spview" />'
+ .'<input type="submit" value="'.&mt('Save and Edit').'" />'
+ .(' ' x3)
+ .'<input type="submit" value="'.&mt('Save and View').'" onclick="javascript:setForceView();" />'
+ .'</p>'
+ );
# Script
if ($qtype eq 'numerical') {
$r->print(&script());
@@ -309,26 +362,26 @@ sub handler {
unless (defined($randomize)) { $randomize='yes'; }
unless ($randomizes{$randomize}) { $randomize='yes'; }
$r->print(
- '<table bgcolor="#00ee44" cellspacing="4" cellpadding="2">'.
+ '<table bgcolor="#00ee44" cellspacing="4" cellpadding="2" style="width:100%">'.
'<tr><td>'.&mt('Max number of foils displayed').
': <input type="text" size="3" name="maxfoils" value="'.$maxfoils.'" /> '.
&Apache::loncommon::select_form(
$randomize,
'randomize',
- &Apache::lonlocal::texthash(%randomizes)).
- '</td></tr><tr><td bgcolor="#AAAAAA">');
+ {&Apache::lonlocal::texthash(%randomizes)}).
+ '</td></tr><tr><td bgcolor="#F0F0F0">');
# Option Response: Options
if ($qtype eq 'option') {
my $options=$qparms{$prefix.'options'};
unless (defined($options)) { $options="('true','false')"; }
my %optionshash=&evaloptionhash($options);
$r->print(
- '<table bgcolor="#ffcc22" cellspacing="4" cellpadding="2">'.
+ '<table bgcolor="#ffcc22" cellspacing="4" cellpadding="2" style="width:100%">'.
'<tr><td><input type="hidden" name="options" value="'.
$options.'" />'.&mt('Add new option').': '.
'<input type="text" name="newopt" size="15" />'.
&mt('Delete an option').': '.
- &Apache::loncommon::select_form('','delopt',('' => '',%optionshash)).
+ &Apache::loncommon::select_form('','delopt',{'' => '',%optionshash}).
'</td></tr><tr><td>');
}
# Foils
@@ -356,43 +409,46 @@ sub handler {
unless (defined($stringtype)) { $stringtype='cs'; }
unless ($stringtypes{$stringtype}) { $stringtype='cs'; }
$r->print(
- '<table bgcolor="#00ee44" cellspacing="4" cellpadding="2">'.
- '<tr><td><label>'.&mt('Correct answer').': <input type="text" size="20" name="stringanswer" value="'.$stringanswer.'" /></label> '.
+ '<table bgcolor="#00ee44" cellspacing="4" cellpadding="2" style="width:100%">'.
+ '<tr><td><label>'.&mt('Correct answer').': <input type="text" size="20" name="stringanswer" value="'.$stringanswer.'" /></label> '.
&Apache::loncommon::select_form(
$stringtype,
'stringtype',
- &Apache::lonlocal::texthash(%stringtypes)).
+ {&Apache::lonlocal::texthash(%stringtypes)}).
'</td></tr></table><br />');
# Hint
$r->print(&hint());
}
if ($qtype eq 'numerical') {
- my $numericalanswer=$qparms{$prefix.'numericalanswer'};
- unless (defined($numericalanswer)) { $numericalanswer=''; }
- my $numericaltolerance=$qparms{$prefix.'numericaltolerance'};
- unless (defined($numericaltolerance)) { $numericaltolerance='5%'; }
- my $numericalsigfigs=$qparms{$prefix.'numericalsigfigs'};
- unless (defined($numericalsigfigs)) { $numericalsigfigs='1,15'; }
+ my $numericalanswer=$qparms{$prefix.'numericalanswer'};
+ unless (defined($numericalanswer)) { $numericalanswer=''; }
+ my $numericaltolerance=$qparms{$prefix.'numericaltolerance'};
+ unless (defined($numericaltolerance)) { $numericaltolerance='5%'; }
+ my $numericalsigfigs=$qparms{$prefix.'numericalsigfigs'};
+ unless (defined($numericalsigfigs)) { $numericalsigfigs='1,15'; }
- $r->print(
- '<table bgcolor="#00ee44" cellspacing="4" cellpadding="2">'.
+ $r->print(
+ '<table bgcolor="#00ee44" cellspacing="4" cellpadding="2" style="width:100%">'.
'<tr><td><label>'.&mt('Correct answer').': <input type="text" size="20" name="numericalanswer" value="'.$numericalanswer.'" /></label> '.
'<label>'.&mt('Unit').': <input type="text" size="5" name="numericalunit" value="'.$qparms{$prefix.'numericalunit'}.'" /></label> '.
'<label>'.&mt('Format').': <input type="text" size="5" name="numericalformat" value="'.$qparms{$prefix.'numericalformat'}.'" /></label> '.
'<label>'.&mt('Tolerance').': <input type="text" size="5" name="numericaltolerance" value="'.$numericaltolerance.'" /></label> '.
'<label>'.&mt('Significant digits').': <input type="text" size="5" name="numericalsigfigs" value="'.$numericalsigfigs.'" /></label>'.
- '</td></tr></table><br />');
- $r->print(&hint());
+ '</td></tr></table><br />');
+ $r->print(&hint());
}
# Store Button
$r->print(
- '<input type="submit" value="'.&mt('Save and Edit').'" /></form>');
+ '<input type="submit" value="'.&mt('Save and Edit').'" />'.
+ (' ' x3).
+ '<input type="submit" value="'.&mt('Save and View').'" onclick="javascript:setForceView();" />'.
+ '</form>');
} else {
$r->print(&mt('Could not identify problem.'));
}
$r->print(&Apache::loncommon::end_page());
return OK;
-}
+}
1;
__END__