ticket 12318: asset error causes asset manager to fail

daviddelikat · daviddelikat · commit bce42bca47a9 · 2012-01-12T00:20:41.000Z
diff --git a/docs/changelog/7.x.x.txt b/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.
diff --git a/lib/WebGUI/Content/AssetManager.pm b/lib/WebGUI/Content/AssetManager.pm
@@ -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,

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`7.10.24`
	`2`	`+ - fixed #12318: asset error causes asset manager to fail`
`2`	`3`	`- fixed #12308: error message used scalar as reference`
`3`	`4`	`- fixed #12256: Calendar Search doesn't show admin controls`
`4`	`5`	`- fixed #12268: Point of sale form missing from cart screen.`