Bug #60726: __identity field will not be rendered if of type @lazy - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #60726

closed

__identity field will not be rendered if of type @lazy

Added by Stefan Froemken over 9 years ago. Updated over 5 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Category:

Fluid

Target version:

next-patchlevel

Start date:

2014-08-01

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

6.2

PHP Version:

5.4

Tags:

Complexity:

easy

Is Regression:

Sprint Focus:

Description

Hello Core-Team,

I have a field like that one in my fluid-template:

<f:form.textfield class="powermail_input" placeholder="ss:mm" id="timeBegin" property="eventTime.timeBegin" />

To prevend it from validation I have added the @lazy annotation in domain model. In renderHiddenIdentityField method you have something like:

if (!is_object($object)
    || !($object instanceof \TYPO3\CMS\Extbase\DomainObject\AbstractDomainObject)
    || ($object->_isNew() && !$object->_isClone())) {
    return '';
}

But \TYPO3\CMS\Extbase\Persistence\Generic\LazyLoadingProxy isn't any type of them, so my required __identity field will not be rendered in template.

Stefan

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Gerrit Code Review over 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 http://review.typo3.org/31934

Actions

Copy link

Updated by Gerrit Code Review over 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 http://review.typo3.org/31934

Actions

Copy link

Updated by Gerrit Code Review over 9 years ago

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

Actions

Copy link

Updated by Gerrit Code Review over 9 years ago

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

Actions

Copy link

Updated by Christian Kuhn over 9 years ago

To prevend it from validation I have added the @lazy annotation in domain model.

Is this sentence correct? If so, @lazy is definately not meant to be used to circumvent validation.

Actions

Copy link

Updated by Stefan Froemken over 9 years ago

I know it is for loading domain models when needed. But in case of MM-Tables a @lazy annotation is the best solution to stop validation over nearly ALL mm-records (tested duration: 17 seconds).

Actions

Copy link

Updated by Gerrit Code Review over 9 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 http://review.typo3.org/32895

Actions

Copy link