Some bugfixes

ccm-cms/src/main/java/com/arsdigita/cms/ui/pages/PageModelAdminPageController.java

@@ -0,0 +1,51 @@
+/*
+ * To change this license header, choose License Headers in Project Properties.
+ * To change this template file, choose Tools | Templates
+ * and open the template in the editor.
+ */
+package com.arsdigita.cms.ui.pages;
+
+import org.libreccm.l10n.GlobalizationHelper;
+import org.libreccm.pagemodel.PageModel;
+import org.libreccm.pagemodel.PageModelRepository;
+import org.librecms.pages.Pages;
+
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
+
+import javax.enterprise.context.RequestScoped;
+import javax.inject.Inject;
+import javax.transaction.Transactional;
+
+/**
+ *
+ * @author <a href="mailto:jens.pelzetter@googlemail.com">Jens Pelzetter</a>
+ */
+@RequestScoped
+public class PageModelAdminPageController {
+
+    @Inject
+    private GlobalizationHelper globalizationHelper;
+
+    @Inject
+    private PageModelRepository pageModelRepository;
+
+    @Transactional(Transactional.TxType.REQUIRED)
+    public Map<String, Object> findDraftPageModelsByApplication(
+        final Pages pages
+    ) {
+        final List<PageModel> pageModels = pageModelRepository
+            .findDraftByApplication(pages);
+        final Map<String, Object> result = new HashMap<>();
+        for (final PageModel pageModel : pageModels) {
+            result.put("pageModelId", pageModel.getPageModelId());
+            final String title = globalizationHelper
+                .getValueFromLocalizedString(pageModel.getTitle());
+            result.put("title", title);
+        }
+        
+        return result;
+    }
+
+}

ccm-cms/src/main/java/com/arsdigita/cms/ui/pages/PagesAdminPage.java

@@ -50,7 +50,6 @@ import org.libreccm.categorization.Category;
 import org.libreccm.categorization.CategoryRepository;
 import org.libreccm.cdi.utils.CdiUtil;
 import org.libreccm.core.UnexpectedErrorException;
-import org.libreccm.l10n.GlobalizationHelper;
 import org.libreccm.pagemodel.PageModel;
 import org.libreccm.pagemodel.PageModelRepository;
 import org.librecms.CmsConstants;
@@ -58,7 +57,7 @@ import org.librecms.pages.PageManager;
 import org.librecms.pages.PageRepository;
 import org.librecms.pages.Pages;
 
-import java.util.List;
+import java.util.Map;
 import java.util.Optional;
 import java.util.TooManyListenersException;
 
@@ -163,7 +162,7 @@ public class PagesAdminPage extends CMSApplicationPage {
 
         pagesContextBar = new PagesContextBar();
         super.add(pagesContextBar);
-        
+
         final TabbedPane tabbedPane = new TabbedPane();
         tabbedPane.addTab(new Label(new GlobalizedMessage(
             "cms.ui.pages.tab.pages", CmsConstants.CMS_BUNDLE)),
@@ -261,22 +260,31 @@ public class PagesAdminPage extends CMSApplicationPage {
                                         "cms.ui.pages.assigned_page_model.inherit",
                                         CmsConstants.CMS_BUNDLE))));
 
-        final CdiUtil cdiUtil = CdiUtil.createCdiUtil();
+//        final CdiUtil cdiUtil = CdiUtil.createCdiUtil();
-        final PageModelRepository pageModelRepo = cdiUtil
+//        final PageModelRepository pageModelRepo = cdiUtil.findBean(
-            .findBean(PageModelRepository.class);
+//            PageModelRepository.class
-        final List<PageModel> pageModels = pageModelRepo
+//        );
-            .findDraftByApplication(pagesInstance);
+//        final List<PageModel> pageModels = pageModelRepo
-        final GlobalizationHelper globalizationHelper = cdiUtil
+//            .findDraftByApplication(pagesInstance);
-            .findBean(GlobalizationHelper.class);
+//        final GlobalizationHelper globalizationHelper = cdiUtil
-
+//            .findBean(GlobalizationHelper.class);
-        for (final PageModel pageModel : pageModels) {
+//        for (final PageModel pageModel : pageModels) {
-            target.addOption(new Option(
+//            target.addOption(
-                Long.toString(pageModel.getPageModelId()),
+//                new Option(
-                new Text(globalizationHelper.getValueFromLocalizedString(
+//                    Long.toString(pageModel.getPageModelId()),
-                    pageModel
+//                    new Text(
-                        .getTitle()))));
+//                        globalizationHelper.getValueFromLocalizedString(
-        }
+//                            pageModel.getTitle()
-
+//                        )
+//                    )
+//                )
+//            );
+//        }
+        final PageModelAdminPageController controller = CdiUtil.createCdiUtil()
+            .findBean(PageModelAdminPageController.class);
+        final Map<String, Object> options = controller
+            .findDraftPageModelsByApplication(pagesInstance);
+        for (final Map.Entry<St)
     }
 
     private void categoryTreeStateChanged(final ChangeEvent event) {

ccm-core/src/main/java/org/libreccm/pagemodel/AbstractPageRenderer.java

@@ -20,6 +20,7 @@ package org.libreccm.pagemodel;
 
 import java.util.HashMap;
 import java.util.Map;
+import java.util.Objects;
 
 import javax.inject.Inject;
 
@@ -76,7 +77,9 @@ public abstract class AbstractPageRenderer implements PageRenderer {
         final Map<String, Object> container = new HashMap<>();
 
         container.put("key", containerModel.getKey());
-        container.put("styles", containerModel.getStyles().toCss());
+        if (containerModel.getStyles() != null) {
+            container.put("styles", containerModel.getStyles().toCss());
+        }
 
         for (final ComponentModel componentModel : containerModel
             .getComponents()) {