Project

General

Profile

Actions

Bug #16682

closed

pid reset to 10000 in Constant Editor

Added by Fredrik no-lastname-given over 17 years ago. Updated about 16 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
-
Start date:
2006-11-01
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

The Constant Editor is limiting pid to max 10000, if entered manually in the constants textfield it will reset them to 10000.

regards
Fredrik Boström

typo3 3.8.0
sr_feuser_register 2.2.6

seems to affect:

plugin.tx_srfeuserregister_pi1.loginPID
plugin.tx_srfeuserregister_pi1.editPID
plugin.tx_srfeuserregister_pi1.registerPID
plugin.tx_srfeuserregister_pi1.confirmPID
plugin.tx_srfeuserregister_pi1.confirmInvitationPID
plugin.tx_srfeuserregister_pi1.pid

(issue imported from #M4455)


Files

t3lib_tsparser_ext.diff (375 Bytes) t3lib_tsparser_ext.diff Administrator Admin, 2006-11-11 08:14
4455.diff (534 Bytes) 4455.diff Administrator Admin, 2008-05-24 21:45
Actions #1

Updated by Karsten Dambekalns over 17 years ago

Changing the limit from 10000 to 100000 is not fixing the problem. This is about making sure it's a positive integer. Either just cast to positive int using
$var=(int)abs($var);
or limit to the maximum range an integer can hold in 32 bits with
$var=t3lib_div::intInRange($var,0,2147483647);
This should be sufficient, although on 64 bit systems the limit is higher.

And yes, I have seen production websites where various uids where above the 100000...

Actions

Also available in: Atom PDF