Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
ticket 12318: asset error causes asset manager to fail
  • Loading branch information
daviddelikat committed Jan 12, 2012
1 parent fabf59f commit bce42bc
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 0 deletions.
1 change: 1 addition & 0 deletions docs/changelog/7.x.x.txt
@@ -1,4 +1,5 @@
7.10.24
- fixed #12318: asset error causes asset manager to fail
- fixed #12308: error message used scalar as reference
- fixed #12256: Calendar Search doesn't show admin controls
- fixed #12268: Point of sale form missing from cart screen.
Expand Down
5 changes: 5 additions & 0 deletions lib/WebGUI/Content/AssetManager.pm
Expand Up @@ -309,6 +309,11 @@ sub www_ajaxGetManagerPage {
for my $assetId ( @{ $p->getPageData } ) {
my $asset = WebGUI::Asset->newByDynamicClass( $session, $assetId );

unless( $asset ) {
$session->log->error('www_ajaxGetManagerPage: assetId="'.$assetId.'" failed in newByDynamicClass');
next;
}

# Populate the required fields to fill in
my %fields = (
assetId => $asset->getId,
Expand Down

0 comments on commit bce42bc

Please sign in to comment.