[mb-commits] branch, beta, updated. MBS-5618: properly URI-escape utf8 for wikipedia extracts

MusicBrainz Git Server <[email protected]>
Newsgroups gmane.comp.audio.musicbrainz.cvs
Message-ID <E1TdPmr-0007e7-Lg@wiley>
The branch, beta has been updated
       via  http://git.musicbrainz.org/gitweb/?p=musicbrainz-server/core.git;a=commit;h=aea7f79a12df01b5ca4d56bd52bbd43ea886b7c7 (commit)
      from  http://git.musicbrainz.org/gitweb/?p=musicbrainz-server/core.git;a=commit;h=fd78521419d77655693a2a64fe1a4c8f48e52dbf (commit)

Summary of changes:
 lib/MusicBrainz/Server/Data/WikipediaExtract.pm |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit aea7f79a12df01b5ca4d56bd52bbd43ea886b7c7
Author: Ian McEwen <[email protected]>
Date:   Tue Nov 27 11:21:11 2012 -0700

    MBS-5618: properly URI-escape utf8 for wikipedia extracts

diff --git a/lib/MusicBrainz/Server/Data/WikipediaExtract.pm b/lib/MusicBrainz/Server/Data/WikipediaExtract.pm
index 2931e4d..59e0509 100644
--- a/lib/MusicBrainz/Server/Data/WikipediaExtract.pm
+++ b/lib/MusicBrainz/Server/Data/WikipediaExtract.pm
@@ -6,7 +6,7 @@ use Readonly;
 use aliased 'MusicBrainz::Server::Entity::WikipediaExtract';
 use JSON;
 use Encode qw( encode );
-use URI::Escape qw( uri_escape );
+use URI::Escape qw( uri_escape_utf8 );
 use List::Util qw( first );
 
 with 'MusicBrainz::Server::Data::Role::Context';
@@ -102,7 +102,7 @@ sub _extract_by_language_callback
 sub _get_cache_and_key
 {
     my ($self, $prefix, $title, $language) = @_;
-    $title = uri_escape($title);
+    $title = uri_escape_utf8($title);
     my $cache = $self->c->cache;
     my $cache_key = "wp:$prefix:$title:$language";
 

-----------------------------------------------------------------------


hooks/post-receive
-- 
mb_server
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.