#72 Changing charset is not working

0.7
closed
Interface (13)
5
2005-03-07
2004-07-15
Softgear
No

I edited modules/CVSMonitor.pm file like
$CHARSET = 'euc-kr';

But, Korean letters were broken. Encoding is
still "Western European (ISO)" in MS-IE.
HTML document has following line.
<meta http-equiv="Content-Type" content="text/html;
charset=euc-kr">

Whenever click "refresh" in IE, encoding was changed to
Western European (ISO).

Discussion

  • Adam Kennedy

    Adam Kennedy - 2004-07-26
    • milestone: 383471 --> 0.7
    • assigned_to: nobody --> adamkennedy
     
  • Adam Kennedy

    Adam Kennedy - 2004-07-26

    Logged In: YES
    user_id=153576

    I'd recommend you check the page headers as well. If the
    application is working correctly, it SHOULD also return the
    correct character set in the HTML headers, as well as the meta.

    Short of this, I'm not sure what I'd have to do to convince
    IE specifically to see it as the different charset...
    suggestions? This might be a bug in IE, although I'm not
    definite on that.

    Does it work correctly for other browsers, such as Firefox
    or Opera?

     
  • OGAWA KenIchi

    OGAWA KenIchi - 2005-03-07

    Logged In: YES
    user_id=1232616

    Although I also encountered this problem, it solved by
    correcting modules/AppLib/Page.pm .
    My patch is shown below. Hope this helps.

    --- Page.pm.org Mon Mar 7 21:54:28 2005
    +++ Page.pm Fri Mar 4 22:03:30 2005
    @@ -161,6 +161,8 @@
    push @params, '-cookie', $cookies;
    }

    + push @params, '-charset', $CVSMonitor::CHARSET;
    +
    # Method form throws warnings sometimes, always use
    function form
    CGI::header( @params );
    }

     
  • Adam Kennedy

    Adam Kennedy - 2005-03-07

    Logged In: YES
    user_id=153576

    Great! Thanks for the fix.

    Added to CVS and will resolve.

     
  • Adam Kennedy

    Adam Kennedy - 2005-03-07
    • status: open --> closed
     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

JavaScript is required for this form.





No, thanks