Bug #60726

__identity field will not be rendered if of type @lazy

Added by Stefan Froemken almost 7 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Fluid
Target version:
Start date:
2014-08-01
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
5.4
Tags:
Complexity:
easy
Is Regression:
No
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

Has duplicate TYPO3 Core - Bug #60913: Cannot edit lazy loaded objectsClosed2014-08-12

Actions
#1

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

#2

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

#3

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

#4

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

#5

Updated by Christian Kuhn over 6 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.

#6

Updated by Stefan Froemken over 6 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).

#7

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

#8

Updated by Stefan Froemken over 6 years ago

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

Updated by Benni Mack over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF