Base Ready Theme - ACAD Bug?

[Ron Frigon]

I just notice this page in my page views report: /OINF,OINF.html. The previous page was ACAD


Form Content on ACAD:

<mvt:if expr="g.Order">
  <mvt:assign name="l.settings:url" value="l.settings:urls:OINF:secure" />
<mvt:elseif expr="g.Redeem">
  <mvt:assign name="l.settings:url" value="l.settings:urls:RGFT:secure" />
<mvt:elseif expr="g.WishList">
  <mvt:assign name="l.settings:url" value="l.settings:urls:WISH:secure" />
<mvt:else>
  <mvt:assign name="l.settings:url" value="l.settings:urls:ACED:secure" />
</mvt:if>
<form method="post" action="&mvte:url;" role="form" id="acad_form" class="form-horizontal clearfix" autocomplete="off">
<input type="hidden" name="Action" value="ICST" />
<input type="hidden" name="Order" value="&mvte:global:Order;" />
<mvt:if expr="g.Order">
  <input type="hidden" name="Screen" value="OINF" />
<mvt:elseif expr="g.Redeem">
  <input type="hidden" name="Screen" value="RGFT" />
<mvt:else>
  <input type="hidden" name="Screen" value="ACED" />
</mvt:if>

It looks like if g.Order then this will set the url of the form action to OINF. But then within the form g.Order is checked again, adding the hidden input for Screen/OINF.

Maybe the mvt:if statement for the hidden form inputs needs to be removed?

[Brennan]

Yes this does look to be a bug when using Legacy SEO settings. You should be able to remove the whole second if statement since the Screen will be determined by the URL instead of the hidden input.

[Ron Frigon]

Thanks Brennan, I am using Legacy SEO settings.

I checked ACAD on a few other Ready Themes and found that they don't include the second if statement at all...