Bug #84588

Int values for field will be converted to unix timestamp in module "DB check" -> Full search

Added by Manuel Selbach about 3 years ago. Updated 23 days ago.

Status:
Under Review
Priority:
Should have
Category:
-
Target version:
-
Start date:
2018-04-03
Due date:
% Done:

100%

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

Description

With the change in https://forge.typo3.org/issues/83675 every value will be converted into a unix timestamp even if you specify e.g. a UID or PID.

Example:

PID: 5134

This will bring back 99854182152 and the if condition is true in the introduced code:

if  (strtotime($conf['inputValue'])) {
    $conf['inputValue'] = strtotime($conf['inputValue']);
    if ($conf['inputValue1'] && strtotime($conf['inputValue1'])) {
        $conf['inputValue1'] = strtotime($conf['inputValue1']);
    }
}

Related issues

Related to TYPO3 Core - Bug #83675: DB Check : Full search : Advanced Query calendar problemClosed2018-01-24

Actions
Related to TYPO3 Core - Bug #89154: QueryGenerator::cleanInputVal() - do not call strtotime on timestampClosed2019-09-12

Actions
Related to TYPO3 Core - Bug #87710: DB check module: Building a database query where sys_language_uid=-1 not possibleClosed2019-02-13

Actions
#1

Updated by Manuel Selbach about 3 years ago

  • Related to Bug #83675: DB Check : Full search : Advanced Query calendar problem added
#2

Updated by Gerrit Code Review about 3 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56530

#3

Updated by Gerrit Code Review about 3 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56530

#4

Updated by Gerrit Code Review about 3 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56530

#5

Updated by Gerrit Code Review about 3 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56530

#6

Updated by Gerrit Code Review about 3 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56530

#7

Updated by Gerrit Code Review about 3 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56530

#8

Updated by Gerrit Code Review about 3 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56530

#9

Updated by Gerrit Code Review about 3 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56530

#10

Updated by Gerrit Code Review about 3 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56530

#11

Updated by Gerrit Code Review about 3 years ago

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56530

#12

Updated by Gerrit Code Review almost 3 years ago

Patch set 12 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56530

#13

Updated by Gerrit Code Review almost 3 years ago

Patch set 13 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56530

#14

Updated by Gerrit Code Review almost 3 years ago

Patch set 14 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56530

#15

Updated by Gerrit Code Review almost 3 years ago

Patch set 15 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56530

#16

Updated by Gerrit Code Review over 2 years ago

Patch set 16 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56530

#17

Updated by Gerrit Code Review about 2 years ago

Patch set 17 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/56530

#18

Updated by Gerrit Code Review almost 2 years ago

Patch set 18 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/56530

#19

Updated by Gerrit Code Review almost 2 years ago

Patch set 19 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/56530

#20

Updated by Gerrit Code Review almost 2 years ago

Patch set 20 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/56530

#21

Updated by Gerrit Code Review over 1 year ago

Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61609

#22

Updated by Sascha Egerer over 1 year ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
#23

Updated by Andreas Fernandez over 1 year ago

  • Related to Bug #89154: QueryGenerator::cleanInputVal() - do not call strtotime on timestamp added
#24

Updated by Daniel Damm over 1 year ago

  • Related to Bug #87710: DB check module: Building a database query where sys_language_uid=-1 not possible added
#25

Updated by Benni Mack over 1 year ago

  • Status changed from Resolved to Closed
#26

Updated by Gerrit Code Review 23 days ago

  • Status changed from Closed to Under Review

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/68770

Also available in: Atom PDF