Changeset 1776

Timestamp:

04/04/08 22:48:02 (5 months ago)

Author:

mpaschal

Message:

Update 'virtual' field of fileinfos directly when dynamicity changes
Don't recreate fileinfos when dynamicity changes, only when publishing path does (but when publishing path changes, recreate fileinfos even if publishing statically, as we always keep fileinfos nowadays)
Don't cache fileinfos, so updating directly is not a problem
BugzID: 67300

Files:

• branches/release-33/lib/MT/CMS/Blog.pm (modified) (3 diffs)
• branches/release-33/lib/MT/FileInfo.pm (modified) (1 diff)

branches/release-33/lib/MT/CMS/Blog.pm

@@ -1371 +1371 @@
-}
-
+sub _update_finfos {
+    my ($app, $new_virtual, $where) = @_;
+    my $finfo_class = MT->model('fileinfo');
+    my $driver = $finfo_class->driver;
+    my $dbd = $driver->dbd;
+
+    my $stmt = MT::ObjectDriver::SQL->new;
+
+    if ($where) {
+        my $new_where = {};
+        while (my ($key, $val) = each %$where) {
+            my $new_key = $dbd->db_column_name($finfo_class->datasource, $key);
+            $new_where->{$new_key} = $val;
+        }
+        $stmt->add_complex_where([ $new_where ]);
+    }
+    my $virtual_col = $dbd->db_column_name($finfo_class->datasource, 'virtual');
+    $stmt->add_complex_where([ { $virtual_col => { op => '!=', value => $new_virtual } } ]);
+
+    my $sql = join q{ }, 'UPDATE', $driver->table_for($finfo_class), 'SET',
+        $virtual_col, '= ?', $stmt->as_sql_where();
+
+    my $dbh = $driver->rw_handle;
+    $dbh->do($sql, {}, $new_virtual, @{ $stmt->{bind} })
+        or return $app->error($dbh->errstr || $DBI::errstr);
+    1;
+}
+
-sub post_save {
-    my $eh = shift;
@@ -1416 +1444 @@
-            }
-
+            if (!$dcty_changed || $dcty eq 'custom') {
+                # do nothing
+            }
+            elsif ($dcty eq 'none') {
+                _update_finfos($app, 0);
+            }
+            elsif ($dcty eq 'all') {
+                _update_finfos($app, 1);
+            }
+            elsif ($dcty eq 'archives') {
+                # Only archives have template maps.
+                _update_finfos($app, 1, { templatemap_id => \'is not null' });
+                _update_finfos($app, 0, { templatemap_id => \'is null' });
+            }
+
-            # If either of the publishing paths changed, rebuild the fileinfos.
-            my $path_changed = 0;
@@ -1422 +1465 @@
-                    && $app->param($path_field) ne $original->$path_field();
-            }
-
+
+
-                $app->rebuild( BlogID => $obj->id, NoStatic => 1 )
-                    or return $app->publish_error();

branches/release-33/lib/MT/FileInfo.pm

@@ -38 +38 @@
-    datasource => 'fileinfo',
-    primary_key => 'id',
+    cacheable => 0,
-});
-