]> www.wagner.pp.ru Git - oss/stilllife.git/blobdiff - forum/forum
Restructured method dispatching. Fixed html escaping in text formatting
[oss/stilllife.git] / forum / forum
index e73f98af81a9aef22cafb80c16275fd66918ca7f..1cff61efef608c52549773f51fda09e4ab2b3bc8 100755 (executable)
@@ -28,101 +28,47 @@ use Net::OpenID::Consumer;
 # Набор поддерживаемых действий. Хэш вида 
 # "имя поля в запросе" =>  "функция обработчик"
 #
-my %actions = (
-       reply => \&reply,
-       edit => \&edit_comment,
-       delete => \&delete_comment,
-       move => \&move_comment,
-       newtopic=> \&new_topic,
-       newforum=> \&new_forum,
-       login => \&login,
-       register=>\&register,
-       profile=>\&profile,
-       setrights=>\&set_rights,
-       openidlogin=>\&openid_login,
-       openidvfy =>\&openid_verify
+my @actions = (
+       {name=>"openidvfy",GET=>\&openid_verify,POST=>\&openid_verify},
+       {name=>"logout",GET=>\&logout,POST=>\&logout},
+       {name=>"reply", POST => \&reply, GET=>\&show_template,rights=>"login"},
+       {name=>"edit", POST => \&edit_item, GET=>\&show_editable,rights=>"author"},
+       {name=>"delete",POST => \&delete_item, GET=>\&delete_item,rights=>"author"},
+       {name=>"move",POST => \&move_item, GET=>\&show_move_dest,rights=>"moderator"},
+       {name=>"newtopic",POST=> \&new_topic,GET=>\&show_template,rights=>"normal"},
+       {name=>"newforum",POST=> \&new_forum,GET=>\&show_template,rights=>"moderator"},
+       {name=>"login",POST => \&login,GET=>\&show_template},
+       {name=>"register",POST=>\&register,GET=>\&show_template},
+       {name=>"profile",POST=>\&profile,GET=>\&show_profile,rights=>"normal"},
+       {name=>"setrights",POST=>\&set_rights,GET=>\&show_rights,rights=>"admin"},
 );     
 #
-#  Уровень прав, которые необходимо иметь пользователю для совершения
-#  определенного действия
-#  иерархия вида undef < banned < normal < author < moderator <admin
-#  Если операция не упомянута в данном массив, то значит можно всем, в
-#  том числе  и анониму.
-# Слово login означает, что вообще-то это normal, но пользователь может
-# логиниться непосредственно в процессе выполнения операции.
-my %permissions = (
-       reply => "login",
-       edit => "author",
-       delete => "author",
-       newtopic => "normal",
-       move => "moderator",
-       newforum => "moderator",
-       profile => "normal",
-       setrights => "admin",
-);     
 our $path_translated; # Untainted value of PATH_TRANSLATED env var
 my $cgi = new CGI;
 print STDERR "--------------------\n";
 my $forum=get_forum_config();
 
 authorize_user($cgi,$forum);
-if ($cgi->request_method ne "POST") {
-# Запрос к скрипту методом GET. Надо показать форму, если только это не
-# редирект от OpenId-сервера 
-       if ($cgi->param('openidvfy')) { 
-               openid_verify($cgi,$forum);
-       } elsif ($cgi->param("logout")) {
-               logout('logout',$cgi,$forum);
-       } elsif ($cgi->param("profile")) {
-               show_profile("profile",$cgi,$forum);
-       } else {
-               for my $param ($cgi->param) {
-# Среди параметров, указанных в URL ищем тот, который задает
-# действие 
-                       if (exists $actions{$param}) {
-# Мы, конечно уже проверили, что в названии параметра
-# нехороших символов нет, но чтобы perl в taint mode не
-# ругался... 
-                               if (allow_operation($param,$cgi,$forum)) {
-                                       print STDERR "Allow_operation completed\n";
-                                       show_template($1,$cgi,$forum) if $param=~/^(\w+)$/;     
-                                       exit;
-                               } else {
-                                       if (!$forum->{"authenticated"}) { 
-                                               $cgi->param("returnto",$cgi->url(-full=>1));
-                                               show_template("login",$cgi,$forum);
-                                               exit;
-
-                                       } else {
-                                               show_error($forum,"У Вас нет прав на  выполнение этой
-                                               операции")
-                                       }
-                               }       
-                       }
-               }
-               if (index($path_translated,$forum->{userdir})==0) {
-                       show_user_page($cgi,$forum);
-               }       
-               show_error($forum,"Некорректный вызов скрипта. Отсутствует параметр
-                               действия");
-       }       
-} else {
-       # Запрос методом POST. Вызываем обработчик
-       for my $param ($cgi->param) {
-               if (exists $actions{$param}) {
-                       if (allow_operation($param,$cgi,$forum)) {
-                               $actions{$param}->($param,$cgi,$forum);
-                               exit;
-                       } else {
-                               show_error($forum,"У Вас нет прав на  выполнение этой
-                               операции")
-                       }
-
+my $method = $cgi->request_method;
+$method = "GET" if ($method eq "HEAD");
+for my $action (@actions) {
+       if ($cgi->param($action->{name})) {
+               if (allow_operation($action,$cgi,$forum)) {
+                       $action->{$method}->($action->{name},$cgi,$forum);
+               } elsif (!$forum->{"authenticated"}) { 
+                               $cgi->param("returnto",$cgi->url(-full=>1));
+                               show_template("login",$cgi,$forum);
+               } else {
+                       show_error($forum,"У Вас нет прав на  выполнение этой операции")
                }
        }
-       print STDERR "Получены параметры ",join(" ",$cgi->param),"\n";
-       show_error($forum,"Некорректный вызов скрипта. Отсутствует параметр действия");
 }      
+if (index($path_translated,$forum->{userdir})==0) {
+       show_user_page($cgi,$forum);
+}      
+show_error($forum,"Некорректный вызов скрипта. Отсутствует параметр
+               действия");
+       
 
 #-------------------------------------------------------------- 
 #-------- Чтение конфигурационного файла и связанные с этим действия
@@ -167,7 +113,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;
@@ -261,7 +206,7 @@ sub show_error {
                print $cgi->header(-type=>'text/html',-charset=>'utf-8');
                print "<html><head><title>Ошибка конфигурации форума</title></head>",
                "<body><h1>Ошибка конфигурации форума</h1><p>",
-               $cgi->escapeHTML($msg),"</p>",
+               escapeHTML($msg),"</p>",
                "<p>При обработке этой ошибки не обнаружен шаблон сообщения об ошибке</p></body></html>";  
        }
        exit;
@@ -283,6 +228,7 @@ sub send_to_user {
        print
        $cgi->header(-type=>"text/html",-charset=>"utf-8",($forum->{cookies}?(-cookie=>$forum->{cookies}):())),
        output_html($tree);
+       exit;
 }      
 sub prepare_template { 
        my ($form,$cgi,$forum) = @_;
@@ -327,8 +273,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 +297,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 +316,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 +435,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 +447,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");
                }       
@@ -514,7 +466,7 @@ if (defined $user) {
 #
 
 sub authorize_user     {
-       ($cgi,$forum) = @_;
+       my ($cgi,$forum) = @_;
        if (my $session=$cgi->cookie("slsession")) {
        # Пользователь имеет куку
                my %sessbase;   
@@ -665,6 +617,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 +644,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 +685,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;
@@ -751,7 +702,38 @@ sub show_profile {
 #
 sub profile {
        my ($formname,$cgi,$forum) = @_; 
-
+       if (!$cgi->param("user")) {
+               show_error($forum,"В форме нет имени пользователя");
+       }
+       my $user = $cgi->param('user');
+       my $rights = getrights($cgi,$forum);
+       if ($user ne $forum->{authenticated}{user} &&
+               $rights ne "admin") {
+               show_error($forum,"У вас нет прав на изменение профиля этого
+               пользователя");
+       }
+       my %base;
+       dbmopen %base,datafile($forum,"passwd"),0644;
+       if (!$base{$user}) {
+               show_error($forum,"Несуществующий пользователь $user");
+       }
+       my $userinfo = thaw $base{$user};
+       $userinfo->{user}=$user;
+       #
+       # If password fields are filled, change password
+       #
+       if ($cgi->param('pass1')) {
+               if ($cgi->param('pass1') eq $cgi->param('pass2')) {
+                       $userinfo->{passwd}=crypt_password($cgi->param('pass1'));
+               } else {
+                       form_error($formname,$cgi,$forum,"Ошибка при вводе пароля");
+               }       
+       }       
+       make_profile($formname,$cgi,$forum,$userinfo,$rights eq "admin");
+       delete $userinfo->{user};
+       $base{$user} = freeze $userinfo;
+       dbmclose %base;
+       show_profile($formname,$cgi,$forum);
 }
 #
 # Обработка результатов заполнения формы регистрации.
@@ -784,6 +766,7 @@ sub register {
                        }
                }       
        }
+       $cgi->delete("required");
        my %userbase;
        dbmopen %userbase,datafile($forum,"passwd"),0644 
                or form_error($formname,$cgi,$forum,"Ошибка открытия файла паролей $!");
@@ -791,35 +774,50 @@ sub register {
                dbmclose %userbase;
                form_error($formname,$cgi,$forum,"Имя пользователя '".$cgi->param("user"). "' уже занято");
        }
-       if ($cgi->param("email") && !  Email::Valid->address($cgi->param("email"))) {
-               form_error($formname,$cgi,$forum,"Некорректный E-Mail адрес");
+       my $userinfo = {passwd=>crypt_password($cgi->param('pass1'))};
+       make_profile($formname,$cgi,$forum,$userinfo,0);
+       $userinfo->{registered} = time;
+       set_default_user_attrs($forum,$userinfo);
+       print STDERR "stilllife forum: registering user $user\n";
+       $userbase{$user} = freeze($userinfo);
+       dbmclose %userbase;
+       if (!defined $forum->{denied_status} || $userinfo->{status} ne
+               $forum->{denied_status}) { 
+               newsession(undef,$forum,$user);
+               forum_redirect($cgi,$forum,$cgi->param("returnto")); 
+       } else {
+               # FIXME Email validation
+               # Email to admin
+               show_template("newuser",$cgi,$forum);
        }
-       my $saltstring = 'ABCDEFGHIJKLMNOPQRSTUVWXUZabcdefghijklmnopqrstuvwxuz0123456789./';
-       my $salt = substr($saltstring,int(rand(64)),1).
-                               substr($saltstring,int(rand(64)),1);
-       my $password=crypt($cgi->param("pass1"),$salt);                 
-       my $userinfo = {passwd=>$password};
+}      
+sub make_profile {
+       my ($formname,$cgi,$forum,$userinfo,$isadmin) =@_;
        # Удаляем лишние поля
-       $cgi->delete("required");
-       $cgi->delete("register");
-       $cgi->delete("user");
-       $cgi->delete("pass1");
-       $cgi->delete("pass2");
        foreach my $field (split(/\s*,\s*/,$cgi->param('ignore'))) {
                if (!$cgi->param($field)) {
                        $cgi->delete($field);
                }
        }       
-       my $returnto = $cgi->param("returnto");
-       $cgi->delete("returnto");
+       if ($cgi->param("email") && !  Email::Valid->address($cgi->param("email"))) {
+               form_error($formname,$cgi,$forum,"Некорректный E-Mail адрес");
+       }
+       my $user = $userinfo->{user};
+       my $userprefix=$user;
+       $userprefix=~tr!\\/:    !_!;
        # Если есть аватар в файле, то сохраняем этот файл и формируем URL
        # на него.
+       $cgi->delete($formname);
+       $cgi->delete("user");
+       $cgi->delete("pass1");
+       $cgi->delete("pass2");
        if ($cgi->param("avatarfile" )) {
                my $f = $cgi->upload("avatarfile");
                binmode $f,":bytes";
                my $out;
                my $filename = $1 if $cgi->param("avatarfile")=~/([^\/\\]+)$/;
-               open $out,">",$forum->{"userdir"}."/".$filename;
+               my $path = $forum->{"userdir"}."/".$userprefix."_".$filename;
+               open $out,">",$path;
                binmode $out,":bytes";
                my $buffer;
                while (my $bytes = read($f,$buffer,4096)) {
@@ -827,16 +825,24 @@ sub register {
                }       
                close $f;
                close $out;
-               my ($w,$h) = imgsize($forum->{"userdir"}."/".$filename);
-               $userinfo->{'avatar'}= {src=>$forum->{"userurl"}."/".$filename,
+               my ($w,$h) = imgsize($path);
+               $userinfo->{'avatar'}= {src=>$forum->{"userurl"}."/".$userprefix."_".$filename,
                        width=>$w,height=>$h};
-               $cgi->delete("avatar");
-               $cgi->delete("avatarfile");
+       } elsif ($cgi->param('avatar')) {
+               if (!ref($userinfo->{'avatar'}) || 
+                       $userinfo->{avatar}{'src'} ne $cgi->param('avatar')) {
+                       $userinfo->{avatar}=get_avatar_info($cgi->param('avatar'));
+               }       
        }
-       my @restrict = split /\s*,\s*/, $forum->{restricted_user_info};
-       set_default_user_attrs($forum,$userinfo);
+       my @restrict=();
+       @restrict = split /\s*,\s*/, $forum->{restricted_user_info}
+               unless $isadmin;
+
        foreach my $param       ($cgi->param) {
                next if  (grep $_ eq $param,@restrict);
+               next if $param eq 'avatar';
+               next if $param eq 'avatarfile';
+               next if $param eq 'returnto';
                next if $param =~ /_format$/;
                if (defined $cgi->param("${param}_format")) {
                        my $tree = input2tree($cgi,$forum,$param);
@@ -846,22 +852,14 @@ sub register {
                        $userinfo->{$param} = $cgi->param($param);
                }
        }
-       $userinfo->{registered} = time;
-       if (exists $forum->{default_status}) {
-               $userinfo->{status} = $forum->{default_status};
-       }
-       print STDERR "stilllife forum: registering user $user\n";
-       $userbase{$user} = freeze($userinfo);
-       dbmclose %userbase;
-       if (!defined $forum->{denied_status} || $userinfo->{status} ne
-               $forum->{denied_status}) { 
-               newsession(undef,$forum,$user);
-               forum_redirect($cgi,$forum,$returnto); 
-       } else {
-               # FIXME Email validation
-               # Email to admin
-               show_template("newuser",$cgi,$forum);
-       }
+}
+sub crypt_password {
+       my $open_password=shift;
+       my $saltstring = 'ABCDEFGHIJKLMNOPQRSTUVWXUZabcdefghijklmnopqrstuvwxuz0123456789./';
+       my $salt = substr($saltstring,int(rand(64)),1).
+                               substr($saltstring,int(rand(64)),1);
+       my $password=crypt($open_password,$salt);                       
+       return $password;
 }      
 
 sub set_default_user_attrs {
@@ -891,7 +889,7 @@ sub show_user_page {
                        my $userinfo =thaw($base{$user});
                        $userinfo->{"user"} = $user;
                        substitute_user_info($block,$forum,$userinfo);
-                       profile_links($block,$user,$rights,$forum);
+                       profile_links($block,$user,$rights,$cgi,$forum);
          }             
          $usertpl->delete;     
        } else {
@@ -905,7 +903,7 @@ sub show_user_page {
                $tree = gettemplate($forum,"user");
                substinfo($tree,[_tag=>"title"],_content=>"Stilllife user: $user");
                substitute_user_info($tree,$forum,$userinfo);
-               profile_links($tree,$user,$rights,$forum);
+               profile_links($tree,$user,$rights,$cgi,$forum);
                unless ($userinfo->{openiduser}) {
                        for my $userlink ($tree->look_down(_tag => "a",class=>"author")) {
                                $userlink->attr("href",undef);
@@ -920,7 +918,7 @@ sub show_user_page {
        $page;
 }
 sub profile_links {
-       my ($tree,$user,$rights,$forum)=@_;
+       my ($tree,$user,$rights,$cgi,$forum)=@_;
        foreach my $profile_link ($tree->look_down(_tag=>"a",
                        href=>qr/profile=/)) {
                if ((defined $rights && $rights eq "admin")|| 
@@ -928,7 +926,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();
@@ -970,24 +968,18 @@ sub logout {
 }      
 sub allow_operation {
        my ($operation,$cgi,$forum) = @_;
-       return 1 if (!exists($permissions{$operation})); 
+       return 1 if (!exists($operation->{rights})); 
        if (!$forum->{authenticated}) {
-               return 1 if ($permissions{$operation} eq "login");
+               return 1 if ($operation->{rights} eq "login");
                return 0;
        }       
        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 0 if ($operation->{rights} 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;
 }
 
@@ -1113,12 +1105,12 @@ 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);
-        
+       forum_redirect($cgi,$forum,$cgi->path_info."#$id");
 }      
 sub update_topic_list {
        my ($forum,$topic,$count,$date) = @_;
@@ -1152,8 +1144,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;
                                }
                        }       
@@ -1386,7 +1378,248 @@ sub new_forum {
 
        forum_redirect($cgi,$forum,$cgi->url(-base=>1).$url);
 }
-       
+#
+# Обработка операций, которые вызываются одинаково,
+# но выполняются по-разному для разных типов объектов
+#
+# Параметры $cgi,$forum, тип => ссылка на функцию ...
+# где тип - message, topic или forum. Определяет обрабатываемый объект
+# и вызывает соответствующую фунцкию. Ожидает что функция завершится 
+#вызовом exit. 
+sub dispatch_objtype {
+       my $cgi=shift;
+       my $forum = shift;
+       my %actions=@_;
+       if (-f $path_translated) {
+               if ($cgi->param("id")) {
+                       $actions{"message"}->($cgi,$forum,$path_translated,$cgi->param("id"))
+                       if exists($actions{'message'});
+               } else {
+                       $actions{topic}->($cgi,$forum,$path_translated) 
+                               if exists($actions{'topic'});
+               }       
+       } elsif (-d $path_translated && -f $path_translated ."/".  $forum->{indexfile}) {
+               $actions{'forum'}->($cgi,$forum,$path_translated) 
+                       if exists($actions{'forum'});
+
+       }
+       return undef;
+}
+#Удаление
+sub delete_item {
+       my ($formname,$cgi,$forum) = @_;
+       #
+       # Сначала разберемся, что мы собственно удаляем 
+       #
+       if ($cgi->param("user")) {
+               delete_user($cgi,$forum,$cgi->param("user"));
+       }       
+        dispatch_objtype($cgi,$forum,topic=>\&delete_topic,
+                       message=>\&delete_comment,
+                       forum=>\&delete_forum);
+       show_error($forum,"Невозможно удалить неопознанный объект");
+}      
+# Показ формы редактирования
+sub show_editable {
+       my ($formname,$cgi,$forum) = @_;
+       dispatch_objtype($cgi,$forum,"message"=>\&show_messageedit,
+               topic=>\&show_topicedit,
+               forum=>\&show_forumedit);
+       show_error($forum,"Невозможно редактировать неопознанный объект");
+}              
+# Применение результатов редактирования
+sub edit_item {
+       my ($formname,$cgi,$forum) = @_;
+       dispatch_objtype($cgi,$forum,"message"=>\&messageedit,
+               topic=>\&topicedit,
+               forum=>\&forumedit);
+       show_error($forum,"Невозможно редактировать неопознанный объект");
+}              
+sub move_item {
+       my ($formname,$cgi,$forum) = @_;
+       dispatch_objtype($cgi,$forum,"message"=>\&move_message,
+               topic=>\&move_topic,
+               forum=>\&move_forum);
+       show_error($forum,"Невозможно переместить неопознанный объект");
+}
+sub show_move_dest {
+       my ($formname,$cgi,$forum) = @_;
+       dispatch_objtype($cgi,$forum,"message"=>\&show_move_message,
+               topic=>\&show_move_topic,
+               forum=>\&show_move_forum);
+       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 moderator_or_author {
+       my ($cgi,$forum,$msg)=@_;
+       return 1 if getrights($cgi,$forum) eq "moderator" 
+               || getrights($cgi,$forum) eq "admin";
+       my $author= $msg->look_down(_tag=>"input",name=>"author");
+       if ($author) {
+               return $author->attr("value") eq $forum->{authenticated}{user};
+       } elsif ($author = $msg->look_down(class=>"author",_tag=>"a")) {
+               return $author->as_text eq $forum->{authenticated}{user};
+       } else {
+               return undef;
+       }
+}      
+#
+# Удаление реплики
+#
+sub delete_comment {
+       my ($cgi,$forum,$topic,$id) = @_;
+       my ($tree,$lockfd) = gettree($topic);
+       my ($msg) = $tree->look_down(id => $id);
+       show_error($forum,"В данной теме нет реплики с id=$id") if (!$msg);
+       show_error($forum,"У вас нет прав на удаление этого сообщения")
+               unless moderator_or_author($cgi,$forum,$msg);           
+       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($forum,"Нельзя удалять непустой форум") 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);      
+}      
+# Показ сообщения для редактирования
+sub show_messageedit {
+       my ($cgi,$forum,$path,$id)=@_;
+       my  ($tree,$lockfd) = gettree($path);
+       my $dirname = $path;
+       $dirname =~ s/\/[^\/]+$//;
+       my ($msg) = $tree->look_down(id => $id);
+       show_error($forum,"В данной теме нет реплики с id=$id") if (!$msg);
+       show_error($forum,"У вас нет прав на редактированиее этого сообщения") 
+               unless moderator_or_author($cgi,$forum,$msg);           
+       # Выбираем данные сообщения
+       my ($text) = $msg->look_down(class=>"mtext")->content_list;
+       my ($subject) = $msg->look_down(class=>"subject");
+       $cgi->param("text"=>$text->as_HTML('<>&"'));
+       $cgi->param("text_format"=>"html");
+       $cgi->param("subject"=> $subject->as_text);
+       discardtree($tree,$lockfd);
+       # Подготавливаем шаблон
+       my $form = prepare_template("edit_message",$cgi,$forum);
+       # Подставляем информацию о картинках.
+       opendir D,$dirname;
+       while ( my $filename=readdir D) {
+               next unless $filename =~/^${id}_/; 
+               my $pic = "$dirname/$filename";
+               my $picitem = newlistelement($form,"picture","picturelist");
+               my ($w,$h) = imgsize($pic);
+               substinfo($picitem,[_tag=>"img", class=>"msgpicture"],
+                       src=>dir2url($cgi,$pic),width=>$w,height=>$h);
+               substinfo($picitem,[_tag=>"input",name=>"delpicture"],
+                       value=>$filename);
+               substinfo($picitem,[class=>"filename"],_content =>$filename);
+       }
+       closedir D;
+       send_to_user($form,"edit_message",$cgi,$forum); 
+}
+# Показ темы для редактирования
+sub show_topicedit {
+       my ($cgi,$forum,$path)=@_;
+}
+# Показ форума для редактирования
+sub show_forumedit {
+       my ($cgi,$forum,$path) = @_;
+}      
 #---------------------------------------------------------- 
 # База пользователей и права доступа
 #----------------------------------------------------------
@@ -1415,6 +1648,8 @@ sub getrights {
        if (!$forum->{authenticated}) {
                return undef;
        }       
+       return $forum->{authenticated}{rights} if 
+               exists $forum->{authenticated}{rights};
        my $user = $forum->{authenticated}{user};
        my $dir = $path_translated;
        $dir =~s/\/$//;
@@ -1433,10 +1668,10 @@ sub getrights {
                                        chomp;
                                        if  ($user eq $_ && defined $status) {
                                                if ($status eq "banned") {
-                                                       return $status;
+                                                       return $forum->{authenticated}{rights}=$status;
                                                } 
                                                if ($status eq "admins" ) {
-                                                       return "admin";
+                                                       return $forum->{authenticated}{rights}="admin";
                                                }
                                                $user_status = "moderator";
                                        }
@@ -1448,7 +1683,7 @@ sub getrights {
                # Strip last path component.
                $dir =~s/\/[^\/]+$// 
        }               
-       return $user_status;
+       return $forum->{authenticated}{rights}=$user_status;
 
 }              
 
@@ -1468,7 +1703,6 @@ sub gettree {
        open $f,"<",$filename or return undef;
        flock $f, LOCK_EX;
        my $tree = treefromfile($f);
-       $tree->parse_file($f);
        return ($tree,$f);
 }      
 #
@@ -1486,6 +1720,14 @@ sub savetree {
        unlink $filename;
        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 вокруг баги в
@@ -1564,6 +1806,7 @@ sub create_openid_consumer {
                required_root => $cgi->url(-base=>1));
 }              
 
+
 # openidstart - вызывается когда обнаружено что текущее имя
 # пользователя, пытающегося аутентифицироваться, содержит http://
 #  
@@ -1599,7 +1842,7 @@ sub openidstart {
 # реплики) 
 #
 sub openid_verify {
-       my ($cgi,$forum) = @_;
+       my ($action,$cgi,$forum) = @_;
        my $csr  = create_openid_consumer($cgi,$forum);
        if (my $setup_url = $csr->user_setup_url) {
                print $cgi->redirect(-location=>$setup_url);
@@ -1626,7 +1869,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;
@@ -1654,6 +1896,20 @@ sub openid_verify {
        }       
 }
 
+sub get_avatar_info {
+       my ($url,$ua) = @_; 
+       $ua = LWP::UserAgent->new( agent => "Stilllife/1.0") unless $ua;
+       my $response = $ua->get($url);
+       if ($response->is_success) {
+                       my $image = $response->content;
+                       my ($w,$h,$type) = imgsize(\$image);
+                       return {width=>$w,height=>$h,type=>$type,src=>$url};
+               } else {
+                       print STDERR "Error getting $url: ".$response->status_line,"\n";
+                       return undef;
+               }       
+}
+
 sub get_foaf {
        my ($ua,$foaf_url) = @_; 
        my $response = $ua->get($foaf_url);
@@ -1664,17 +1920,11 @@ sub get_foaf {
        my $foaf = $response->content;
        my %info = foaf_parse($foaf);
        if ($info{avatar}) {
-               $response = $ua->get($info{avatar});
-               if ($response->is_success) {
-                       my $image = $response->content;
-                       my ($w,$h,$type) = imgsize(\$image);
-                       $info{avatar}={width=>$w,height=>$h,type=>$type,src=>$info{avatar}};
-               } else {
-                       print STDERR "Error getting $info{avatar}: ".$response->status_line,"\n";
-               }       
+               $info{avatar} = get_avatar_info($info{avatar},$ua);
        }       
        return %info;
 }
+
 sub foaf_parse {
        my $foaf = shift;
        my ($starttag) = $foaf =~ /<(\w+(:\w+)?[^>]+)>/sg;
@@ -1702,9 +1952,10 @@ sub input2tree {
                my $parser = HTML::BBReverse->new(); 
                $text="<div class=\"bbcode\">".$parser->parse($text)."</div>";
        } elsif ($format eq "text") {
-               $text=~s/\r?\n\r?\n/<\/p><p class=\"text\">/;
-               $text=~s/\r?\n/<br>/;
-               $text = "<div><p class=\"text\">".$text."</p></div>";
+               $text = escapeHTML($text);
+               $text=~s/\r?\n\r?\n/<\/p><p class=\"text\">/sg;
+               $text=~s/\r?\n/<br>/sg;
+               $text = "<div><p class=\"text\">$text</p></div>";
        } 
        my $txtree = str2tree($text);
        for my $badtag
@@ -1714,7 +1965,7 @@ sub input2tree {
                        $element->delete() if defined $element;
                }       
        }       
-       # Проверяем на наличие URL-ок не оформленных ссылками.
+       # FIXME Проверяем на наличие URL-ок не оформленных ссылками.
        return $txtree;
 }      
 
@@ -1723,12 +1974,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;
@@ -1779,6 +2035,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;
+}      
+#
 # Найти все элементы, удоволетворяющие заданному критерию и подставить в
 # них указанные атрибуты
 # 
@@ -1835,3 +2109,11 @@ sub newlistelement {
                return undef;
        }
 }      
+
+sub escapeHTML {
+       local $_ = shift;
+       s/\&/&amp;/g;
+       s/\</&lt;/g;
+       s/\>/&gt;/g;
+       return $_;
+}