[LON-CAPA-cvs] cvs: loncom /interface portfolio.pm

raeburn raeburn@source.lon-capa.org
Fri, 26 Mar 2010 22:27:06 -0000


This is a MIME encoded message

--raeburn1269642426
Content-Type: text/plain

raeburn		Fri Mar 26 22:27:06 2010 EDT

  Modified files:              
    /loncom/interface	portfolio.pm 
  Log:
  - Accommodate Communities.
  - Remove html from phrases to be translated. 
  
  
--raeburn1269642426
Content-Type: text/plain
Content-Disposition: attachment; filename="raeburn-20100326222706.txt"

Index: loncom/interface/portfolio.pm
diff -u loncom/interface/portfolio.pm:1.220 loncom/interface/portfolio.pm:1.221
--- loncom/interface/portfolio.pm:1.220	Fri Feb 19 10:20:31 2010
+++ loncom/interface/portfolio.pm	Fri Mar 26 22:27:06 2010
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # portfolio browser
 #
-# $Id: portfolio.pm,v 1.220 2010/02/19 10:20:31 bisitz Exp $
+# $Id: portfolio.pm,v 1.221 2010/03/26 22:27:06 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -38,7 +38,7 @@
 use Apache::longroup;
 use Apache::lonhtmlcommon;
 use HTML::Entities;
-use LONCAPA;
+use LONCAPA qw(:DEFAULT :match);
 
 sub group_args {
     my $output;
@@ -880,7 +880,7 @@
 sub explain_conditionals {
     return
         &mt('Conditional files are accessible to logged-in users with accounts in the LON-CAPA network, who satisfy the conditions you set.').'<br />'."\n".
-        &mt('The conditions can include affiliation with a particular course, or a user account in a specific domain.').'<br />'."\n".
+        &mt('The conditions can include affiliation with a particular course or community, or a user account in a specific domain.').'<br />'."\n".
         &mt('Alternatively access can be granted to people with specific LON-CAPA usernames and domains.');
 }
 
@@ -921,7 +921,7 @@
                       guest => 'Passphrase-protected',
                       domains => 'Conditional: domain-based',
                       users => 'Conditional: user-based',
-                      course => 'Conditional: course-based',
+                      course => 'Conditional: course/community-based',
                      );
     my @allscopes = ('public','guest','domains','users','course');
     foreach my $scope (@allscopes) {
@@ -946,11 +946,16 @@
                 }
             }
             $r->print('<td>'.&mt($scope_desc{$scope}));
+            my $crstype;
             if ($scope eq 'course') {
                 if ($chg ne 'delete') {
                     my $cid = $content->{'domain'}.'_'.$content->{'number'};
                     my %course_description = &Apache::lonnet::coursedescription($cid);
                     $r->print('<br />('.$course_description{'description'}.')');
+                    $crstype = 'Course';
+                    if ($course_description{'type'} ne '') {
+                        $crstype = $course_description{'type'};
+                    }
                 }
             }
             $r->print('</td><td>'.&mt('Start: ').$showstart.
@@ -970,8 +975,6 @@
                         foreach my $item ('role','access','section','group') {
                             $r->print('<td>');
                             if ($item eq 'role') {
-                                my $ucscope = $scope;
-                                $ucscope =~ s/^(\w)/uc($1)/e;
                                 my $role_output;
                                 foreach my $role (@{$content->{'roles'}{$id}{$item}}) {
                                     if ($role eq 'all') {
@@ -979,7 +982,7 @@
                                     } elsif ($role =~ /^cr/) {
                                         $role_output .= (split('/',$role))[3].',';
                                     } else {
-                                        $role_output .= &Apache::lonnet::plaintext($role,$ucscope).',';
+                                        $role_output .= &Apache::lonnet::plaintext($role,$crstype).',';
                                     }
                                 }
                                 $role_output =~ s/,$//;
@@ -1105,16 +1108,22 @@
     my $totalnew = 0;
     my $status = 'new';
     my ($firstitem,$lastitem);
-    foreach my $newitem ('course','domains','users') {
+    my @types = ('course','domains','users');
+    foreach my $newitem (@types) {
         $allnew += $env{'form.new'.$newitem};
     }
     if ($allnew > 0) {
         my $now = time;
         my $then = $now + (60*60*24*180); # six months approx.
         &open_form($r,$url);
-        foreach my $newitem ('course','domains','users') {
+        my %showtypes = &Apache::lonlocal::texthash (
+           course  => 'course/community',
+           domains => 'domain',
+           users   => 'user',
+        );
+        foreach my $newitem (@types) {
             if ($env{'form.new'.$newitem} > 0) {
-                $r->print('<br />'.&mt('Add new <b>[_1]-based</b> access control for portfolio file: <b>[_2]</b>',&mt($newitem),$env{'form.currentpath'}.$env{'form.selectfile'}).'<br /><br />');
+                $r->print('<br />'.&mt('Add new [_1]-based[_2] access control for portfolio file: [_3]','<b>'.$showtypes{$newitem},'</b>','<b>'.$env{'form.currentpath'}.$env{'form.selectfile'}.'</b>').'<br /><br />');
                 $firstitem = $totalnew;
                 $lastitem = $totalnew + $env{'form.new'.$newitem};
                 $totalnew = $lastitem;
@@ -1391,10 +1400,12 @@
 
 sub access_element {
     my ($r,$type,$acl_count,$items,$access_controls,$now,$then) = @_;
-    my $title = $type;
-    $title =~ s/s$//;
-    $title =~ s/^(\w)/uc($1)/e;
-    $r->print('<h3>'.&mt('[_1]-based conditional access: ',&mt($title)));
+    my %typetext = &Apache::lonlocal::texthash(
+        domains => 'Domain',
+        users   => 'User',
+        course  => 'Course/Community'
+    );
+    $r->print('<h3>'.&mt('[_1]-based conditional access: ',$typetext{$type}).':&nbsp;');
     if ($$acl_count{$type}) {
         $r->print($$acl_count{$type}.' ');
         if ($$acl_count{$type} > 1) {
@@ -1412,17 +1423,24 @@
 
 sub display_access_row {
     my ($r,$status,$type,$items,$access_controls,$now,$then) = @_;
+    my $showtype;
+    if ($type eq 'course') {
+        $showtype = &mt('Courses/Communities');
+    } elsif ($type eq 'domains') {
+        $showtype = &mt('Domains');
+    } elsif ($type eq 'users') {
+        $showtype = &mt('Users');
+    }
     if (@{$items} > 0) {
         my @all_doms;
         my $colspan = 3;
-        my $uctype = $type;
-        $uctype =~ s/^(\w)/uc($1)/e;
         $r->print(&Apache::loncommon::start_data_table());
         $r->print(&Apache::loncommon::start_data_table_header_row());
-        $r->print('<th>'.&mt('Action?').'</th><th>'.&mt($uctype).'</th><th>'.
+        $r->print('<th>'.&mt('Action?').'</th><th>'.$showtype.'</th><th>'.
               &mt('Dates available').'</th>');
         if ($type eq 'course' && $status eq 'old') {
-            $r->print('<th>'.&mt('Allowed [_1] member affiliations',$type).
+            
+            $r->print('<th>'.&mt('Allowed course/community affiliations').
                       '</th>');
             $colspan ++;
         } elsif ($type eq 'domains') {
@@ -1449,8 +1467,8 @@
         }
         $r->print(&Apache::loncommon::end_data_table());
     } else {
-        $r->print(&mt('No [_1]-based conditions defined.',&mt($type)).'<br />'
-                  .&additional_item($type));
+        $r->print(&mt('No [_1]-based conditions defined.',$showtype).'<br />'.
+                  &additional_item($type));
     }
     return;
 }
@@ -1511,17 +1529,20 @@
     }
     my $js = &Apache::loncommon::coursebrowser_javascript($defdom)
 	.&course_js();
-    my $uctype = $type;
-    $uctype =~ s/^(\w)/uc($1)/e;
+    my $showtype = &mt('Course/Community');
+    my $crstype = 'Course';
     my ($num,$scope,$end,$start) = &set_identifiers($status,$item,$now,$then,
                                                     $type);
     $r->print('<td>'.$js.&actionbox($status,$num,$scope).'</td>');
     if ($status eq 'old') {
         my $cid = $content->{'domain'}.'_'.$content->{'number'};
         my %course_description = &Apache::lonnet::coursedescription($cid);
+        if ($course_description{'type'} ne '') {
+            $crstype = $course_description{'type'};
+        }
         $r->print('<td><input type="hidden" name="crsdom_'.$num.'" value="'.$content->{'domain'}.'" /><input type="hidden" name="crsnum_'.$num.'" value="'.$content->{'number'}.'" />'.$course_description{'description'}.'</td>');
     } elsif ($status eq 'new') {
-        $r->print('<td>'.&Apache::loncommon::selectcourse_link('portform','crsnum_'.$num,'crsdom_'.$num,'description_'.$num,$num.'_1',undef,$uctype).'&nbsp;&nbsp;<input type="text" name="description_'.$num.'" size="30" /><input type="hidden" name="crsdom_'.$num.'" /><input type="hidden" name="crsnum_'.$num.'" /></td>');
+        $r->print('<td>'.&Apache::loncommon::selectcourse_link('portform','crsnum_'.$num,'crsdom_'.$num,'description_'.$num,$num.'_1',undef,$showtype).'&nbsp;&nbsp;<input type="text" name="description_'.$num.'" size="30" /><input type="hidden" name="crsdom_'.$num.'" /><input type="hidden" name="crsnum_'.$num.'" /></td>');
     }
     $r->print('<td>'.&dateboxes($num,$start,$end));
     my $newrole_id = 1;
@@ -1539,7 +1560,7 @@
                     $max_id = $role_id;
                 }
                 $max_id ++;
-                my $role_selects = &role_selectors($num,$role_id,$type,$content,'display');
+                my $role_selects = &role_selectors($num,$role_id,$crstype,$content,'display');
                 $r->print('<tr><td><span class="LC_nobreak"><label><input type="checkbox" name="delete_role_'.$num.'" value="'.$role_id.'" />'.&mt('Delete').'</label></span><br /><input type="hidden" name="preserve_role_'.$num.'" value="'.$role_id.'" /></td>'.$role_selects.'</tr>');
             }
             $r->print('</table>');
@@ -1548,7 +1569,7 @@
                   '&nbsp;<input type="checkbox" name="add_role_'.
                   $num.'" onClick="javascript:setRoleOptions('."'$num',
                   '$max_id','$content->{'domain'}','$content->{'number'}',
-                  '$uctype'".')" value="" />');
+                  '$showtype'".')" value="" />');
         $newrole_id = $max_id;
     } else {
         $r->print('<input type="hidden" name="add_role_'.$num.'" value="" />');
@@ -1609,7 +1630,15 @@
 
 sub additional_item {
     my ($type) = @_;
-    my $output = &mt('Add new [_1] condition(s)?',&mt($type)).'&nbsp;'.&mt('Number to add: ').'<input type="text" name="new'.$type.'" size="3" value="0" />';
+    my $showtype;
+    if ($type eq 'course') {
+        $showtype = &mt('course/community');
+    } elsif ($type eq 'domains') {
+        $showtype = &mt('domains');
+    } elsif ($type eq 'users') {
+        $showtype = &mt('users');
+    }
+    my $output = &mt('Add new [_1] condition(s)?',$showtype).'&nbsp;'.&mt('Number to add: ').'<input type="text" name="new'.$type.'" size="3" value="0" />';
     return $output;
 }
 
@@ -1680,10 +1709,12 @@
          $cdom = $env{'form.cdom'};
          $cnum = $env{'form.cnum'};
     }
-    my $uctype = $type;
-    $uctype =~ s/^(\w)/uc($1)/e;
+    my $crstype = 'Course';
+    if ($cnum =~ /^$match_community$/) {
+        $crstype = 'Community'
+    }
     my ($sections,$groups,$allroles,$rolehash,$accesshash) =
-            &Apache::loncommon::get_secgrprole_info($cdom,$cnum,1,$uctype);
+            &Apache::loncommon::get_secgrprole_info($cdom,$cnum,1,$crstype);
     if (!@{$sections}) {
         @{$sections} = ('none');
     } else {

--raeburn1269642426--