Lava comments (single line & multiline) are being displayed in HTML output

[stanalyst]

Please go through all the tasks below

• Check this box only after you have successfully completed both the above tasks

Please provide a brief description of the problem. Please do not forget to attach the relevant screenshots from your side.

While preforming alpha testing for v16, we noticed that some of our lava comments were appearing on the output page. It appears to be limited to the Single Line //- & Multiline /- -/ Lava comment types which were added for v12. The behavior was inconsistent though - with some comments from the block showing and others not. So far we have only seen this behavior in HTML blocks.

Expected Behavior

We would expect that all single & multiline comments would NOT be visible on the output page content.

Actual Behavior

Some of the single line and multiline comments are visible on the output page...

Steps to Reproduce

Put an HTML block on the page.
Add content that includes single and/or multiline comments.
Observe output of that block on the page.

Rock Version

1.16.0.10

Client Culture Setting

en-US

[mikedotmundy]

I will add that this is not just an issue in v16.0 Alpha. We have seen this issue in v15.1 too. Trouble is I could never get it to duplicate on the demo server and couldn't find a way to duplicate it consistently in my environment.

[stanalyst]

Oh - good call - just confirmed we're seeing it under v15.1 also.

[nairdo]

@stanalyst We've not been able to reproduce this yet (v15.1 demo site: https://www.rocksolidchurchdemo.com/intranet/shared-documents nor v16.0 pre-alpha site: https://prealpha.rocksolidchurchdemo.com/intranet/shared-documents) using this HTML/Lava:

<div class="alert alert-info">
    <h4>Lava Engine</h4>
    <p>
        This system is currently running the <b>{{ 'LavaEngine' | RockInstanceConfig }}</b> Lava engine.
    </p>    
</div>

<hr>

<h3>Testing issue 5560</h3>

{% comment %}By Jim M...{% endcomment %}
{% comment %}By Stan Y...{% endcomment %}

/- GroupType 67 = Explo Online - assume person is in only 1 group of this type -/

Did you see those comments ^^^

{% assign groupMember = CurrentPerson | Groups: "25" | First %}
{% assign grp = groupMember.Group.Id| GroupById %}

//- proceed if we found a group

{% if grp != null and grp != empty %}
    <b>Welcome...</b>
{% endif %}

We've tried each of the engines, but perhaps there is a particular block/spot where this is occurring in your environment or something particular to your block's settings?

Would you be willing to share some more screenshots of the block settings, full content example, etc? Perhaps there is something else in the earlier content which is causing the problem.

[JimMichael]

Is it possibly related to this old bug?

#4975

[stanalyst]

It does appear to be related to that old bug ... for the example above ... look at the very first line

{% comment %} By Jim M Jan 2021. This block gets a person's Explo Online group, Zoom Link, schedule, and Leader details.{% endcomment %}

That single quote in "person's" caused the problem ... as soon as I removed it the page rendered correctly.

But this still seems like a bug to me. At least now we know the cause and can reproduce it.