--- loncom/publisher/lonrights.pm	2007/07/13 18:35:28	1.24
+++ loncom/publisher/lonrights.pm	2009/12/02 18:33:35	1.29
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler to show and edit custom distribution rights
 #
-# $Id: lonrights.pm,v 1.24 2007/07/13 18:35:28 albertel Exp $
+# $Id: lonrights.pm,v 1.29 2009/12/02 18:33:35 bisitz Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -55,9 +55,24 @@ sub handler {
   &Apache::loncommon::content_type($r,'text/html');
   $r->send_http_header;
 
+  # Breadcrumbs
+  &Apache::lonhtmlcommon::clear_breadcrumbs();
+  &Apache::lonhtmlcommon::add_breadcrumb({
+      'text'  => 'Construction Space',
+      'href'  => &Apache::loncommon::authorspace(),
+      });
+  &Apache::lonhtmlcommon::add_breadcrumb({
+      'text'  => 'Custom Rights Editor',
+      'title' => 'Custom Distribution Rights Editor',
+      'href'  => '',});
+
   my $js = &Apache::loncommon::coursebrowser_javascript().
       &Apache::loncommon::studentbrowser_javascript();
-  $r->print(&Apache::loncommon::start_page('Custom Distribution Rights',$js));
+  # Breadcrumbs are included by &start_page
+  $r->print(&Apache::loncommon::start_page('Custom Distribution Rights',$js)
+           .&Apache::loncommon::head_subbox(
+                &Apache::loncommon::CSTR_pageheader())
+  );
   $r->rflush();
 
   my $uri=$r->uri;
@@ -174,7 +189,9 @@ sub handler {
   my $token;
   my $rulecounter=0;
   my $colzero=&mt($constructmode?'Edit action':'Rule');
-  my %lt=&Apache::lonlocal::texthash('ef' => 'Effect',
+  my %lt=&Apache::lonlocal::texthash(
+				     'ef' => 'Effect',
+				     'ty' => 'Type',
 				     'do' => 'Domain',
 				     'co' => 'Course / User',
 				     'se' => 'Section',
@@ -182,7 +199,7 @@ sub handler {
 # ---------------------------------------------------------- Start table output
   $r->print(&Apache::loncommon::start_data_table().
 	    &Apache::loncommon::start_data_table_header_row().
-	    "<th>$colzero</th><th>$lt{'ef'}</th><th>Type</th><th>$lt{'do'}</th>".
+	    "<th>$colzero</th><th>$lt{'ef'}</th><th>$lt{'ty'}</th><th>$lt{'do'}</th>".
 	    "<th>$lt{'co'}</th><th>$lt{'se'}</th><th>$lt{'ro'}</th>".
 	    &Apache::loncommon::end_data_table_header_row());
 # --------------------------------------------------------------------- Default
@@ -194,7 +211,7 @@ sub handler {
   if ($constructmode) {
      $r->print(&Apache::loncommon::select_form('','action_0',
                                   ('' => '',
-				   'insertbelow' => 'Insert rule below    ')));
+                                   'insertbelow' => &mt('Insert rule below'))));
                                                 
   } else {
       $r->print('&nbsp;');
@@ -203,6 +220,7 @@ sub handler {
   if ($constructmode) {
       $r->print(&Apache::loncommon::select_form
                                  ($token->[2]->{'effect'},'effect_0',
+                                  &Apache::lonlocal::texthash
                                    ('allow' => 'allow',
                                     'deny'  => 'deny')));
   } else {
@@ -225,10 +243,11 @@ sub handler {
           if ($constructmode) {
              $r->print(&Apache::loncommon::select_form(
                     '','action_'.$rulecounter,
+                    &Apache::lonlocal::texthash
                     ('' => '', 
                      'delete' => 'Delete this rule',
                      'insertabove' => 'Insert rule above',
-                     'insertbelow' => 'Insert rule below    ',
+                     'insertbelow' => 'Insert rule below',
                      'moveup'      => 'Move rule up',
                      'movedown'    => 'Move rule down')));
 	  }
@@ -239,6 +258,7 @@ sub handler {
              $r->print(&Apache::loncommon::select_form
                                  ($token->[2]->{'effect'},
                                   'effect_'.$rulecounter,
+                                  &Apache::lonlocal::texthash
                                    ('allow' => 'allow',
                                     'deny'  => 'deny')));
           } else {
@@ -251,6 +271,7 @@ sub handler {
           if ($constructmode) {
               $r->print(&Apache::loncommon::select_form($type,
 							'type_'.$rulecounter,
+                            &Apache::lonlocal::texthash
 							('course' => 'Course',
 							 'user'   => 'User')));
           } else {