If logged in statement failing - template not parsing properly

Question

I have an Expresso:Store site and I am using the following conditionals in the header:

{exp:store:cart}
{if no_items}
    <a href="{path='store/checkout'}">Your bag is empty</a>

        <div class="details">
            {if logged_in}
                <br /><a href="{path='store/my_orders'}">My Orders</a>
                <!-- template parsing fails -->
            {if:else}
                <br /><a href="{path='account/login'}">Login</a>
            {/if}
         </div>
    {/if}

<a href="{path='store/checkout'}">My Shopping Bag ({order_qty})</a>
    <div class="details">
         <span class="price">{order_subtotal}</span><a href="{path='store/checkout'}">Checkout</a>
     {if logged_in}
         <br /><a href="{path='store/my_orders'}">My Orders</a>
     {if:else}
         <br /><a href="{path='account/login'}">Login</a>
     {/if}
     </div>
{/exp:store:cart}

When I am logged in, the template parses fine and everything works.

If I am NOT logged in, the template parses up to the commented line and the header template fails to output anything after that line, breaking the remaining page layout.

It's as if it just stops after that line, but I can't work out what I have done wrong, there are no template errors I can see in the EE template editor.

Answer 1

Perhaps it's to do with parse order? From my understanding, the {if:else} conditional (particularly of the logged_in variety) is counted as an advanced conditional, so gets parsed later than modules, such as the Expresso:store.

This is just guessing, but perhaps when parsing exp:store:cart, it's interpreting the {if:else} on line 9 as being part of the {if no_items} conditional, rather than the {if logged_in}.

Hope that helps

Answer 2

Try this instead:

 {if logged_in}
     <br /><a href="{path='store/my_orders'}">My Orders</a>
 {/if}
 {if logged_out}
     <br /><a href="{path='account/login'}">Login</a>
 {/if}

Not sure if it's an EE version thing, but {if:else} never used to work for me on older EE versions, but does with the latest. Either that or it's where it's used (EE introduced new conditional parsing rules with a more recent update), so the {if:else} may work in channel:entries, but not outside. Just theoretical.

Answer 3

I have worked out that the issue is something to do with nesting {if logged_in} within the {if no_items} loop.

The code is all working fine without the {if} statement in the expresso:store {no_items} - a bit of reworking the code to not need any nested loops worked much better.

I can't explain the issue, but it's resolved my problem.