Bug #48423

Inspector: Allow to edit multiple images within a NodeType

Added by Berit Hlubek about 2 years ago. Updated about 2 years ago.

Status:Resolved Start date:2013-05-21
Priority:Should have Due date:
Assigned To:Christopher Hlubek % Done:

100%

Category:-
Target version:Base Distribution - 1.0 alpha 5

Description

At the moment it is not possible to add more than one image to a NodeType. A JavaScript error is thrown because the ID image-width is used twice.

To have the chance to create fully flexible NodeTypes it has to be possible to put multiple properties of the type image inside one NodeType.

Associated revisions

Revision 63081d12
Added by Christopher Hlubek about 2 years ago

[BUGFIX] Fix multiple image editing and file upload in inspector

This change updates the id generation to prevent duplicate element ids.
The file upload editor also had a bug through the usage of GUID_KEY as
an Ember property which no longer works.

Change-Id: Id6c0d8056d49b2002ecdf1f41b04a68f7265f039
Fixes: #48423
Reviewed-on: https://review.typo3.org/22305
Reviewed-by: Christopher Hlubek
Tested-by: Christopher Hlubek
Tested-by: Mr. Hudson
Reviewed-by: Sebastian Kurfuerst
Reviewed-by: Aske Ertmann
Tested-by: Aske Ertmann

History

#1 Updated by Aske Ertmann about 2 years ago

  • Target version set to 1.0 alpha 5

This seems to happen because the inspector tries to use the same elementId twice which Ember does not like

#2 Updated by Christopher Hlubek about 2 years ago

  • Status changed from New to Accepted

#3 Updated by Aske Ertmann about 2 years ago

  • Assigned To set to Christopher Hlubek

#4 Updated by Gerrit Code Review about 2 years ago

  • Status changed from Accepted to Under Review

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

#5 Updated by Gerrit Code Review about 2 years ago

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

#6 Updated by Christopher Hlubek about 2 years ago

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

Also available in: Atom PDF