--- loncom/interface/lonfeedback.pm 2012/01/04 19:27:04 1.319 +++ loncom/interface/lonfeedback.pm 2012/01/04 20:12:40 1.320 @@ -1,7 +1,7 @@ # The LearningOnline Network # Feedback # -# $Id: lonfeedback.pm,v 1.319 2012/01/04 19:27:04 www Exp $ +# $Id: lonfeedback.pm,v 1.320 2012/01/04 20:12:40 www Exp $ # # Copyright Michigan State University Board of Trustees # @@ -2787,27 +2787,28 @@ sub send_msg { } } -# Records of number of postings, etc, are kept under the "symb" called "_feedback" +# Records of number of feedback messages are kept under the "symb" called "_feedback" # There are two entries within the framework of a course: # - the URLs for which feedback was provided # - the total number of contributions - my %record=&getdiscrecords(); - my ($temp)=keys(%record); - unless ($temp=~/^error\:/) { - my %newrecord=(); - $newrecord{'resource'}=$feedurl; - $newrecord{'subnumber'}=$record{'subnumber'}+1; - unless (&Apache::lonnet::cstore(\%newrecord,'_feedback') eq 'ok') { - $status.='
'.&mt('Not registered').'
'; - } + if ($sendsomething) { + my %record=&getfeedbackrecords(); + my ($temp)=keys(%record); + unless ($temp=~/^error\:/) { + my %newrecord=(); + $newrecord{'resource'}=$feedurl; + $newrecord{'subnumber'}=$record{'subnumber'}+1; + unless (&Apache::lonnet::cstore(\%newrecord,'_feedback') eq 'ok') { + $status.='
'.&mt('Not registered').'
'; + } + } } - return ($status,$sendsomething); } -# Routine to get the complete discussion records +# Routine to get the complete feedback records -sub getdiscrecords { +sub getfeedbackrecords { my ($uname,$udom,$course)=@_; unless ($uname) { $uname=$env{'user.name'}; } unless ($udom) { $udom=$env{'user.domain'}; } @@ -2816,10 +2817,10 @@ sub getdiscrecords { return %record; } -# Routine to get discussion statistics +# Routine to get feedback statistics -sub getdiscstats { - my %record=&getdiscrecords(@_); +sub getfeedbackstats { + my %record=&getfeedbackrecords(@_); my $subnumber=$record{'subnumber'}; my $points=0; my $likes=0; @@ -2834,9 +2835,9 @@ sub getdiscstats { return ($subnumber,$points,$likes); } -# Store discussion credit +# Store feedback credit -sub storediscpoints { +sub storefeedbackpoints { my ($points,$uname,$udom,$course)=@_; unless ($points) { $points=0; } unless ($uname) { $uname=$env{'user.name'}; } @@ -2848,9 +2849,9 @@ sub storediscpoints { return &Apache::lonnet::cstore(\%record,'_feedback'); } -# Store discussion "likes" +# Store feedback "likes" -sub storedisclikes { +sub storefeedbacklikes { my ($likes,$uname,$udom,$course)=@_; unless ($likes) { $likes=0; } if ($likes>0) { $likes=1; }