tontof/kriss_feed

$kfcmaxupdate forced to maxlength="3" in config panel

Closed this issue · 1 comments

If I understand correctly, $kfcmaxupdate is the maximum amount of minutes KF can delay an update of a feed.
However in the config panel, it's lengh is restricted (and type is text):
<input type="text" maxlength="3" id="maxUpdate" name="maxUpdate" value="<?php echo $kfcmaxupdate;?>">
which means the average user can only set a maximum number of 999 minutes (~16 hours). Or.. K7£ minutes...

In my use case, some feeds are updated once every... 3-4 months ? And near-realtime updates are not useful to me, so I edited config.php to manually set a maxUpdate of 15 days:
$this->maxUpdate = '21600';

I suggest the following change in the HTML code of the config panel, allowing the user to set arbitrary delay minutes :)
<input type="number" id="maxUpdate" name="maxUpdate" min="1" value="<?php echo $kfcmaxupdate;?>">

I guess you are right :-)
1ac380b
I think, it's ok now