Project

General

Profile

Actions
Copy link

Bug #91553

closed

Risk of non-unique field in DatePickerViewHelper

Added by Tizian Schmidlin over 4 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Form Framework
Target version:
11.0
Start date:
2020-06-02
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
11
PHP Version:
Tags:
Complexity:
no-brainer
Is Regression:
Sprint Focus:

Description

Hello there,

I've discovered an issue with the DatePickerViewHelper.

The method to calculate the field id when rendering the datepicker uses this method to do so:
$id = 'field' . md5(uniqid());

This has some downsides since PHP on Windows does not use microseconds to create the uniqid by default. Therefore the second parameter to uniqid has to be set to ensure its uniqueness.

This bug can be traced back to very old versions of TYPO3.

Best Regards
Tizian

Related issues 3 (0 open — 3 closed)

Related to TYPO3 Core - Task #69730: Remove dot in uniqid('NEW', TRUE)ClosedNicole Cordes2015-09-12

Actions
Related to TYPO3 Core - Bug #58602: Datepicker issue on multiple datetime fields in BEClosed2014-05-07

Actions
Related to TYPO3 Core - Bug #59701: uniqid() not returning unique valuesClosedTymoteusz Motylewski2014-06-20

Actions
Actions
Copy link
 #1

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/64652

Actions
Copy link
 #2

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/64652

Actions
Copy link
 #3

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/64652

Actions
Copy link
 #4

Updated by Stephan Großberndt over 4 years ago

  • Related to Task #69730: Remove dot in uniqid('NEW', TRUE) added
Actions
Copy link
 #5

Updated by Stephan Großberndt over 4 years ago

  • Related to Bug #58602: Datepicker issue on multiple datetime fields in BE added
Actions
Copy link
 #6

Updated by Stephan Großberndt over 4 years ago

  • Related to Bug #59701: uniqid() not returning unique values added
Actions
Copy link
 #7

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/64652

Actions
Copy link
 #8

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/64652

Actions
Copy link
 #9

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/64652

Actions
Copy link
 #10

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/64652

Actions
Copy link
 #11

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/64652

Actions
Copy link
 #12

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/64652

Actions
Copy link
 #13

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/64652

Actions
Copy link
 #14

Updated by Gerrit Code Review over 4 years ago

Patch set 11 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/+/64652

Actions
Copy link
 #15

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/64652

Actions
Copy link
 #16

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/64652

Actions
Copy link
 #17

Updated by Gerrit Code Review over 4 years ago

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

Actions
Copy link
 #18

Updated by Tizian Schmidlin over 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions
Copy link
 #19

Updated by Benni Mack about 4 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF