Changeset 2128 for branches/release-36/lib/MT/CMS/Template.pm

Timestamp:

04/28/08 07:25:19 (19 months ago)

Author:

fumiakiy

Message:

Properly handle system level and blog level widgetsets and widgets. BugId:79534

Files:

• branches/release-36/lib/MT/CMS/Template.pm (modified) (5 diffs)

branches/release-36/lib/MT/CMS/Template.pm

@@ -2190 +2190 @@
     while (my $m = $iter->()) {
         next unless $m;
-        $all_widgets{ $m->id } = $m->name;
+        $all_widgets{ $m->id }{name} = $m->name;
+        $all_widgets{ $m->id }{blog_id} = $m->blog_id;
     }
 
@@ -2222 +2223 @@
             my @modules = split ',', $modulesets;
             foreach my $mid ( @modules ) {
-                push @inst_modules, { id => $mid, name => $all_widgets{$mid} };
+                push @inst_modules, {
+                    id => $mid,
+                    name => $all_widgets{$mid}{name},
+                    blog_id => $all_widgets{$mid}{blog_id},
+                };
                 delete $all_widgets{$mid};
             }
@@ -2228 +2233 @@
     }
     $param->{installed} = \@inst_modules if @inst_modules;
-    my @avail_modules = map { { id => $_, name => $all_widgets{$_} } }
-        keys %all_widgets;
+    my @avail_modules = map { {
+        id => $_, name => $all_widgets{$_}{name}, blog_id => $all_widgets{$_}{blog_id}
+    } } keys %all_widgets;
     $param->{available} = \@avail_modules;
 
@@ -2259 +2265 @@
         ],
     );
-    
-    my $param = {
-        widget_table   => $widget_loop,
-        object_type    => "widgetset",
-        search_type    => "template",
-        search_label   => MT::Template->class_label_plural,
-        listing_screen => 1,
-        screen_id      => "list-widget-set",
-        $blog_id ? ( blog_view => 1, blog_id => $blog_id ) : (),
-        exists($opt{rebuild}) ? ( rebuild => $opt{rebuild} ) : (),
-        exists($opt{error}) ? ( error => $opt{error} ) : (),
-        exists($opt{deleted}) ? ( saved => $opt{deleted} ) : ()
-    };
 
     my $iter = $app->model('template')->load_iter(
-        { type => 'widgetset', blog_id => $blog_id ? [ $blog_id, 0 ] : 0 },
+        { type => 'widgetset', blog_id => $blog_id ? $blog_id : 0 },
         { sort => 'name', direction => 'ascend' }
     );
@@ -2295 +2288 @@
         push @widgetmanagers, $ws;
     }
-    $param->{object_loop} = \@widgetmanagers if @widgetmanagers;
+
+    my @widget_loop;
+    if ( $blog_id ) {
+        # Remove system level widgets from the listing
+        @widget_loop = grep { $_->{blog_id} == $blog_id } @$widget_loop;
+    }
+    else {
+        @widget_loop = @$widget_loop;
+    }
+
+    my $param = {
+        @widgetmanagers ? ( object_loop  => \@widgetmanagers ) : (),
+        @widget_loop    ? ( widget_table => \@widget_loop ) : (),
+        object_type    => "widgetset",
+        search_type    => "template",
+        search_label   => MT::Template->class_label_plural,
+        listing_screen => 1,
+        screen_id      => "list-widget-set",
+        $blog_id ? ( blog_view => 1, blog_id => $blog_id ) : (),
+        exists($opt{rebuild}) ? ( rebuild => $opt{rebuild} ) : (),
+        exists($opt{error}) ? ( error => $opt{error} ) : (),
+        exists($opt{deleted}) ? ( saved => $opt{deleted} ) : ()
+    };
 
     $app->load_tmpl('list_widget.tmpl', $param);