[ 
http://jira.magnolia.info/browse/MAGNOLIA-2361?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Gregory Joseph updated MAGNOLIA-2361:
-------------------------------------

          Summary: Content2bean: get a more explicit message when class 
property exists but is empty  (was: Content2bean: fallback to default 
implementation when class property is empty or wrongly set ?)
    Fix Version/s: 3.6.2
                       (was: 3.6.x)
      Description: 
original title: fallback to default implementation when class property is empty 
or wrongly set ?

To be discussed: as per MAGNOLIA-1688, it might be useful to fallback to the 
default implementation class when a {{class}} property is set to an unexisting 
class or is empty. I am not entirely convinced, because this is typically a 
configuration mistake and it should not be ignored. Maybe a precise 
warning/error log before falling back would be good. (depending on what 
component is being configured, this could otherwise block the system - i.e for 
filters for instance)

  was:To be discussed: as per MAGNOLIA-1688, it might be useful to fallback to 
the default implementation class when a {{class}} property is set to an 
unexisting class or is empty. I am not entirely convinced, because this is 
typically a configuration mistake and it should not be ignored. Maybe a precise 
warning/error log before falling back would be good. (depending on what 
component is being configured, this could otherwise block the system - i.e for 
filters for instance)

         Assignee: Gregory Joseph  (was: Philipp Bracher)

My main concern was actually for when the class property exists but does not 
have any value; I just realized that in fact the error log was still pretty 
clear in this case:
{noformat}
WARN  info.magnolia.content2bean.impl.Content2BeanProcessorImpl 
Content2BeanProcessorImpl.java(toBean:88) 12.09.2008 14:20:27  can't resolve 
class for node /modules/adminInterface/virtualURIMapping/default
java.lang.ClassNotFoundException:
at java.lang.Class.forName0(Native Method)
[...]
{noformat}
However I've just added a little check that will behave exactly the same way if 
the class property is empty, but with a more explicit message in the 
ClassNotFoundException than an empty string.

> Content2bean: get a more explicit message when class property exists but is 
> empty
> ---------------------------------------------------------------------------------
>
>                 Key: MAGNOLIA-2361
>                 URL: http://jira.magnolia.info/browse/MAGNOLIA-2361
>             Project: Magnolia
>          Issue Type: Bug
>          Components: core
>            Reporter: Gregory Joseph
>            Assignee: Gregory Joseph
>             Fix For: 3.6.2
>
>
> original title: fallback to default implementation when class property is 
> empty or wrongly set ?
> To be discussed: as per MAGNOLIA-1688, it might be useful to fallback to the 
> default implementation class when a {{class}} property is set to an 
> unexisting class or is empty. I am not entirely convinced, because this is 
> typically a configuration mistake and it should not be ignored. Maybe a 
> precise warning/error log before falling back would be good. (depending on 
> what component is being configured, this could otherwise block the system - 
> i.e for filters for instance)

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.magnolia.info/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

----------------------------------------------------------------
for list details see
http://documentation.magnolia.info/
----------------------------------------------------------------

Reply via email to