Project

General

Profile

Actions

Bug #84588

closed

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

Added by Manuel Selbach almost 6 years ago. Updated over 2 years ago.

Status:
Closed
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 3 (0 open3 closed)

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
Actions #1

Updated by Manuel Selbach almost 6 years ago

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

Updated by Gerrit Code Review almost 6 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

Actions #3

Updated by Gerrit Code Review almost 6 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

Actions #4

Updated by Gerrit Code Review almost 6 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

Actions #5

Updated by Gerrit Code Review almost 6 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

Actions #6

Updated by Gerrit Code Review almost 6 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

Actions #7

Updated by Gerrit Code Review almost 6 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

Actions #8

Updated by Gerrit Code Review almost 6 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

Actions #9

Updated by Gerrit Code Review almost 6 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

Actions #10

Updated by Gerrit Code Review almost 6 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

Actions #11

Updated by Gerrit Code Review almost 6 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

Actions #12

Updated by Gerrit Code Review almost 6 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

Actions #13

Updated by Gerrit Code Review almost 6 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

Actions #14

Updated by Gerrit Code Review almost 6 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

Actions #15

Updated by Gerrit Code Review over 5 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

Actions #16

Updated by Gerrit Code Review over 5 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

Actions #17

Updated by Gerrit Code Review almost 5 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

Actions #18

Updated by Gerrit Code Review over 4 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

Actions #19

Updated by Gerrit Code Review over 4 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

Actions #20

Updated by Gerrit Code Review over 4 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

Actions #21

Updated by Gerrit Code Review over 4 years 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

Actions #22

Updated by Sascha Egerer over 4 years ago

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

Updated by Andreas Kienast over 4 years ago

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

Updated by Daniel Damm over 4 years ago

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

Updated by Benni Mack about 4 years ago

  • Status changed from Resolved to Closed
Actions #26

Updated by Gerrit Code Review almost 3 years 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

Actions #27

Updated by Tymoteusz Motylewski over 2 years ago

  • Status changed from Under Review to Closed
Actions

Also available in: Atom PDF