[LON-CAPA-cvs] cvs: loncom /homework bridgetask.pm structuretags.pm /xml lontexconvert.pm
albertel
lon-capa-cvs@mail.lon-capa.org
Thu, 09 Nov 2006 21:31:55 -0000
This is a MIME encoded message
--albertel1163107915
Content-Type: text/plain
albertel Thu Nov 9 16:31:55 2006 EDT
Modified files:
/loncom/homework bridgetask.pm structuretags.pm
/loncom/xml lontexconvert.pm
Log:
- some cleanups to the generated html to make it easier to read
--albertel1163107915
Content-Type: text/plain
Content-Disposition: attachment; filename="albertel-20061109163155.txt"
Index: loncom/homework/bridgetask.pm
diff -u loncom/homework/bridgetask.pm:1.199 loncom/homework/bridgetask.pm:1.200
--- loncom/homework/bridgetask.pm:1.199 Thu Nov 9 15:42:43 2006
+++ loncom/homework/bridgetask.pm Thu Nov 9 16:31:11 2006
@@ -1,7 +1,7 @@
# The LearningOnline Network with CAPA
# definition of tags that give a structure to a document
#
-# $Id: bridgetask.pm,v 1.199 2006/11/09 20:42:43 albertel Exp $
+# $Id: bridgetask.pm,v 1.200 2006/11/09 21:31:11 albertel Exp $
#
# Copyright Michigan State University Board of Trustees
#
@@ -234,36 +234,43 @@
if (scalar(keys(%sections)) < 3) {
$size=scalar(keys(%sections))+2;
}
- my $sec_select = '<select multiple="multiple" name="chosensections" size="'.$size.'">'."\n";
- $sec_select .= "<option value='all' selected='selected'>all</option>\n";
+ my $sec_select = "\n".'<select multiple="multiple" name="chosensections" size="'.$size.'">'."\n";
+ $sec_select .= "\t<option value='all' selected='selected'>all</option>\n";
foreach my $sec (sort {lc($a) cmp lc($b)} (keys(%sections))) {
- $sec_select .= "<option value=\"$sec\">$sec</option>\n";
+ $sec_select .= "\t<option value=\"$sec\">$sec</option>\n";
}
- $sec_select .= "<option value='none'>none</option></select>\n";
+ $sec_select .= "\t<option value='none'>none</option>\n</select>\n";
- my $result=' <input type="submit" name="gradeasubmission" value="'.
+ my $result="\n\t".'<input type="submit" name="gradeasubmission" value="'.
&mt("Get a submission to grade").'" />';
- $result.='<input type="hidden" name="grade_target" value="webgrade" />';
+ $result.="\n\t".'<input type="hidden" name="grade_target" value="webgrade" />';
if (&Apache::lonnet::allowed('mgq',$env{'request.course.id'})) {
my ($entries,$ready,$locks)=&get_queue_counts('gradingqueue');
- $result.='<table><tr>';
- $result.='<td rowspan="4">Specify a section: </td><td rowspan="4">'.$sec_select.'</td>';
- $result.='<td>'.' <input type="submit" name="reviewagrading" value="'.
+ $result.="\n\t".'<table>'."\n\t\t".'<tr>';
+ $result.="\n\t\t\t".'<td rowspan="4">Specify a section: </td>'.
+ "\n\t\t\t".'<td rowspan="4">'.$sec_select."\n\t\t\t".'</td>';
+ $result.="\n\t\t\t".'<td>'.'<input type="submit" name="reviewagrading" value="'.
&mt("Select an entry from the grading queue:").'" /> ';
- $result.= &mt("[_1] entries, [_2] ready, [_3] being graded",$entries,$ready,$locks).' </td></tr>'."\n";
+ $result.= "\n\t\t\t\t".&mt("[_1] entries, [_2] ready, [_3] being graded",$entries,$ready,$locks).'</td>'."\n\t\t".'</tr>'."\n";
($entries,$ready,$locks)=&get_queue_counts('reviewqueue');
- $result.='<tr><td>'.
- ' <input type="submit" name="reviewasubmission" value="'.
+ $result.="\n\t\t".'<tr>'.
+ "\n\t\t\t".'<td>'.
+ "\n\t\t\t\t".'<input type="submit" name="reviewasubmission" value="'.
&mt("Select an entry from the review queue:").'" /> ';
$result.=&mt("[_1] entries, [_2] ready, [_3] being graded",
- $entries,$ready,$locks).'</td></tr>'."\n";
- $result.='<tr><td> <input type="submit" name="regradeasubmission" value="'.
- &mt("List of user's grade status").'" /> </td></tr></table>'."\n";
- $result.='<p> <input type="submit" name="regradeaspecificsubmission" value="'.
- &mt("Regrade specific user:").'" />'."\n";
- $result.='<input type="text" size="12" name="gradinguser" />';
+ $entries,$ready,$locks).'</td>'."\n\t\t".'</tr>'."\n";
+ $result.="\n\t\t".'<tr>'.
+ "\n\t\t\t".'<td>'.
+ "\n\t\t\t\t".'<input type="submit" name="regradeasubmission" value="'.
+ &mt("List of user's grade status").'" /> </td>'
+ ."\n\t\t".'</tr>'
+ ."\n\t".'</table>'."\n";
+ $result.="\n\t".'<p>'.
+ "\n\t\t".'<input type="submit" name="regradeaspecificsubmission" value="'.
+ &mt("Regrade specific user:").'" />';
+ $result.= "\n\t\t".'<input type="text" size="12" name="gradinguser" />';
$result.=&Apache::loncommon::select_dom_form($env{'user.domain'},
'gradingdomain');
$result.=' '.
@@ -271,7 +278,7 @@
'gradinguser',
'gradingdomain');
$result.=&Apache::loncommon::studentbrowser_javascript();
- $result.= '</p>';
+ $result.= '</p>'."\n";
}
return $result;
}
@@ -295,18 +302,19 @@
my $description=&Apache::slotrequest::get_description($slot_name,
$slot);
$result.=(<<STUFF);
-<p> Will be next available: $description </p>
+ <p> Will be next available: $description </p>
STUFF
}
if ($env{'request.enc'}) { $symb=&Apache::lonenc::encrypted($symb); }
$symb=&escape($symb);
- $result.='<form method="post" action="/adm/slotrequest">'.
- '<input type="hidden" name="symb" value="'.$symb.'" />'.
- '<input type="hidden" name="command" value="'.$action.'" />'.
+ $result.=
+ "\n\t".'<form method="post" action="/adm/slotrequest">'."\n\t\t".
+ '<input type="hidden" name="symb" value="'.$symb.'" />'."\n\t\t".
+ '<input type="hidden" name="command" value="'.$action.'" />'."\n\t\t".
'<input type="submit" name="requestattempt" value="'.
- &mt($text).'" />'.
- '</form>';
+ &mt($text).'" />'."\n\t".
+ '</form>'."\n";
return $result;
}
@@ -584,7 +592,7 @@
$result.='<form name="gradesubmission" method="post" action="';
my $uri=$env{'request.uri'};
if ($env{'request.enc'}) { $uri=&Apache::lonenc::encrypted($uri); }
- $result.=$uri.'">'.&add_grading_button()."</form>";
+ $result.=$uri.'">'.&add_grading_button()."</form>\n";
my $symb=&Apache::lonnet::symbread();
if (&Apache::lonnet::allowed('mgq',$env{'request.course.id'})) {
$result.='<form method="post" name="slotrequest" action="/adm/slotrequest">'.
@@ -683,7 +691,7 @@
$result.=&preserve_grade_info();
$result.=&internal_location();
- $result.=$form_tag_start.
+ $result.=$form_tag_start."\t".
'<input type="hidden" name="submitted" value="yes" />';
&Apache::lonxml::startredirection();
}
@@ -1042,7 +1050,8 @@
}
$start_time=&Apache::lonlocal::locallocaltime($start_time);
- my $status = "\n<div class='LC_$bt_status LC_criteria'>\n";
+ my $status =
+ "\n<div class='LC_$bt_status LC_criteria'>\n\t";
if ($bt_status eq 'pass') {
$status.='<h2>You passed the '.$title.' given on '.
@@ -1055,13 +1064,13 @@
$status.=&add_request_another_attempt_button();
}
}
- $status.='</div>';
+ $status.="\n".'</div>'."\n";
my $dim = $top;
my %counts = &get_counts($dim,undef,$parstack,
$safeeval);
- $status.="\n<div class='LC_$bt_status LC_criteria LC_task_overall_status'><p>".
+ $status.="\n<div class='LC_$bt_status LC_criteria LC_task_overall_status'>\n\t<p>".
&question_status_message(\%counts,-1).
- "</p></div>\n";
+ "</p>\n</div>\n";
foreach my $id (@{$dimension{$dim}{'criterias'}}) {
@@ -2215,7 +2224,8 @@
if (&Apache::lonxml::get_param('Mandatory',$parstack,$safeeval) eq 'N') {
$mandatory='Optional';
}
- my $dim_info="<div class='LC_$dim_status LC_question_grade'>\n";
+ my $dim_info=
+ "\n<div class='LC_$dim_status LC_question_grade'>\n\t";
my $question = ('sub' x $dimension{$dim}{'depth'}).'question';
$question =~ s/^(.)/uc($1)/e;
if ($dim_status eq 'pass') {
@@ -2227,10 +2237,10 @@
my %counts = &get_counts($dim,$instance,$parstack,
$safeeval);
- $dim_info.="\n<p>"
+ $dim_info.="\n\t<p>"
.&question_status_message(\%counts,
$dimension{$dim}{'depth'})
- ."</p>\n</div>";
+ ."</p>\n</div>\n";
foreach my $id (@{$dimension{$dim}{$instance.'.criterias'}},
@{$dimension{$dim}{'criterias'}}) {
@@ -2504,11 +2514,12 @@
$entry.=' comment: "'.$Apache::lonhomework::history{"$t:$scope.comment"}.'"';
}
if ($entry) {
- $result.= "<li>$grader : $entry </li>";
+ $result.= "\n\t\t<li>\n\t\t\t$grader :\n\t\t\t $entry \n\t\t</li>";
}
}
if ($result) {
- return '<ul class="LC_GRADING_pastgrading">'.$result.'</ul>';
+ return "\n\t".'<ul class="LC_GRADING_pastgrading">'.$result.
+ "\n\t".'</ul>'."\n";
}
return '';
}
@@ -2621,15 +2632,18 @@
my $status_display=$status;
$status_display=~s/^([a-z])/uc($1)/e;
my $criteria_info.=
- '<div class="LC_'.$status.' LC_criteria"><h4>'
- .$mandatory.' Criteria</h4><p>';
+ '<div class="LC_'.$status.' LC_criteria">'."\n\t".'<h4>'
+ .$mandatory.' Criteria</h4>'."\n\t".'<p>'."\n";
$criteria_info.= $criteria;
- $criteria_info.='</p><p class="LC_grade">'.$status_display.'</p>';
+ $criteria_info.="\n\t".'</p>'.
+ "\n\t".'<p class="LC_grade">'.$status_display.'</p>';
if ($comment =~ /\w/) {
- $criteria_info.='<p class="LC_comment">'.
- &mt('Comment: [_1]',$comment).'</p>';
+ $criteria_info.=
+ "\n\t".
+ '<p class="LC_comment">'.&mt('Comment: [_1]',$comment).'</p>';
}
- $criteria_info.='</div>';
+ $criteria_info.="\n".'</div>'."\n";
+
return $criteria_info;
}
@@ -2638,34 +2652,51 @@
my $link=&link($id);
my $version = &get_version();
my $status = &get_criteria('status',$version,$dim,$id);
- my $result =
- '<div class="LC_GRADING_criteria">'."\n".
- '<div class="LC_GRADING_criteriatext">'."\n".
- $criteria.
- '</div>'."\n".
- '<div class="LC_GRADING_grade">'."\n".
- '<label class="LC_GRADING_ungraded">
- <input type="radio" name="HWVAL_'.$link.'" value="ungraded" '.($status eq 'ungraded' || !$status ? 'checked="checked"':'').' />'.
- &mt('Ungraded').
- '</label>'."\n".
- '<label class="LC_GRADING_fail">
- <input type="radio" name="HWVAL_'.$link.'" value="fail" '.($status eq 'fail' ? 'checked="checked"':'').' />'.
- &mt('Fail').
- '</label>'."\n".
- '<label class="LC_GRADING_pass">
- <input type="radio" name="HWVAL_'.$link.'" value="pass" '.($status eq 'pass' ? 'checked="checked"':'').' />'.
- &mt('Pass').
- '</label>'."\n".
- '<label class="LC_GRADING_review">
- <input type="radio" name="HWVAL_'.$link.'" value="review" '.($status eq 'review' ? 'checked="checked"':'').' />'.
- &mt('Review').
- '</label>'."\n".
- '</div>'."\n".
- '<label class="LC_GRADING_comment">'.&mt('Additional Comment for Student')."\n".
- '<textarea class="LC_GRADING_comment_area" name="HWVAL_comment_'.$link.'">'.&HTML::Entities::encode(&get_criteria('comment',$version,$dim,$id),'<>"&').'</textarea>'."\n".
- '</label>'."\n".
- '</div>'."\n".
- &grading_history($version,$dim,$id);
+ my %lt = ( 'ungraded' => 'Ungraded',
+ 'fail' => 'Fail',
+ 'pass' => 'Pass',
+ 'review' => 'Review',
+ 'comment' => 'Additional Comment for Student',
+ );
+ %lt = &Apache::lonlocal::texthash(%lt);
+ my $comment = &get_criteria('comment',$version,$dim,$id);
+ $comment = &HTML::Entities::encode($comment,'<>"&');
+ my %checked;
+ foreach my $which ('ungraded','fail','pass','review') {
+ if ($status eq $which) { $checked{$which} = 'checked="checked"'; }
+ }
+ if (!%checked) { $checked{'ungraded'} = 'checked="checked"'; }
+
+ my $result = <<END_CRITERIA;
+ <div class="LC_GRADING_criteria">
+ <div class="LC_GRADING_criteriatext">
+ $criteria
+ </div>
+ <div class="LC_GRADING_grade">
+ <label class="LC_GRADING_ungraded">
+ <input type="radio" name="HWVAL_$link" value="ungraded" $checked{'ungraded'} />
+ $lt{'ungraded'}
+ </label>
+ <label class="LC_GRADING_fail">
+ <input type="radio" name="HWVAL_$link" value="fail" $checked{'fail'} />
+ $lt{'fail'}
+ </label>
+ <label class="LC_GRADING_pass">
+ <input type="radio" name="HWVAL_$link" value="pass" $checked{'pass'} />
+ $lt{'pass'}
+ </label>
+ <label class="LC_GRADING_review">
+ <input type="radio" name="HWVAL_$link" value="review" $checked{'review'} />
+ $lt{'review'}
+ </label>
+ </div>
+ <label class="LC_GRADING_comment">
+ $lt{'comment'}
+ <textarea class="LC_GRADING_comment_area" name="HWVAL_comment_$link">$comment</textarea>
+ </label>
+ </div>
+END_CRITERIA
+ $result .= &grading_history($version,$dim,$id);
return $result;
}
Index: loncom/homework/structuretags.pm
diff -u loncom/homework/structuretags.pm:1.367 loncom/homework/structuretags.pm:1.368
--- loncom/homework/structuretags.pm:1.367 Tue Oct 17 11:11:12 2006
+++ loncom/homework/structuretags.pm Thu Nov 9 16:31:11 2006
@@ -1,7 +1,7 @@
# The LearningOnline Network with CAPA
# definition of tags that give a structure to a document
#
-# $Id: structuretags.pm,v 1.367 2006/10/17 15:11:12 albertel Exp $
+# $Id: structuretags.pm,v 1.368 2006/11/09 21:31:11 albertel Exp $
#
# Copyright Michigan State University Board of Trustees
#
@@ -162,14 +162,14 @@
if ($target eq 'edit') {
$form_tag_start.=&Apache::edit::form_change_detection();
}
- $form_tag_start.='>';
+ $form_tag_start.='>'."\n";
my $symb=&Apache::lonnet::symbread();
if ($symb =~ /\S/) {
$symb=
&HTML::Entities::encode(&Apache::lonenc::check_encrypt($symb));
$form_tag_start.=
- '<input type="hidden" name="symb" value="'.$symb.'" />';
+ "\t".'<input type="hidden" name="symb" value="'.$symb.'" />'."\n";
}
}
return ($page_start,$form_tag_start);
@@ -744,7 +744,7 @@
&Apache::bridgetask::check_in('problem',undef,undef,
$slot_name);
}
- $result.="\n $form_tag_start".
+ $result.="\n $form_tag_start \t".
'<input type="hidden" name="submitted" value="yes" />';
# create a page header and exit
if ($env{'request.state'} eq "construct") {
Index: loncom/xml/lontexconvert.pm
diff -u loncom/xml/lontexconvert.pm:1.80 loncom/xml/lontexconvert.pm:1.81
--- loncom/xml/lontexconvert.pm:1.80 Mon Oct 23 21:28:47 2006
+++ loncom/xml/lontexconvert.pm Thu Nov 9 16:31:54 2006
@@ -1,7 +1,7 @@
# The LearningOnline Network with CAPA
# TeX Conversion Module
#
-# $Id: lontexconvert.pm,v 1.80 2006/10/24 01:28:47 albertel Exp $
+# $Id: lontexconvert.pm,v 1.81 2006/11/09 21:31:54 albertel Exp $
#
# Copyright Michigan State University Board of Trustees
#
@@ -200,7 +200,8 @@
sub jsMath_process {
my $state = pop(@jsMath_sent_header);
return '' if !$state;
- return '<script type="text/javascript">jsMath.Process()</script>';
+ return "\n".
+ '<script type="text/javascript">jsMath.Process()</script>'."\n";
}
}
--albertel1163107915--