Skip to content

Commit

Permalink
remove Mojolicious::Plugin::Charset
Browse files Browse the repository at this point in the history
  • Loading branch information
kraih committed Jun 22, 2016
1 parent 74ff057 commit 503dec9
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 76 deletions.
1 change: 1 addition & 0 deletions Changes
@@ -1,6 +1,7 @@

7.0 2016-06-22
- Code name "Nerd Face", this is a major release.
- Removed Mojolicious::Plugin::Charset.
- Removed squish method from Mojo::ByteStream.
- Removed squish function from Mojo::Util.
- Removed support for smart whitespace trimming from all_text and text methods
Expand Down
2 changes: 0 additions & 2 deletions lib/Mojolicious/Guides.pod
Expand Up @@ -397,8 +397,6 @@ This is the class hierarchy of the L<Mojolicious> distribution.

=over 2

=item * L<Mojolicious::Plugin::Charset>

=item * L<Mojolicious::Plugin::Config>

=over 2
Expand Down
68 changes: 0 additions & 68 deletions lib/Mojolicious/Plugin/Charset.pm

This file was deleted.

4 changes: 0 additions & 4 deletions lib/Mojolicious/Plugins.pm
Expand Up @@ -78,10 +78,6 @@ examples.
=over 2
=item L<Mojolicious::Plugin::Charset>
Change the application charset.
=item L<Mojolicious::Plugin::Config>
Perl-ish configuration files.
Expand Down
9 changes: 7 additions & 2 deletions t/mojolicious/charset_lite_app.t
Expand Up @@ -10,8 +10,13 @@ use Test::Mojo;
my $yatta = 'やった';
my $yatta_sjis = b($yatta)->encode('shift_jis')->to_string;

# Charset plugin
plugin Charset => {charset => 'Shift_JIS'};
app->types->type(html => 'text/html;charset=Shift_JIS');
app->renderer->encoding('Shift_JIS');
app->hook(
before_dispatch => sub {
shift->req->default_charset('Shift_JIS')->url->query->charset('Shift_JIS');
}
);

# UTF-8 text renderer
app->renderer->add_handler(
Expand Down

2 comments on commit 503dec9

@s1037989
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why was this plugin removed?

@kraih
Copy link
Member Author

@kraih kraih commented on 503dec9 Jun 22, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It doesn't serve a purpose anymore.

Please sign in to comment.