Uploaded image for project: '[Read Only] - Hippo Site Toolkit 2'
  1. [Read Only] - Hippo Site Toolkit 2
  2. HSTTWO-1724

Content Bean annotated classes should be able to be configured in both external configuration and local classes in a component without warning

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed
    • Normal
    • Resolution: Fixed
    • None
    • 2.22.04, 2.23.02
    • None
    • None

    Description

      When it register local annotated classes which is retrieved by invoking overriden #getLocalAnnotatedClasses(), BaseHstComponent leaves a warning log if there already exists that annotated class as follows:

      08.08.2011 13:34:22 WARN [org.hippoecm.hst.component.support.bean.BaseHstComponent.getObjectConverter():449] local added class 'org.onehippo.forge.tcmp.beans.AbstractTagBean' already present. Skipping

      However, I found an example where the annotated beans should be configured in both web.xml and local annotated classes in a component.
      In the former case, for example, a news content bean can refer a tag child bean. In this case, the news component is basically unaware of the local annotated classes.
      So, in this case, the annotated beans should be registered through web.xml as it does for news beans.

      Of course, the dedicated tag component provided by tcmp forge project is still able to register its own local annotated beans without configuring web.xml.

      Therefore, I think the warning log is not necessary; should be a debug log.

      Attachments

        Activity

          People

            wko Woonsan Ko (Inactive)
            wko Woonsan Ko (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: