Uploaded image for project: 'CMS'
  1. CMS
  2. CMS-12457

NPE should be better logged when migrating CMS-12289

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • 4.9.0
    • None
    • None
    • None

      If a searchservice is refering an unexisting content type, it lead to an NPE in

      Caused by: org.ametys.core.migration.MigrationException: Error while executing script to upgrade component 'org.ametys.plugins.web.upgrade' to version '20250402T0000'. Error : '', message : 'Cannot invoke "org.ametys.cms.contenttype.ContentType.getModelItems()" because "contentType" is null', stacktrace : 'Cannot invoke "org.ametys.cms.contenttype.ContentType.getModelItems()" because "contentType" is null
      at org.ametys.web.frontoffice.search.metamodel.impl.AbstractContentBasedSearchable#_getModelItemCriterionDefinitions:308
      at java.base/java.util.stream.ReferencePipeline$3$1#accept:197
      at java.base/java.util.HashMap$KeySpliterator#forEachRemaining:1715
      at java.base/java.util.stream.AbstractPipeline#copyInto:509
      at java.base/java.util.stream.AbstractPipeline#wrapAndCopyInto:499
      at java.base/java.util.stream.ReduceOps$ReduceOp#evaluateSequential:921
      at java.base/java.util.stream.AbstractPipeline#evaluate:234
      at java.base/java.util.stream.ReferencePipeline#collect:682
      at org.ametys.web.frontoffice.search.metamodel.impl.AbstractContentBasedSearchable#_getModelItemCriterionDefinitions:292
      at org.ametys.web.frontoffice.search.metamodel.impl.AbstractContentBasedSearchable#getCriteria:261
      at org.ametys.web.frontoffice.search.metamodel.SearchServiceCreationHelper#lambda$getCriterionDefinitions$0:256
      at java.base/java.util.stream.ReferencePipeline$3$1#accept:197
      at java.base/java.util.ArrayList$ArrayListSpliterator#forEachRemaining:1708
      at java.base/java.util.stream.AbstractPipeline#copyInto:509
      at java.base/java.util.stream.AbstractPipeline#wrapAndCopyInto:499
      at java.base/java.util.stream.ForEachOps$ForEachOp#evaluateSequential:151
      at java.base/java.util.stream.ForEachOps$ForEachOp$OfRef#evaluateSequential:174
      at java.base/java.util.stream.AbstractPipeline#evaluate:234
      at java.base/java.util.stream.ReferencePipeline#forEach:596
      at org.ametys.web.frontoffice.search.metamodel.SearchServiceCreationHelper#getCriterionDefinitions:259
      at org.ametys.web.frontoffice.search.instance.SearchServiceInstanceFactory#createSearchServiceInstance:117
      at <js> :anonymous(generated script:53:2283-2356)
      at com.oracle.truffle.polyglot.PolyglotFunctionProxyHandler#invoke:151
      at jdk.proxy1/jdk.proxy1.$Proxy61#accept (Unknown Source)
      at java.base/java.lang.Iterable#forEach:75
      at <js> __migrateSearchServicesCriteria(generated script:52-83:2185-3785)
      at <js> main(generated script:35:1619-1669)
      at <js> :program(generated script:2251:77640-77645)
      at org.graalvm.polyglot.Context#eval:402
      at org.ametys.plugins.core.ui.script.ScriptHandler#_executeScript:239
      at org.ametys.plugins.core.ui.script.ScriptHandler#executeScript:173
      at org.ametys.plugins.core.ui.script.ScriptHandler#executeScript:151
      at org.ametys.core.migration.action.ScriptAction#act:77
      at org.ametys.core.migration.MigrationEngine#doAction:451
      

          [CMS-12457] NPE should be better logged when migrating CMS-12289

          There are no comments yet on this issue.

            raphael Raphaël Franchet
            raphael Raphaël Franchet
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: