Uploaded image for project: 'Google Sitemap (closed)'
  1. Google Sitemap (closed)
  2. MGNLGS-94

Displaying a sitemap that is markedAsDeleted will result in NPE

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Neutral Neutral
    • 2.0.5
    • 2.0.4
    • None

      To reproduce:

      1. Create sitemap entry
      2. Delete it
      3. Display the table
      4. Logs should show an NullPointerException

      From jenkins console:

      [INFO] [talledLocalContainer] 2014-07-20 20:11:45,323 WARN  info.magnolia.event.SimpleEventBus                : Exception caught when dispatching a class info.magnolia.ui.api.event.ContentChangedEvent event with class info.magnolia.ui.contentapp.browser.BrowserPresenter$1 eventHandler.
      [INFO] [talledLocalContainer] com.vaadin.ui.Table$CacheUpdateException: Error during Table cache update.
      [INFO] [talledLocalContainer] 	at com.vaadin.ui.Table.maybeThrowCacheUpdateExceptions(Table.java:1716)
      [INFO] [talledLocalContainer] 	at com.vaadin.ui.Table.refreshRenderedCells(Table.java:1705)
      [INFO] [talledLocalContainer] 	at com.vaadin.ui.Table.refreshRowCache(Table.java:2639)
      [INFO] [talledLocalContainer] 	at com.vaadin.ui.Table.containerItemSetChange(Table.java:4445)
      [INFO] [talledLocalContainer] 	at com.vaadin.ui.TreeTable.containerItemSetChange(TreeTable.java:612)
      [INFO] [talledLocalContainer] 	at info.magnolia.ui.workbench.container.AbstractJcrContainer.fireItemSetChange(AbstractJcrContainer.java:197)
      [INFO] [talledLocalContainer] 	at info.magnolia.ui.workbench.list.ListPresenter.refresh(ListPresenter.java:110)
      [INFO] [talledLocalContainer] 	at info.magnolia.ui.workbench.WorkbenchPresenter.refresh(WorkbenchPresenter.java:265)
      [INFO] [talledLocalContainer] 	at info.magnolia.ui.contentapp.browser.BrowserPresenter$1.onContentChanged(BrowserPresenter.java:160)
      [INFO] [talledLocalContainer] 	at info.magnolia.ui.api.event.ContentChangedEvent.dispatch(ContentChangedEvent.java:88)
      [INFO] [talledLocalContainer] 	at info.magnolia.ui.api.event.ContentChangedEvent.dispatch(ContentChangedEvent.java:44)
      [INFO] [talledLocalContainer] 	at info.magnolia.event.SimpleEventBus.fireEvent(SimpleEventBus.java:78)
      [INFO] [talledLocalContainer] 	at info.magnolia.event.ResettableEventBus.fireEvent(ResettableEventBus.java:70)
      [INFO] [talledLocalContainer] 	at info.magnolia.event.ResettableEventBus.fireEvent(ResettableEventBus.java:70)
      [INFO] [talledLocalContainer] 	at info.magnolia.ui.framework.action.DeleteAction.onPostExecute(DeleteAction.java:119)
      [INFO] [talledLocalContainer] 	at info.magnolia.ui.framework.action.AbstractCommandAction.executeOnItem(AbstractCommandAction.java:278)
      [INFO] [talledLocalContainer] 	at info.magnolia.ui.framework.action.DeleteAction.executeOnItem(DeleteAction.java:125)
      [INFO] [talledLocalContainer] 	at info.magnolia.ui.framework.action.AbstractMultiItemAction.execute(AbstractMultiItemAction.java:110)
      [INFO] [talledLocalContainer] 	at info.magnolia.ui.framework.action.DeleteAction.execute(DeleteAction.java:113)
      [INFO] [talledLocalContainer] 	at info.magnolia.ui.api.action.AbstractActionExecutor.execute(AbstractActionExecutor.java:78)
      [INFO] [talledLocalContainer] 	at info.magnolia.ui.framework.action.ConfirmationAction$1.onSuccess(ConfirmationAction.java:100)
      [INFO] [talledLocalContainer] 	at info.magnolia.ui.framework.overlay.OverlayPresenter$4.onConfirmation(OverlayPresenter.java:241)
      [INFO] [talledLocalContainer] 	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
      [INFO] [talledLocalContainer] 	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
      [INFO] [talledLocalContainer] 	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
      [INFO] [talledLocalContainer] 	at java.lang.reflect.Method.invoke(Method.java:597)
      [INFO] [talledLocalContainer] 	at com.vaadin.event.ListenerMethod.receiveEvent(ListenerMethod.java:508)
      [INFO] [talledLocalContainer] 	at com.vaadin.event.EventRouter.fireEvent(EventRouter.java:167)
      [INFO] [talledLocalContainer] 	at com.vaadin.server.AbstractClientConnector.fireEvent(AbstractClientConnector.java:969)
      [INFO] [talledLocalContainer] 	at info.magnolia.ui.vaadin.dialog.ConfirmationDialog.confirm(ConfirmationDialog.java:109)
      [INFO] [talledLocalContainer] 	at info.magnolia.ui.vaadin.dialog.ConfirmationDialog$1.buttonClick(ConfirmationDialog.java:76)
      [INFO] [talledLocalContainer] 	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
      [INFO] [talledLocalContainer] 	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
      [INFO] [talledLocalContainer] 	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
      [INFO] [talledLocalContainer] 	at java.lang.reflect.Method.invoke(Method.java:597)
      [INFO] [talledLocalContainer] 	at com.vaadin.event.ListenerMethod.receiveEvent(ListenerMethod.java:508)
      [INFO] [talledLocalContainer] 	at com.vaadin.event.EventRouter.fireEvent(EventRouter.java:167)
      [INFO] [talledLocalContainer] 	at com.vaadin.server.AbstractClientConnector.fireEvent(AbstractClientConnector.java:969)
      [INFO] [talledLocalContainer] 	at com.vaadin.ui.Button.fireClick(Button.java:368)
      [INFO] [talledLocalContainer] 	at com.vaadin.ui.Button$1.click(Button.java:57)
      [INFO] [talledLocalContainer] 	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
      [INFO] [talledLocalContainer] 	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
      [INFO] [talledLocalContainer] 	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
      [INFO] [talledLocalContainer] 	at java.lang.reflect.Method.invoke(Method.java:597)
      [INFO] [talledLocalContainer] 	at com.vaadin.server.ServerRpcManager.applyInvocation(ServerRpcManager.java:168)
      [INFO] [talledLocalContainer] 	at com.vaadin.server.ServerRpcManager.applyInvocation(ServerRpcManager.java:118)
      [INFO] [talledLocalContainer] 	at com.vaadin.server.communication.ServerRpcHandler.handleBurst(ServerRpcHandler.java:214)
      [INFO] [talledLocalContainer] 	at com.vaadin.server.communication.ServerRpcHandler.handleRpc(ServerRpcHandler.java:111)
      [INFO] [talledLocalContainer] 	at com.vaadin.server.communication.UidlRequestHandler.synchronizedHandleRequest(UidlRequestHandler.java:91)
      [INFO] [talledLocalContainer] 	at com.vaadin.server.SynchronizedRequestHandler.handleRequest(SynchronizedRequestHandler.java:37)
      [INFO] [talledLocalContainer] 	at com.vaadin.server.VaadinService.handleRequest(VaadinService.java:1371)
      [INFO] [talledLocalContainer] 	at com.vaadin.server.VaadinServlet.service(VaadinServlet.java:238)
      [INFO] [talledLocalContainer] 	at info.magnolia.ui.admincentral.AdmincentralVaadinServlet.service(AdmincentralVaadinServlet.java:132)
      [INFO] [talledLocalContainer] 	at javax.servlet.http.HttpServlet.service(HttpServlet.java:820)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.ServletDispatchingFilter.doFilter(ServletDispatchingFilter.java:127)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:82)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:82)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:82)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:82)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.CompositeFilter.doFilter(CompositeFilter.java:65)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.VirtualUriFilter.doFilter(VirtualUriFilter.java:68)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
      [INFO] [talledLocalContainer] 	at info.magnolia.module.cache.executor.Bypass.processCacheRequest(Bypass.java:58)
      [INFO] [talledLocalContainer] 	at info.magnolia.module.cache.executor.CompositeExecutor.processCacheRequest(CompositeExecutor.java:66)
      [INFO] [talledLocalContainer] 	at info.magnolia.module.cache.filter.CacheFilter.doFilter(CacheFilter.java:153)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:58)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.i18n.I18nContentSupportFilter.doFilter(I18nContentSupportFilter.java:73)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.RangeSupportFilter.doFilter(RangeSupportFilter.java:84)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.security.BaseSecurityFilter.doFilter(BaseSecurityFilter.java:57)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.security.SecurityCallbackFilter.doFilter(SecurityCallbackFilter.java:83)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:58)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.security.LogoutFilter.doFilter(LogoutFilter.java:93)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:58)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
      [INFO] [talledLocalContainer] 	at info.magnolia.module.templatingkit.filters.SiteMergeFilter.doFilter(SiteMergeFilter.java:112)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MultiChannelFilter.doFilter(MultiChannelFilter.java:82)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:58)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
      [INFO] [talledLocalContainer] 	at info.magnolia.module.cache.filter.GZipFilter.doFilter(GZipFilter.java:73)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:58)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:82)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.security.auth.login.LoginFilter.doFilter(LoginFilter.java:104)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:82)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:82)
      [INFO] [talledLocalContainer] 	at info.magnolia.module.devicedetection.filter.DeviceDetectionFilter.doFilter(DeviceDetectionFilter.java:71)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:58)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.ContentTypeFilter.doFilter(ContentTypeFilter.java:103)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.ContextFilter.doFilter(ContextFilter.java:129)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.CompositeFilter.doFilter(CompositeFilter.java:65)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.SafeDestroyMgnlFilterWrapper.doFilter(SafeDestroyMgnlFilterWrapper.java:106)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterDispatcher.doDispatch(MgnlFilterDispatcher.java:66)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlMainFilter.doFilter(MgnlMainFilter.java:107)
      [INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlMainFilter.doFilter(MgnlMainFilter.java:93)
      [INFO] [talledLocalContainer] 	at org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1157)
      [INFO] [talledLocalContainer] 	at org.mortbay.jetty.servlet.ServletHandler.handle(ServletHandler.java:388)
      [INFO] [talledLocalContainer] 	at org.mortbay.jetty.security.SecurityHandler.handle(SecurityHandler.java:216)
      [INFO] [talledLocalContainer] 	at org.mortbay.jetty.servlet.SessionHandler.handle(SessionHandler.java:182)
      [INFO] [talledLocalContainer] 	at org.mortbay.jetty.handler.ContextHandler.handle(ContextHandler.java:765)
      [INFO] [talledLocalContainer] 	at org.mortbay.jetty.webapp.WebAppContext.handle(WebAppContext.java:418)
      [INFO] [talledLocalContainer] 	at org.mortbay.jetty.handler.ContextHandlerCollection.handle(ContextHandlerCollection.java:230)
      [INFO] [talledLocalContainer] 	at org.mortbay.jetty.handler.HandlerCollection.handle(HandlerCollection.java:114)
      [INFO] [talledLocalContainer] 	at org.mortbay.jetty.handler.HandlerWrapper.handle(HandlerWrapper.java:152)
      [INFO] [talledLocalContainer] 	at org.mortbay.jetty.Server.handle(Server.java:326)
      [INFO] [talledLocalContainer] 	at org.mortbay.jetty.HttpConnection.handleRequest(HttpConnection.java:542)
      [INFO] [talledLocalContainer] 	at org.mortbay.jetty.HttpConnection$RequestHandler.content(HttpConnection.java:938)
      [INFO] [talledLocalContainer] 	at org.mortbay.jetty.HttpParser.parseNext(HttpParser.java:755)
      [INFO] [talledLocalContainer] 	at org.mortbay.jetty.HttpParser.parseAvailable(HttpParser.java:218)
      [INFO] [talledLocalContainer] 	at org.mortbay.jetty.HttpConnection.handle(HttpConnection.java:404)
      [INFO] [talledLocalContainer] 	at org.mortbay.io.nio.SelectChannelEndPoint.run(SelectChannelEndPoint.java:409)
      [INFO] [talledLocalContainer] 	at org.mortbay.thread.BoundedThreadPool$PoolThread.run(BoundedThreadPool.java:451)
      [INFO] [talledLocalContainer] Caused by: java.lang.NullPointerException
      [INFO] [talledLocalContainer] 	at info.magnolia.module.googlesitemap.app.subapp.sitemapdetail.formatter.FolderNameColumnFormatter.generateCell(FolderNameColumnFormatter.java:81)
      [INFO] [talledLocalContainer] 	at com.vaadin.ui.Table.parseItemIdToCells(Table.java:2312)
      [INFO] [talledLocalContainer] 	at com.vaadin.ui.Table.getVisibleCellsNoCache(Table.java:2173)
      [INFO] [talledLocalContainer] 	at com.vaadin.ui.Table.refreshRenderedCells(Table.java:1694)
      [INFO] [talledLocalContainer] 	... 132 more
      

        Acceptance criteria

              pmundt Philip Mundt
              pmundt Philip Mundt
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: