Bug #36959

Image title should be settable via form field

Added by Nikolas Hagelstein about 3 years ago. Updated almost 3 years ago.

Status:Resolved Start date:2012-05-08
Priority:Must have Due date:
Assigned To:Adrian Föder % Done:

0%

Category:-
Target version:-

Description

At them moment it is not possible to set the image title via form field,
since the typeConverter handles nothing but the ressource property (not sure if that's the reason, just guessing)

Associated revisions

Revision 27d2c527
Added by Adrian Föder almost 3 years ago

[BUGFIX] Fix Image TypeConverter

The Image TypeConverter now takes care about a given `title` property
and takes it into account for the mapping; additionally the handling of
mapping persisted Images (i.e. sources with an ``__identity`` property)
works as it used to be for persisted entities.

This is covered by some additional Functional Tests, which have,
besides, been augmented a bit and refactored to an abstract functional
test case for reusable methods.

Change-Id: I83e0953278b0dac42b151b2859606206d7f1cf8f
Fixes: #36959
Fixes: #37230

History

#1 Updated by Adrian Föder about 3 years ago

  • Status changed from New to Needs Feedback

Nicolas, I seem to remember we talked about this, maybe in IRC; is this solvable with on-board-methods; means: can this be closed?

#2 Updated by Adrian Föder almost 3 years ago

  • Status changed from Needs Feedback to Accepted
  • Assigned To set to Adrian Föder
  • Priority changed from Should have to Must have

#3 Updated by Gerrit Code Review almost 3 years ago

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/15134

#4 Updated by Gerrit Code Review almost 3 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/15134

#5 Updated by Gerrit Code Review almost 3 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/15134

#6 Updated by Gerrit Code Review almost 3 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/15134

#7 Updated by Adrian Föder almost 3 years ago

  • Status changed from Accepted to Resolved

Also available in: Atom PDF