Changeset 1823 for branches/release-34/lib/MT/CMS/Entry.pm

Timestamp:

04/09/08 10:05:12 (20 months ago)

Author:

takayama

Message:

Fixed BugId:67959
* Added check for result of object loading

Files:

• branches/release-34/lib/MT/CMS/Entry.pm (modified) (8 diffs)

branches/release-34/lib/MT/CMS/Entry.pm

@@ -139 +139 @@
         my $blog_timezone = 0;
         if ($blog_id) {
-            my $blog = $blog_class->load($blog_id);
+            my $blog = $blog_class->load($blog_id)
+                or return $app->error($app->translate('Can\'t load blog #[_1].', $blog_id));
             $blog_timezone = $blog->server_offset();
             if ( $type eq 'entry' ) {
@@ -836 +837 @@
         $fullscreen = 1;
     }
+    return $app->error($app->translate('Can\'t load template.')) unless $tmpl;
 
     # translates naughty words when PublishCharset is NOT UTF-8
@@ -1079 +1081 @@
     require MT::Blog;
     my $blog_id = $app->param('blog_id');
-    my $blog    = MT::Blog->load($blog_id);
-
+    my $blog    = MT::Blog->load($blog_id)
+        or return $app->error($app->translate('Can\'t load blog #[_1].', $blog_id));
+    
     my $archive_type;
 
@@ -1349 +1352 @@
 
         # blog_id sanity check
-        next if $cat->blog_id != $obj->blog_id;
+        next if !$cat || $cat->blog_id != $obj->blog_id;
 
         my $place = MT::Placement->new;
@@ -1462 +1465 @@
         next unless $p =~ /^category_id_(\d+)/;
         my $id    = $1;
-        my $entry = MT::Entry->load($id);
+        my $entry = MT::Entry->load($id)
+            or next;
         return $app->error( $app->translate("Permission denied.") )
             unless $perms
@@ -1822 +1826 @@
     my $entry_id = $app->param('entry_id');
     require MT::Entry;
-    my $entry = MT::Entry->load($entry_id);
+    my $entry = MT::Entry->load($entry_id)
+        or return $app->error($app->translate('Can\'t load entry #[_1].', $entry_id));
     $param{url_loop} = [ map { { url => $_ } } @{ $entry->pinged_url_list } ];
     $param{failed_url_loop} =
@@ -2071 +2076 @@
     my ($type)  = @_;
     my $blog_id = $app->blog->id;
-    my $blog    = $app->model('blog')->load($blog_id);
+    my $blog    = $app->model('blog')->load($blog_id)
+        or return $app->error($app->translate('Can\'t load blog #[_1].', $blog_id));
     my %args    = ( '_type' => $type, blog_id => $blog_id, qp => 1 );
     my $uri = $app->base . $app->uri( 'mode' => 'view', args => \%args );
@@ -2286 +2292 @@
     my $q       = $app->param;
     my $blog_id = $q->param('blog_id');
-    my $blog    = MT::Blog->load($blog_id);
+    my $blog    = MT::Blog->load($blog_id)
+        or return $app->error($app->translate('Can\'t load blog #[_1].', $blog_id));
 
     my $can_background =