Announcement

Collapse
No announcement yet.

Levels ReadyTheme dated July 9, 2018 customer.htm page_update for OCST incorrect

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

    Levels ReadyTheme dated July 9, 2018 customer.htm page_update for OCST incorrect

    The selection for the billing address on OCST doesn't work as expected and so when returning to checkout sometimes customer can't actually do anything to the billing and can't set a new address, and new addresses keep getting created in the address book. When looking at the html output of the OCST page as I checked out of the store, I noticed that the id in the billing address selection was at the same as the id in the shipping selection above. I also noticed in the template code that the billing selection only referred to ship variables.
    Look in the ]<div id="billing_fields"> at the select with name=BillID

    This should be updated asap in the readytheme.

    Code:
     <mvt:if expr="g.Basket:cust_id AND g.Customer_Session_Verified">
                    <div class="column whole form-row &mvte:global:BillID_Row;">
                        <div class="floating-placeholder">
                            <select name="BillID" id="billing_selector">
                                <option value="-1">&lt;New&gt;</option>
                                <mvt:foreach iterator="address" array="customer:addresses">
                                    <mvt:if expr="toupper( l.settings:address:fname ) EQ toupper( g.ShipFirstName ) AND
                                                     toupper( l.settings:address:lname ) EQ toupper( g.ShipLastName ) AND
                                                     toupper( l.settings:address:email ) EQ toupper( g.ShipEmail ) AND
                                                     toupper( l.settings:address:comp ) EQ toupper( g.ShipCompany ) AND
                                                     toupper( l.settings:address:phone ) EQ toupper( g.ShipPhone ) AND
                                                     toupper( l.settings:address:fax ) EQ toupper( g.ShipFax ) AND
                                                     toupper( l.settings:address:addr1 ) EQ toupper( g.ShipAddress1 ) AND
                                                     toupper( l.settings:address:addr2 ) EQ toupper( g.ShipAddress2 ) AND
                                                     toupper( l.settings:address:city ) EQ toupper( g.ShipCity ) AND
                                                     (
                                                        ( ( NOT ISNULL g.ShipStateSelect ) AND ( toupper( l.settings:address:state ) EQ toupper( g.ShipStateSelect ) ) ) OR
                                                        ( ( ISNULL g.ShipStateSelect ) AND ( toupper( l.settings:address:state ) EQ toupper( g.ShipState ) ) )
                                                     ) AND
                                                     toupper( l.settings:address:zip ) EQ toupper( g.ShipZip ) AND
                                                     toupper( l.settings:address:cntry ) EQ toupper( g.ShipCountry )">
                                        <option value="&mvte:address:id;" selected>&mvte:address:descrip;</option>
                                    <mvt:else>
                                        <option value="&mvte:address:id;">&mvte:address:descrip;</option>
                                    </mvt:if>
                                </mvt:foreach>
                            </select>
                            <label class="required" for="ShipID">Saved Address:</label>
                        </div>
                    </div>
                    <div class="clear clearfix"></div>
                </mvt:if>

    Should be
    Code:
                <mvt:if expr="g.Basket:cust_id AND g.Customer_Session_Verified">
                    <div class="column whole form-row &mvte:global:BillID_Row;">
                        <div class="floating-placeholder">
                            <select name="BillID" id="billing_selector">
                                <option value="-1">&lt;New&gt;</option>
                                <mvt:foreach iterator="address" array="customer:addresses">
                                    <mvt:if expr="toupper( l.settings:address:fname ) EQ toupper( g.BillFirstName ) AND
                                                     toupper( l.settings:address:lname ) EQ toupper( g.BillLastName ) AND
                                                     toupper( l.settings:address:email ) EQ toupper( g.BillEmail ) AND
                                                     toupper( l.settings:address:comp ) EQ toupper( g.BillCompany ) AND
                                                     toupper( l.settings:address:phone ) EQ toupper( g.BillPhone ) AND
                                                     toupper( l.settings:address:fax ) EQ toupper( g.BillFax ) AND
                                                     toupper( l.settings:address:addr1 ) EQ toupper( g.BillAddress1 ) AND
                                                     toupper( l.settings:address:addr2 ) EQ toupper( g.BillAddress2 ) AND
                                                     toupper( l.settings:address:city ) EQ toupper( g.BillCity ) AND
                                                     (
                                                        ( ( NOT ISNULL g.BillStateSelect ) AND ( toupper( l.settings:address:state ) EQ toupper( g.BillpStateSelect ) ) ) OR
                                                        ( ( ISNULL g.BillStateSelect ) AND ( toupper( l.settings:address:state ) EQ toupper( g.BillState ) ) )
                                                     ) AND
                                                     toupper( l.settings:address:zip ) EQ toupper( g.BillZip ) AND
                                                     toupper( l.settings:address:cntry ) EQ toupper( g.BillCountry )">
                                        <option value="&mvte:address:id;" selected>&mvte:address:descrip;</option>
                                    <mvt:else>
                                        <option value="&mvte:address:id;">&mvte:address:descrip;</option>
                                    </mvt:if>
                                </mvt:foreach>
                            </select>
                            <label class="required" for="BillID">Saved Address:</label>
                        </div>
                    </div>
                    <div class="clear clearfix"></div>
                </mvt:if>

    #2
    Thank you for finding this bug. I have updated the demonstration site and will make a note of this issue in the GitHub repository [ https://github.com/mivaecommerce/lev...heme/issues/62 ].
    Matt Zimmermann

    Miva Web Developer
    Alchemy Web Development
    https://www.alchemywebdev.com
    Site Development - Maintenance - Consultation

    Miva Certified Developer
    Miva Professional Developer

    https://www.dev4web.net | Twitter

    Comment

    Working...
    X