--- loncom/interface/lonfeedback.pm 2005/04/07 07:34:52 1.158 +++ loncom/interface/lonfeedback.pm 2005/04/11 01:07:26 1.159 @@ -1,7 +1,7 @@ # The LearningOnline Network # Feedback # -# $Id: lonfeedback.pm,v 1.158 2005/04/07 07:34:52 albertel Exp $ +# $Id: lonfeedback.pm,v 1.159 2005/04/11 01:07:26 raeburn Exp $ # # Copyright Michigan State University Board of Trustees # @@ -2866,9 +2866,9 @@ sub has_discussion { my @allres=$navmap->retrieveResources(); foreach my $resource (@allres) { if ($resource->hasDiscussion()) { - my $ressymb; - if ($resource->symb() =~ m-(___adm/\w+/\w+)/(\d+)/bulletinboard$-) { - $ressymb = 'bulletin___'.$2.$1.'/'.$2.'/bulletinboard'; + my $ressymb = $resource->symb(); + if ($resource->symb() =~ m-(___adm/\w+/\w+/)(\d+)(/bulletinboard)$-) { + $ressymb = 'bulletin___'.$2.$1.$2.$3; } else { $ressymb = $resource->symb(); } @@ -3030,13 +3030,11 @@ END 'twnp' => 'There are currently no resources or bulletin boards with unread discussion postings.' ); foreach (@resources) { -# backward compatibility (bulletin boards used to be 'wrapped') my $ressymb=$_; &Apache::lonenc::check_decrypt(\$ressymb); +# backward compatibility (bulletin boards used to be 'wrapped') if ($ressymb =~ m/bulletin___\d+___/) { - unless ($ressymb =~ m|bulletin___\d+___adm/wrapper|) { - $ressymb=~s|(bulletin___\d+___)|$1adm/wrapper/|; - } + $ressymb = &wrap_symb($ressymb); } my $lastkey = $ressymb.'_lastread'; $discinfo{$lastkey} = $env{'form.navtime'}; @@ -3258,9 +3256,7 @@ ENDREDIR unless ($symb) { $goahead=0; } } # backward compatibility (bulletin boards used to be 'wrapped') - if ($feedurl=~m|^/adm/wrapper/adm/.*/bulletinboard$|) { - $feedurl=~s|^/adm/wrapper||; - } + &dewrapper(\$feedurl); if (!$goahead) { # Ambiguous Problem Resource $r->internal_redirect('/adm/ambiguous');