Merge pull request #26 from daniel-rose/add-menu-item-by-config

show/hide top menu item by configuration

Author: magefan

Observer/PageBlockHtmlTopmenuBethtmlBeforeObserver.php

@@ -16,17 +16,34 @@
  */
 class PageBlockHtmlTopmenuBethtmlBeforeObserver implements ObserverInterface
 {
+    /**
+     * Show top menu item config path
+     */
+    const XML_PATH_TOP_MENU_SHOW_ITEM = 'mfblog/top_menu/show_item';
+
+    /**
+     * Top menu item text config path
+     */
+    const XML_PATH_TOP_MENU_ITEM_TEXT = 'mfblog/top_menu/item_text';
+
     /**
      * @var \Magefan\Blog\Model\Url
      */
     protected $_url;
 
+    /**
+     * @var \Magento\Framework\App\Config\ScopeConfigInterface
+     */
+    protected $_scopeConfig;
+
     /**
      * @param \Magefan\Blog\Model\Url $url
      */
     public function __construct(
-        \Magefan\Blog\Model\Url $url
+        \Magefan\Blog\Model\Url $url,
+        \Magento\Framework\App\Config\ScopeConfigInterface $scopeConfig
     ) {
+        $this->_scopeConfig = $scopeConfig;
         $this->_url = $url;
     }
 
@@ -39,19 +56,22 @@ public function __construct(
      */
     public function execute(\Magento\Framework\Event\Observer $observer)
     {
+        if (!$this->_scopeConfig->isSetFlag(static::XML_PATH_TOP_MENU_SHOW_ITEM, \Magento\Store\Model\ScopeInterface::SCOPE_STORE)) {
+            return;
+        }
+
         /** @var \Magento\Framework\Data\Tree\Node $menu */
         $menu = $observer->getMenu();
         $block = $observer->getBlock();
 
         $tree = $menu->getTree();
         $data = [
-            'name'      => __('Blog'),
+            'name'      => $this->_scopeConfig->getValue(static::XML_PATH_TOP_MENU_ITEM_TEXT, \Magento\Store\Model\ScopeInterface::SCOPE_STORE),
             'id'        => 'magefan-blog',
             'url'       => $this->_url->getBaseUrl(),
             'is_active' => ($block->getRequest()->getModuleName() == 'blog'),
         ];
         $node = new Node($data, 'id', $tree, $menu);
         $menu->addChild($node);
-        return $this;
     }
 }

etc/adminhtml/system.xml

@@ -275,6 +275,16 @@
                     <label>AddThis Language</label>
                 </field>
             </group>
+            <group id="top_menu" translate="label" type="text" sortOrder="90" showInDefault="1" showInWebsite="1" showInStore="1">
+                <label>Top Menu</label>
+                <field id="show_item" translate="label comment" type="select" sortOrder="10" showInDefault="1" showInWebsite="1" showInStore="1">
+                    <label>Show Item</label>
+                    <source_model>Magento\Config\Model\Config\Source\Yesno</source_model>
+                </field>
+                <field id="item_text" type="text" translate="label" sortOrder="20" showInDefault="1" showInWebsite="1" showInStore="1">
+                    <label>Item Text</label>
+                </field>
+            </group>
         </section>
     </system>
 </config>

etc/config.xml

@@ -75,6 +75,10 @@
                 <add_this_pubid>ra-4def63cd3cf37569</add_this_pubid>
                 <add_this_language>en</add_this_language>
             </social>
+            <top_menu>
+                <show_item>0</show_item>
+                <item_text>Blog</item_text>
+            </top_menu>
         </mfblog>
     </default>
 </config>