Browse files

Don't republish an asset that is curently being expired. Yes, the Exp…

…ire job may fail if the asset does not exist at the Destination(s), but we really don't need to take time to publish an asset just so the expiration won't fail. Also talked to theory about possibly updating Trans classes to ignore the 'no such file at destination' case upon expiration.
  • Loading branch information...
1 parent f39fd99 commit e7471c8d873d882f660b5298a4617b7ed1ca48c1 nlegg committed Sep 23, 2011
Showing with 1 addition and 1 deletion.
  1. +1 −1 lib/Bric/Util/
@@ -1311,7 +1311,7 @@ sub publish {
- if ($exp_date && $exp_date lt $publish_date) {
+ if ($exp_date && ($exp_date lt $publish_date || !$ba->is_active())) {
# Don't really publish it, just expire it.
return 1 unless $ba->get_publish_status;
(my $search_path = $base_path) =~ s/([_%])/\\$1/g;

0 comments on commit e7471c8

Please sign in to comment.