From e5d01f5516b5802bb6ac5256e03cf57aa0de6962 Mon Sep 17 00:00:00 2001 From: joey Date: Tue, 11 Jul 2006 21:25:16 +0000 Subject: [PATCH] on second though, just remove done patches --- doc/patchqueue.mdwn | 2 - doc/patchqueue/decode-from.mdwn | 108 ----------------------------- doc/patchqueue/done.mdwn | 4 -- doc/patchqueue/webcommit-from.mdwn | 81 ---------------------- 4 files changed, 195 deletions(-) delete mode 100644 doc/patchqueue/decode-from.mdwn delete mode 100644 doc/patchqueue/done.mdwn delete mode 100644 doc/patchqueue/webcommit-from.mdwn diff --git a/doc/patchqueue.mdwn b/doc/patchqueue.mdwn index aa04c1e47..29631fe9d 100644 --- a/doc/patchqueue.mdwn +++ b/doc/patchqueue.mdwn @@ -5,7 +5,5 @@ Feel free to either copy the patch inline, or link to one elsewhere (or nag Joey to open up anonymous svn access to this wiki so you can check in the patches directly). -Link patches to [[patchqueue/done]] when done. - [[inline pages="patchqueue/* !patchqueue/done !link(patchqueue/done) !*/Discussion" rootpage="patchqueue" show="30" archive="yes"]] diff --git a/doc/patchqueue/decode-from.mdwn b/doc/patchqueue/decode-from.mdwn deleted file mode 100644 index 9f1dd8df2..000000000 --- a/doc/patchqueue/decode-from.mdwn +++ /dev/null @@ -1,108 +0,0 @@ -Wow, big hammer! [[patchqueue/Done]] though, with minor mods -- [[Joey]]. - -From [[Faidon]]: - -I consolidated all decode_utf8 in FormBuilder's fields -to make the code more readable and avoid future mistakes. The patch -seems to work and for the first time I have a UTF-8 username ;-) -I think that with this patch and the previous one about locales, we're -done with UTF-8 support. Yay! - - Index: IkiWiki/CGI.pm - =================================================================== - --- IkiWiki/CGI.pm (revision 904) - +++ IkiWiki/CGI.pm (working copy) - @@ -41,6 +41,17 @@ - return 0; - } #}}} - - +sub decode_form_utf8 ($) { #{{{ - + my $form = shift; - + foreach my $f ($form->field) { - + next if Encode::is_utf8(scalar $form->field($f)); - + $form->field(name => $f, - + value => decode_utf8($form->field($f)), - + force => 1, - + ); - + } - +} #}}} - + - sub cgi_recentchanges ($) { #{{{ - my $q=shift; - - @@ -166,6 +177,8 @@ - } - - if ($form->submitted && $form->validate) { - + decode_form_utf8($form); - + - if ($form->submitted eq 'Login') { - $session->param("name", $form->field("name")); - if (defined $form->field("do") && - @@ -282,6 +295,8 @@ - value => userinfo_get($user_name, "locked_pages")); - } - - + decode_form_utf8($form); - + - if ($form->submitted eq 'Logout') { - $session->delete(); - redirect($q, $config{url}); - @@ -326,7 +341,7 @@ - ); - my @buttons=("Save Page", "Preview", "Cancel"); - - - # This untaint is safe because titlepage removes any problimatic - + # This untaint is safe because titlepage removes any problematic - # characters. - my ($page)=decode_utf8($form->param('page')); - $page=titlepage(possibly_foolish_untaint(lc($page))); - @@ -367,16 +382,16 @@ - force => 1); - } - - + decode_form_utf8($form); - + - if ($form->submitted eq "Cancel") { - redirect($q, "$config{url}/".htmlpage($page)); - return; - } - elsif ($form->submitted eq "Preview") { - require IkiWiki::Render; - - # Apparently FormBuilder doesn't not treat input as - - # utf-8, so decode from it. - - my $content=decode_utf8($form->field('editcontent')); - - my $comments=decode_utf8($form->field('comments')); - + my $content=$form->field('editcontent'); - + my $comments=$form->field('comments'); - $form->field(name => "editcontent", - value => $content, force => 1); - $form->field(name => "comments", - @@ -463,8 +478,7 @@ - # save page - page_locked($page, $session); - - - # Decode utf-8 since FormBuilder does not - - my $content=decode_utf8($form->field('editcontent')); - + my $content=$form->field('editcontent'); - - $content=~s/\r\n/\n/g; - $content=~s/\r/\n/g; - @@ -480,7 +494,7 @@ - } - if (defined $form->field('comments') && - length $form->field('comments')) { - - $message.=": ".decode_utf8($form->field('comments')); - + $message.=": ".$form->field('comments'); - } - - if ($config{rcs}) { - @@ -499,7 +513,7 @@ - force => 1); - $form->tmpl_param("page_conflict", 1); - $form->field("editcontent", value => $conflict, force => 1); - - $form->field(name => "comments", value => decode_utf8($form->field('comments')), force => 1); - + $form->field(name => "comments", value => $form->field('comments'), force => 1); - $form->field("do", "edit)"); - $form->tmpl_param("page_select", 0); - $form->field(name => "page", type => 'hidden'); diff --git a/doc/patchqueue/done.mdwn b/doc/patchqueue/done.mdwn deleted file mode 100644 index 0f3dc2f83..000000000 --- a/doc/patchqueue/done.mdwn +++ /dev/null @@ -1,4 +0,0 @@ -Old applied patches: - -[[inline pages="patchqueue/* !patchqueue/done link(patchqueue/done) !*/Discussion" rootpage="patchqueue" show="30" archive="yes"]] - diff --git a/doc/patchqueue/webcommit-from.mdwn b/doc/patchqueue/webcommit-from.mdwn deleted file mode 100644 index ea13c4b44..000000000 --- a/doc/patchqueue/webcommit-from.mdwn +++ /dev/null @@ -1,81 +0,0 @@ -Looks good, [[patchqueue/done]] -- [[Joey]] - -From [[Faidon]]: - -Match 'web commit from' in SVN and git, these are web -commits too. Show the IP instead of the user 'www-data' (which is kind -of security fix too, since the user Apache is running as was visible). - -Note that the git backend patch is untested. -Recai, could you test it? (if Joey finds the change acceptable that is). - - Index: IkiWiki/Rcs/svn.pm - =================================================================== - --- IkiWiki/Rcs/svn.pm (revision 904) - +++ IkiWiki/Rcs/svn.pm (working copy) - @@ -7,7 +7,7 @@ - - package IkiWiki; - - -my $svn_webcommit=qr/^web commit by (\w+):?(.*)/; - +my $svn_webcommit=qr/^web commit (by (\w+)|from (\d+\.\d+\.\d+\.\d+)):?(.*)/; - - sub svn_info ($$) { #{{{ - my $field=shift; - @@ -136,8 +136,8 @@ - my $committype="web"; - if (defined $message[0] && - $message[0]->{line}=~/$svn_webcommit/) { - - $user="$1"; - - $message[0]->{line}=$2; - + $user=defined $2 ? "$2" : "$3"; - + $message[0]->{line}=$4; - } - else { - $committype="svn"; - @@ -178,8 +178,8 @@ - chomp $user; - my $message=`svnlook log $config{svnrepo} -r $rev`; - if ($message=~/$svn_webcommit/) { - - $user="$1"; - - $message=$2; - + $user=defined $2 ? "$2" : "$3"; - + $message=$4; - } - - my @changed_pages; - Index: IkiWiki/Rcs/git.pm - =================================================================== - --- IkiWiki/Rcs/git.pm (revision 904) - +++ IkiWiki/Rcs/git.pm (working copy) - @@ -12,7 +12,7 @@ - my $master_branch = 'master'; # working branch - my $sha1_pattern = qr/[0-9a-fA-F]{40}/; # pattern to validate Git sha1sums - my $dummy_commit_msg = 'dummy commit'; # message to skip in recent changes - -my $web_commit_msg = qr/^web commit by (\w+):?(.*)/; # pattern for web commits - +my $web_commit_msg = qr/^web commit (by (\w+)|from (\d+\.\d+\.\d+\.\d+)):?(.*)/; - - sub _safe_git (&@) { #{{{ - # Start a child process safely without resorting /bin/sh. - @@ -377,8 +377,8 @@ - - if (defined $message[0] && - $message[0]->{line} =~ m/$web_commit_msg/) { - - $user = "$1"; - - $message[0]->{line} = $2; - + $user=defined $2 ? "$2" : "$3"; - + $message[0]->{line}=$4; - } else { - $type ="git"; - $user = $ci->{'author_username'}; - @@ -426,8 +426,8 @@ - - my ($user, $message); - if (@{ $ci->{'comment'} }[0] =~ m/$web_commit_msg/) { - - $user = "$1"; - - $message = $2; - + $user = defined $2 ? "$2" : "$3"; - + $message = $4; - } else { - $user = $ci->{'author_username'}; - $message = join "\n", @{ $ci->{'comment'} }; -- 2.32.0.93.g670b81a890