View Issue Details

IDProjectCategoryView StatusLast Update
0010754mantisbtbugtrackerpublic2010-02-22 14:34
Reportermados Assigned Todhx  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version1.2.0rc1 
Target Version1.2.0Fixed in Version1.2.0 
Summary0010754: One-digit redirect delay can be too short
Description

Please reopen 0003167.

The input field is still limited in account_prefs_inc.php.

In 1.1.8:
<input type="text" name="redirect_delay" size="1" maxlength="1" ...

In 1.2.0rc1:
<input type="text" name="redirect_delay" size="4" maxlength="1" ...

The "maxlength" attribute means it is not possible to enter more than a single digit (0 to 9 seconds). Please change all "1" to "4".

TagsNo tags attached.

Activities

dhx

dhx

2009-10-28 21:09

reporter   ~0023460

Fixed. I did however limit it to 3 characters instead of 4... I couldn't understand why anyone would want a redirect delay longer than 40 minutes.

Thanks for reporting!

mados

mados

2009-10-29 06:13

reporter   ~0023470

Best current practice is to set size one digit larger then maxlength to get more space for the text cursor.

In this case: size="4" maxlength="3"

Related Changesets

MantisBT: master-1.2.x 00a4d6cb

2009-10-28 21:04

dhx


Details Diff
Fix 0010754: Allow user to enter redirect delays above 9 seconds

Users were prevented from entering redirect delays larger than 9 because
the field which accepted the redirect delay was limited to a single
character of input.

I've changed the allowable number of characters in the input field to 3
to allow for redirect delays above 2 minutes. Users shouldn't have a
need to set the redirect delay larger than 40 minutes (the maximum this
field can take with just 3 characters allowed in the field). However
it's quite reasonable for a user to expect a redirect delay of 2 or 5
minutes.
Affected Issues
0010754
mod - account_prefs_inc.php Diff File

MantisBT: master c026051e

2009-10-28 21:04

dhx


Details Diff
Fix 0010754: Allow user to enter redirect delays above 9 seconds

Users were prevented from entering redirect delays larger than 9 because
the field which accepted the redirect delay was limited to a single
character of input.

I've changed the allowable number of characters in the input field to 3
to allow for redirect delays above 2 minutes. Users shouldn't have a
need to set the redirect delay larger than 40 minutes (the maximum this
field can take with just 3 characters allowed in the field). However
it's quite reasonable for a user to expect a redirect delay of 2 or 5
minutes.
Affected Issues
0010754
mod - account_prefs_inc.php Diff File