kiridaruma / Onset

TRPG向けの簡単なチャット
MIT License
13 stars 1 forks source link

write.phpにおいて文字数制限が機能しない #7

Closed akarregi closed 8 years ago

akarregi commented 8 years ago

この問題はプルリクエスト https://github.com/kiridaruma/Onset/pull/5 で解決されています。 ですが、一応イシューを建てておきます。

write.php : 20 - 27

if($config['maxNick'] <= $nick){
    echo Onset::errorJson("名前が長すぎます(".mb_strlen($nick).")");
    die();
}
if($config['maxText'] <= $text){
    echo Onset::errorJson("テキストが長すぎます(".mb_strlen($text).")");
    die();
}

$nick, $textがそのまま文字列として評価されているので、このコードは正常に動作しません。 以下のように修正すると動作します。

// mb_strlen()を追加
if($config['maxNick'] <= mb_strlen($nick)){
    echo Onset::errorJson("名前が長すぎます(".mb_strlen($nick).")");
    die();
}
// mb_strlen()を追加
if($config['maxText'] <= mb_strlen($text)){
    echo Onset::errorJson("テキストが長すぎます(".mb_strlen($text).")");
    die();
}
kiridaruma commented 8 years ago

修正確認しました、ありがとうございます