Нет опции 'sidebar' в $theme_settings.category_nav_mode Принято

1

В theme.xml темы "Дефолт" приложения магазин есть только два вариант навигации категорий товаров: dropdown и inline.

    <setting var="category_nav_mode" control_type="radio">
      <value>dropdown</value>
      <name locale="ru_RU">Каталог товаров</name>
      <name locale="en_US">Category list</name>
      <options>
        <option value="dropdown">
          <name locale="ru_RU">Выпадающее меню &laquo;Каталог товаров&raquo;</name>
          <name locale="en_US">Drop-down menu &ldquo;Shop by Department&rdquo;</name>
          <description locale="ru_RU">Выпадающее развернутое меню с полным каталогом товаров.</description>
          <description locale="en_US">Category list will be available under the "Shop by Department" header link.</description>
        </option>
        <option value="inline">
          <name locale="ru_RU">Список категорий</name>
          <name locale="en_US">Inline list</name>
          <description locale="ru_RU">Рекомендуется, если в корне не более четырех-пяти категорий, и все они помещаются в один ряд.</description>
          <description locale="en_US">Great for menus with up to four or five top-level categories.</description>
        </option>
      </options>
    </setting>

Однако, в шаблоне header.html есть такой код:

...
        {if $theme_settings.category_nav_mode != 'sidebar'}
...
...
...
        {/if}
...

Такой опции как sidebar нет, выходит, что выражение всегда исполнится в True, а значит, эту условную обёртку можно убрать.

Возможно я что-то упускаю. Что думаете?

1 комментарий

  • +1
    devellopah devellopah 23 ноября 2020 03:07 #

    кстати, чуть выше есть строка

    {if $shop_pages || $theme_settings.category_nav_mode != 'sidebar'}

    которую можно сократить до

    {if $shop_pages || true}

    которую можно упростить до

    {if true}

    То есть, в итоге и эту условную обёртку можно убрать из шаблона.

    Добавить комментарий

    Чтобы добавить комментарий, зарегистрируйтесь или войдите