Uploaded image for project: 'Hippo CMS'
  1. Hippo CMS
  2. CMS-13274

Improve ComparingController logging and error handling

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed
    • Normal
    • Resolution: Fixed
    • 14.0.1
    • 14.3.0, 100.4.0
    • None

    Description

      If you have data inconsistency, for example you change a compound primary type in your template from myproject:video into myproject:image, and you leave existing data as is, NPE error is thrown and document cannot be rendered at all.
      Main problem is that it is very hard to find out what / which field is being compared (so, what is causing the error).
      Another problem is that end user ends up with a blank page and refreshing it throws 500 server error.
      Not very user and developer friendly.
      Logging extra information about which fields are being compared would already be a big improvement.

      Caused by: java.lang.NullPointerException
      at org.hippoecm.frontend.editor.plugins.field.ComparingController.start(ComparingController.java:270) ~[hippo-cms-editor-frontend-5.6.9.jar:5.6.9]
      at org.hippoecm.frontend.editor.plugins.field.AbstractFieldPlugin.onBeforeRender(AbstractFieldPlugin.java:333) ~[hippo-cms-editor-frontend-5.6.9.jar:5.6.9]
      at org.hippoecm.frontend.editor.plugins.field.NodeFieldPlugin.onBeforeRender(NodeFieldPlugin.java:130) ~[hippo-cms-editor-frontend-5.6.9.jar:5.6.9]
      at org.apache.wicket.Component.internalBeforeRender(Component.java:949) ~[wicket-core-6.30.0.jar:6.30.0]
      at org.apache.wicket.Component.beforeRender(Component.java:1017) ~[wicket-core-6.30.0.jar:6.30.0]
      at org.apache.wicket.MarkupContainer.onBeforeRenderChildr
      
      

      Attachments

        Activity

          People

            clientserviceteam Client Service Team
            mmilicevic Marijan Milicevic
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: