--- loncom/interface/groupboards.pm 2006/06/30 18:27:19 1.4 +++ loncom/interface/groupboards.pm 2013/08/17 00:34:29 1.19 @@ -1,6 +1,8 @@ # The LearningOnline Network # Group Bulletin Boards Manager # +# $Id: groupboards.pm,v 1.19 2013/08/17 00:34:29 raeburn Exp $ +# # Copyright Michigan State University Board of Trustees # # This file is part of the LearningOnline Network with CAPA (LON-CAPA). @@ -28,11 +30,10 @@ package Apache::groupboards; use strict; use Apache::Constants qw(:common :http); -use Apache::loncommon; +use Apache::loncommon(); use Apache::lonnet; -use Apache::lonnavmaps; -use Apache::lonuserstate; -use Apache::lonratedt; +use Apache::lonuserstate(); +use LONCAPA::map(); use Apache::lonlocal; use LONCAPA; @@ -51,40 +52,78 @@ sub handler { } &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'}, - ['group']); + ['group','ref']); my $cdom = $env{'course.'.$env{'request.course.id'}.'.domain'}; my $cnum = $env{'course.'.$env{'request.course.id'}.'.num'}; my $now = time; my $crstype = &Apache::loncommon::course_type(); my $gpterm = &Apache::loncommon::group_term(); - my $bodytitle = &mt('[_1] Discussion Boards',$crstype); + my $ucgpterm = $gpterm; + $ucgpterm =~ s/^(\w)/uc($1)/e; + my $bodytitle = $ucgpterm.' Discussion Boards'; my $group = $env{'form.group'}; + $group =~ s/\W//g; + my ($description,$earlyout,$refarg); + + if (exists($env{'form.ref'})) { + $refarg = 'ref='.$env{'form.ref'}; + } + + &Apache::lonhtmlcommon::clear_breadcrumbs(); if (!defined($group)) { - $r->print(&Apache::loncommon::start_page($bodytitle)); - $r->print(&mt('No [_1] defined, so there are no [_1] discussion boards to display',$gpterm)); - $r->print(&Apache::loncommon::end_page()); + $earlyout = &mt('No [_1] defined, so there are no [_1] discussion boards to display',$gpterm); + $r->print(&display_error($cdom,$cnum,$group,$description,$gpterm, + $ucgpterm,$bodytitle,$earlyout,$refarg)); return OK; } + my %curr_groups = &Apache::longroup::coursegroups($cdom,$cnum,$group); + if (!defined($curr_groups{$group})) { + $earlyout = &mt('Invalid group'); + $r->print(&display_error($cdom,$cnum,$group,$description,$gpterm, + $ucgpterm,$bodytitle,$earlyout,$refarg)); + return OK; + } + my %content = &Apache::longroup::get_group_settings($curr_groups{$group}); + $description = &unescape($content{'description'}); + my $can_create=&Apache::lonnet::allowed('cgb',$env{'request.course.id'}. '/'.$group); - my $can_view=&Apache::lonnet::allowed('pgd',$env{'request.course.id'}. + my $can_view = &Apache::lonnet::allowed('vcg',$env{'request.course.id'}.($env{'request.course.sec'}?'/'.$env{'request.course.sec'}:'')); + + if (!$can_view) { + $can_view = &Apache::lonnet::allowed('vgb',$env{'request.course.id'}. '/'.$group); + } + if (!$can_view) { + $earlyout=&mt('You do not have privileges to view discussion boards in this [_1]',$gpterm); + $r->print(&display_error($cdom,$cnum,$group,$description,$gpterm, + $ucgpterm,$bodytitle,$earlyout,$refarg)); + return OK; + } if (defined($env{'form.newbul'})) { if (($can_create) || (&Apache::lonnet::allowed('mdg',$env{'request.course.id'}))) { - $r->print(&Apache::loncommon::start_page($bodytitle)); - my ($outcome,$newurl,$bbtitle) = + $r->print(&boards_header($cdom,$cnum,$group,$description,$gpterm, + $ucgpterm,$bodytitle,$refarg)); + my ($outcome,$newurl,$bbtitle,$lockfreed) = &create_board($cdom,$cnum,$group,$env{'form.newbul'}); if ($outcome eq 'ok') { my ($furl,$ferr)= &Apache::lonuserstate::readmap($cdom.'/'.$cnum); - $r->print(&mt('The new discussion board was added successfully.
')); + $r->print(&mt('The new discussion board was added successfully.').'
'); $r->print(''. + ''.&mt('Edit [_1] board',$bbtitle).''. '
'. - ''. - &mt('Edit [_1] board',$bbtitle).'  '. - ''. - &mt('View all group discussion boards'). + ''.&mt('View all group discussion boards'). '
'); + if ($lockfreed ne 'ok') { + $r->print(&mt('There was a problem removing a lockfile for the group ([_1]).',$description).'
'. + &mt('This may prevent creation of additional bulletin boards in this group.').'
'. + &mt('Please contact the [_1]helpdesk[_2] for assistance.', + '','') + ); + } } else { $r->print(&mt('There was a problem creating the new discussion board - [_1]',''.$outcome.'').'
'. &mt('Return to discussion boards').''); @@ -96,66 +135,97 @@ sub handler { my $jscript; if (($can_create) || (&Apache::lonnet::allowed('mdg',$env{'request.course.id'}))) { $jscript = qq| + |; } - $r->print(&Apache::loncommon::start_page($bodytitle, - '')); - if (!$can_view) { - $r->print(&mt('You do not have privileges to view discussion boards in this [_1]',$crstype)); - return OK; - } - my $navmap = Apache::lonnavmaps::navmap->new(); + $r->print(&boards_header($cdom,$cnum,$group,$description,$gpterm,$ucgpterm, + $bodytitle,$refarg,$jscript)); my ($groupboards,$boards) = &Apache::longroup::get_group_bbinfo($cdom,$cnum, $group); if (($can_create) || (&Apache::lonnet::allowed('mdg',$env{'request.course.id'}))) { - $r->print('
'. + $r->print('
'. "\n".''."\n". - ' '."\n". - ' '. + '" onclick="javascript:makebulboard();" />'."\n". + ''."\n". + ''. + ''. "\n".'

'); } if (@{$groupboards} > 0) { + $r->print('
' + .&Apache::loncommon::start_data_table() + .&Apache::loncommon::start_data_table_header_row() + .''.&mt('Discussion Boards').'' + .&Apache::loncommon::end_data_table_header_row() + ); foreach my $board (@{$groupboards}) { - $r->print(''.$$boards{$board}{'title'}.'
'); + my $board_url = $$boards{$board}{'url'}; + if ($board_url =~ /\?/) { + $board_url .= '&group='.$group; + } else { + $board_url .= '?group='.$group; + } + $r->print(&Apache::loncommon::start_data_table_row() + .'' + .'' + .$$boards{$board}{'title'}.'' + .'' + .&Apache::loncommon::end_data_table_row() + ); } + $r->print(&Apache::loncommon::end_data_table()); } else { - $r->print(&mt('There are currently no discussion boards in this [_1].', - $gpterm)); + $r->print('

' + .&mt('There are currently no discussion boards in this '.$gpterm.'.',) + .'

' + ); } $r->print(&Apache::loncommon::end_page()); return OK; } sub create_board { - my ($cdom,$cnum,$group,$newboard) = @_; - my ($bbtitle,$newurl)=split(/\=/,$newboard); + my ($cdom,$cnum,$group,$bbtitle) = @_; + my ($outcome,$newurl,$idtype); $bbtitle=&unescape($bbtitle); - $newurl=&unescape($newurl); + $idtype = 'inc'; + my ($boardid,$dellock,$error) = + &Apache::lonnet::get_timebased_id($group,'boardids','groupboards', + $cdom,$cnum); + if ($boardid) { + $newurl = '/adm/'.$cdom.'/'.$cnum.'/'.$boardid.'/bulletinboard'; + } else { + return ($error,$newurl,$bbtitle,$dellock); + } + + # need to check here if group_boards_$group.sequence is in the course + # if not - add it as an item in group_folder_$group.sequence my $allbbsmap = &Apache::longroup::get_bbfolder_url($cdom,$cnum,$group); - my ($outcome); if ($allbbsmap =~ m|^/uploaded|) { - my ($errtext,$fatal)=&Apache::lonratedt::mapread($allbbsmap); + my ($errtext,$fatal)=&LONCAPA::map::mapread($allbbsmap); if (!$fatal) { - my $newidx=&Apache::lonratedt::getresidx($newurl); - $Apache::lonratedt::resources[$newidx]=$bbtitle.':'.$newurl. + my $newidx=&LONCAPA::map::getresidx($newurl); + $LONCAPA::map::resources[$newidx]=$bbtitle.':'.$newurl. ':false:normal:res'; - push(@Apache::lonratedt::order,$newidx); - my ($errtext,$fatal)=&Apache::lonratedt::storemap($allbbsmap,1); + push(@LONCAPA::map::order,$newidx); + my ($errtext,$fatal)=&LONCAPA::map::storemap($allbbsmap,1,1); if ($fatal) { - $outcome = "error: failed to store discussion boards map - $errtext\n"; + $outcome = "error: failed to save discussion boards map - $errtext\n"; } else { - $outcome = 'ok'; + my %boardinfo = ( + 'group' => $group, + ); + $outcome = &Apache::lonnet::put('bulletinpage_'.$boardid, + \%boardinfo,$cdom,$cnum); } } else { $outcome = "error: failed to read all discussion boards map - $errtext\n"; @@ -164,7 +234,38 @@ sub create_board { $outcome = 'error: discussion boards folder absent, '. 'or in unexpected location - '.$allbbsmap."\n"; } - return ($outcome,$newurl,$bbtitle); + return ($outcome,$newurl,$bbtitle,$dellock); +} + +sub display_error { + my ($cdom,$cnum,$group,$description,$gpterm,$ucgpterm,$bodytitle,$earlyout, + $refarg)=@_; + my $output = &boards_header($cdom,$cnum,$group,$description,$gpterm, + $ucgpterm,$bodytitle,$refarg); + $output .= $earlyout; + $output .= &Apache::loncommon::end_page(); + return $output; +} + +sub boards_header { + my ($cdom,$cnum,$group,$description,$gpterm,$ucgpterm,$bodytitle,$refarg, + $jscript)=@_; + my $output = &Apache::loncommon::start_page($bodytitle,$jscript); + if ($refarg) { + &Apache::lonhtmlcommon::add_breadcrumb + ({href=>"/adm/coursegroups", + text=>"Groups", + title=>"View course groups"},); + } + &Apache::lonhtmlcommon::add_breadcrumb + ({href=>"/adm/$cdom/$cnum/$group/smppg?$refarg", + text=>"$ucgpterm: $description", + title=>"Go to group's home page"}, + {href=>"/adm/groupboards?group=$group&$refarg", + text=>"Discussion Boards", + title=>"Display group discussion boards"},); + $output .= &Apache::lonhtmlcommon::breadcrumbs(&mt('[_1] discussion boards - [_2]',$gpterm,$description)); + return $output; } 1;