X-Git-Url: https://git.decadent.org.uk/gitweb/?a=blobdiff_plain;f=Memories%2FPhoto.pm;h=e2175c5ca80737387fae96525b7f4e559369a8b5;hb=33a3af1b640ffb40912f0a9b4de60ca460587712;hp=ce0cd2003a777277b06e5e535389e14c1be033ba;hpb=0462f25b10e5adb4eacc0e11f4ced5165784f38f;p=memories.git diff --git a/Memories/Photo.pm b/Memories/Photo.pm index ce0cd20..e2175c5 100644 --- a/Memories/Photo.pm +++ b/Memories/Photo.pm @@ -1,26 +1,42 @@ package Memories::Photo; +use File::Basename; +use File::Copy; +use Archive::Any; use strict; use Carp qw(cluck confess); use base qw(Memories::DBI Maypole::Model::CDBI::Plain); +use constant INTERESTINGNESS_ALGORITHM => '((rating+3)/(rated+1))*(1+rated/hit_count)*(1+hit_count/(select max(hit_count) from photo))'; use Time::Piece; use Image::Seek; use constant PAGER_SYNTAX => "LimitXY"; __PACKAGE__->columns(Essential => qw(id title uploader uploaded x y rating rated hit_count)); __PACKAGE__->untaint_columns(printable => [qw/title/]); __PACKAGE__->columns(TEMP => qw/exif_object/); -__PACKAGE__->set_sql(recent => q{ -SELECT __ESSENTIAL__ -FROM __TABLE__ -ORDER BY uploaded DESC -LIMIT 4 -}); -__PACKAGE__->set_sql(popular => q{ -SELECT __ESSENTIAL__ -FROM __TABLE__ -ORDER BY hit_count DESC -LIMIT 4 -}); +BEGIN { +my %order_by = ( + recent => "uploaded", + popular => "hit_count", + loved => "rating/(1+rated)", + interesting => INTERESTINGNESS_ALGORITHM, + random => "rand()" +); + +while (my($label, $how) = each %order_by) { + __PACKAGE__->set_sql($label => qq{ + SELECT __ESSENTIAL__ + FROM __TABLE__ + ORDER BY $how DESC + LIMIT 4 + }); + no strict; + *$label = sub { + my ($self, $r) = @_; + $self->view_paged_ordered($r, "$how desc"); + }; + __PACKAGE__->MODIFY_CODE_ATTRIBUTES(*{$label}{CODE}, "Exported"); # Hack +} +} __PACKAGE__->has_many(comments => "Memories::Comment"); __PACKAGE__->has_a(uploader => "Memories::User"); __PACKAGE__->has_a(uploaded => "Time::Piece", @@ -30,53 +46,50 @@ __PACKAGE__->has_a(uploaded => "Time::Piece", sub do_upload :Exported { my ($self, $r) = @_; - my %upload = $r->upload("photo"); + my $upload = $r->{ar}->upload("photo"); + # Check $self->type + my @photos = $self->upload_jpeg($upload->tempname, ($r->params->{title}||basename($upload->filename)), $r->params->{tags}, $r); + my @quarantined = grep { !$_->tags } @photos; + # Set it up to go again + $r->objects(\@photos); + $r->template("view"); + $r->message("Thanks for the upload!"); + # Deal with the quarantined +} - # XXX Stop anonymous uploads! +sub upload_jpeg { + my ($self, $filename, $title, $tags, $r) = @_; + my $quarantine; my $photo = $self->create({ uploader => $r->user, uploaded => Time::Piece->new(), - title => $r->params->{title}, + title => $title, hit_count => 0, rating => 0, - rated => 0, # Oh, the potential for divide by zero errors... + rated => 0, }); - - # Dump content - if (!open OUT, ">". $photo->path("file")) { - die "Can't write ".$photo->path("file")." because $!"; + if (!copy($filename, $photo->path("file"))) { + $photo->delete(); die "Couldn't copy photo: $!"; } - # XXX Check it's a JPEG, etc. - # XXX Unzip ZIP file - print OUT $upload{content}; - close OUT; my ($x, $y) = dim(image_info($photo->path)); $photo->x($x); $photo->y($y); # Rotate? $photo->unrotate(); if (!$photo->title){ - $photo->title($photo->title_exif || $upload{filename}); + $photo->title($photo->title_exif || basename($filename)); } $photo->make_thumb; - $r->{params}{tags} ||= join " ", map { qq{"$_"} } $photo->tags_exif; - $photo->add_tags($r->{params}{tags}); + $tags ||= join " ", map { qq{"$_"} } $photo->tags_exif; + $photo->add_tags($tags); $photo->add_to_imageseek_library; Memories->zap_cache(); # Add system tags here my $tag = "date:".$photo->shot->ymd; $photo->add_to_system_tags({tag => Memories::SystemTag->find_or_create({name =>$tag}) }); - - # Set it up to go again - $r->objects([$photo]); - $r->template("view"); - $r->message("Thanks for the upload! ". - ($r->{params}{tags} ? "" - : "Don't forget to config->uri_base."photo/view/".$photo->id."?active=tagedit\">tag your photo" - ) - ); + return $photo; } sub approx_rating { @@ -120,6 +133,7 @@ sub exif :Exported {} sub comment :Exported {} sub tagedit :Exported {} sub similar :Exported {} +sub sized :Exported {} use Class::DBI::Plugin::Pager; use Class::DBI::Plugin::AbstractCount; @@ -139,16 +153,6 @@ sub view_paged_ordered { $r->template("paged"); # Set the what using the action name } -sub recent :Exported { - my ($self, $r) = @_; - $self->view_paged_ordered($r, "uploaded desc"); -} - -sub popular :Exported { - my ($self, $r) = @_; - $self->view_paged_ordered($r, "hit_count desc"); -} - sub add_comment :Exported { my ($self, $r, $photo) = @_; $r->template("view"); @@ -303,7 +307,6 @@ sub scale { } } -use Text::Balanced qw(extract_multiple extract_quotelike); sub edit_tags :Exported { my ($self, $r) = @_; my $photo = $r->objects->[0]; @@ -321,7 +324,7 @@ sub edit_tags :Exported { sub add_tags { my ($photo, $tagstring) = @_; - for my $tag (map { s/^"|"$//g; $_} extract_multiple(lc $tagstring, [ \&extract_quotelike, qr/([^\s]+)/ ], undef,1)) { + for my $tag (Tagtools->separate_tags($tagstring)) { $photo->add_to_tags({tag => Memories::Tag->find_or_create({name =>$tag}) }) } }