Hi team, I'm in the process of the WEBPAGE --> WEBLOG_CUSTOM_TEMPLATE work (https://issues.apache.org/jira/browse/ROL-2039) and confirming that the 5.0.4 -> 5.1 migration will work with the changes. (I did custom_theme_rendition earlier today and it seems to be in good shape.) Question: The webpage table has a boolean "navbar" column that appears always "false", even in 5.0.x, I don't know what it's for, ThemeTemplate.java says this:

    /**
     * Is the Template to be included in the navbar?
     */
    boolean isNavbar();

Can we get rid of this column? There appear to be three other columns we can potentially remove from WEBPAGE/WEBLOG_CUSTOM_TEMPLATE: decorator, template, and templatelang (the latter two columns, which are now in custom_template_rendition, can be deleted after the 500-to-510 migration script successfully copies those values from webpage into custom_template_rendition.) ThemeManagerImpl says "decorators are deprecated starting in 4.0" so it looks safe to remove.

One more column we can possibly remove from the webpage table is "outputtype" -- AFAICT it's always null in 5.0.4 & 5.1, anyone know if this is still useful? This is what WeblogTemplate and TemplateEdit.java have to say about it:

    /**
* Content-type rendered by template or null for auto-detection by link extension.
     */

                if (getBean().getAutoContentType() == null ||
                        !getBean().getAutoContentType()) {
templateToSave.setOutputContentType(getBean().getManualContentType());
                } else {
// empty content-type indicates that template uses auto content-type detection
                    templateToSave.setOutputContentType(null);
                }

If this is something that, if it ever became useful in the future, it would be at the rendition level, not the higher-level template level, I can pull it out of WEBLOG_CUSTOM_TEMPLATE, and later, if there's ever a demand for it, place it in custom_template_rendition.

Regards,
Glen

Reply via email to