Project

General

Profile

Actions

Bug #74294

closed

TCA: range upper don't work with all numbers

Added by Gernot Ploiner about 8 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2016-03-05
Due date:
% Done:

0%

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

Description

Mysterious...
If you use the following TCA, you can only enter 1 as valid number in your backend form. Otherwise the Javascript validation fails:

'config' => array(
    'type' => 'input',
    'size' => 4,
    'eval' => 'int',
    'range'    => array(
        'upper' => '2000',
        'lower' => '0'
    ),
)

If you use (for example) 86400 instead of 2000, all works fine.
Other "bad" values for upper: 1000, 1001
I couldn't figure out a system which numbers are "good" or "bad" ;)

For easier reconstruction, i attached a little extension to reproduce. Just install it and create a record in your list module.

I use TYPO3 7.6.4


Files

mytcatest_0.0.0_201603050823.zip (9.82 KB) mytcatest_0.0.0_201603050823.zip Gernot Ploiner, 2016-03-05 08:26
Actions #1

Updated by Gernot Ploiner about 8 years ago

This works at all (without: '):

'config' => array(
    'type' => 'input',
    'size' => 4,
    'eval' => 'int',
    'range'    => array(
        'upper' => 2000,
        'lower' => 0
    ),
)

My TCA is from an old extension, initially generated with kickstarter. So I'm not sure if this is a bug or wrong code-format...

Actions #2

Updated by Wouter Wolters almost 8 years ago

  • Status changed from New to Needs Feedback

I could not reproduce this in master. This could have been a bug that is solved now in master.
https://review.typo3.org/#/c/48336/ this is a WIP to fix some stuff in validation part.

Can you try out master and see if it works there for you as expected?
For 7.6 we need to wait if the current work improves the situation in this area.

Actions #3

Updated by Riccardo De Contardi over 7 years ago

  • Status changed from Needs Feedback to Closed

90+ days without feedback > closing this issue.
If you think that this is the wrong decision or experience the issue again, please open a new issue with a reference to this one .
Thank you

Actions #4

Updated by Gernot Ploiner over 7 years ago

Sorry for the delay.
Yes, it ist resolved.
Tested in TYO3 7.6.14

Actions

Also available in: Atom PDF