]> 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 7e02a0008206151effcd8cadaa1d255f34f75301..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;
@@ -975,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();
@@ -1017,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;
 }
 
@@ -1160,8 +1105,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 +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;
                                }
                        }       
@@ -1432,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) = @_;
+}      
 #---------------------------------------------------------- 
 # База пользователей и права доступа
 #----------------------------------------------------------
@@ -1516,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);
 }      
 #
@@ -1534,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 вокруг баги в
@@ -1648,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);
@@ -1675,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;
@@ -1759,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
@@ -1771,7 +1965,7 @@ sub input2tree {
                        $element->delete() if defined $element;
                }       
        }       
-       # Проверяем на наличие URL-ок не оформленных ссылками.
+       # FIXME Проверяем на наличие URL-ок не оформленных ссылками.
        return $txtree;
 }      
 
@@ -1780,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;
@@ -1836,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;
+}      
+#
 # Найти все элементы, удоволетворяющие заданному критерию и подставить в
 # них указанные атрибуты
 # 
@@ -1892,3 +2109,11 @@ sub newlistelement {
                return undef;
        }
 }      
+
+sub escapeHTML {
+       local $_ = shift;
+       s/\&/&amp;/g;
+       s/\</&lt;/g;
+       s/\>/&gt;/g;
+       return $_;
+}