Shopify / dawn

Shopify's first source available reference theme, with Online Store 2.0 features and performance built-in.
Other
2.36k stars 3.13k forks source link

Theme Check Summary has 2 errors and 14 warnings #3456

Open skmohotasin opened 4 weeks ago

skmohotasin commented 4 weeks ago

╭─ info ───────────────────────────────────────────────────────────────────────╮ │ │ │ layout/password.liquid │ │ │ │ │ │ [warning]: UndefinedObject │ │ Unknown object 'scheme_classes' used. │ │ │ │ 40 {% assign scheme_classes = scheme_classes | append: ', .color-' | │ │ append: scheme.id %} │ │ │ ╰──────────────────────────────────────────────────────────────────────────────╯

╭─ info ───────────────────────────────────────────────────────────────────────╮ │ │ │ layout/theme.liquid │ │ │ │ │ │ [warning]: UndefinedObject │ │ Unknown object 'scheme_classes' used. │ │ │ │ 54 {% assign scheme_classes = scheme_classes | append: ', .color-' | │ │ append: scheme.id %} │ │ │ ╰──────────────────────────────────────────────────────────────────────────────╯

╭─ info ───────────────────────────────────────────────────────────────────────╮ │ │ │ sections/announcement-bar.liquid │ │ │ │ │ │ [warning]: UndefinedObject │ │ Unknown object 'block' used. │ │ │ │ 34 {{ block.shopify_attributes }} │ │ │ │ │ │ [warning]: UndefinedObject │ │ Unknown object 'block' used. │ │ │ │ 23 {{ block.shopify_attributes }} │ │ │ ╰──────────────────────────────────────────────────────────────────────────────╯

╭─ info ───────────────────────────────────────────────────────────────────────╮ │ │ │ sections/email-signup-banner.liquid │ │ │ │ │ │ [error]: ValidSchema │ │ Property templates is not allowed. │ │ │ │ 388 "templates": ["password"] │ │ │ ╰──────────────────────────────────────────────────────────────────────────────╯

╭─ info ───────────────────────────────────────────────────────────────────────╮ │ │ │ sections/header.liquid │ │ │ │ │ │ [error]: LiquidHTMLSyntaxError │ │ SyntaxError: expected a letter, "{{", "wbr" (case-insensitive), "track" │ │ (case-insensitive), "source" (case-insensitive), "param" │ │ (case-insensitive), "meta" (case-insensitive), "link" (case-insensitive), │ │ "keygen" (case-insensitive), "input" (case-insensitive), "img" │ │ (case-insensitive), "hr" (case-insensitive), "embed" (case-insensitive), │ │ "command" (case-insensitive), "col" (case-insensitive), "br" │ │ (case-insensitive), "base" (case-insensitive), "area" (case-insensitive), │ │ "svg", "style", or "script" │ │ │ │ 137 <{% if section.settings.sticky_header_type != 'none' %}sticky-header │ │ data-sticky-type="{{ section.settings.sticky_header_type }}"{% else │ │ %}div{% endif %} class="header-wrapper color-{{ │ │ section.settings.color_scheme }} gradient{% if │ │ section.settings.show_line_separator %} header-wrapper--border-bottom{% │ │ endif %}"> │ │ │ ╰──────────────────────────────────────────────────────────────────────────────╯

╭─ info ───────────────────────────────────────────────────────────────────────╮ │ │ │ sections/main-article.liquid │ │ │ │ │ │ [warning]: UndefinedObject │ │ Unknown object 'comment' used. │ │ │ │ 110 {{ comment.content }} │ │ │ │ │ │ [warning]: UndefinedObject │ │ Unknown object 'comment' used. │ │ │ │ 112 {{ │ │ comment.author }} │ │ │ │ │ │ [warning]: UndefinedObject │ │ Unknown object 'comment' used. │ │ │ │ 109 <article id="{{ comment.id }}" │ │ class="article-template__comments-comment"> │ │ │ │ │ │ [warning]: UndefinedObject │ │ Unknown object 'comment' used. │ │ │ │ 108 {%- if comment.status == 'pending' and comment.content -%} │ │ │ │ │ │ [warning]: UndefinedObject │ │ Unknown object 'comment' used. │ │ │ │ 108 {%- if comment.status == 'pending' and comment.content -%} │ │ │ │ │ │ [warning]: UndefinedObject │ │ Unknown object 'comment' used. │ │ │ │ 135 if blog.moderated? and comment.status == 'unapproved' │ │ │ ╰──────────────────────────────────────────────────────────────────────────────╯

╭─ info ───────────────────────────────────────────────────────────────────────╮ │ │ │ sections/main-product.liquid │ │ │ │ │ │ [warning]: UndefinedObject │ │ Unknown object 'continue' used. │ │ │ │ 559 offset: continue │ │ │ ╰──────────────────────────────────────────────────────────────────────────────╯

╭─ info ───────────────────────────────────────────────────────────────────────╮ │ │ │ sections/main-search.liquid │ │ │ │ │ │ [warning]: UnusedAssign │ │ The variable 'product_settings' is assigned but not used │ │ │ │ 271 {%- capture product_settings -%}{%- if │ │ section.settings.product_show_vendor -%}vendor,{%- endif -%}title,price{%- │ │ endcapture -%} │ │ │ ╰──────────────────────────────────────────────────────────────────────────────╯

╭─ info ───────────────────────────────────────────────────────────────────────╮ │ │ │ snippets/product-variant-options.liquid │ │ │ │ │ │ [warning]: UnusedAssign │ │ The variable 'checked' is assigned but not used │ │ │ │ 75 assign checked = true │ │ │ │ │ │ [warning]: UnusedAssign │ │ The variable 'help_text' is assigned but not used │ │ │ │ │ {{ value | escape }} │ │ │ │ ╰──────────────────────────────────────────────────────────────────────────────╯

╭─ info ───────────────────────────────────────────────────────────────────────╮ │ │ │ Theme Check Summary. │ │ │ │ 203 files inspected with 16 total offenses found across 9 files. │ │ 2 errors. │ │ 14 warnings.

skmohotasin commented 4 weeks ago

https://github.com/skmohotasin/dawn/tree/liquid-render-issue-fixed I've fixed it