]> www.wagner.pp.ru Git - oss/stilllife.git/blobdiff - forum/forum
Отрефакторена подстановка атрибутов в дерево. Поправлено распознавание
[oss/stilllife.git] / forum / forum
index c9bb2c6caa4e92a6c173f0a909ea2b86bfc63fe3..e7cbc9218f6e23ae34691ed508adc207a72147c9 100755 (executable)
@@ -19,6 +19,9 @@ use HTML::TreeBuilder;
 use Storable qw(freeze thaw);
 use Date::Parse;
 use Email::Valid;
+use Image::Size;
+use HTML::BBReverse;
+use POSIX;
 use LWP::UserAgent;
 use Net::OpenID::Consumer;
 #
@@ -345,7 +348,7 @@ sub show_template {
                }
 
        }
-       $f->attr("type","POST");
+       $f->attr("method","POST");
        for my $required ($form,"returnto") {
                if (!$substituted{$required}) {
                        my $element = new HTML::Element('input',
@@ -377,6 +380,18 @@ sub fix_forum_links {
                } else {
                        $attr ="src";
                }
+               
+               # Обрабатываем наши специальные link rel=""
+               if ($element->tag eq "link") {
+                       if ($element->attr("rel") eq "forum-user-list") {
+                               $element->attr("href" => $cgi->url(-absolute=>1,
+                                       -path_info=>0,-query_string=>0).$forum->{userurl});
+                               next ELEMENT;   
+                       } elsif ($element->attr("rel") eq "forum-script")  {
+                               $element->attr("href" => $script_with_path);
+                               next ELEMENT;
+                       }       
+               }
                my $link = $element->attr($attr);
                # Абсолютная ссылка - оставляем как есть. 
                next ELEMENT if (! defined $link || $link=~/^\w+:/); 
@@ -409,39 +424,27 @@ my %userinfo = %{$forum->{"authenticated"}};
 #
 # Специально обрабатываем поля user (должна быть ссылка) и avatar  
 # (должен быть img).
-my @userlink = $tree->look_down("_tag"=>"a","class"=>"author");
-if (@userlink) {
        my $userpage;
-       if ($userinfo{"user"}=~/^http:/) {
-               $userpage = $userinfo{"user"};
+       if ($userinfo{"openiduser"}) {
+               $userpage = "http://".$userinfo{"user"};
        } else {
                $userpage =
                $cgi->url(-absolute=>1).$forum->{"userurl"}."/".$cgi->escape($userinfo{"user"});
        }       
-       for my $element (@userlink) {
-               $element->attr(href=>$userpage);
+       substinfo($tree,["_tag"=>"a","class"=>"author"],
+        href=>$userpage,_content=>$userinfo{"user"});
+       delete $userinfo{"user"};
+       substinfo($tree,["_tag"=>"img","class"=>"avatar"],
+       src=>$userinfo{"avatar"}||$forum->{templatesurl}."/1x1.gif");
+       delete $userinfo{"avatar"};
+
+       for my $element ( $tree->look_down("class",qr/^ap-/)) {
+               my $field=$1 if $element->attr("class")=~/^ap-(.*)$/;   
                $element->delete_content();
-               $element->push_content($userinfo{"user"});
-       }
-}      
-delete $userinfo{"userpage"};
-delete $userinfo{"user"};
-my $avatar = $tree->look_down("_tag"=>"img","class"=>"avatar");
-if ($avatar) {
-       $avatar->attr(src=>$userinfo{"avatar"});
-}
-delete $userinfo{"avatar"};
-
-while (my ($field,$value)=each %userinfo) {
-       my $element = $tree->look_down("class","a".$field);
-       if ($element) {
-               $element->delete_content();
-               # 
-               # FixME - allow HTML in author attributes
-               $element->push_content($value);
+               $element->push_content(str2tree($userinfo{$field})) 
+                               if $userinfo{$field};
        }
 
-}
 
 }
 #
@@ -457,10 +460,18 @@ sub authorize_user        {
                dbmopen %sessbase,datafile($forum,"session"),0644;
                        if ($sessbase{$session})  {
                                my ($user,$expires,$ip)=split(";", $sessbase{$session});
+                               my $user_cookie = $cgi->cookie("sluser");
+                               if ($user_cookie ne $user && $user_cookie ne
+                               "http://".$user) {
+                                       clear_user_cookies($cgi,$forum);
+                                       show_error($forum,"Некорректная пользовательская сессия");
+                                       exit;
+                               }       
                                if (!defined $ip|| $ip eq $ENV{'REMOTE_ADDR'}) {
                                        my %userbase;
                                        dbmopen %userbase,datafile($forum,"passwd"),0644;
                                        if ( $userbase{$user}) {
+                                               print STDERR "getting user info for $user\n";
                                                my $userinfo = thaw($userbase{$user});
                                                delete $userinfo->{"passwd"};
                                                $userinfo->{"user"} = $user;
@@ -470,13 +481,16 @@ sub authorize_user        {
                                                }
                                                print STDERR "user $user restored session $session\n";
                                                $forum->{"authenticated"}=$userinfo;
-                                               print STDERR "authorize_user:
-                                               ",$forum->{authenticated}{user},
+                                               print STDERR "authorize_user: ",$forum->{authenticated}{user},
                                                $forum->{authenticated},"\n";
                                        }       
                                        dbmclose %userbase; 
                                }       
-                       }       
+                       } else {
+                               clear_user_cookies($cgi,$forum);
+                               show_error($forum,"Некорректная пользовательская сессия");
+                               exit;
+                       }
                dbmclose %sessbase;
        }
 }
@@ -510,7 +524,9 @@ sub newsession {
        } while ($base->{$sessname});
        my $cookie = $cgi->cookie(-name=>"slsession",
                -expires => $forum->{"authperiod"},-value=> $sessname);
-       $base->{$sessname}=$user.";".str2time($cookie->expires()).
+       my $username = $user;
+       $username =~ s/^http:\/\///; #Remoove http:// from OpenID user names 
+       $base->{$sessname}=$username.";".str2time($cookie->expires()).
                ($ip?";$ENV{'REMOTE_ADDR'}":"");
                
        $forum->{'cookies'}=[ $cookie,
@@ -692,15 +708,19 @@ sub login {
                show_template(@_);
        }       
 }      
+sub clear_user_cookies {
+       my ($cgi,$forum) = @_;
+       $forum->{cookies}=[ $cgi->cookie(-name=>"sluser", -value=>"0",
+       -expires=>"-1m"),$cgi->cookie(-name=>"slsession", -value=>"0",
+                       -expires => "-1m")];
+}                      
 #
 # Обработчик формы logout. В отличие от большинства обработчиков форм,
 # поддерживает обработку методом GET
 #
 sub logout {
        my ($form,$cgi,$forum) = @_;
-       $forum->{cookies}=[ $cgi->cookie(-name=>"sluser", -value=>"0",
-       -expires=>"-1m"),$cgi->cookie(-name=>"slsession", -value=>"0",
-                       -expires => "-1m")];
+       clear_user_cookies($cgi,$forum);
        if (defined (my $session_id = $cgi->cookie("slsession"))) {
                my %sessiondb;
                dbmopen %sessiondb,datafile($forum,"session"),0644;
@@ -740,37 +760,131 @@ sub reply {
        #
        # Находим файл дискуссии, в который надо поместить реплику
        #
+       my ($tree,$lockfd)=gettree($ENV{'PATH_TRANSLATED'}); 
+       my $messagetpl = $tree->look_down(class=>"message");
+       if (!$messagetpl) {
+               show_error($forum,"Шаблон темы не содержит элемента с классом
+               message");
+               exit;
+       }       
        
+       #       
+       # Генерируем идентификатор записи.
        #
-       # Сохраняем приаттаченную картинку, если есть.
+       my $id = get_uid($forum);
+
+
        #
-       
-       # Генерируем идентификатор записи.
+       # Сохраняем приаттаченные картинки, если есть.
        #
-       
+       my $dir = $1 if $ENV{PATH_TRANSLATED}=~/^(.*)$/;
+       $dir=~ s/[^\/]+$// if (-f $dir);
+       my %attached;
+       for (my $i=1;$cgi->param("image$i"); $i++) {
+               my $userpath=$cgi->param("image$i");
+               my $filename=lc($1) if $userpath =~ /([^\/\\]+)$/;
+               $attached{$filename} = $id."_".$filename;
+               my $in = $cgi->upload("image$i");
+               if (!$in) {
+                       show_error($forum,"Ошибка при загрузке картинки $filename");
+                       exit;
+               }       
+               my $out;
+               open $out,">$dir/$attached{$filename}";
+               binmode $out,":bytes";
+               local $/=undef;
+               my $data = <$in>;
+               print $out $data;
+               close $in;
+               close $out;
+       }
        #
        # Преобразуем текст записи в html и чистим его
        #
-       my $txtree = undef;
-       if ($cgi->param("format") eq "bbcode") {
-
-       } elsif ($cgi->param("format") eq "text") {
-               my $text = $cgi->escapeHTML($cgi->param("text"));
-               $text=~s/\r?\n\r?\n/<p>/;
-               $text=~s/\n/<br>/;
-               $txtree =
-               HTML::TreeBuilder->new_from_content("<div>$text</div>");
-       } else { # Default - html
-               $txtree =
-               HTML::TreeBuilder->new_from_content("<div>".$cgi->param("text")."</div>");      
-               for my $badtag
-               ("script","style","head","html","object","embed","iframe","frameset","frame",
-               ($forum->{forbid_tags}?split(/\s*,\s*/,$forum->{forbid_tags}):())) {
-                       for my $element ($txtree->find_by_tag_name($badtag)) {
-                               $element->delete() if defined $element;
-                       }       
+       my $txtree = input2tree($cgi,$forum,"text");
+       #
+       # Находим в тексте URL на приаттаченные картинки и меняем на те
+       # имена, под которыми мы их сохранили.
+       #
+       for my $image ($txtree->find_by_tag_name("img")) {
+               my $file=lc($image->attr("src"));
+               if ( exists $attached{$file}) {
+                       $image->attr("src" => $attached{$file});
+                       my ($width,$height) = imgsize($dir ."/".$attached{$file});              
+                       $image->attr("width" =>$width);
+                       $image->attr("height" => $height);
                }       
        }       
+       #
+       # Копируем элемент с классом message
+       #
+       my $newmsg = $messagetpl->clone;
+       my $parent = $messagetpl->parent;
+       $parent->push_content($newmsg);
+       #
+       # Подставляем данные сообщения 
+       #
+       $newmsg->attr("id"=>$id);
+       substinfo($newmsg,[class=>"subject"],_content=>$cgi->param("subject"));
+       my $textnode=$newmsg->look_down("class"=>"mtext");
+       if (!$textnode) {
+               show_error($forum,"В шаблоне реплики нет места для текста"); 
+       }       
+       $textnode->delete_content();
+       $textnode->push_content($txtree);
+       if ($forum->{authenticated}{signature}) {
+               $textnode->push_content(new HTML::Element("br"),"--",
+               new HTML::Element("br"),str2tree($forum->{authenticated}{signature}));
+       }
+       substitute_user_info($newmsg,$forum);
+       #
+       # Подставляем данные в форму msginfo
+       #
+       my $editform=$newmsg->look_down(_tag=>"form","class"=>"msginfo");
+       if ($editform) {
+               substinfo($editform,[_tag=>"input",name=>"id"],value=>$id) ||
+                       show_error($forum,"В форме управления сообщением нет поля id");
+               substinfo($editform,[_tag=>"input",name=>"author"],value=>
+                       $forum->{authenticated}{user}) ||
+                       show_error($forum,"В форме управления сообщением нет поля author");
+       }
+       # Подставляем mdate
+       substinfo($newmsg,["class"=>"mdate"],
+               _content =>strftime("%d.%m.%Y %H:%M",localtime()));
+       # Подставляем mreply
+       substinfo($newmsg,[_tag=>"a","class"=>"mreply"],"href" =>
+        $cgi->url(-absolute=>1,-path_info=>1)."?reply=1&id=$id");
+       # Подставляем manchor
+       substinfo($newmsg,[_tag=>"a","class"=>"manchor"],
+               "name"=>"#$id","href"=>undef) or
+               show_error($forum,"В шаблоне сообщения отсутствует якорь для ссылок на него");
+       # подставляем mlink
+       substinfo($newmsg,[_tag=>"a","class"=>"mlink"],
+               href=>$cgi->path_info."#id");
+       # подставляем mparent
+       my $parent_id=$cgi->param("id");
+       if ($parent_id) {
+               substinfo($newmsg,[_tag => "a",class=>"mparent"], 
+                       "href"=>$cgi->path_info."#$parent_id");
+       } else {
+               substinfo($newmsg,[_tag => "a",class=>"mparent"], 
+                       "_content"=>"");
+       }       
+
+       #
+       # Проверяем видимость списка сообщений 
+       #
+       my $msglist = $tree->look_down("class"=>"messagelist");
+       if ($msglist) {
+               my $style = $msglist->attr("style");
+               $msglist->attr("style",$style) if $style && $style =~ s/display: none;//;
+       }       
+       #
+       # Делаем Уфф и сохраняем то, что получилось 
+       #
+       savetree($ENV{PATH_TRANSLATED},$tree,$lockfd);
+       forum_redirect($cgi,$forum);
+        
 }      
 #
 # читает файлы прав доступа в дереве форума, и возвращает
@@ -828,7 +942,7 @@ sub gettree {
        my $f;
        open $f,"<",$filename or return undef;
        flock $f, LOCK_EX;
-       my $tree = HTML::TreeBuider->new_from_file($f);
+       my $tree = HTML::TreeBuilder->new_from_file($f);
        return ($tree,$f);
 }      
 #
@@ -837,8 +951,9 @@ sub gettree {
 #
 
 sub savetree {
-       my ($filename,$tree,$lockfd) = shift;
+       my ($filename,$tree,$lockfd) = @_;
        my $f;
+       $filename = $1 if $filename =~ /^(.*)$/;
        open $f,">",$filename . ".new" or return undef;
        print $f $tree->as_HTML("<>&");
        close $f;
@@ -856,18 +971,19 @@ sub savetree {
 sub get_uid {
        my $forum = shift;
        my $f;
-       open $f,"<+",datafile($forum,"sequence");
+       open $f,"+<",datafile($forum,"sequence") or 
        flock $f,LOCK_EX;
        my $id=<$f> || "0";
        $id++;
        seek $f,0,0;
        printf $f "%8s\n",$id;
        close $f;
-       return sprintf ("%08s",$id);
+       $id=~/(\d+)/;
+       return sprintf ("%08s",$1);
 }
-#
-# ----------------- OpenID registration -----------------------------
-# 
+# --------------------------------------------------------------------
+#  OpenID registration
+# -------------------------------------------------------------------
 sub create_openid_consumer {
        my ($cgi,$forum) = @_;
        return Net::OpenID::Consumer ->new(
@@ -929,13 +1045,15 @@ sub openid_verify {
                $user=~s/\/$//;
                my %userbase;
                dbmopen %userbase,datafile($forum,"passwd"),0664;
-               if (!$userbase{$user}) {
-                       $userbase{$user} = $forum->{authenticated}={};
+               my $username = $user; 
+               $username =~ s/^http:\/\///;
+               if (!$userbase{$username}) {
+                       $userbase{$username} = freeze($forum->{authenticated}={"openiduser"=>1});
                } else {
-                       $forum->{authenticated} = thaw ($userbase{$user});
+                       $forum->{authenticated} = thaw ($userbase{$username});
                }
                dbmclose %userbase;
-               $forum->{"authenticated"}{"user"} = $user;
+               $forum->{"authenticated"}{"user"} = $username;
                newsession(undef,$forum,$user);
                # Если указан параметр reply, вызываем обработку реплики
                if ($cgi->param("reply")) {     
@@ -949,3 +1067,90 @@ sub openid_verify {
                exit;
        }       
 }
+#-----------------------------------------------------------------
+# Обработка форматированных текстовых полей
+#-----------------------------------------------------------------
+
+sub input2tree {
+       my ($cgi,$forum,$field_name) = @_;
+       my $format = $cgi->param($field_name."_format");
+       my $text = $cgi->param($field_name);
+       if ($format eq "bbcode") {
+               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>";
+       } 
+       my $txtree = str2tree($text);
+       for my $badtag
+       ("script","style","head","html","object","embed","iframe","frameset","frame",
+       ($forum->{forbid_tags}?split(/\s*,\s*/,$forum->{forbid_tags}):())) {
+               for my $element ($txtree->find_by_tag_name($badtag)) {
+                       $element->delete() if defined $element;
+               }       
+       }       
+       # Проверяем на наличие URL-ок не оформленных ссылками.
+       return $txtree;
+}      
+
+
+
+sub str2tree {
+       my ($data)=@_;
+       my $tree = HTML::TreeBuilder->new();
+       # 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") {
+       }
+       $element->detach;
+       $tree->destroy;
+       return $element;
+}      
+
+sub tree2str {
+       my ($tree)=@_;
+       return $tree->as_HTML("<>&");
+}
+
+#------------------------------------------------------------------------
+# Подстановка в дереве
+#------------------------------------------------------------------------
+
+#
+# Найти все элементы, удоволетворяющие заданному критерию и подставить в
+# них указанные атрибуты
+# 
+# Параметры 1. Дерево (класса HTML::Element)
+# 2. Запрос - ссылка на список вида атрибут=>значение. 
+#    Этот список будет непосредственно передан в
+#    HTML::Element::look_down
+# 3. Далее пары имя-атрибута, значение. Если вместо имени атрибута
+#    использовать слово _content, заменено будет содержимое элемента.
+#    Значение для _content - ссылка на HTML::Element. Если там строка,
+#    она будет вставлена как одиночный текстовый узел.
+# 4. Возвращает число выполненных подстановок (0, если искомых элементов   
+#   не найдено.
+#
+sub substinfo {
+       my ($tree,$query,@attrs) = @_;
+       my $count;
+       foreach my $element ($tree->look_down(@$query)) {
+               $count ++;
+               while (@attrs) {
+                       my $attr = shift @attrs;
+                       my $value = shift @attrs;
+                       if ($attr eq "_content") {
+                               $element->delete_content;
+                               $element->push_content($value);
+                       } else {        
+                               $element->attr($attr,$value);
+                       }       
+               }       
+       }
+       return $count;  
+}
+