Project

General

Profile

Actions

Bug #72654

closed

Date and Datetime cause error at page copy

Added by Michael Fritz almost 9 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Backend API
Target version:
-
Start date:
2016-01-12
Due date:
% Done:

100%

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

Description

When I copy a page, I got an error from the typo3_src-6.2.17/typo3/sysext/core/Classes/DataHandling/DataHandler.php telling me, getTimestamp causes an error.

This is somehow, because the $dateTime here has the value FALSE, but it should have at least the value 0000-00-00.

Problem Line ~1729: $value = $value === $emptyValue: $dateTime->getTimestamp();
Fix: $value = $value === $emptyValue || $dateTime === false ? 0 : $dateTime->getTimestamp();

This might be due to a customised TCA in my table PAGES using DATE as a database field.

"tx_mask_eventstart": {
"config": {
"type": "input",
"dbType": "date",
"size": "8",
"max": "20",
"eval": "date"
},
"key": "eventstart",
"l10n_mode": "exclude"
},
Actions #1

Updated by Michael Fritz almost 9 years ago

The issue is not big but when a page is being copied, there is a lot of trouble:

- a duplicate page with the same ID as the original appears in the page tree.
- the content in fal fields are beeing duplicated within the original page (which is very problematic)
- no roll back due to the error...

Actions #2

Updated by Gerrit Code Review almost 9 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/46426

Actions #3

Updated by Gerrit Code Review almost 9 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/46426

Actions #4

Updated by Nicole Cordes over 8 years ago

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

Updated by Gerrit Code Review about 8 years ago

  • Status changed from Resolved to Under Review

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

Actions #6

Updated by Gerrit Code Review about 8 years ago

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

Actions #7

Updated by Gerrit Code Review about 8 years ago

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

Actions #8

Updated by Gerrit Code Review about 8 years ago

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

Actions #9

Updated by Nicole Cordes almost 8 years ago

  • Status changed from Under Review to Resolved
Actions #10

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF