--- loncom/homework/lonhomework.pm	2025/01/17 15:51:31	1.344.2.10.4.11
+++ loncom/homework/lonhomework.pm	2015/06/15 20:23:19	1.349.2.4
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # The LON-CAPA Homework handler
 #
-# $Id: lonhomework.pm,v 1.344.2.10.4.11 2025/01/17 15:51:31 raeburn Exp $
+# $Id: lonhomework.pm,v 1.349.2.4 2015/06/15 20:23:19 musolffc Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -49,23 +49,16 @@ use Apache::matchresponse();
 use Apache::chemresponse();
 use Apache::functionplotresponse();
 use Apache::drawimage();
-use Apache::loncapamath();
-use Apache::loncourseuser();
-use Apache::grades();
 use Apache::Constants qw(:common);
 use Apache::loncommon();
 use Apache::lonparmset();
-use Apache::lonnavmaps();
 use Apache::lonlocal;
-use LONCAPA qw(:DEFAULT :match);
-use LONCAPA::ltiutils();
 use Time::HiRes qw( gettimeofday tv_interval );
 use HTML::Entities();
 use File::Copy();
 
 # FIXME - improve commenting
 
-my $registered_cleanup;
 
 BEGIN {
     &Apache::lonxml::register_insert();
@@ -163,19 +156,9 @@ sub get_target {
                 return ('web','answer');
             } elsif (($env{'form.problemmode'} eq 'saveedit') ||
                      ($env{'form.problemmode'} eq 'undo')) {
-                my %editors = &Apache::loncommon::permitted_editors();
-                if ($editors{'edit'}) {
-                    return ('modified','no_output_web','edit');
-                } else {
-                    return ('web');
-                }
+                return ('modified','no_output_web','edit');
             } elsif ($env{'form.problemmode'} eq 'edit') {
-                my %editors = &Apache::loncommon::permitted_editors();
-                if ($editors{'edit'}) {
-		    return ('no_output_web','edit');
-                } else {
-                    return ('web');
-                }
+		return ('no_output_web','edit');
 	    } else {
 		return ('web');
 	    }
@@ -206,7 +189,7 @@ sub proctor_checked_in {
     if ($type eq 'Task') {
 	my $version=$Apache::lonhomework::history{'resource.0.version'};
 	$key ="resource.$version.0.checkedin";
-    } elsif (($type eq 'problem') || ($type eq 'tool')) {
+    } elsif ($type eq 'problem') {
 	$key ='resource.0.checkedin';
     }
     # backward compatability, used to be username@domain, 
@@ -221,16 +204,17 @@ sub proctor_checked_in {
 	    return 1;
 	}
     }
+    
     return 0;
 }
 
 sub check_slot_access {
-    my ($id,$type,$symb,$partlist)=@_;
+    my ($id,$type)=@_;
 
     # does it pass normal muster
-    my ($status,$datemsg)=&check_access($id,$symb);
-
-    my $useslots = &Apache::lonnet::EXT("resource.0.useslots",$symb);
+    my ($status,$datemsg)=&check_access($id);
+    
+    my $useslots = &Apache::lonnet::EXT("resource.0.useslots");
     if ($useslots ne 'resource' && $useslots ne 'map' 
 	&& $useslots ne 'map_map') {
 	return ($status,$datemsg);
@@ -252,16 +236,10 @@ sub check_slot_access {
 	    $Apache::lonhomework::history{"resource.$version.0.status"} eq 'pass') {
 	    return ('SHOW_ANSWER');
 	}
-    } elsif (($type eq 'problem') &&
-             ($Apache::lonhomework::browse eq 'F') &&
-             ($ENV{'REMOTE_ADDR'} eq '127.0.0.1') &&
-             ($env{'form.grade_courseid'} eq $env{'request.course.id'}) &&
-             (&Apache::lonnet::allowed('mgr',$env{'request.course.id'}))) {
-        return ($status,$datemsg);
     }
 
-    my $availablestudent = &Apache::lonnet::EXT("resource.0.availablestudent",$symb);
-    my $available = &Apache::lonnet::EXT("resource.0.available",$symb);
+    my $availablestudent = &Apache::lonnet::EXT("resource.0.availablestudent");
+    my $available = &Apache::lonnet::EXT("resource.0.available");
     my @slots= (split(':',$availablestudent),split(':',$available));
 
 #    if (!@slots) {
@@ -271,9 +249,6 @@ sub check_slot_access {
     my ($returned_slot,$slot_name);
     my $now = time;
     my $num_usable_slots = 0;
-    unless ($symb) {
-        ($symb) = &Apache::lonnet::whichuser();
-    }
     foreach my $slot (@slots) {
 	$slot =~ s/(^\s*|\s*$)//g;
 	&Apache::lonxml::debug("getting $slot");
@@ -297,10 +272,9 @@ sub check_slot_access {
 	$slotstatus=$status;
     }
 
-    my ($is_correct,$got_grade,$checkin,$checkinslot,$checkedin,$consumed_uniq);
+    my ($is_correct,$got_grade,$checkedin);
     if ($type eq 'Task') {
 	my $version=$Apache::lonhomework::history{'resource.0.version'};
-        $checkin = "resource.$version.0.checkedin";
 	$got_grade = 
 	    ($Apache::lonhomework::history{"resource.$version.0.status"} 
 	     =~ /^(?:pass|fail)$/);
@@ -309,58 +283,11 @@ sub check_slot_access {
 	     || $Apache::lonhomework::history{"resource.0.solved"} =~ /^correct_/ );
 	$checkedin =
 	    $Apache::lonhomework::history{"resource.$version.0.checkedin"};
-    } elsif (($type eq 'problem') || ($type eq 'tool')) {
-        $checkin = 'resource.0.checkedin';
-	$checkedin  = $Apache::lonhomework::history{$checkin};
-    }
-    if ($checkedin) {
-        $checkinslot = $Apache::lonhomework::history{"$checkin.slot"};
-        my %slot=&Apache::lonnet::get_slot($checkinslot);
-        $consumed_uniq = $slot{'uniqueperiod'};
-    }
-    if (($type eq 'problem') || ($type eq 'tool')) {
-        if ((ref($partlist) eq 'ARRAY') && (@{$partlist} > 0)) {
-            my ($numcorrect,$numgraded) = (0,0);
-            foreach my $part (@{$partlist}) {
-                my $currtries = $Apache::lonhomework::history{"resource.$part.tries"};
-                my $maxtries = &Apache::lonnet::EXT("resource.$part.maxtries",$symb);
-                my $probstatus = &Apache::structuretags::get_problem_status($part);
-                my $earlyout;
-                unless (($probstatus eq 'no') ||
-                        ($probstatus eq 'no_feedback_ever')) {
-                    if ($Apache::lonhomework::history{"resource.$part.solved"} =~/^correct_/) {
-                        $numcorrect ++;
-                    } else {
-                        $earlyout = 1;
-                    }
-                }
-                if ($currtries == $maxtries) {
-                    $earlyout = 1;
-                } else {
-                    $numgraded ++;
-                }
-                last if ($earlyout);
-            }
-            my $numparts = scalar(@{$partlist});
-            if ($numparts == $numcorrect) {
-                $is_correct = 1;
-            }
-            if ($numparts == $numgraded) {
-                $got_grade = 1;
-            }
-        } else {
-            my $currtries = $Apache::lonhomework::history{"resource.0.tries"};
-            my $maxtries = &Apache::lonnet::EXT("resource.0.maxtries",$symb);
-            my $probstatus = &Apache::structuretags::get_problem_status('0');
-            unless (($probstatus eq 'no') ||
-                    ($probstatus eq 'no_feedback_ever')) {
-                $is_correct =
-                    ($Apache::lonhomework::history{"resource.0.solved"} =~/^correct_/);
-            }
-            unless (($currtries == $maxtries) || ($is_correct)) {
-                $got_grade = 1;
-            }
-        }
+    } elsif ($type eq 'problem') {
+	$got_grade  = 1;
+	$checkedin  = $Apache::lonhomework::history{"resource.0.checkedin"};
+	$is_correct =
+	    ($Apache::lonhomework::history{"resource.0.solved"} =~/^correct_/);
     }
     
     &Apache::lonxml::debug(" slot is $slotstatus checkedin ($checkedin) got_grade ($got_grade) is_correct ($is_correct)");
@@ -377,71 +304,29 @@ sub check_slot_access {
     # However, the problem is not closed, and potentially, another slot might be
     # used to gain access to it to work on it, until the due date is reached, and the
     # problem then becomes CLOSED.  Therefore return the slotstatus - 
-    # (which will be one of: NOT_IN_A_SLOT, RESERVABLE, RESERVABLE_LATER, or NOTRESERVABLE).
-
-    if (!defined($slot_name) && (($type eq 'problem') || ($type eq 'tool'))) {
+    # (which will be one of: NOT_IN_A_SLOT, RESERVABLE, RESERVABLE_LATER, or NOTRESERVABLE.
+    if (!defined($slot_name) && $type eq 'problem') {
         if ($slotstatus eq 'NOT_IN_A_SLOT') {
             if (!$num_usable_slots) {
                 if ($env{'request.course.id'}) {
                     my $cdom = $env{'course.'.$env{'request.course.id'}.'.domain'};
                     my $cnum = $env{'course.'.$env{'request.course.id'}.'.num'};
+                    my ($symb)=&Apache::lonnet::whichuser();
                     $slotstatus = 'NOTRESERVABLE';
                     my ($reservable_now_order,$reservable_now,$reservable_future_order,
                         $reservable_future) = 
                         &Apache::loncommon::get_future_slots($cnum,$cdom,$now,$symb);
                     if ((ref($reservable_now_order) eq 'ARRAY') && (ref($reservable_now) eq 'HASH')) {
                         if (@{$reservable_now_order} > 0) {
-                            if ((!$checkedin) || (ref($consumed_uniq) ne 'ARRAY')) {
-                                $slotstatus = 'RESERVABLE';
-                                $datemsg = $reservable_now->{$reservable_now_order->[-1]}{'endreserve'};
-                            } else {
-                                my ($uniqstart,$uniqend,$useslot);
-                                if (ref($consumed_uniq) eq 'ARRAY') {
-                                    ($uniqstart,$uniqend)=@{$consumed_uniq};
-                                }
-                                foreach my $slot (reverse(@{$reservable_now_order})) {
-                                    if ($reservable_now->{$slot}{'uniqueperiod'} =~ /^(\d+)\,(\d+)$/) {
-                                        my ($new_uniq_start,$new_uniq_end) = ($1,$2);
-                                        next if (!
-                                            ($uniqstart < $new_uniq_start && $uniqend < $new_uniq_start) ||
-                                            ($uniqstart > $new_uniq_end   &&  $uniqend > $new_uniq_end  ));
-                                    }
-                                    $useslot = $slot;
-                                    last;
-                                }
-                                if ($useslot) {
-                                    $slotstatus = 'RESERVABLE';
-                                    $datemsg = $reservable_now->{$useslot}{'endreserve'};
-                                }
-                            }
+                            $slotstatus = 'RESERVABLE';
+                            $datemsg = $reservable_now->{$reservable_now_order->[-1]}{'endreserve'};
                         }
                     }
                     unless ($slotstatus eq 'RESERVABLE') {
                         if ((ref($reservable_future_order) eq 'ARRAY') && (ref($reservable_future) eq 'HASH')) {
                             if (@{$reservable_future_order} > 0) {
-                                if ((!$checkedin) || (ref($consumed_uniq) ne 'ARRAY')) {
-                                    $slotstatus = 'RESERVABLE_LATER';
-                                    $datemsg = $reservable_future->{$reservable_future_order->[0]}{'startreserve'};
-                                } else {
-                                    my ($uniqstart,$uniqend,$useslot);
-                                    if (ref($consumed_uniq) eq 'ARRAY') {
-                                        ($uniqstart,$uniqend)=@{$consumed_uniq};
-                                    }
-                                    foreach my $slot (@{$reservable_future_order}) {
-                                        if ($reservable_future->{$slot}{'uniqueperiod'} =~ /^(\d+),(\d+)$/) {
-                                            my ($new_uniq_start,$new_uniq_end) = ($1,$2);
-                                            next if (!
-                                               ($uniqstart < $new_uniq_start && $uniqend < $new_uniq_start) ||
-                                               ($uniqstart > $new_uniq_end   &&  $uniqend > $new_uniq_end  ));
-                                        }
-                                        $useslot = $slot;
-                                        last;
-                                    }
-                                    if ($useslot) {
-                                        $slotstatus = 'RESERVABLE_LATER';
-                                        $datemsg = $reservable_future->{$useslot}{'startreserve'};
-                                    }
-                                }
+                                $slotstatus = 'RESERVABLE_LATER';
+                                $datemsg = $reservable_future->{$reservable_future_order->[0]}{'startreserve'};
                             }
                         }
                     }
@@ -463,7 +348,7 @@ sub check_slot_access {
     }
 
     if ( $is_correct) {
-	if (($type eq 'problem') || ($type eq 'tool')) {
+	if ($type eq 'problem') {
 	    return ($status);
 	}
 	return ('SHOW_ANSWER');
@@ -480,7 +365,7 @@ sub check_slot_access {
 # JB, 9/24/2002: Any changes in this function may require a change
 # in lonnavmaps::resource::getDateStatus.
 sub check_access {
-    my ($id,$symb) = @_;
+    my ($id) = @_;
     my $date ='';
     my $status;
     my $datemsg = '';
@@ -510,13 +395,11 @@ sub check_access {
     &Apache::lonxml::debug("checking for part :$id:");
     &Apache::lonxml::debug("time:".time);
 
-    unless ($symb) {
-        ($symb)=&Apache::lonnet::whichuser();
-    }
+    my ($symb)=&Apache::lonnet::whichuser();
     &Apache::lonxml::debug("symb:".$symb);
     #if ($env{'request.state'} ne "construct" && $symb ne '') {
     if ($env{'request.state'} ne "construct") {
-        my $idacc = &Apache::lonnet::EXT("resource.$id.acc",$symb);
+        my $idacc = &Apache::lonnet::EXT("resource.$id.acc");
 	my $allowed=&Apache::loncommon::check_ip_acc($idacc);
 	if (!$allowed && ($Apache::lonhomework::browse ne 'F')) {
 	    $status='INVALID_ACCESS';
@@ -532,12 +415,12 @@ sub check_access {
 	foreach my $temp ("opendate","duedate","answerdate") {
 	    $lastdate = $date;
 	    if ($temp eq 'duedate') {
-		$date = &due_date($id,$symb);
+		$date = &due_date($id);
 	    } else {
-		$date = &Apache::lonnet::EXT("resource.$id.$temp",$symb);
+		$date = &Apache::lonnet::EXT("resource.$id.$temp");
 	    }
 	    
-	    my $thistype = &Apache::lonnet::EXT("resource.$id.$temp.type",$symb);
+	    my $thistype = &Apache::lonnet::EXT("resource.$id.$temp.type");
 	    if ($thistype =~ /^(con_lost|no_such_host)/ ||
 		$date     =~ /^(con_lost|no_such_host)/) {
 		$status='UNAVAILABLE';
@@ -546,10 +429,10 @@ sub check_access {
 	    }
 	    if ($thistype eq 'date_interval') {
 		if ($temp eq 'opendate') {
-		    $date=&Apache::lonnet::EXT("resource.$id.duedate",$symb)-$date;
+		    $date=&Apache::lonnet::EXT("resource.$id.duedate")-$date;
 		}
 		if ($temp eq 'answerdate') {
-		    $date=&Apache::lonnet::EXT("resource.$id.duedate",$symb)+$date;
+		    $date=&Apache::lonnet::EXT("resource.$id.duedate")+$date;
 		}
 	    }
 	    &Apache::lonxml::debug("found :$date: for :$temp:");
@@ -583,7 +466,7 @@ sub check_access {
 	(($Apache::lonhomework::browse eq 'F') && ($status eq 'CLOSED'))) {
 	#check #tries, and if correct.
 	my $tries = $Apache::lonhomework::history{"resource.$id.tries"};
-	my $maxtries = &Apache::lonnet::EXT("resource.$id.maxtries",$symb);
+	my $maxtries = &Apache::lonnet::EXT("resource.$id.maxtries");
 	if ( $tries eq '' ) { $tries = '0'; }
 	if ( $maxtries eq '' && 
 	     $env{'request.state'} ne 'construct') { $maxtries = '2'; } 
@@ -592,7 +475,7 @@ sub check_access {
 	if ( ($Apache::lonhomework::history{"resource.$id.solved"}=~/^correct/)
 	      && (&show_problem_status()) ) {
             if (($Apache::lonhomework::history{"resource.$id.awarded"} >= 1) ||
-                (&Apache::lonnet::EXT("resource.$id.retrypartial",$symb) !~/^1|on|yes$/i)) {
+                (&Apache::lonnet::EXT("resource.$id.retrypartial") !~/^1|on|yes$/i)) {
 	        $status = 'CANNOT_ANSWER';
             }
         } elsif ($Apache::lonhomework::history{"resource.$id.solved"}=~/^excused/) {
@@ -604,14 +487,14 @@ sub check_access {
 	}
     }
     if ($status eq 'CAN_ANSWER' || $status eq 'CANNOT_ANSWER') {
-	my @interval=&Apache::lonnet::EXT("resource.$id.interval",$symb);
+	my @interval=&Apache::lonnet::EXT("resource.$id.interval");
 	&Apache::lonxml::debug("looking for interval @interval");
 	if ($interval[0]) {
-	    my $first_access=&Apache::lonnet::get_first_access($interval[1],$symb);
+	    my $first_access=&Apache::lonnet::get_first_access($interval[1]);
 	    &Apache::lonxml::debug("looking for accesstime $first_access");
 	    if (!$first_access) {
 		$status='NOT_YET_VIEWED';
-		my $due_date = &due_date($id,$symb);
+		my $due_date = &due_date($id);
 		my $seconds_left = $due_date - time;
 		if ($seconds_left > $interval[0] || $due_date eq '') {
 		    $seconds_left = $interval[0];
@@ -734,9 +617,6 @@ sub setuppermissions {
 	$env{'request.course.sec'} !~ /^\s*$/) {
 	$viewgrades = &Apache::lonnet::allowed('vgr',$env{'request.course.id'}.
                                                '/'.$env{'request.course.sec'});
-        if ($viewgrades) {
-            $Apache::lonhomework::viewgradessec = $env{'request.course.sec'};
-        }
     }
     $Apache::lonhomework::viewgrades = $viewgrades;
 
@@ -754,9 +634,6 @@ sub setuppermissions {
 	$modifygrades = 
 	    &Apache::lonnet::allowed('mgr',$env{'request.course.id'}.
 				     '/'.$env{'request.course.sec'});
-        if ($modifygrades) {
-            $Apache::lonhomework::modifygradessec = $env{'request.course.sec'};
-        }
     }
     $Apache::lonhomework::modifygrades = $modifygrades;
 
@@ -775,9 +652,7 @@ sub setuppermissions {
 sub unset_permissions {
     undef($Apache::lonhomework::queuegrade);
     undef($Apache::lonhomework::modifygrades);
-    undef($Apache::lonhomework::modifygradessec);
     undef($Apache::lonhomework::viewgrades);
-    undef($Apache::lonhomework::viewgradessec);
     undef($Apache::lonhomework::browse);
 }
 
@@ -876,7 +751,7 @@ STATE
 
 sub analyze_header {
     my ($request) = @_;
-    my $js = &Apache::lonxml::setmode_javascript();
+    my $js = &Apache::structuretags::setmode_javascript();
 
     # Breadcrumbs
     my $brcrum = [{'href' => &Apache::loncommon::authorspace($request->uri),
@@ -892,32 +767,24 @@ sub analyze_header {
                                        {'bread_crumbs' => $brcrum,})
        .&Apache::loncommon::head_subbox(
                 &Apache::loncommon::CSTR_pageheader());
-    my %lt = &Apache::lonlocal::texthash(
-                 edit => 'Edit',
-                 editxml => 'EditXML',
-             );
     $result .= 
-            '<form name="lonhomework" method="post" action="'.
+	    '<form name="lonhomework" method="post" action="'.
 	    &HTML::Entities::encode($env{'request.uri'},'<>&"').'">'.
             '<input type="hidden" name="problemmode" value="'.
             $env{'form.problemmode'}.'" />'.
 	    &Apache::structuretags::remember_problem_state().'
-            <div class="LC_edit_problem_analyze_header">';
-     my %editors = &Apache::loncommon::permitted_editors();
-     foreach my $item ('editxml','edit') {
-         next unless ($editors{$item});
-         $result .= '<input type="button" name="submitmode" value="'.$lt{$item}.'" '.
-                    'onclick="javascript:setmode(this.form,'."'$item'".')" />'.
-                    "\n";
-     }
-     $result .=
-            '<hr />
+            <div class="LC_edit_problem_analyze_header">
+            <input type="button" name="submitmode" value="'.&mt("EditXML").'" '.
+            'onclick="javascript:setmode(this.form,'."'editxml'".')" />
+            <input type="button" name="submitmode" value="'.&mt('Edit').'" '.
+            'onclick="javascript:setmode(this.form,'."'edit'".')" />
+            <hr />
             <input type="button" name="submitmode" value="'.&mt("View").'" '.
             'onclick="javascript:setmode(this.form,'."'view'".')" />
             <hr />
             </div>'
-            .&Apache::lonxml::message_location().'
-            </form>';
+            .&Apache::lonxml::message_location().
+            '</form>';
     &Apache::lonxml::add_messages(\$result);
     $request->print($result);
     $request->rflush();
@@ -1116,7 +983,7 @@ sub editxmlmode {
 	my $js =
 	    &Apache::edit::js_change_detection(). 
 	    &Apache::loncommon::resize_textarea_js().
-            &Apache::lonxml::setmode_javascript().
+            &Apache::structuretags::setmode_javascript().
             &Apache::lonhtmlcommon::dragmath_js("EditMathPopup");
 
     # Breadcrumbs
@@ -1146,86 +1013,70 @@ sub editxmlmode {
             <div class="LC_edit_problem_header">
               <div class="LC_edit_problem_header_title">'.
                 &mt('Problem Editing').' '.&Apache::loncommon::help_open_topic('Problem_Editor_XML_Index').
-                '</div><div class="LC_edit_actionbar" id="actionbar">';
+              '</div><div class="LC_edit_actionbar" id="actionbar">';
 
-    $result.='<input type="hidden" name="problemmode" value="saveedit" />'.
+        $result.='<input type="hidden" name="problemmode" value="saveedit" />'.
                   &Apache::structuretags::problem_edit_buttons('editxml');
-    $result.='<div>';
-
-    $result .= '<ol class="LC_primary_menu" style="display:inline-block;font-size:90%;vertical-align:middle;">';
-
-    my $nocodemirror = &Apache::loncommon::nocodemirror();
-    unless ($nocodemirror) {
-        # dropdown menus
-        $result .= Apache::lonmenu::create_submenu("#", "",
-            &mt("Problem Templates"), template_dropdown_datastructure());
-
-        $result .= Apache::lonmenu::create_submenu("#", "",
-            &mt("Response Types"), responseblock_dropdown_datastructure());
-
-        $result .= Apache::lonmenu::create_submenu("#", "",
-            &mt("Conditional Blocks"), conditional_scripting_datastructure());
-
-        $result .= Apache::lonmenu::create_submenu("#", "",
-            &mt("Miscellaneous"), misc_datastructure());
-    }
-
-    $result .= Apache::lonmenu::create_submenu("#", "",
-        &mt("Help") . ' <img src="/adm/help/help.png" alt="' . &mt("Help") .
-        '" style="vertical-align:text-bottom; height: auto; margin:0; "/>',
-        helpmenu_datastructure(),"");
-
-    $result.="</ol></div>";
-
-    $result .= '</div></div>' .
-        &Apache::lonxml::message_location() .
-        &Apache::loncommon::xmleditor_js() .
-        '<textarea ' . &Apache::edit::element_change_detection() .
-        ' rows="'.$rows.'" cols="'.$cols.'" style="width:100%" ' .
-        ' name="editxmltext" id="LC_editxmltext">' .
-        &HTML::Entities::encode($problem,'<>&"') .
-        '</textarea> <div id="LC_aftertextarea"> </div> </form>';
-
-    my $resource = $env{'request.ambiguous'};
-    unless ($nocodemirror) {
-        $result .= '<link rel="stylesheet" href="/adm/codemirror/codemirror-combined-xml.css">
-        <script src="/adm/codemirror/codemirror-compressed-xml.js"></script>
-        <script>
-            CodeMirror.defineMode("mixedmode", function(config) {
-                return CodeMirror.multiplexingMode(
-                    CodeMirror.getMode(config, "xml"),
-                    {
-                        open: "\<script type=\"loncapa/perl\"\>", close: "\</script\>",
-                        mode: CodeMirror.getMode(config, "perl"),
-                        delimStyle: "tag",
-                    }
-              );
-            });
-            var cm = CodeMirror.fromTextArea(document.getElementById("LC_editxmltext"),
-            {
-                mode: "mixedmode",
-                lineWrapping: true,
-                lineNumbers: true,
-                tabSize: 4,
-                indentUnit: 4,
-
-                autoCloseTags: true,
-                autoCloseBrackets: true,
-                height: "auto",
-                styleActiveLine: true,
-
-                extraKeys: {
-                    "Tab": "indentMore",
-                    "Shift-Tab": "indentLess",
-                }
-            });
-            restoreScrollPosition("'.$resource.'");
-        </script>';
-    }
+        $result.='<div class="LC_edit_problem_discards">';
 
-    $result .= &Apache::loncommon::end_page();
-    &Apache::lonxml::add_messages(\$result);
-    $request->print($result);
+		unless ($env{'environment.nocodemirror'}) {
+		# dropdown menues
+    		$result .= '<ol class="LC_primary_menu LC_floatleft">'.
+    		&Apache::lonmenu::create_submenu("#", "", &mt("Insert Menu"), &Apache::structuretags::insert_menu_datastructure(),"").'</ol>';
+		}
+    $result .= '<ol class="LC_primary_menu LC_floatleft">'.
+    Apache::lonmenu::create_submenu("#", "", &mt("Help"), &Apache::structuretags::helpmenu_datastructure(),"").'</ol>';
+    $result.="</div>";
+         
+         $result.='<hr style="clear:both;visibility:hidden" /></div></div>'.&Apache::lonxml::message_location().
+                  &Apache::loncommon::xmleditor_js().
+		  '<textarea '.&Apache::edit::element_change_detection().
+	              ' rows="'.$rows.'" cols="'.$cols.'" style="width:100%" '.
+		      ' name="editxmltext" id="LC_editxmltext">'.
+		      &HTML::Entities::encode($problem,'<>&"').'</textarea>
+            <div id="LC_aftertextarea">
+            </div>
+	        </form>';
+	    my $resource = $env{'request.ambiguous'};
+	    unless($env{'environment.nocodemirror'}){
+	        
+	        $result .= '<link rel="stylesheet" href="/adm/codemirror/codemirror-combined-xml.css">
+	        <script src="/adm/codemirror/codemirror-compressed-xml.js"></script>
+	        <script>
+	            CodeMirror.defineMode("mixedmode", function(config) {
+	                return CodeMirror.multiplexingMode(
+	                    CodeMirror.getMode(config, "xml"),
+	                    {
+	                        open: "\<script type=\"loncapa/perl\"\>", close: "\</script\>",
+	                        mode: CodeMirror.getMode(config, "perl"),
+	                        delimStyle: "tag",
+	                    }
+	              );
+	            });
+	            var cm = CodeMirror.fromTextArea(document.getElementById("LC_editxmltext"),
+	            {
+	                mode: "mixedmode",
+	                lineWrapping: true,
+	                lineNumbers: true,
+	                tabSize: 4,
+	                indentUnit: 4,
+
+	                autoCloseTags: true,
+	                autoCloseBrackets: true,
+	                height: "auto",
+	                styleActiveLine: true,
+	                
+	                extraKeys: {
+	                    "Tab": "indentMore",
+	                    "Shift-Tab": "indentLess",
+	                }
+	            });
+	            restoreScrollPosition("'.$resource.'");
+	        </script>';
+	    }
+        $result .= &Apache::loncommon::end_page();
+        &Apache::lonxml::add_messages(\$result);
+        $request->print($result);
     }
     return '';
 }
@@ -1234,7 +1085,7 @@ sub editxmlmode {
 #    Render the page in whatever target desired.
 #
 sub renderpage {
-    my ($request,$file,$targets,$return_string,$donebuttonmsg,$viewasuser,$symb) = @_;
+    my ($request,$file,$targets,$return_string) = @_;
 
     my @targets = @{$targets || [&get_target()]};
     &Apache::lonhomework::showhashsubset(\%env,'form.');
@@ -1260,8 +1111,10 @@ sub renderpage {
 	    $problem='';
 	    my $filename=(split('/',$file))[-1];
 	    my $error =
-                &mt('Unable to find [_1]',
-			   '<span class="LC_filename">'.$filename.'</span>');
+		'<p class="LC_error">'
+               .&mt('Unable to find [_1]',
+			   '<span class="LC_filename">'.$filename.'</span>')
+		."</p>";
 	    $result.=
 		&Apache::loncommon::simple_error_page($request,'Not available',
 						      $error,{'no_auto_mt_msg' => 1});
@@ -1273,10 +1126,6 @@ sub renderpage {
 	if ($target eq 'answer') { &showhash(%Apache::lonhomework::history); }
 	if ($target eq 'web') {&Apache::lonhomework::showhashsubset(\%env,'^form');}
 
-        if (($target eq 'web') && ($viewasuser ne '') && ($symb ne '')) {
-            $env{'request.user_in_effect'} = $viewasuser;
-        }
-
 	&Apache::lonxml::debug("Should be parsing now");
 	$result .= &Apache::lonxml::xmlparse($request, $target, $problem,
 					     &setup_vars($target),%mystyle);
@@ -1289,11 +1138,6 @@ sub renderpage {
 	    if ($target eq 'analyze') {
 		$result=&Apache::lonnet::hashref2str(\%Apache::lonhomework::analyze);
 		undef(%Apache::lonhomework::analyze);
-            } elsif ($target eq 'web') {
-                if ($donebuttonmsg) {
-                    $result =~ s{</body>}{};
-                    $result.= &Apache::loncommon::confirmwrapper(&Apache::lonhtmlcommon::confirm_success($donebuttonmsg,1))."\n</body>";
-                }
 	    }
 	    #my $td=&tv_interval($t0);
 	    #if ( $Apache::lonxml::debug) {
@@ -1302,23 +1146,15 @@ sub renderpage {
 	    #}
 #	    $request->print($result);
 	    $overall_result.=$result;
-            if (($target eq 'web') && ($viewasuser ne '') && ($symb ne '')) {
-                my ($vuname,$vudom) = split(/:/,$viewasuser);
-                $overall_result .= &Apache::grades::view_as_user($symb,$vuname,$vudom).
-                                   '</body></html>';
-            }
 #	    $request->rflush();
 	}
-        if (($target eq 'web') && ($viewasuser ne '') && ($symb ne '')) {
-            undef($env{'request.user_in_effect'});
-        }
 	#$request->print(":Result ends");
 	#my $td=&tv_interval($t0);
     }
     if (!$return_string) {
 	&Apache::lonxml::add_messages(\$overall_result);
-	$request->print($overall_result);
-	$request->rflush();
+	$request->print($overall_result);   
+	$request->rflush();   
     } else {
 	return $overall_result;
     }
@@ -1335,14 +1171,14 @@ sub finished_parsing {
 # value 1: title
 # value 2: category
 # value 3: name of help topic ???
-sub get_template_list {
+sub get_template_list{
     my ($extension) = @_;
-
+    
     my @files = glob($Apache::lonnet::perlvar{'lonIncludes'}.
-		     '/templates/*.'.$extension);
+                     '/templates/*.'.$extension);
     @files = map {[$_,&mt(&Apache::lonnet::metadata($_, 'title')),
                       (&Apache::lonnet::metadata($_, 'category')?&mt(&Apache::lonnet::metadata($_, 'category')):&mt('Miscellaneous')),
-                     &mt(&Apache::lonnet::metadata($_, 'help'))]} (@files);
+                      &mt(&Apache::lonnet::metadata($_, 'help'))]} (@files);
     @files = sort {$a->[2].$a->[1] cmp $b->[2].$b->[1]} (@files);
     return @files;
 }
@@ -1354,7 +1190,7 @@ sub get_template_html {
     &Apache::lonxml::debug("Looking for :$extension:");
     my $glob_extension  = $extension;
     if ($extension eq 'survey' || $extension eq 'exam') {
-        $glob_extension = 'problem';
+	$glob_extension = 'problem';
     }
     my @files = &get_template_list($extension);
     my ($midpoint,$seconddiv,$numfiles);
@@ -1418,39 +1254,18 @@ sub get_template_html {
 sub newproblem {
     my ($request) = @_;
 
-    if ($env{'form.mode'} eq 'blank'){
+	if ($env{'form.mode'} eq 'blank'){
         my $dest = &Apache::lonnet::filelocation("",$request->uri);
-        my $templatefilename =
-            $request->dir_config('lonIncludes').'/templates/blank.problem';
-        &File::Copy::copy($templatefilename,$dest);
+        &File::Copy::copy('/home/httpd/html/res/adm/includes/templates/blank.problem',$dest);
         &renderpage($request,$dest);
         return;
     }
-    my $errormsg;
     if ($env{'form.template'}) {
-        my $file;
-        my ($extension) = ($env{'form.template'} =~ /\.(\w+)$/);
-        if ($extension) {
-            my @files = &get_template_list($extension);
-            foreach my $poss (@files) {
-                if (ref($poss) eq 'ARRAY') {
-                    if ($env{'form.template'} eq $poss->[0]) {
-                        $file = $env{'form.template'};
-                        last;
-                    }
-                }
-            }
-            if ($file) {
-                my $dest = &Apache::lonnet::filelocation("",$request->uri);
-                &File::Copy::copy($file,$dest);
-                &renderpage($request,$dest);
-                return;
-            } else {
-                $errormsg = '<p class="LC_error">'.&mt('Invalid template file.').'</p>';
-            }
-        } else {
-            $errormsg = '<p class="LC_error">'.&mt('Invalid template file; template needs to be a .problem, .library, or .task file.').'</p>';
-        }
+	my $file = $env{'form.template'};
+	my $dest = &Apache::lonnet::filelocation("",$request->uri);
+	&File::Copy::copy($file,$dest);
+	&renderpage($request,$dest);
+	return;
     }
 
     my ($extension) = ($request->uri =~ m/\.(\w+)$/);
@@ -1467,6 +1282,7 @@ sub newproblem {
     } else {
 	my $url=&HTML::Entities::encode($request->uri,'<>&"');
 	my $dest = &Apache::lonnet::filelocation("",$request->uri);
+	my $errormsg;
 	my $instructions;
         my $brcrum = [{'href' => &Apache::loncommon::authorspace($request->uri),
                        'text' => 'Authoring Space'},
@@ -1517,88 +1333,56 @@ sub update_construct_style {
 }
 
 #
-# Sets interval for current user so time left will be zero, either for the entire folder
+# Sets interval for current user so time left will be zero, either for the entire folder 
 # containing the current resource, or just the resource, depending on value of first item
 # in interval array retrieved from EXT("resource.0.interval");
 #
 sub zero_timer {
     my ($symb) = @_;
     my ($hastimeleft,$first_access,$now);
-    my @interval=&Apache::lonnet::EXT("resource.0.interval",$symb);
+    my @interval=&Apache::lonnet::EXT("resource.0.interval");
     if (@interval > 1) {
         if ($interval[1] eq 'course') {
-            return ('fail',&mt('Ending of timed events not supported for intervals set course-wide'));
+            return;
         } else {
             my $now = time;
             my $first_access=&Apache::lonnet::get_first_access($interval[1],$symb);
             if ($first_access > 0) {
-                my ($timelimit,$donesuffix) = split(/_/,$interval[0],2);
-                if ($donesuffix =~ /^done(?:|\:[^\:]+\:)(.*)$/) {
-                    my ($dummy,$proctor,$secret) = split(/_/,$1);
-                    if (($proctor) && ($secret ne '')) {
-                        my $key = $env{'form.LC_interval_done_proctorpass'};
-                        $key =~ s/^\s+//;
-                        $key =~ s/\s+$//;
-                        if ($env{'form.LC_interval_done_proctorpass'} ne $secret) {
-                            return ('fail',
-                                   &mt('Incorrect key entered by proctor'));
-                        }
-                    }
-                    if ($first_access+$timelimit > $now) {
-                        my $done_time = $now - $first_access;
-                        my $snum = 1;
-                        if ($interval[1] eq 'map') {
-                            $snum = 2;
-                        }
-                        my $result =
-                            &Apache::lonparmset::storeparm_by_symb_inner($symb,'0_interval',
-                                                                         $snum,$done_time,
-                                                                         'date_interval',
-                                                                         $env{'user.name'},
-                                                                         $env{'user.domain'});
-                        if ($result eq '') {
-                            # Record action in "User Notes"
-                            &Apache::lonmsg::store_instructor_comment(
-                                'Pressed Done button for symb:<br />'.$symb,
-                                $env{'user.name'}, $env{'user.domain'});
-                            return ('ok');
-                        } else {
-                            return ('fail',&mt('Error ending timed event: [_1]',$result));
-                        }
-                    } else {
-                        return ('fail',&mt('Timed event already ended'));
+                if ($first_access+$interval[0] > $now) {
+                    my $done_time = $now - $first_access;
+                    my $snum = 1;
+                    if ($interval[1] eq 'map') {
+                        $snum = 2;
                     }
-                } else {
-                    return ('fail',&mt('Timed event can not be ended before the time limit'));
+                    my $result = 
+                        &Apache::lonparmset::storeparm_by_symb_inner($symb,'0_interval',
+                                                                     $snum,$done_time,
+                                                                     'date_interval',
+                                                                     $env{'user.name'},
+                                                                     $env{'user.domain'});
+                    return $result;
                 }
-            } else {
-                return ('fail',&mt('Timer not yet started for this timed event'));
             }
         }
-    } else {
-        return ('fail',&mt('No timer in use'));
     }
-    return();
+    return;
 }
 
 sub handler {
     #my $t0 = [&gettimeofday()];
     my $request=$_[0];
+
     $Apache::lonxml::request=$request;
     $Apache::lonxml::debug=$env{'user.debug'};
     $env{'request.uri'}=$request->uri;
     &setuppermissions();
 
-# -------------------------------------- Flag and buffer for registered cleanup
-    $registered_cleanup=0;
-    undef(@Apache::lonhomework::ltipassback);
-
     my $file=&Apache::lonnet::filelocation("",$request->uri);
 
     #check if we know where we are
-    if ($env{'request.course.fn'} && !&Apache::lonnet::symbread('','',1,1)) {
+    if ($env{'request.course.fn'} && !&Apache::lonnet::symbread()) { 
 	# if we are browsing we might not be able to know where we are
-	if ($Apache::lonhomework::browse ne 'F' &&
+	if ($Apache::lonhomework::browse ne 'F' && 
 	    $env{'request.state'} ne "construct") {
 	    #should know where we are, so ask
 	    &unset_permissions();
@@ -1627,13 +1411,7 @@ sub handler {
                      ($env{'form.problemmode'} eq 'saveeditxml') ||
                      ($env{'form.problemmode'} eq 'saveviewxml') ||
                      ($env{'form.problemmode'} eq 'undoxml')) {
-                my %editors = &Apache::loncommon::permitted_editors();
-                if (($editors{'xml'}) || ($env{'form.problemmode'} eq 'saveviewxml') || ($env{'form.problemmode'} eq 'undoxml')) {
-		    &editxmlmode($request,$file);
-                } else {
-                    &update_construct_style();
-                    &renderpage($request,$file);
-                }
+		&editxmlmode($request,$file);
 	    } elsif ($env{'form.problemmode'} eq 'calcanswers') {
 		&analyze($request,$file);
 	    } else {
@@ -1641,55 +1419,21 @@ sub handler {
 		&renderpage($request,$file);
 	    }
 	} else {
-            &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},
-                                                    ['mode']);
+		&Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},
+						    ['mode']);
 	    # requested file doesn't exist in contruction space
 	    &newproblem($request);
 	}
     } else {
         # Set the event timer to zero if the "done button" was clicked.  The button is
         # part of the doneButton form created in lonmenu.pm
-        my ($donebuttonresult,$donemsg,$viewasuser);
-        if ($symb && $env{'form.LC_interval_done'} eq 'true') {
-            ($donebuttonresult,$donemsg) = &zero_timer($symb);
+        if ($symb && $env{'form.LC_interval_done'} eq 'true') {  
+            &zero_timer($symb);
             undef($env{'form.LC_interval_done'});
-            undef($env{'form.LC_interval_done_proctorpass'});
-        }
-        if (($env{'form.LC_viewas'} ne '') && $symb && $env{'request.course.id'} &&
-            ($Apache::lonhomework::viewgrades || $Apache::lonhomework::modifygrades)) {
-            if ($env{'form.LC_viewas'} =~ /^($match_username):($match_domain)$/) {
-                my ($possuname,$possudom) = ($1,$2);
-                my $cdom = $env{'course.'.$env{'request.course.id'}.'.domain'};
-                my $cnum = $env{'course.'.$env{'request.course.id'}.'.num'};
-                my ($canview,$posssec);
-                if ($env{'request.course.sec'} ne '') {
-                    if ($Apache::lonhomework::modifygradessec eq $env{'request.course.sec'}) {
-                        $canview = 'section';
-                        $posssec = $env{'request.course.sec'};
-                    } elsif ($Apache::lonhomework::viewgradessec eq $env{'request.course.sec'}) {
-                        $canview = 'section';
-                        $posssec = $env{'request.course.sec'};
-                    }
-                }
-                my $crstype = &Apache::loncommon::course_type();
-                if (&Apache::loncourseuser::is_course_user($possudom,$possuname,$cdom,$cnum,
-                                                           $canview,$crstype,$posssec)) {
-                    $viewasuser = $possuname.':'.$possudom;
-                }
-            }
-            undef($env{'form.LC_viewas'});
         }
 	# just render the page normally outside of construction space
 	&Apache::lonxml::debug("not construct");
-	&renderpage($request,$file,undef,undef,$donemsg,$viewasuser,$symb);
-        if (@Apache::lonhomework::ltipassback) {
-            unless ($registered_cleanup) {
-                my $handlers = $request->get_handlers('PerlCleanupHandler');
-                $request->set_handlers('PerlCleanupHandler' =>
-                                       [\&do_ltipassback,@{$handlers}]);
-                $registered_cleanup=1;
-            }
-        }
+	&renderpage($request,$file);
     }
     #my $td=&tv_interval($t0);
     #&Apache::lonxml::debug("Spent $td seconds processing");
@@ -1700,387 +1444,5 @@ sub handler {
 
 }
 
-sub template_dropdown_datastructure {
-    # gathering the all templates and their path, title, category and help topic
-    my @templates = get_template_list('problem');
-    # template category => title
-    my %tmplthash = ();
-    # template title => path
-    my %tmpltcontent = ();
-
-    foreach my $template (@templates){
-        # put in hash if the template is not empty
-        unless ($template->[1] eq ''){
-            push(@{$tmplthash{$template->[2]}}, $template->[1]);
-            push(@{$tmpltcontent{$template->[1]}},$template->[0]);
-        }
-    }
-
-        my $catList = [];
-    foreach my $cat (sort keys %tmplthash) {
-                my $catItems = [];
-        foreach my $title (sort @{$tmplthash{$cat}}) {
-            my $path = $tmpltcontent{$title}->[0];
-            my $code;
-            open(FH, "<$path");
-            while(<FH>){
-                $code.= $_ unless $_ =~ /(<problem>)|(<\/problem>)/;
-            }
-            close(FH);
-
-                        if ($code ne '') {
-                my $href = 'javascript:insertText(\'' . &convert_for_js(&HTML::Entities::encode($code,'<>&"')) . '\')';
-                                my $currItem = [$href, $title, undef];
-                                push @{$catItems}, $currItem;
-                        }
-        }
-                push @{$catList}, [$catItems, $cat, undef];
-    }
-
-    return $catList;
-}
-
-sub responseblock_dropdown_datastructure {
-
-    my $mathCat = [
-                    [
-                        ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::edit::insert_formularesponse())) . "\')", &mt("Formula Response"), undef],
-                        ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::edit::insert_functionplotresponse())) . "\')", &mt("Function Plot Response"), undef],
-                        ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::edit::insert_mathresponse())) . "\')", &mt("Math Response"), undef],
-                        ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::edit::insert_numericalresponse())) . "\')", &mt("Numerical Response"), undef]
-                    ],
-                    &mt("Math"),
-                    undef
-        ];
-
-    my $miscCat = [
-                    [
-            ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::edit::insert_imageresponse())) . "\')", &mt("Click on Image"), undef],
-            ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::edit::insert_customresponse())) . "\')", &mt("Custom Response"), undef],
-            ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::edit::insert_externalresponse())) . "\')", &mt("External Response"), undef],
-            ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::edit::insert_matchresponse())) . "\')", &mt("Match Two Lists"), undef],
-            ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::edit::insert_radiobuttonresponse())) . "\')", &mt("One out of N statements"), undef],
-            ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::edit::insert_optionresponse())) . "\')", &mt("Select from Options"), undef],
-                        ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::edit::insert_rankresponse())) . "\')", &mt("Rank Values"), undef]
-                ],
-                &mt("Miscellaneous"),
-                undef
-        ];
-
-    my $chemCat = [
-                    [
-                        ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::edit::insert_reactionresponse())) . "\')", &mt("Chemical Reaction"), undef],
-                        ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::edit::insert_organicresponse())) . "\')", &mt("Organic Chemical Structure"), undef]
-                     ],
-                     &mt("Chemistry"),
-                     undef
-                   ];
-
-    my $textCat = [
-                    [
-                      ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::edit::insert_stringresponse())) . "\')", &mt("String Response"), undef],
-                      ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::edit::insert_essayresponse())) . "\')", &mt("Essay"), undef]
-                     ],
-                     &mt("Text"),
-                     undef
-                   ];
-
-    return [$mathCat, $miscCat, $chemCat, $textCat];
-}
-
-
-sub conditional_scripting_datastructure {
-# TODO: corresponding routines should be used for the javascript:insertText parts
-# instead of the placeholder routine default_xml_tag with the tags
-# e.g. &default_xml_tag("postanswerdate") should be replaced with a routine which
-# returns the corresponding content for this case
-
-#TODO translated is currently temporarily here, another solution should be found where the
-# needed string can be retrieved
-
-    my $translatedTag = '
-<translated>
-    <lang which="en"></lang>
-    <lang which="default"></lang>
-</translated>';
-    return [
-             ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode($translatedTag)) . "\')", &mt("Translated Block"), undef],
-             ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&default_xml_tag("block"))) . "\')", &mt("Conditional Block"), undef],
-             ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&default_xml_tag("postanswerdate"))) . "\')", &mt("After Answer Date Block"), undef],
-             ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&default_xml_tag("preduedate"))) . "\')", &mt("Before Due Date Block"), undef],
-             ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&default_xml_tag("solved"))) . "\')", &mt("Block For After Solved"), undef],
-             ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&default_xml_tag("notsolved"))) . "\')", &mt("Block For When Not Solved"), undef]
-        ];
-}
-
-sub misc_datastructure {
-    return [
-        ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::edit::insert_img())) . "\')", &mt("Image"), undef],
-        ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::lonplot::insert_gnuplot())) . "\')", &mt("GNU Plot"), undef],
-        ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::edit::insert_organicstructure())) . "\')", &mt("Organic Structure"), undef],
-        ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::edit::insert_script())) . "\')", &mt("Script Block"), undef],
-        ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&default_xml_tag("allow"))) . "\')", &mt("File Dependencies"), undef],
-        ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&default_xml_tag("import"))) . "\')", &mt("Import a File"), undef],
-        ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&Apache::londefdef::insert_meta())) . "\')", &mt("Custom Metadata"), undef],
-        ["javascript:insertText(\'" . &convert_for_js(&HTML::Entities::encode(&default_xml_tag("part"))) . "\')", &mt("Problem Part"), undef]
-    ];
-}
-
-# helper routine for the datastructure building subroutines
-sub default_xml_tag {
-    my ($tag) = @_;
-    return "\n<$tag></$tag>";
-}
-
-sub helpmenu_datastructure {
-
-    # filename, title, width, height
-    my $helpers = [
-                    ['Problem_LON-CAPA_Functions.hlp', &mt('Script Functions'), 800, 600],
-                    ['Greek_Symbols.hlp', &mt('Greek Symbols'), 500, 600],
-                    ['Other_Symbols.hlp', &mt('Other Symbols'), 500, 600],
-                    ['Authoring_Output_Tags.hlp', &mt('Output Tags'), 800, 600],
-                    ['Authoring_Multilingual_Problems.hlp', &mt('Languages'), 800, 600],
-                    ['loncapa.html', &mt('Reference'), 800, 600],
-                   ];
-
-    my $help_structure = [];
-
-    foreach my $count (0..(scalar(@{$helpers})-1)) {
-        my $filename = $helpers->[$count]->[0];
-        my $title = $helpers->[$count]->[1];
-        my $width = $helpers->[$count]->[2];
-        my $height = $helpers->[$count]->[3];
-        if ($width eq '') {
-            $width = 500;
-        }
-        if ($height eq '') {
-            $height = 600;
-        }
-        my $href = &HTML::Entities::encode("javascript:openMyModal('/adm/help/$filename',$width,$height,'yes');");
-        push @{$help_structure}, [$href, $title, undef];
-    }
-
-    return $help_structure;
-}
-
-# we need substitution to not break javascript code
-sub convert_for_js {
-    my $return = shift;
-    $return =~ s|script|ESCAPEDSCRIPT|g;
-    $return =~ s|\\|\\\\|g;
-    $return =~ s|\n|\\r\\n|g;
-    $return =~ s|'|\\'|g;
-    $return =~ s|&#39;|\\&#39;|g;
-    return $return;
-}
-
-sub do_ltipassback {
-    if (@Apache::lonhomework::ltipassback) {
-        my $lonhost = $Apache::lonnet::perlvar{'lonHostID'};
-        my $ip = &Apache::lonnet::get_host_ip($lonhost);
-        foreach my $item (@Apache::lonhomework::ltipassback) {
-            &Apache::lonhomework::run_passback($item,$lonhost,$ip);
-        }
-        undef(@Apache::lonhomework::ltipassback);
-    }
-    return OK;
-}
-
-sub run_passback {
-    my ($item,$lonhost,$ip) = @_;
-    if (ref($item) eq 'HASH') {
-        if ((ref($item->{'lti'}) eq 'HASH') && ($item->{'cid'} =~ /^($match_domain)_($match_courseid)$/)) {
-            my ($cdom,$cnum) = ($1,$2);
-            my $msgformat = $item->{'lti'}->{'passbackformat'};
-            my $sigmethod = 'HMAC-SHA1';
-            my $ltinum = $item->{'ltinum'};
-            my $id = $item->{'pbid'};
-            my $url = $item->{'pburl'};
-            my $type = $item->{'pbtype'};
-            my $pbscope = $item->{'pbscope'};
-            my $map = $item->{'pbmap'};
-            my $symb = $item->{'pbsymb'};
-            my $uname = $item->{'uname'};
-            my $udom = $item->{'udom'};
-            my $uhome = $item->{'uhome'};
-            my $usec = $item->{'usec'};
-            my $keynum = $item->{'lti'}->{'cipher'};
-            my $crsdef = $item->{'crsdef'};
-            my $scoretype = $item->{'format'};
-            my $scope = $item->{'scope'};
-            my $clientip = $item->{'clientip'};
-            my ($total,$possible,%total_by_symb,%possible_by_symb);
-            if ((exists($item->{'total_s'})) && (ref($item->{'total_s'}) eq 'HASH')) {
-                %total_by_symb = %{$item->{'total_s'}};
-                if ($pbscope eq 'resource') {
-                    if (exists($total_by_symb{$symb})) {
-                        $total = $total_by_symb{$symb};
-                    } else {
-                        $total = $item->{'total'};
-                    }
-                }
-            } elsif ($pbscope eq 'resource') {
-                $total = $item->{'total'};
-            }
-            if ((exists($item->{'possible_s'})) && (ref($item->{'possible_s'}) eq 'HASH')) {
-                %possible_by_symb = %{$item->{'possible_s'}};
-                if ($pbscope eq 'resource') {
-                    if (exists($possible_by_symb{$symb})) {
-                        $possible = $possible_by_symb{$symb};
-                    } else {
-                        $possible = $item->{'possible'};
-                    }
-                }
-            } elsif ($pbscope eq 'resource') {
-                $possible = $item->{'possible'};
-            }
-            if (($pbscope eq 'map') || ($pbscope eq 'nonrec')) {
-                if ((keys(%total_by_symb)) && (keys(%possible_by_symb))) {
-                    ($total,$possible) =
-                    &get_lti_score($uname,$udom,$usec,$map,$pbscope,\%total_by_symb,\%possible_by_symb);
-                } else {
-                   ($total,$possible) = &get_lti_score($uname,$udom,$usec,$map,$pbscope);
-                }
-            } elsif ($pbscope eq 'course') {
-                ($total,$possible) = &get_lti_score($uname,$udom,$usec);
-            }
-            $item->{'total'} = $total;
-            $item->{'possible'} = $possible;
-            if (($id ne '') && ($url ne '') && ($possible)) {
-                my ($sent,$score,$code,$result) =
-                    &LONCAPA::ltiutils::send_grade($cdom,$cnum,$crsdef,$type,$ltinum,$keynum,$id,
-                                                   $url,$scoretype,$sigmethod,$msgformat,$total,$possible);
-                $item->{'score'} = $score;
-                my ($linkprotector,$linkuri,$no_passback,$appname);
-                if ($item->{'linkprot'}) {
-                    ($linkprotector,$linkuri) = split(/:/,$item->{'linkprot'});
-                }
-                if ($sent) {
-                    if ($code == 200) {
-                        if ($item->{'linkprot'}) {
-                            my $skey = join("\0",($linkuri,$linkprotector,$scope));
-                            my $namespace = $cdom.'_'.$cnum.'_lp_passback';
-                            my $store = {
-                                          'score' => $score,
-                                          'ip' => $ip,
-                                          'host' => $Apache::lonnet::perlvar{'lonHostID'},
-                                          'protector' => $linkprotector,
-                                          'deeplink' => $linkuri,
-                                          'scope' => $scope,
-                                          'url' => $url,
-                                          'id' => $id,
-                                          'clientip' => $clientip,
-                                          'whodoneit' => $env{'user.name'}.':'.$env{'user.domain'},
-                                        };
-                            my $value='';
-                            foreach my $key (keys(%{$store})) {
-                                $value.=&escape($key).'='.&Apache::lonnet::freeze_escape($store->{$key}).'&';
-                            }
-                            $value=~s/\&$//;
-                            &Apache::lonnet::courselog(&escape($linkuri).':'.$uname.':'.$udom.':EXPORT:'.$value);
-                            &Apache::lonnet::store_userdata({'score' => $score},$skey,$namespace,$udom,$uname,$ip);
-                        }
-                    } else {
-                        if ($item->{'linkprot'}) {
-                           $no_passback = "Passback response was $code ($result).";
-                        }
-                    }
-                } else {
-                    if ($item->{'linkprot'}) {
-                        $no_passback = 'No passback of scores.';
-                    }
-                }
-                if ($no_passback) {
-                    if ($item->{'linkprot'}) {
-                        my ($ltinum,$ltitype) = ($linkprotector =~ /^(\d+)(c|d)$/);
-                        if ($ltitype eq 'c') {
-                            my %lti = &Apache::lonnet::get_course_lti($cnum,$cdom,'provider');
-                            if (ref($lti{$ltinum}) eq 'HASH') {
-                                $appname = $lti{$ltinum}{'name'};
-                            }
-                        } elsif ($ltitype eq 'd') {
-                            my %lti = &Apache::lonnet::get_domain_lti($cdom,'linkprot');
-                            if (ref($lti{$ltinum}) eq 'HASH') {
-                                $appname = $lti{$ltinum}{'name'};
-                            }
-                        }
-                        $no_passback .= " LTI launcher $linkprotector ($appname) for $linkuri (${cdom}_${cnum})";
-                        &Apache::lonnet::logthis($no_passback." for $uname:$udom");
-                        &Apache::lonnet::log($udom,$uname,$uhome,"$no_passback score=$score total=$total poss=$possible");
-                        if ($item->{'linkprot'}) {
-                            my $pendingkey = &Time::HiRes::time().':'.$uname.':'.$udom.':'.
-                                             "$linkuri\0$linkprotector\0$scope";
-                            &Apache::lonnet::put('linkprot_passback_pending',{$pendingkey => $item},$cdom,$cnum);
-                        }
-                    }
-                }
-            }
-        }
-    }
-    return;
-}
-
-sub get_lti_score {
-    my ($uname,$udom,$usec,$mapurl,$pbscope,$totals,$possibles) = @_;
-    my $navmap = Apache::lonnavmaps::navmap->new($uname,$udom,$usec);
-    if (ref($navmap)) {
-        my $iterator;
-        if ($mapurl ne '') {
-            my $map = $navmap->getResourceByUrl($mapurl);
-            if (ref($map)) {
-                my $firstres = $map->map_start();
-                my $finishres = $map->map_finish();
-                my $recursive = 1;
-                if ($pbscope eq 'nonrec') {
-                    $recursive = 0;
-                }
-                $iterator = $navmap->getIterator($firstres,$finishres,undef,$recursive);
-            }
-        } else {
-            $iterator = $navmap->getIterator(undef,undef,undef,1);
-        }
-        if (ref($iterator)) {
-            my $depth = 1;
-            my $total = 0;
-            my $possible = 0;
-            my (%totals_by_symb,%possibles_by_symb);
-            if (ref($totals) eq 'HASH') {
-                %totals_by_symb = %{$totals};
-            }
-            if (ref($possibles) eq 'HASH') {
-                %possibles_by_symb = %{$possibles};
-            }
-            $iterator->next(); # ignore first BEGIN_MAP
-            my $curRes = $iterator->next();
-            while ( $depth > 0 ) {
-                if ($curRes == $iterator->BEGIN_MAP()) {$depth++;}
-                if ($curRes == $iterator->END_MAP()) { $depth--; }
-                if (ref($curRes) && $curRes->is_gradable() && !$curRes->randomout) {
-                    my $currsymb = $curRes->symb();
-                    if (($currsymb) && (exists($totals_by_symb{$currsymb})) &&
-                        (exists($possibles_by_symb{$currsymb}))) {
-                        $total += $totals_by_symb{$currsymb};
-                        $possible += $possibles_by_symb{$currsymb};
-                    } else {
-                        my $parts = $curRes->parts();
-                        foreach my $part (@{$parts}) {
-                            next if ($curRes->solved($part) eq 'excused');
-                            $total += $curRes->weight($part) * $curRes->awarded($part);
-                            $possible += $curRes->weight($part);
-                        }
-                    }
-                }
-                $curRes = $iterator->next();
-            }
-            if ($total > $possible) {
-                $total = $possible;
-            }
-            return ($total,$possible);
-        }
-    }
-    return;
-}
-
 1;
 __END__