Differences

This shows you the differences between two versions of the page.

Link to this comparison view

Both sides previous revision Previous revision
Next revision
Previous revision
litespeed_wiki:cache:litemage:troubleshooting:login-link-still-show-after-logged-in [2017/01/30 18:50]
Michael Alegre Edited for grammer.
litespeed_wiki:cache:litemage:troubleshooting:login-link-still-show-after-logged-in [2018/07/16 13:06] (current)
Jackson Zhang [Solution]
Line 1: Line 1:
-In the sm_market yt theme, with the default LiteMage installation, ​"Login or Create an Account" ​links in the header are still appearing even after a user has logged in.+~~NOTOC~~ 
 +====== "​Login/​Create an Account"​ Links Appearing While Logged In (block hole punch required) ====== 
 + 
 +==== Problem ==== 
 +In the sm_market yt theme, with the default LiteMage installation, ​**Login or Create an Account** links in the header are still appearing even after a user has logged in.
  
 To investigate this issue, ​ To investigate this issue, ​
   - [[litespeed_wiki:​cache:​litemage:​troubleshooting:​enabling-debug-log|Turn on the LiteMage Debug Log]].   - [[litespeed_wiki:​cache:​litemage:​troubleshooting:​enabling-debug-log|Turn on the LiteMage Debug Log]].
-  - Add query string ​"?​LITEMAGE_DEBUG=SHOWHOLES" ​to the end of a URL on your site. You should see that only welcome is inside a punched hole, while the login links are not. If you do not use an HTML minifier, you can also see the hole punched block when inspecting the HTML source. In the source, you will see something similar to ''<​p class="​welcome-msg"><​!--Litemage esi started welcome-->​Welcome<​!--Litemage esi ended welcome-->​ </​p>''​ +  - Add query string ​''​?​LITEMAGE_DEBUG=SHOWHOLES'' ​to the end of a URL on your site. You should see that only welcome is inside a punched hole, while the login links are not. If you do not use an HTML minifier, you can also see the hole punched block when inspecting the HTML source. In the source, you will see something similar to ''<​p class="​welcome-msg"><​!--Litemage esi started welcome-->​Welcome<​!--Litemage esi ended welcome-->​ </​p>''​ 
-  - Check the logs output for LiteMage related messages with he following command, ''​tail -f var/​log/​system.log | grep LiteMage''​. You should again see that the welcome block is hole punched while the login link is not. The ESI request should be similar to ''/​litemage/​esi/​getBlock/​t/​welcome/​bi/​welcome/​h/​D/​s/​1/​dp/​sm_market/​dt/​default/''​+  - Check the logs output for LiteMage related messages with the following command, ''​tail -f var/​log/​system.log | grep LiteMage''​. You should again see that the welcome block is hole punched while the login link is not. The ESI request should be similar to ''/​litemage/​esi/​getBlock/​t/​welcome/​bi/​welcome/​h/​D/​s/​1/​dp/​sm_market/​dt/​default/''​
  
-Now that we have confirmed the issue, it is time to fix it by punching a hole for the login link block, since this is private information. 
-  * In your sm_market config, check the header style selection. For this site, it is header4 
-  * Looking at the header4.phtml code, we can see that the logged in link is mixed into the header template. We need to take that logic out and put in its own block and template so that we can punch a hole for that block. 
  
-  [app/​design/​frontend/​sm_market/​default/​template/​page/​html]#​ vi header4.phtml+==== Cause ==== 
 +Login links are not hole punched.
  
- <div class="​yt-header-top">​ 
-     <div class="​container">​ 
-         <div class="​row">​ 
-             <div class="​header-top-1 col-lg-4 col-md-4 col-sm-6 col-xs-12">​ 
-                 <div class="​inner">​ 
-                     <p class="​welcome-msg"><?​php echo $this->​getChildHtml('​welcome'​) ?> <?php echo $this->​getAdditionalHtml() ?></​p>​ 
-                     <div class="​login-regis">​ 
-                        <?​php if(!$this->​helper('​customer'​)->​isLoggedIn() ){ ?> 
-                             <a title="<?​php echo $this->​__("​Login"​);​ ?>" class="​btn-head"​ href="<?​php echo $this->​getUrl('​customer/​account/​login/'​) ?>">​ 
-                                 <?php echo $this->​__("​Login"​);​ ?> 
-                             </a> 
-                             <?php echo $this->​__("​or"​);​ ?> 
-                         <?php } else{ ?> 
-                             <a title="<?​php echo $this->​__("​Logout"​);​ ?>" class="​btn-head"​ href="<?​php echo $this->​getUrl('​customer/​account/​logout'​) ?>">​ 
-                                 <?php echo $this->​__("​Logout"​);​ ?> 
-                             </a> 
-                         <?php } ?> 
-                         <?php 
-                         if(!$this->​helper('​customer'​)->​isLoggedIn() ){ ?> 
-                             <a title="<?​php echo $this->​__("​Create an Account"​);​ ?>" class="​btn-head"​ href="<?​php echo Mage::​getBaseUrl();​ ?>​customer/​account/​create/">​ 
-                             <?php echo $this->​__("​Create an Account"​);​ ?></​a>​ 
-                         <?php } ?> 
-                     </​div>​ 
-                 </​div>​ 
-             </​div>​ 
-             <div class="​header-top-2 col-lg-4 col-md-4 col-sm-6 col-xs-12">​ 
-                 <?php if($this->​getLayout()->​createBlock('​cms/​block'​)->​setBlockId('​v4-call-us'​)->​toHtml()) { ?> 
-                     <?php echo $this->​getLayout()->​createBlock('​cms/​block'​)->​setBlockId('​v4-call-us'​)->​toHtml();​ ?> 
-                 <?php } ?> 
-             </​div>​ 
-             <div class="​header-top-3 col-lg-4 col-md-4 col-sm-6 col-xs-12">​ 
-                 <div class="​inner">​ 
-                     <div class="​head-quicklink">​ 
-                         <?php echo $this->​getChildHtml('​topLinks'​);?>​ 
-                     </​div>​ 
-                 </​div>​ 
-             </​div>​ 
-         </​div>​ 
-     </​div>​ 
- </​div>​ 
  
-  * Here we call the new block “welcomelogin”,​ Modify the layout file, and add our new “welcomelogin” block inside the header block. ​ 
  
-  [app/​design/​frontend/​sm_market/​default/​layout]#​ vi page.xml+===== Solution =====
  
-  <block type="​page/​html_header"​ name="​header"​ as="​header">​ +==== Quick fix: Punch a big hole instead of small one through LiteMage configuration ====
-  …. +
-  …. After the welcome block +
-                 <​block type="​core/​template"​ name="​welcomelogin"​ as="​welcomelogin"​ template="​page/​html/​welcomelogin.phtml"/>​+
  
-  * Move the logic out of the header4.phtml and put in our new welcomelogin.phtml+You can try to punch a big hole instead ​of small one
  
-  [app/​design/​frontend/​sm_market/​default/​template/​page/​html]#​ vi welcomelogin.phtml+Logging into the Magento Admin and entering LiteMage config, we added ''​header''​ in **Customized Block Names for "​toplinks"​ Tag**.  
 +We also saw "​compare"​ in a toplinks block, so we added ''​compare''​ to **Additional Purge Tags for "​toplinks"​ Blocks** as well
  
 +{{ :​litespeed_wiki:​cache:​litemage:​troubleshooting:​litemage-see-minicompare-tag-add-purge.png?​nolink |}}
  
-                        <?php if(!$this->​helper('​customer'​)->​isLoggedIn() ){ ?> +For more info on this step, check out [[litespeed_wiki:​cache:​litemage:​troubleshooting:​logged-in-user-info-shown-to-others|Logged In Usernames/Cart Items Shown On Other User's Pages]].
-                            <a title="<?​php echo $this->​__("​Login"​);​ ?>" class="​btn-head" href="<?​php echo $this->​getUrl('​customer/​account/​login/'​) ?>">​ +
-                                <?php echo $this->​__("​Login"​);​ ?> +
-                            </a> +
-                            <?php echo $this->​__("​or"​);​ ?> +
-                        <?php } else{ ?> +
-                            <a title="<?​php echo $this->​__("​Logout"​);​ ?>" class="​btn-head"​ href="<?​php echo $this->​getUrl('customer/​account/​logout'​) ?>">​ +
-                                <?php echo $this->​__("​Logout"​);​ ?> +
-                            </​a>​ +
-                        <?php } ?> +
-                        <?php +
-                        if(!$this->​helper('​customer'​)->​isLoggedIn() ){ ?> +
-                            <a title="<?​php echo $this->​__("​Create an Account"​);​ ?>" class="​btn-head"​ href="<?​php echo Mage::​getBaseUrl();​ ?>​customer/​account/​create/">​ +
-                            <?php echo $this->​__("​Create an Account"​);​ ?></​a>​ +
-                        <?php } ?>+
  
 +This quick fix solution may or may not work. If not, please try to modify the theme template instead.
  
-  * Update header4.phtml using the new "​welcomelogin"​ block+==== Modify ​the template ====
  
-  ​[app/​design/​frontend/​sm_market/​default/​template/​page/​html]#​ vi header4.phtml +Now that we have confirmed the issue, it is time to fix it by punching a hole for the login link block, since this is private information. 
-  <div class="​yt-header-top">​ +  * In your sm_market config, check the header style selection. For this site, it is ''​header4''​ 
-      <div class="​container">​ +  * Looking at the ''​header4.phtml''​ code, we can see that the logged in link is mixed into the header template. We need to take that logic out and put in its own block and template so that we can punch a hole for that block: 
-        <div class="​row">​ + 
-            <div class="​header-top-1 col-lg-4 col-md-4 col-sm-6 col-xs-12">​ +**[app/​design/​frontend/​sm_market/​default/​template/​page/​html]#​** ''​vi header4.phtml''​ 
-                <div class="​inner">​ + 
-                    <p class="​welcome-msg"><?​php echo $this->​getChildHtml('​welcome'​) ?> <?php echo $this->​getAdditionalHtml() ?></​p>​ +<​code>​ 
-                    <div class="​login-regis">​ +<div class="​yt-header-top">​ 
-                    <?php echo $this->getChildHtml('welcomelogin'); ?> + <div class="​container">​ 
-                    </​div>​ + <div class="​row">​ 
-                </​div>​ + <div class="​header-top-1 col-lg-4 col-md-4 col-sm-6 col-xs-12">​ 
-            </​div>​ + <div class="​inner">​ 
-            <div class="​header-top-2 col-lg-4 col-md-4 col-sm-6 col-xs-12">​ + <p class="​welcome-msg"><?​php echo $this->​getChildHtml('​welcome'​) ?> <?php echo $this->​getAdditionalHtml() ?></​p>​ 
-                <?php if($this->​getLayout()->​createBlock('​cms/​block'​)->​setBlockId('​v4-call-us'​)->​toHtml()) { ?> + <div class="​login-regis">​ 
-                    <?php echo $this->​getLayout()->​createBlock('​cms/​block'​)->​setBlockId('​v4-call-us'​)->​toHtml();​ ?> +    <?​php if(!$this->​helper('​customer'​)->​isLoggedIn() ){ ?> 
-                <?php } ?> + <​a title="​<?php echo $this->__("​Login"​);​ ?>" class="​btn-head"​ href="<?​php echo $this->​getUrl('customer/​account/​login/​') ?>">​ 
-            </​div>​ + <?​php echo $this->​__("​Login"​); ?> 
-            <div class="​header-top-3 col-lg-4 col-md-4 col-sm-6 col-xs-12">​ + </​a>​ 
-                <div class="​inner">​ + <?​php echo $this->​__("​or"​);​ ?> 
-                    <div class="​head-quicklink">​ + <?​php } else{ ?> 
-                        <?php echo $this->​getChildHtml('​topLinks'​);?>​ + <​a title="<?​php echo $this->​__("​Logout"​);​ ?>" class="​btn-head"​ href="<?​php echo $this->​getUrl('​customer/​account/​logout'​) ?>">​ 
-                    </​div>​ + <?​php echo $this->​__("​Logout"​);​ ?> 
-                </​div>​ + </​a>​ 
-            </​div>​ + <?​php } ?> 
-        </​div>​ + <?​php 
-      </​div>​+ if(!$this->​helper('​customer'​)->​isLoggedIn() ){ ?> 
 + <​a title="<?​php echo $this->​__("​Create an Account"​);​ ?>" class="​btn-head"​ href="<?​php echo Mage::​getBaseUrl();​ ?>​customer/​account/​create/">​ 
 + <?​php echo $this->​__("​Create an Account"​);​ ?></​a>​ 
 + <?​php } ?> 
 + </​div>​ 
 + </​div>​ 
 + </​div>​ 
 + <div class="​header-top-2 col-lg-4 col-md-4 col-sm-6 col-xs-12">​ 
 + <?php if($this->​getLayout()->​createBlock('​cms/​block'​)->​setBlockId('​v4-call-us'​)->​toHtml()) { ?> 
 + <?php echo $this->​getLayout()->​createBlock('​cms/​block'​)->​setBlockId('​v4-call-us'​)->​toHtml();​ ?> 
 + <?php } ?> 
 + </​div>​ 
 + <div class="​header-top-3 col-lg-4 col-md-4 col-sm-6 col-xs-12">​ 
 + <div class="​inner">​ 
 + <div class="​head-quicklink">​ 
 + <?php echo $this->​getChildHtml('​topLinks'​);?>​ 
 + </​div>​ 
 + </​div>​ 
 + </​div>​ 
 + </​div>​ 
 + </​div>​ 
 +</​div>​ 
 +</​code>​ 
 + 
 +  * Move the logic out of ''​header4.phtml''​ and put it in our new ''​welcomelogin.phtml'':​ 
 + 
 +**[app/​design/​frontend/​sm_market/​default/​template/​page/​html]#​** ''​vi welcomelogin.phtml''​ 
 +<​code>​ 
 +<?php if(!$this->​helper('​customer'​)->​isLoggedIn() ){ ?> 
 + <a title="<?​php echo $this->​__("​Login"​);​ ?>" class="​btn-head"​ href="<?​php echo $this->​getUrl('​customer/​account/​login/'​) ?>">​ 
 + <?php echo $this->​__("​Login"​);​ ?> 
 + </​a>​ 
 + <?php echo $this->​__("​or"​);​ ?> 
 +<?php } else{ ?> 
 + <a title="<?​php echo $this->​__("​Logout"​);​ ?>" class="​btn-head"​ href="<?​php echo $this->​getUrl('​customer/​account/​logout'​) ?>">​ 
 + <?php echo $this->​__("​Logout"​);​ ?> 
 + </​a>​ 
 +<?php } ?> 
 +<?php 
 +if(!$this->​helper('​customer'​)->​isLoggedIn() ){ ?> 
 + <a title="<?​php echo $this->​__("​Create an Account"​);​ ?>" class="​btn-head"​ href="<?​php echo Mage::​getBaseUrl();​ ?>​customer/​account/​create/">​ 
 + <?php echo $this->​__("​Create an Account"​);​ ?></​a>​ 
 +<?php } ?> 
 +</​code>​ 
 + 
 +  * Next we call the new block ''​welcomelogin'',​ modify the layout file, and add our new welcomelogin block inside the header block:  
 + 
 +**[app/​design/​frontend/​sm_market/​default/​layout]#​** ''​vi page.xml''​ 
 + 
 +<​code>​ 
 +<block type="​page/​html_header"​ name="​header"​ as="​header">​ 
 +…. 
 +…. After the welcome block 
 +<block type="​core/​template"​ name="​welcomelogin"​ as="​welcomelogin"​ template="​page/​html/​welcomelogin.phtml"/>​ 
 +</​code>​ 
 + 
 +  * Update ''​header4.phtml''​ using the new welcomelogin block: 
 + 
 +**[app/​design/​frontend/​sm_market/​default/​template/​page/​html]#​** ''​vi header4.phtml''​ 
 +<​code>​ 
 +<div class="​yt-header-top">​ 
 +  <div class="​container">​ 
 + <div class="​row">​ 
 + <div class="​header-top-1 col-lg-4 col-md-4 col-sm-6 col-xs-12">​ 
 + <div class="​inner">​ 
 + <p class="​welcome-msg"><?​php echo $this->​getChildHtml('​welcome'​) ?> <?php echo $this->​getAdditionalHtml() ?></​p>​ 
 + <div class="​login-regis">​ 
 +   <?php echo $this->​getChildHtml('​welcomelogin'​);​ ?>  
 + </​div>​ 
 + </​div>​ 
 + </​div>​ 
 + <div class="​header-top-2 col-lg-4 col-md-4 col-sm-6 col-xs-12">​ 
 + <?php if($this->​getLayout()->​createBlock('​cms/​block'​)->​setBlockId('​v4-call-us'​)->​toHtml()) { ?> 
 + <?​php echo $this->​getLayout()->​createBlock('​cms/​block'​)->​setBlockId('​v4-call-us'​)->​toHtml();​ ?> 
 + <?php } ?> 
 + </​div>​ 
 + <div class="​header-top-3 col-lg-4 col-md-4 col-sm-6 col-xs-12">​ 
 + <div class="​inner">​ 
 + <div class="​head-quicklink">​ 
 + <?​php echo $this->​getChildHtml('​topLinks'​);?>​ 
 + </​div>​ 
 + </​div>​ 
 + </​div>​ 
 + </​div>​
   </​div>​   </​div>​
-  ​ +</​div>​ 
-  * Go to the LiteMage Configuration section of your Magento admin panel and add "welcomelogin" ​to the Customized Block Names for "​welcome"​ Tag field.  ​+</​code>​ 
 + 
 +  * Go to the **LiteMage Configuration** section of your Magento admin panel and add ''​welcomelogin'' ​to the **Customized Block Names for "​welcome"​ Tag** field.  ​
  
-  * Flush LiteMage cache, and verify that "welcomelogin" ​has been properly hole punched.+  * Flush LiteMage cache, and verify that welcomelogin has been properly hole punched.
  • Admin
  • Last modified: 2017/01/30 18:50
  • by Michael Alegre