[MGNLUI-819] Creating a new "Promo" component raises exception Created: 06/Mar/13  Updated: 09/Jul/13  Resolved: 09/Jul/13

Status: Closed
Project: Magnolia UI
Component/s: None
Affects Version/s: 5.0
Fix Version/s: 5.0

Type: Bug Priority: Major
Reporter: Eric Hechinger Assignee: Eric Hechinger
Resolution: Fixed Votes: 0
Labels: alpha3
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
relation
is related to MGNLUI-829 Field with property extends=override ... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:

 Description   

When trying to create an New Promos Component this generate the following exception:
WARN info.magnolia.ui.model.builder.FactoryBase : No matching implementation class was found for definition class [info.magnolia.ui.model.field.definition.ConfiguredFieldDefinition]. Please check your configuration.
Mar 5, 2013 8:38:55 PM com.vaadin.server.DefaultErrorHandler doDefault
SEVERE:
com.vaadin.server.ServerRpcManager$RpcInvocationException: Unable to invoke method fireAction in info.magnolia.ui.vaadin.gwt.client.dialog.rpc.ActionFiringServerRpc
at com.vaadin.server.ServerRpcManager.applyInvocation(ServerRpcManager.java:172)
at com.vaadin.server.ServerRpcManager.applyInvocation(ServerRpcManager.java:118)



 Comments   
Comment by Eric Hechinger [ 06/Mar/13 ]

Seems to be related to bad Dialog definition from STK.

In STK: dialogs50/components/promos/stkPromo/formDefinition/tabs/fields/hideTeaserImage
This field definition has a property extends = override
--> this means that we override the hideTeaserImage properties defined
in '/modules/standard-templating-kit/dialogs50/generic/master/baseTeaserInternal'
by the properties defined in hideTeaserImage.

hideTeaserImage has no properties.... so at the end we have a hideTeaserImage without properties (no field class definition...)

Node2BeanTransformerImpl will create a Node for this hideTeaserImage and define default properties (as none are defiend) and set
as class properties the ConfigureFieldDefinition
based on

<type-mapping>
<type>info.magnolia.ui.model.field.definition.FieldDefinition</type>
<implementation>info.magnolia.ui.model.field.definition.ConfiguredFieldDefinition</implementation>
</type-mapping>

defining an extends=override propertie on a field that do not define other properties is wrong.

Comment by Jozef Chocholacek [ 06/Mar/13 ]

Happens also in the Extras column.

Comment by Jozef Chocholacek [ 07/Mar/13 ]

Fixed, but it's rather a quick hack for Alpha3, so the proper solution should be created by resolving MGNLUI-829.

Generated at Mon Feb 12 08:40:36 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.