--- loncom/interface/lonmeta.pm 2004/04/13 14:42:24 1.64
+++ loncom/interface/lonmeta.pm 2004/04/14 16:14:29 1.67
@@ -1,7 +1,7 @@
# The LearningOnline Network with CAPA
# Metadata display handler
#
-# $Id: lonmeta.pm,v 1.64 2004/04/13 14:42:24 matthew Exp $
+# $Id: lonmeta.pm,v 1.67 2004/04/14 16:14:29 matthew Exp $
#
# Copyright Michigan State University Board of Trustees
#
@@ -40,9 +40,6 @@ use Apache::lonlocal;
use Apache::lonmysql;
use Apache::lonmsg;
-# MySQL table columns
-
-my @columns;
# Fetch and evaluate dynamic metadata
sub dynamicmeta {
@@ -60,6 +57,15 @@ sub dynamicmeta {
#
# Deal with 'count' separately
$Data{'count'} = &access_count($url,$aauthor,$adomain);
+ #
+ # Debugging code I will probably need later
+ if (0) {
+ &Apache::lonnet::logthis('Dynamic Metadata');
+ while(my($k,$v)=each(%Data)){
+ &Apache::lonnet::logthis(' "'.$k.'"=>"'.$v.'"');
+ }
+ &Apache::lonnet::logthis('-------------------');
+ }
return %Data;
}
@@ -104,7 +110,9 @@ sub authordisplay {
# Pretty display
sub evalgraph {
my $value=shift;
- unless ($value) { return ''; }
+ if (! $value) {
+ return '';
+ }
my $val=int($value*10.+0.5)-10;
my $output='
';
if ($val>=20) {
@@ -126,7 +134,9 @@ sub evalgraph {
sub diffgraph {
my $value=shift;
- unless ($value) { return ''; }
+ if (! $value) {
+ return '';
+ }
my $val=int(40.0*$value+0.5);
my @colors=('#FF9933','#EEAA33','#DDBB33','#CCCC33',
'#BBDD33','#CCCC33','#DDBB33','#EEAA33');
@@ -142,15 +152,6 @@ sub diffgraph {
return $output;
}
-# Turn MySQL row into hash
-sub metadata_col_to_hash {
- my @cols=@_;
- my %hash=();
- for (my $i=0; $i<=$#columns; $i++) {
- $hash{$columns[$i]}=$cols[$i];
- }
- return %hash;
-}
# The field names
sub fieldnames {
@@ -200,7 +201,9 @@ sub fieldnames {
sub prettyprint {
my ($type,$value)=@_;
- unless (defined($value)) { return ' '; }
+ if (! defined($value)) {
+ return ' ';
+ }
# Title
if ($type eq 'title') {
return ''.$value.'';
@@ -239,7 +242,9 @@ sub prettyprint {
return &Apache::loncommon::gradeleveldescription($value);
}
# Only for advance users below
- unless ($ENV{'user.adv'}) { return '- '.&mt('not displayed').' -' };
+ if (! $ENV{'user.adv'}) {
+ return '- '.&mt('not displayed').' -';
+ }
# File
if (($type eq 'customdistributionfile') ||
($type eq 'obsoletereplacement') ||
@@ -284,7 +289,9 @@ sub direct {
sub selectbox {
my ($name,$value,$functionref,@idlist)=@_;
- unless (defined($functionref)) { $functionref=\&direct; }
+ if (! defined($functionref)) {
+ $functionref=\&direct;
+ }
my $selout='