Allow configuration of API limit instead of hardcoding at 100

[sanak]

https://www.redmine.org/issues/16069

以前、弊社のRest APIクライアント開発サイドで、Redmine側で対応されていれば...という話が出てましたので、こちらも登録いたします。

[fkdkent]

https://github.com/redmine/redmine/blob/52cc3fdc24dbb3814b6924323c032051ddac941e/app/controllers/application_controller.rb#L660-L661

ここで上限を100件に制限してそう。

[ishikawa999]

制限はIssueで提案されているとおりAdministrationの設定で変えるようにするのが良さそう

[fkdkent]

https://www.redmine.org/issues/16069#note-5

100件未満にすることを許さないほうがいいという話

でも設定画面で入力値をバリデーションするのは実装大変そう。

[ishikawa999]

でも設定画面で入力値をバリデーションするのは実装大変そう。

既存の仕組みの上でバリデーションするなら、Email Addressのバリデーションと同じようにvalidate_all_from_paramsの中で実行する形になりそうです。
(今後このメソッドが膨らんでいくと混ぜるの嫌がられそうですが...)
https://github.com/redmine/redmine/blob/master/app/models/setting.rb#L186

[ishikawa999]

でも設定画面で入力値をバリデーションするのは実装大変そう。

既存の仕組みの上でバリデーションするなら、Email Addressのバリデーションと同じようにvalidate_all_from_paramsの中で実行する形になりそうです。
(今後このメソッドが膨らんでいくと混ぜるの嫌がられそうですが...)
https://github.com/redmine/redmine/blob/master/app/models/setting.rb#L186

https://www.redmine.org/issues/16069#note-35
とりあえずその手法でのパッチを添付しました。