Fix submenu when path name is same

[jiangtj]

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines.
• Tests for the changes have been added (for bug fixes / features).
  • Muse | Mist have been tested.
  • Pisces | Gemini have been tested.
• Docs have been added / updated (for bug fixes / features).

PR Type

What kind of change does this PR introduce?

• Bugfix.
• Feature.
• Code style update (formatting, local variables).
• Refactoring (no functional changes, no api changes).
• Build related changes.
• CI related changes.
• Documentation content changes.
• Other... Please describe:

What is the current behavior?

When submenu path name is same, submenu2 is rendered twice

Issue Number(s): N/A

What is the new behavior?

• Screens with this changes: N/A
• Link to demo site with this changes: N/A

Does this PR introduce a breaking change?

• Yes.
• No.

[ivan-nginx]

Please, test it with this configuration:

menu:
  News: / || info-circle
  Docs:
    default: /docs/ || book
    Getting Started: /getting-started/ || flag
    Theme Settings: /theme-settings/ || star
    Third Party Services:
      default: /third-party-services/ || plug
      Algolia Search: /algolia-search/ || adn
    Tag Plugins: /tag-plugins/ || rocket
    Advanced Settings: /advanced-settings/ || rocket
    FAQ's: /faqs/ || life-ring
    Platforms: /platforms/ || retweet

https://github.com/theme-next/theme-next.org/blob/57db8ec03bc6da495ea94de733c27cc297988568/_config.yml#L158-L188

[jiangtj]

@ivan-nginx
Tested.
Project https://gitlab.com/JiangTJ/next-test deploy twice branch this and menu-external-url

• Muse https://jiangtj.gitlab.io/next-test/muse
• Mist https://jiangtj.gitlab.io/next-test/mist
• Pisces https://jiangtj.gitlab.io/next-test/pisces
• Gemini https://jiangtj.gitlab.io/next-test/gemini

[ivan-nginx]

@jiangtj here is bug founded: go to test sub-menu and then push second more breadcrumbs – will be 404 error.

BTW, breadcrumbs file generated here:

hexo-theme-next/layout/_partials/breadcrumb.swig

Lines 1 to 23 in 87f515c

	{% set paths = page.path.split('/') %}
	{% set count = paths.length %}
	{% if count > 2 %}
	{% set current = 0 %}
	{% set link = '' %}
	<ul class="breadcrumb">
	{% for path in paths %}
	{% set current += 1 %}
	{% if path != 'index.html' %}
	{% if current == count - 1 %}
	<li>{{ path | upper }}</li>
	{% else %}
	<li><a href="{{ link }}{{ url_for(path) }}/">{{ path | upper }}</a></li>
	{% if link == '' %}
	{% set link = '/' + path %}
	{% else %}
	{% set link += '/' + path %}
	{% endif %}
	{% endif %}
	{% endif %}
	{% endfor %}
	</ul>
	{% endif %}

[jiangtj]

May need change position
I try to fix it

[ivan-nginx]

Here it's description for subdirectory: https://github.com/hexojs/hexo-starter/blob/f04af381af085de6d3f19518efa433c585791688/_config.yml#L14-L17

And, yes, without subdirectory it's worked (on localhost).

[jiangtj]

@ivan-nginx Okay! Fixed.

[ivan-nginx]

Yeah? GJ and HF, dude! 👍