isNavBar is used by the #showPageMenu() macro and to determine which page
templates to show in the page menu. Do not remove it or you will break lots
of themes, including mine. You can set the value in the advanced settings
part of the page template edit page.

- Dave



On Fri, Jul 18, 2014 at 6:51 PM, Glen Mazza <glen.ma...@gmail.com> wrote:

> 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