ウェブリンク編集画面のリンク先入力欄について

[shobu]

こんにちは。

これは以前からのようですが、ウェブリンク編集画面での「ウェブリンク」入力欄は<input type="text" ですが、maxlengthが255となっています。255字越えるようなケースは滅多に無いでしょうから、私も今まで気にしたことが無かったのですが、RFC的にもURLの長さは無制限で扱えるべきとなっているようですし、現実的にも255字以上のURL（パラメータが付いたり、最近はページタイトルそのままのURLも多いので）が存在したりします（ついさっき、そういうケースがあったのですがｗ）。
保存先のcontentに255字の制限があるわけではないので、maxlengthさえどうにかしてもらえれば、この問題が解消します。

manager/actions/mutate_content.dynamic.php の 500～600行目あたりにinput_text()があってこれで構築されているみたいです。オプションが取れるようなので

コード: 全て選択

<?php echo input_text('ta',$content['content']);?>
<?php echo tooltip($_lang['resource_weblink_help']);?>

これを

コード: 全て選択

<?php echo input_text('ta',$content['content'],'','9999');?>
<?php echo tooltip($_lang['resource_weblink_help']);?>

とでもすると maxlength="9999" という感じで緩和できます。
input_textの第4パラメータを''やNULLにしたらmaxlength付けないような処理ならば、それがベストと思いますが、とりあえず上記の形で回避出来るので採用して頂けないでしょうか。

[yama]

提案ありがとうございます、そのほうがよさそうですね。

http://forum.modx.jp/viewtopic.php?p=6687#p6687
もしよければ上記手順でプルリクエストをいただけたら受理させていただきます。
（もしプルリクエストがなくてもこちらで修正して次リリースに積み込みます）

[shobu]

http://forum.modx.jp/viewtopic.php?p=6687#p6687
もしよければ上記手順でプルリクエストをいただけたら受理させていただきます。
（もしプルリクエストがなくてもこちらで修正して次リリースに積み込みます）

input_text の処理はそのままですが、とりあえず文字数9999として書き換えしておきました。
よろしくお願いいたします。

[yama]

確認してみましたが、プルリクエストが届いてませんでした。もしかするとGitHubの仕様が最近になって変わったのかもしれません。よく変わるので・・
もしよければ、すでにプロジェクトをフォークされているようなので、そこで適当にブランチを作ってプルリクエストをお送りいただければと思います。
（数文字書き換えるだけのようなので、分かりにくければこちらで修正します）

[shobu]

EDITの時点で自動的にforkまではされているみたいですが、その先、Pull Requestsはソース右上のアイコンから自分で行なう必要があるみたいです。
今、やり直してみました。
よろしくお願いいたします。

[yama]

https://github.com/modxcms-jp/evolution ... ec51730bfc
ありがとうございます、さっそくマージさせていただきました。
他にも何か思いついたことがあれば、いつでもPRをお送りいただければと思います。