fix: search does not find the contents of the table

[sy-records]

Summary
The previous commit #1015 only fixed the situation where there was no content before the table. like this:

## Errno List

| C Name  | Value | Description       |
| ------- | ----- | ----------------- |
| Success | 0     | Success           |
| EPERM   | 1     | Operation not per |

But this one below is not searchable.

## Errno List

Here are some descriptions....

| C Name  | Value | Description       |
| ------- | ----- | ----------------- |
| Success | 0     | Success           |
| EPERM   | 1     | Operation not per |

What kind of change does this PR introduce? (check at least one)

• Bugfix
• Feature
• Code style update
• Refactor
• Docs
• Build-related changes
• Other, please describe:

If changing the UI of default theme, please provide the before/after screenshot:

Does this PR introduce a breaking change? (check one)

• Yes
• No

If yes, please describe the impact and migration path for existing applications:

The PR fulfills these requirements:

• When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)

You have tested in the following browsers: (Providing a detailed version will be better.)

• Chrome
• Firefox
• Safari
• Edge
• IE

If adding a new feature, the PR's description includes:

• A convincing reason for adding this feature
• Related documents have been updated
• Related tests have been updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.

Other information:

---

• DO NOT include files inside lib directory.

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/docsify-core/docsify-preview/qfd7yuok4
✅ Preview: https://docsify-preview-git-fork-sy-records-fix-search.docsify-core.now.sh

[anikethsaha]

Thanks for the PR.

Can you check the commit message format

[sy-records]

Is now okay? @anikethsaha

[anikethsaha]

remove the scope, search , currenlty we dont have scope.

Also, change the commit message as well.

[Koooooo-7]

Good Job. 👍

Litte suggestion.
I think this block could refine.

 if (!token.text && token.type === 'table') {
          token.text = token.cells
            .map(function(rows) {
              return rows.join(' | ');
            })
            .join(' |\n ');
        }

        index[slug].body += '\n' + (token.text || '');
      } else {
        if (!token.text && token.type === 'table') {
          token.text = token.cells
            .map(function(rows) {
              return rows.join(' | ');
            })
            .join(' |\n ');
        }

        index[slug].body = index[slug].body
          ? index[slug].body + token.text
          : token.text;
      }

what the the difference is solving the index[slug].body.
it seems too much duplicate here.

if (!token.text && token.type === 'table') {
          token.text = token.cells
            .map(function(rows) {
              return rows.join(' | ');
            })
            .join(' |\n ');
        }

[Koooooo-7]

and I prefer to use return in some cases direcly to avoid useing more if else branch.
Idk whether it should be applied to docsify.
WDYT ?

[sy-records]

@Koooooo-7 Is there a problem with using return in forEach?

docsify/src/plugins/search/search.js

Lines 79 to 112 in d0ecc42

	tokens.forEach(token => {
	if (token.type === 'heading' && token.depth <= depth) {
	const { str, config } = getAndRemoveConfig(token.text);
	if (config.id) {
	slug = router.toURL(path, { id: slugify(config.id) });
	} else {
	slug = router.toURL(path, { id: slugify(escapeHtml(token.text)) });
	}
	index[slug] = { slug, title: str, body: '' };
	} else {
	if (!slug) {
	return;
	}
	if (!index[slug]) {
	index[slug] = { slug, title: '', body: '' };
	} else if (index[slug].body) {
	token.text = getTableData(token);
	index[slug].body += '\n' + (token.text || '');
	} else {
	token.text = getTableData(token);
	index[slug].body = index[slug].body
	? index[slug].body + token.text
	: token.text;
	}
	}
	});
	slugify.clear();
	return index;
	}

nah, it's okay.
I wonder how to make it simple（cuz there r too much if else branch.）:100:
never mind.

[anikethsaha]

is there any issue which is being fixed by this PR ? feel free to close that one

[sy-records]

No relevant issue. I found the problem, fixed it by the way.

[anikethsaha]

great