Matthieu Moy <matthieu....@imag.fr> writes:

> Signed-off-by: Matthieu Moy <matthieu....@imag.fr>
> ---
>  contrib/mw-to-git/git-remote-mediawiki | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/contrib/mw-to-git/git-remote-mediawiki 
> b/contrib/mw-to-git/git-remote-mediawiki
> index a6ad8cf..a2da52f 100755
> --- a/contrib/mw-to-git/git-remote-mediawiki
> +++ b/contrib/mw-to-git/git-remote-mediawiki
> @@ -51,6 +51,9 @@ use constant EMPTY_CONTENT => "<!-- empty page -->\n";
>  # used to reflect file creation or deletion in diff.
>  use constant NULL_SHA1 => "0000000000000000000000000000000000000000";
>  
> +# Used on Git's side to reflect empty edit messages on the wiki
> +use constant EMPTY_MESSAGE => '*Empty MediaWiki Message*';
> +

Is there a reason why this sentinel value cannot be a real empty
string?

Not that I am complaining, as I find it highly unlikely for a
message stored in mw to be exactly "*Empty MediaWiki Message*", but
just wondering.

>  my $remotename = $ARGV[0];
>  my $url = $ARGV[1];
>  
> @@ -935,7 +938,7 @@ sub mw_import_revids {
>  
>               my %commit;
>               $commit{author} = $rev->{user} || 'Anonymous';
> -             $commit{comment} = $rev->{comment} || '*Empty MediaWiki 
> Message*';
> +             $commit{comment} = $rev->{comment} || EMPTY_MESSAGE;
>               $commit{title} = mediawiki_smudge_filename($page_title);
>               $commit{mw_revision} = $rev->{revid};
>               $commit{content} = mediawiki_smudge($rev->{'*'});
> @@ -1050,6 +1053,10 @@ sub mw_push_file {
>       my $oldrevid = shift;
>       my $newrevid;
>  
> +     if ($summary eq EMPTY_MESSAGE) {
> +             $summary = '';
> +     }
> +
>       my $new_sha1 = $diff_info_split[3];
>       my $old_sha1 = $diff_info_split[2];
>       my $page_created = ($old_sha1 eq NULL_SHA1);
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to