Changed default behavior to return nothing instead of the macro tag when no
matching collateral is found.
This commit is contained in:
parent
e8c305d919
commit
47015a8b26
6 changed files with 37 additions and 16 deletions
|
|
@ -18,8 +18,14 @@ use WebGUI::Session;
|
|||
#-------------------------------------------------------------------
|
||||
sub process {
|
||||
my @param = WebGUI::Macro::getParams($_[0]);
|
||||
my $collateral = WebGUI::Collateral->find($param[0]);
|
||||
return '<a href="'.$collateral->getURL.'"><img src="'.$collateral->getIcon.'" align="middle" border="0" /> '.$collateral->get("name").'</a>';
|
||||
if (my $collateral = WebGUI::Collateral->find($param[0])) {
|
||||
return '<a href="' . $collateral->getURL .
|
||||
'"><img src="' . $collateral->getIcon .
|
||||
'" align="middle" border="0" /> ' .
|
||||
$collateral->get("name") . '</a>';
|
||||
} else {
|
||||
return undef;
|
||||
}
|
||||
}
|
||||
|
||||
1;
|
||||
|
|
|
|||
|
|
@ -28,8 +28,11 @@ sub process {
|
|||
}
|
||||
my @images = WebGUI::SQL->buildArray("select collateralId from collateral
|
||||
where collateralType='image' and collateralFolderId=".$collateralFolderId);
|
||||
my $collateral = WebGUI::Collateral->new($images[rand($#images+1)]);
|
||||
return '<img src="'.$collateral->getURL.'" '.$collateral->get("parameters").' />';
|
||||
if (my $collateral = WebGUI::Collateral->new($images[rand($#images+1)])) {
|
||||
return '<img src="'.$collateral->getURL.'" '.$collateral->get("parameters").' />';
|
||||
} else {
|
||||
return undef;
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
|
|
|
|||
|
|
@ -28,8 +28,11 @@ sub process {
|
|||
}
|
||||
my @snippets = WebGUI::SQL->buildArray("select collateralId from collateral
|
||||
where collateralType='snippet' and collateralFolderId=".$collateralFolderId);
|
||||
my $collateral = WebGUI::Collateral->new($snippets[rand($#snippets+1)]);
|
||||
return $collateral->get("parameters");
|
||||
if (my $collateral = WebGUI::Collateral->new($snippets[rand($#snippets+1)])) {
|
||||
return $collateral->get("parameters");
|
||||
} else {
|
||||
return undef;
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
|
|
|
|||
|
|
@ -20,8 +20,11 @@ use WebGUI::Session;
|
|||
sub process {
|
||||
my (@param, $temp);
|
||||
@param = WebGUI::Macro::getParams($_[0]);
|
||||
my $collateral = WebGUI::Collateral->find($param[0]);
|
||||
return $collateral->get("parameters");
|
||||
if (my $collateral = WebGUI::Collateral->find($param[0])) {
|
||||
return $collateral->get("parameters");
|
||||
} else {
|
||||
return undef;
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
|
|
|
|||
|
|
@ -18,8 +18,11 @@ use WebGUI::Session;
|
|||
#-------------------------------------------------------------------
|
||||
sub process {
|
||||
my @param = WebGUI::Macro::getParams($_[0]);
|
||||
my $collateral = WebGUI::Collateral->find($param[0]);
|
||||
return $collateral->getThumbnail;
|
||||
if (my $collateral = WebGUI::Collateral->find($param[0])) {
|
||||
return $collateral->getThumbnail;
|
||||
} else {
|
||||
return undef;
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
|
|
|
|||
|
|
@ -18,12 +18,15 @@ use WebGUI::Session;
|
|||
#-------------------------------------------------------------------
|
||||
sub process {
|
||||
my @param = WebGUI::Macro::getParams($_[0]);
|
||||
my $collateral = WebGUI::Collateral->find($param[0]);
|
||||
my $output = '<a href="'.$collateral->getURL.'"';
|
||||
$output .= ' target="_blank"' if ($param[1]);
|
||||
$output .= '><img src="'.$collateral->getThumbnail.
|
||||
'" border="0"></a><br><b>'.$param[0].'</b><p>';
|
||||
return $output;
|
||||
if (my $collateral = WebGUI::Collateral->find($param[0])) {
|
||||
my $output = '<a href="'.$collateral->getURL.'"';
|
||||
$output .= ' target="_blank"' if ($param[1]);
|
||||
$output .= '><img src="' . $collateral->getThumbnail;
|
||||
$output .= '" border="0"></a><br><b>'.$param[0].'</b><p>';
|
||||
return $output;
|
||||
} else {
|
||||
return undef;
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue