--- loncom/interface/groupboards.pm	2006/06/30 18:27:19	1.4
+++ loncom/interface/groupboards.pm	2006/07/17 14:52:00	1.7
@@ -30,7 +30,6 @@ use strict;
 use Apache::Constants qw(:common :http);
 use Apache::loncommon;
 use Apache::lonnet;
-use Apache::lonnavmaps;
 use Apache::lonuserstate;
 use Apache::lonratedt;
 use Apache::lonlocal;
@@ -51,39 +50,70 @@ 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 $ucgpterm = $gpterm;
+    $ucgpterm =~ s/^(\w)/uc($1)/e;
     my $bodytitle = &mt('[_1] Discussion Boards',$crstype);
     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 [_1]',$gpterm);
+        $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));
+            $r->print(&boards_header($cdom,$cnum,$group,$description,$gpterm,
+                                     $ucgpterm,$bodytitle,$refarg));
             my ($outcome,$newurl,$bbtitle) = 
 		&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>');
             } 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.'">'.
@@ -96,6 +126,7 @@ sub handler {
     my $jscript;
     if (($can_create) || (&Apache::lonnet::allowed('mdg',$env{'request.course.id'}))) {
         $jscript = qq|
+<script type="text/javascript">
 function makebulboard() {
    var title=prompt('Discussion Board Title');
    if (title) {
@@ -104,33 +135,37 @@ function makebulboard() {
     this.document.forms.newbb.submit();
    }
 }
+</script>
 |;
     }
-    $r->print(&Apache::loncommon::start_page($bodytitle,
-                                       '<script type="text/javascript">'.
-                                       $jscript.'</script>'));
-    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('<form method="post" name="newbb" action="/adm/groupboards">'.
+        $r->print('<br /><form method="post" name="newbb" action="/adm/groupboards">'.
                   "\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) {
+        $r->print('<br />');
         foreach my $board (@{$groupboards}) {
-            $r->print('<a href="'.$$boards{$board}{'url'}.'?group='.$group.'">'.$$boards{$board}{'title'}.'</a><br />');
+            my $board_url = $$boards{$board}{'url'};
+            if ($board_url =~ /\?/) {
+                $board_url .= '&amp;group='.$group;
+            } else {
+                $board_url .= '?group='.$group;
+            }
+            $r->print('<a href="'.$board_url.'&amp;'.$refarg.'">'.
+                      $$boards{$board}{'title'}.'</a><br />');
         }
     } else {
-        $r->print(&mt('There are currently no discussion boards in this [_1].',
+        $r->print('<br />'.&mt('There are currently no discussion boards in this [_1].',
                       $gpterm));
     }
     $r->print(&Apache::loncommon::end_page());
@@ -140,10 +175,15 @@ 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);
+    }
     $bbtitle=&unescape($bbtitle);
     $newurl=&unescape($newurl);
     my $allbbsmap = &Apache::longroup::get_bbfolder_url($cdom,$cnum,$group);
-    my ($outcome);
     if ($allbbsmap =~ m|^/uploaded|) {
         my ($errtext,$fatal)=&Apache::lonratedt::mapread($allbbsmap);
         if (!$fatal) {
@@ -155,7 +195,11 @@ sub create_board {
             if ($fatal) {
                 $outcome = "error: failed to store 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";
@@ -167,4 +211,35 @@ sub create_board {
     return ($outcome,$newurl,$bbtitle);
 }
 
+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&amp;$refarg",
+          text=>"Discussion Boards",
+          title=>"Display group discussion boards"},);
+    $output .= &Apache::lonhtmlcommon::breadcrumbs(&mt('[_1] discussion boards - [_2]',$gpterm,$description));
+    return $output;
+}
+
 1;