]> wagner.pp.ru Git - oss/stilllife.git/commitdiff
Implemented delete operation
authorVictor Wagner <vitus@wagner.pp.ru>
Fri, 4 Apr 2008 07:25:43 +0000 (07:25 +0000)
committerVictor Wagner <vitus@wagner.pp.ru>
Fri, 4 Apr 2008 07:25:43 +0000 (07:25 +0000)
forum/forum

index 7e02a0008206151effcd8cadaa1d255f34f75301..7cb882b48c9c9234da43ab5d6d0ec89cd5dadeb5 100755 (executable)
@@ -31,7 +31,7 @@ use Net::OpenID::Consumer;
 my %actions = (
        reply => \&reply,
        edit => \&edit_comment,
-       delete => \&delete_comment,
+       delete => \&delete_item,
        move => \&move_comment,
        newtopic=> \&new_topic,
        newforum=> \&new_forum,
@@ -75,6 +75,8 @@ if ($cgi->request_method ne "POST") {
                logout('logout',$cgi,$forum);
        } elsif ($cgi->param("profile")) {
                show_profile("profile",$cgi,$forum);
+       } elsif ($cgi->param('delete')) {
+               delete_item("delete",$cgi,$forum);
        } else {
                for my $param ($cgi->param) {
 # Среди параметров, указанных в URL ищем тот, который задает
@@ -167,7 +169,6 @@ sub get_forum_config {
        $path_translated=~s/\/+$//;
        my @path=split("/",$path_translated);
        while (@path>1) {
-               print STDERR "Searching for config in ",join("/",@path),"\n";
                if (-r (my $config=join("/",@path,".forum")) ) {
                        open F,"<",$config;
                        my %config;
@@ -327,8 +328,6 @@ sub prepare_template {
        ELEMENT:
        for my $element ($f->find_by_tag_name("textarea","input","select")) {
                my $name = $element->attr("name");
-               #print STDERR "Found element <".$element->tag()." name=\"$name\">\n" ;
-               #print STDERR "Corresponding \$cgi->param($name)=\"",$cgi->param($name),"\"\n"; 
                $substituted{$name} = 1;
                if (defined  $cgi->param($name)) {
                        if ($element->tag eq "input") {
@@ -353,6 +352,18 @@ sub prepare_template {
                                $element->attr("value",$cgi->param($name));
                                }
                        } elsif ($element->tag eq "textarea") {
+                               my $data=$cgi->param($name);
+                               if ($data=~/^<(div|p)\b/ && !defined($cgi->param($name."_format"))) {
+                                       if ($data=~/^<div class="bbcode">(.*)<\/div>$/) {
+                                               $cgi->param($name."_format","bbcode");
+                                               my $parser = HTML::BBReverse->new();
+                                               $data = $parser->reverse($1);
+                                       # FIXME elsif ($data=~/<p class="text">)        
+                                       } else {
+                                               $cgi->param($name."_format","html");
+                                       }       
+
+                               }
                                $element->delete_content;
                                $element->push_content($cgi->param($name));
                        } elsif ($element->tag eq "select") {
@@ -360,7 +371,7 @@ sub prepare_template {
                                        my $value = $option->attr("value") ||
                                                $option->as_text();
                                        if (grep($value eq $_, $cgi->param($name))) {
-                                               $option->attr("selected","");
+                                               $option->attr("selected","selected");
                                        } else {        
                                                $option->attr("selected",undef);
                                        }       
@@ -479,7 +490,6 @@ if (defined $user) {
                        src=>$forum->{templatesurl}."/1x1.gif",
                        width=>1,height=>1);
        }               
-       delete $userinfo{"avatar"};
 
        for my $element ( $tree->look_down("class",qr/^ap-/)) {
                my $field=$1 if $element->attr("class")=~/^ap-(.*)$/;   
@@ -492,15 +502,12 @@ if (defined $user) {
                        my $data;
                        if ($field eq "registered" || substr($field,0,5) eq "last_") {
                        $data = strftime("%d.%m.%Y %H:%M",localtime($userinfo{$field}))
-                       } elsif ($field=~/^<\w+/) {
+                       } elsif ($userinfo{$field}=~/^<\w+/) {
                                $data = str2tree($userinfo{$field});
                        } else {
                                $data = $userinfo{$field}
                        }
                        $element->push_content($data);
-                       if (ref($data)) {
-                               $data->delete;
-                       }       
                        $element->attr(href=>"mailto:$userinfo{$field}") 
                                if ($element->tag eq "a" && $field eq "email");
                }       
@@ -665,6 +672,7 @@ sub forum_redirect {
                $url =
                $cgi->url(-base=>1).($cgi->path_info()||$forum->{forumtop}) if !$url ;
        }
+       $url = $cgi->url(-base=>1).$url if substr($url,0,1) eq "/";
        print $cgi->redirect(-url=>$url,
                ($forum->{cookies}?(-cookie=>$forum->{cookies}):()));
        exit;   
@@ -691,7 +699,6 @@ sub show_profile {
        dbmclose(%base); 
        delete $userinfo->{passwd};
        $userinfo->{user}=$user;
-       print STDERR "Substituting userinfo for $user\n";
        while(my ($field,$value) = each %$userinfo) {   
                $value = $value->{src} if ($field eq 'avatar' && ref($value));
                $cgi->param($field,$value);
@@ -733,7 +740,6 @@ sub show_profile {
                }       
        }
        # Подставляем аватарку
-       print STDERR "avatar=",$userinfo->{avatar},"\n";
        substinfo($tree,[_tag=>'img',class=>'avatar'],(ref($userinfo->{avatar})?(%{$userinfo->{avatar}}):(src=>$userinfo->{avatar})));
        for my $userlink ($tree->look_down(_tag => "a",class=>"author")) {
                $userlink->delete_content;
@@ -975,7 +981,7 @@ sub profile_links {
                         $forum->{authenticated}{user} eq $user)) {
 
                                $profile_link->attr("href",
-                                       $cgi->url(-absolute=>1,-path_info=>0).$forum->{userurl}.
+                                       $ENV{'SCRIPT_NAME'}.$forum->{userurl}.
                                        "/".$user."?profile=1");
                } else {        
                        $profile_link->delete();
@@ -1024,17 +1030,11 @@ sub allow_operation {
        }       
        my $user = $forum->{authenticated}{user} ;
        my $accesslevel=getrights($cgi,$forum);
-       # Если permissions{$operation} равны author, нам нужно извлечь
-       # текст из соответствующего файла и положить его в
-       # cgi->param("text"); Заодно определим и автора
-       my ($itemauthor,$itemtext)=get_message_by_id($cgi->param("id")) if
-               $permissions{$operation} eq "author";
        
        return 1 if ($accesslevel eq "admin");
        return 0 if ($permissions{$operation} eq "admin");      
        return 1 if ($accesslevel eq "moderator");
        return 0 if $accesslevel eq "banned";   
-       return 0 if $permissions{$operation} eq "author" && $user ne $itemauthor;
        return 1;
 }
 
@@ -1160,8 +1160,9 @@ sub reply {
        #
        # Делаем Уфф и сохраняем то, что получилось 
        #
-       record_as_recent($forum,$newmsg->clone);
+       $newmsg = $newmsg->clone;
        savetree($path_translated,$tree,$lockfd);
+       record_as_recent($forum,$newmsg);
        record_statistics($forum,"message"),
        update_topic_list($forum,$path_translated,$msgcount,$posted);
        forum_redirect($cgi,$forum,$cgi->path_info."#$id");
@@ -1198,8 +1199,8 @@ sub record_as_recent {
                        $msglist->look_down(class=>"message")->replace_with($msg);
                } else {
                        my @msgs = $msglist->look_down("class"=>"message");
-                       if (@msgs > $forum->{replies_per_page}) {
-                               for (my $i=$#msgs;$i>=$forum->{replies_per_page};$i--) {
+                       if (@msgs >= $forum->{replies_per_page}) {
+                               for (my $i=$#msgs;$i>=$forum->{replies_per_page}-1;$i--) {
                                        $msgs[$i]->delete;
                                }
                        }       
@@ -1432,7 +1433,149 @@ sub new_forum {
 
        forum_redirect($cgi,$forum,$cgi->url(-base=>1).$url);
 }
-       
+
+#
+# Обработка операции удаления всего на свете
+#
+sub delete_item {
+       my ($formname,$cgi,$forum) = @_;
+       #
+       # Сначала разберемся, что мы собственно удаляем 
+       #
+       if ($cgi->param("user")) {
+               delete_user($cgi,$forum,$cgi->param("user"));
+       } elsif (-f $path_translated) {
+               if ($cgi->param("id")) {
+                       delete_comment($cgi,$forum,$path_translated,$cgi->param("id"));
+               } else {
+                       delete_topic($cgi,$forum,$path_translated);
+               }       
+       } elsif (-d $path_translated && -f $path_translated ."/".  $forum->{indexfile}) {
+               delete_forum($cgi,$forum,$path_translated);
+       } else {
+               show_error($forum,"Невозможно удалить неопознанный объект");
+       }
+}      
+# 
+# Удаление пользователя
+#
+sub delete_user {
+       my ($cgi,$forum,$user) = @_;
+       if ($forum->{authenticated}{user} ne $user &&
+               getrights($cgi,$forum) ne "admin") {
+                       show_error($forum,"У вас недостаточно прав для удаления
+                       пользователя $user"); 
+       }               
+       my %base;
+       dbmopen %base,datafile($forum,"passwd"),0644;
+       delete $base{$user};
+       dbmclose %base;
+       forum_redirect($cgi,$forum,$forum->{forumtop});
+}      
+#
+# Удаление реплики
+#
+sub delete_comment {
+       my ($cgi,$forum,$topic,$id) = @_;
+       my ($tree,$lockfd) = gettree($topic);
+       my ($msg) = $tree->look_down(id => $id);
+       show_error("В данной теме нет реплики с id=$id") if (!$msg);
+       if (getrights($cgi,$forum) ne "moderator" 
+               && getrights($cgi,$forum) ne "admin") {
+               my $author= $msg->look_down(_tag=>"input",name=>"author");
+               if ($author->attr("value") ne $forum->{authenticated}{user}) {
+                       show_error($forum,"У вас нет прав на удаление этого сообщения");
+               }       
+       }
+       delete_from_list($tree,"messagelist","message",$msg);
+       savetree($topic,$tree,$lockfd);
+       $tree->delete;
+       ($tree,$lockfd) = gettree($forum->{forumroot}."/recent.html");
+       ($msg) = $tree->look_down(id => $id);
+       if ($msg) {
+               delete_from_list($tree,"messagelist","message",$msg); 
+               savetree($forum->{forumroot}."/recent.html",$tree,$lockfd);
+       } else {
+               discardtree($tree,$lockfd);
+       }
+       # Если у реплики были картинки, то их тоже надо грохнуть
+       for my $pic (<$id."_*.*">) {
+               unlink $pic;
+       }       
+       forum_redirect($cgi,$forum,$cgi->path_info);
+}      
+#
+# Удаление темы
+#
+sub delete_topic {
+       my ($cgi,$forum,$topic) = @_;
+       # Если модератор, то в тему и не заглядываем
+       my ($dir,$id) = ($1,$2) if $topic=~/^(.*)\/([^\/]+).html$/;  
+       show_error($forum,"Неверное имя темы $topic") unless ($dir && $id);
+       if (getrights($cgi,$forum) ne "moderator" &&
+               getrights($cgi,$forum) ne "admin") {
+               # Проверяем авторство темы и наличие в ней сообщений
+               my ($tree,$lockfd) = gettree($topic);
+               my $info = $tree->look_down(class=>"topic");
+               my $author = $tree->look_down(class=>"author");
+               if ($author->as_text() ne $forum->{authenticated}{user}) {
+                       show_error($forum,"У вас нет права удалять эту тему");
+               }
+               $info = $tree->look_down(class=>"messagelist");
+               if ($info->attr("style") !~ /display: none;/) {
+                       show_error($forum,"Невозможно удалить тему, содержащую сообщения");
+               }       
+               discardtree($tree,$lockfd);
+       } # FIXME при удалении непустой темы модератором почистить recent.html       
+       my ($tree,$lockfd) = gettree($dir . "/".$forum->{indexfile});
+       delete_from_list($tree,"topiclist","topic",$id);
+       savetree($tree,$dir ."/".$forum->{indexfile});
+       unlink $topic;
+       my $redirect_url = $cgi->path_info;
+       $redirect_url =~ s/\/[^\/]*$//;
+       forum_redirect($cgi,$forum,$redirect_url);
+}
+#
+# Удаление форума
+#
+sub delete_forum {
+       my ($cgi,$forum,$dir) = @_;
+       if (getrights($cgi,$forum) ne "moderator" && getrights($cgi,$forum)
+       ne "admin") {
+               show_error($forum,"У вас нет права удалять форумы");
+       }       
+       opendir DIR,$dir;
+       my $count = 0;
+       while ( my $entry=readdir DIR) {
+               next if $entry eq $forum->{indexfile};
+               next if $entry eq "perms.txt";
+               next if $entry =~ /^logo.[^.]+/;
+               show_error($forum,"Нельзя удалять корень форума" )
+                       if $entry eq ".forum";
+               $count ++;      
+       }               
+       closedir DIR;
+       show_error("Нельзя удалять непустой форум") if $count;
+       # Находим родительский форум
+       my $upper = $dir;
+       $upper=~s/([\/]+)$/$forum->{indexfile}/;
+       my $id = $1 if $dir =~ /\/([\/]+)$/;
+       # Удаляем форум из списка.
+       my ($tree,$lockfd) = gettree($upper);
+       delete_from_list($tree,"forumlist","forum",$id);
+       savetree($upper,$tree,$lockfd);
+       # Удаляем из файловой системы
+       opendir DIR,$dir;
+       for my $entry ( readdir DIR) {
+               unlink "$dir/$entry";
+       }       
+       rmdir $dir;
+       my $redirect_url = $cgi->path_info;
+       $redirect_url =~s/\/$//;
+       $redirect_url =~ s/\/[^\/]*$//;
+       forum_redirect($cgi,$forum,$redirect_url);      
+}      
+
 #---------------------------------------------------------- 
 # База пользователей и права доступа
 #----------------------------------------------------------
@@ -1535,6 +1678,13 @@ sub savetree {
        rename $filename.".new",$filename;
        close $lockfd if defined($lockfd);
 }      
+
+sub discardtree {
+       my ($tree,$lockfd) = @_;
+       flock $lockfd,LOCK_UN;
+       close $lockfd;
+       $tree->delete;
+}      
 #
 # Cериализовать HTML-документ с DOCTYPE (workaround вокруг баги в
 # HTML::TreeBuilder)
@@ -1675,7 +1825,6 @@ sub openid_verify {
                        }       
                        $info{"openiduser"}=1;
                        $info{"registered"}=time; 
-                       print STDERR "forum $forum info ".\%info."\n";
                        set_default_user_attrs($forum,\%info);
                        $info{'status'} = $forum->{openid_status} if $forum->{openid_status};
                        $forum->{authenticated}=\%info;
@@ -1780,12 +1929,17 @@ sub input2tree {
 sub str2tree {
        my ($data)=@_;
        my $tree = make_tree();
+       $tree->ignore_ignorable_whitespace(1);
        # Set parser options here
        $tree->parse("<html><body><div>$data</div></body></html>");
        $tree->eof;
        my $element=$tree->find("body");
        while (($element =($element->content_list)[0])->tag ne "div") {
        }
+       while ($element->content_list==1&& $element->tag eq "div" &&
+               !defined $element->attr("style")) {
+               $element = ($element->content_list)[0]
+       }
        $element->detach;
        $tree->destroy;
        return $element;
@@ -1836,6 +1990,24 @@ sub hide_list {
        return 1;
 }      
 #
+# Удаляет из списка элемент, если он не последий. Если последний -
+# скрывает весь список. Элемент можно указывать как ссылкой на объект 
+# HTML::Element, так и значением атрибута id 
+#
+sub delete_from_list {
+       my ($tree,$listclass,$itemclass,$item) = @_;
+       my $list = $tree->look_down(class =>$listclass);
+       ($item) = $list->look_down(id =>$item) unless ref($item);
+       return undef unless $item;
+       my (@items)=$tree->look_down(class=>$itemclass); 
+       if (@items == 1) {
+               hide_list($tree,$listclass);
+       } else {
+               $item->delete;
+       }
+       return 1;
+}      
+#
 # Найти все элементы, удоволетворяющие заданному критерию и подставить в
 # них указанные атрибуты
 # 
@@ -1892,3 +2064,4 @@ sub newlistelement {
                return undef;
        }
 }      
+