yktsr / Text2Frame-MV

テキストファイル(.txtファイルなど)から「文章の表示」イベントコマンドに簡単に変換するための、RPGツクールMV・MZ用の開発支援プラグインです。/ Plugin for RPG Maker MV/MZ to convert text to event command
MIT License
9 stars 1 forks source link

画像の表示関係のパラメータに 0 を指定した場合に反映されない不具合の修正 #114

Closed yktsr closed 9 months ago

yktsr commented 9 months ago

fix #113

問題の概要

Number(0) || 100

が、100になってしまう。

> 1 || 255
< 1
> 0 || 255
< 255

ボツ案

  1. isNaNを使う案 色々検討したが、下記の場合、

    !isNaN(values[0]) ? Number(values[0]) : 100;

    空文字のときに、isNaN('') がfalseになってしまい、0になるため、空文字のときは特別視する。

  2. parseIntを使う案

    Number.isFinite(parseInt(values[0])) ? parseInt(values[0]) : 100;

    では、 parseInt('255f') が 255 になるため、都合が悪かった。

HidetoshiKawaguchi commented 9 months ago

@yktsr 修正ありがとうございます。Blendだけではなく、ScaleやDurationのテストケースも追加した上でチェックしました。値が0というのはちょっと警戒したほうが良さそうですね。