Changeset 6525

Timestamp:

Jun 14, 2010, 9:29:37 AM (14 years ago)

Author:

ronosman

Message:

Fix destination_category issue : move the property in the manager to avoid thread data copy.

Location:

extensions/pLoader/trunk/src/Uploader

Files:

• Image.pm (modified) (1 diff)
• Images.pm (modified) (1 diff)
• TransferManager.pm (modified) (2 diffs)
• TransferWorker.pm (modified) (4 diffs)

extensions/pLoader/trunk/src/Uploader/Image.pm

@@ -38 +38 @@
     exif_metadata
     add_rank
-    site_categories
     site_tags
     image_id

extensions/pLoader/trunk/src/Uploader/Images.pm

@@ -102 +102 @@
                 site_author       => $self->author,
                 add_rank          => $i,
-                site_categories   => [],
                 site_tags         => [],
                 site_high_file    => $file->{ANSIPathName},

extensions/pLoader/trunk/src/Uploader/TransferManager.pm

@@ -9 +9 @@
 __PACKAGE__->mk_accessors(qw/
     pwg
+    destination_category
 /);
-
+use Data::Dumper;
 
 sub new_worker {
@@ -23 +24 @@
 
 
-sub destination_category {
-    my ( $self, $category ) = @_;
-
-    $self->worker->destination_category($category);
-}
-
-
 sub destination_category_is_empty {
     my ( $self ) = @_;
 
-    $self->worker->destination_category_is_empty;
+    !scalar @{$self->destination_category};
 }
 
-
 1;

extensions/pLoader/trunk/src/Uploader/TransferWorker.pm

@@ -11 +11 @@
 __PACKAGE__->mk_accessors(qw/
     preferences
-    destination_category
     pwg
     event_handler
@@ -22 +21 @@
     my ( $self ) =@_;
 
-    $self->destination_category([]);
     $self->pwg->log_progress_cbk(
         sub { $self->post_image_progress_event(@_) }
@@ -165 +163 @@
 sub update_image {
     my ( $self, $image ) = @_;
-
+print Dumper $image;
     $self->log_image_progress($image, "updating", 0);
 
@@ -225 +223 @@
 
 
-sub destination_category_is_empty {
-    my ( $self ) = @_;
-
-    !scalar @{$self->destination_category};
-}