Announcement

Collapse
No announcement yet.

Base Ready Theme - ACAD Bug?

Collapse
X
 
  • Filter
  • Time
  • Show
Clear All
new posts

    Base Ready Theme - ACAD Bug?

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


    Form Content on ACAD:

    Code:
    <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?
    Ron Frigon
    The Web Dude at J. Rousek Toy Co.

    #2
    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.

    Brennan Heyde
    VP Product
    Miva, Inc.
    [email protected]
    https://www.miva.com

    Comment


      #3
      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...
      Ron Frigon
      The Web Dude at J. Rousek Toy Co.

      Comment

      Working...
      X