--- loncom/interface/groupboards.pm	2006/07/14 17:23:49	1.6
+++ loncom/interface/groupboards.pm	2006/11/02 21:06:06	1.10
@@ -28,10 +28,10 @@ package Apache::groupboards;
 
 use strict;
 use Apache::Constants qw(:common :http);
-use Apache::loncommon;
+use Apache::loncommon();
 use Apache::lonnet;
-use Apache::lonuserstate;
-use Apache::lonratedt;
+use Apache::lonuserstate();
+use LONCAPA::map();
 use Apache::lonlocal;
 use LONCAPA;
 
@@ -50,7 +50,7 @@ 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'};
@@ -59,23 +59,27 @@ sub handler {
     my $gpterm =  &Apache::loncommon::group_term();
     my $ucgpterm = $gpterm;
     $ucgpterm =~ s/^(\w)/uc($1)/e;
-    my $bodytitle = &mt('[_1] Discussion Boards',$crstype);
+    my $bodytitle = $crstype.' Discussion Boards';
     my $group = $env{'form.group'};
     $group =~ s/\W//g;
-    my ($description,$earlyout);
+    my ($description,$earlyout,$refarg);
+
+    if (exists($env{'form.ref'})) {
+        $refarg = 'ref='.$env{'form.ref'};
+    }
 
     &Apache::lonhtmlcommon::clear_breadcrumbs();
     if (!defined($group)) {
         $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));
+                                 $ucgpterm,$bodytitle,$earlyout,$refarg));
         return OK;
     }
     my %curr_groups = &Apache::longroup::coursegroups($cdom,$cnum,$group);
     if (!defined($curr_groups{$group})) {
-        $earlyout = &mt('Invalid [_1]',$gpterm);
+        $earlyout = &mt('Invalid group');
         $r->print(&display_error($cdom,$cnum,$group,$description,$gpterm,
-                                 $ucgpterm,$bodytitle,$earlyout));
+                                 $ucgpterm,$bodytitle,$earlyout,$refarg));
         return OK;
     }
     my %content = &Apache::longroup::get_group_settings($curr_groups{$group});
@@ -83,34 +87,37 @@ sub handler {
 
     my $can_create=&Apache::lonnet::allowed('cgb',$env{'request.course.id'}.
                                             '/'.$group);
-    my $can_view = &Apache::lonnet::allowed('vcg',$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('pgd',$env{'request.course.id'}.
+        $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));
+                                 $ucgpterm,$bodytitle,$earlyout,$refarg));
         return OK;
     }
     if (defined($env{'form.newbul'})) {
         if (($can_create) || (&Apache::lonnet::allowed('mdg',$env{'request.course.id'}))) {
             $r->print(&boards_header($cdom,$cnum,$group,$description,$gpterm,
-                                     $ucgpterm,$bodytitle));
-            my ($outcome,$newurl,$bbtitle) = 
+                                     $ucgpterm,$bodytitle,$refarg));
+            my ($outcome,$newurl,$bbtitle,$dellockoutcome) = 
 		&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.<br />'));
                 $r->print('<table border="0"><tr><td>'.
-                          '<a href="'.$newurl.'?group='.$group.'">'.
-                          &mt('Edit [_1] board',$bbtitle).'</a></td>'.
+                          '<a href="'.$newurl.'?group='.$group.'&amp;'.$refarg.
+                          '">'.&mt('Edit [_1] board',$bbtitle).'</a></td>'.
                            '<td>&nbsp;&nbsp;</td><td>'.
-                          '<a href="/adm/groupboards?group='.$group.'">'.
-                          &mt('View all group discussion boards').
+                          '<a href="/adm/groupboards?group='.$group.'&amp;'.
+                          $refarg.'">'.&mt('View all group discussion boards').
                           '</a></td></tr></table>');
+                if ($dellockoutcome ne 'ok') {
+                    $r->print(&mt('There was a problem removing a lockfile for the group ([_1]). This may prevent creation of additional bulletin boards in this group.  Please contact the system administrator for your LON-CAPA domain.'));
+                }
             } else {
                 $r->print(&mt('There was a problem creating the new discussion board - [_1]','<span class="LC_error">'.$outcome.'</span>').'<br /><a href="/adm/groupboards?group='.$group.'">'.
                          &mt('Return to discussion boards').'</a>');
@@ -126,8 +133,7 @@ sub handler {
 function makebulboard() {
    var title=prompt('Discussion Board Title');
    if (title) {
-    this.document.forms.newbb.newbul.value=
-        title+'=/adm/$cdom/$cnum/$now/bulletinboard';
+    this.document.forms.newbb.newbul.value=title;
     this.document.forms.newbb.submit();
    }
 }
@@ -135,7 +141,7 @@ function makebulboard() {
 |;
     }
     $r->print(&boards_header($cdom,$cnum,$group,$description,$gpterm,$ucgpterm,
-                             $bodytitle,$jscript));
+                             $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'}))) {
@@ -143,8 +149,9 @@ function makebulboard() {
                   "\n".'<input type="button" name="bbbutton" value="'.
                   &mt('New Discussion Board').
                   '" onClick="javascript:makebulboard();" />'."\n".
-                  '  <input type="hidden" name="newbul" />'."\n".
-                  '  <input type="hidden" name="group" value="'.$group.'" />'.
+                  '<input type="hidden" name="newbul" />'."\n".
+                  '<input type="hidden" name="group" value="'.$group.'" />'.
+                  '<input type="hidden" name="ref" value="'.$env{'form.ref'}.'" />'.
                   "\n".'</form><br />');
     }
     if (@{$groupboards} > 0) {
@@ -156,7 +163,8 @@ function makebulboard() {
             } else {
                 $board_url .= '?group='.$group;
             }
-            $r->print('<a href="'.$board_url.'">'.$$boards{$board}{'title'}.'</a><br />');
+            $r->print('<a href="'.$board_url.'&amp;'.$refarg.'">'.
+                      $$boards{$board}{'title'}.'</a><br />');
         }
     } else {
         $r->print('<br />'.&mt('There are currently no discussion boards in this [_1].',
@@ -167,25 +175,68 @@ function makebulboard() {
 }
 
 sub create_board {
-    my ($cdom,$cnum,$group,$newboard) = @_;
-    my ($bbtitle,$newurl)=split(/\=/,$newboard);
-    my $outcome;
-    my ($boardid) = ($newurl =~ m-/adm/\Q$cdom\E/\Q$cnum\E/(\d+)/bulletinboard-);
-    if (!$boardid) {
-        $outcome = ('error: the URL for new board was invalid');
-        return ($outcome,$newurl,$bbtitle);
-    }
+    my ($cdom,$cnum,$group,$bbtitle) = @_;
+    my ($outcome,$boardid,$newurl);
     $bbtitle=&unescape($bbtitle);
+    # get lock on nohist_groupboards file
+    my $lockhash = {
+                  $group."\0".'locked_boardids' => $env{'user.name'}.
+                                                       ':'.$env{'user.domain'},
+                   };
+    my $tries = 0;
+    my $gotlock = &Apache::lonnet::newput('nohist_groupboards',$lockhash,$cdom,$cnum);
+    my $dellockoutcome;
+    while (($gotlock ne 'ok') && $tries <3) {
+        $tries ++;
+        sleep 1;
+        $gotlock = &Apache::lonnet::newput('nohist_groupboards',$lockhash,$cdom,$cnum);
+    }
+    if ($gotlock eq 'ok') {
+       my %curr_boards = &Apache::lonnet::dump('nohist_groupboards',$cdom,$cnum,$group);
+       $boardid = time;
+       my $idtries = 0;
+       while(exists($curr_boards{$group."\0".$boardid}) && $idtries < 20) {
+           $boardid ++;
+           $idtries ++;
+       }
+       if (!exists($curr_boards{$group."\0".$boardid})) {
+           my %new_board = (
+                          $group."\0".$boardid => $env{'user.name'}.':'.
+                                                  $env{'user.domain'},
+           );
+           my $putresult = &Apache::lonnet::put('nohist_groupboards',\%new_board,
+                                                $cdom,$cnum);
+           if ($putresult ne 'ok') {
+               $outcome = 'error storing new board: '.$putresult;
+           } else {
+               $newurl = '/adm/'.$cdom.'/'.$cnum.'/'.$boardid.
+                         '/bulletinboard';
+           }
+       } else {
+            $outcome = ('error: no unique ID for the new board available.');
+       }
+       #  remove lock
+       my @del_lock = ($group."\0".'locked_boardids');
+       $dellockoutcome = &Apache::lonnet::del('nohist_groupboards',\@del_lock,$cdom,$cnum);
+    } else {
+        $outcome = "error: could not obtain lockfile\n";
+        $dellockoutcome = 'ok';
+    }
+    if (!$newurl) {
+        return ($outcome,$newurl,$bbtitle,$dellockoutcome);
+    }
     $newurl=&unescape($newurl);
+    # 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);
     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);
             if ($fatal) {
                 $outcome = "error: failed to store discussion boards map - $errtext\n";
             } else {
@@ -202,26 +253,34 @@ sub create_board {
         $outcome = 'error: discussion boards folder absent, '.
                    'or in unexpected location - '.$allbbsmap."\n";
     }
-    return ($outcome,$newurl,$bbtitle);
+    return ($outcome,$newurl,$bbtitle,$dellockoutcome);
 }
 
 sub display_error {
-    my ($cdom,$cnum,$group,$description,$gpterm,$ucgpterm,$bodytitle,$earlyout)=@_;
+    my ($cdom,$cnum,$group,$description,$gpterm,$ucgpterm,$bodytitle,$earlyout,
+        $refarg)=@_;
     my $output = &boards_header($cdom,$cnum,$group,$description,$gpterm,
-                                $ucgpterm,$bodytitle);
+                                $ucgpterm,$bodytitle,$refarg);
     $output .= $earlyout;
     $output .= &Apache::loncommon::end_page();
     return $output;
 }
 
 sub boards_header {
-    my ($cdom,$cnum,$group,$description,$gpterm,$ucgpterm,$bodytitle,$jscript)=@_;
+    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",
+        ({href=>"/adm/$cdom/$cnum/$group/smppg?$refarg",
           text=>"$ucgpterm: $description",
           title=>"Go to group's home page"},
-         {href=>"/adm/groupboards?group=$group",
+         {href=>"/adm/groupboards?group=$group&amp;$refarg",
           text=>"Discussion Boards",
           title=>"Display group discussion boards"},);
     $output .= &Apache::lonhtmlcommon::breadcrumbs(&mt('[_1] discussion boards - [_2]',$gpterm,$description));