fix(nerv-server): extend the method of renderVodeToString to support void and array vnode #158

huyang2229 · 2020-04-05T03:47:02Z

这个 PR 做了什么? (简要描述所做更改)
使用Taro时，当用包裹多个条件表达式时，这时输出的快照不正确，原因在于Block包裹的子元素的vnode为Array类型且没有vType，而nerv-server的renderVnodeToString没有对array类型的Vnode进行处理，具体截图如下：

打了断点，相应的输出截图如下所示：

为了解决上述问题，在nerv-server库中的renderVnodeToString方法中增加了对Array类型vnode的支持，此外对为覆盖到的void类型的vnode也进行了处理，避免snapshot中出现undefined字眼引起困惑。

对变动增加了单测用例，并确保提交前所有测试用例均正常通过

这个 PR 是什么类型? (至少选择一个)

错误修复(Bugfix) issue id #
新功能(Feature)
代码重构(Refactor)
TypeScript 类型定义修改(Typings)
文档修改(Docs)
代码风格更新(Code style update)
其他，请描述(Other, please describe):

这个 PR 满足以下需求:

提交到 master 分支
Commit 信息遵循 Angular Style Commit Message Conventions
所有测试用例已经通过
代码遵循相关包中的 .eslintrc, .tslintrc, .stylelintrc 所规定的规范
在本地测试可用，不会影响到其它功能

coveralls · 2020-04-05T03:52:36Z

Coverage remained the same at 94.294% when pulling 96593ea on huyang2229:master into 69e5c1e on NervJS:master.

fix(nerv-server): support void and array vnode renderToString

Loading status checks…

96593ea

Nov	DEC	Jan
	05
2019	2020	2021

NervJS / nerv

fix(nerv-server): extend the method of renderVodeToString to support void and array vnode #158

fix(nerv-server): extend the method of renderVodeToString to support void and array vnode #158

huyang2229 commented Apr 5, 2020 •

edited

coveralls commented Apr 5, 2020

NervJS / nerv

Join GitHub today

GitHub is where the world builds software

fix(nerv-server): extend the method of renderVodeToString to support void and array vnode #158

fix(nerv-server): extend the method of renderVodeToString to support void and array vnode #158

Conversation

huyang2229 commented Apr 5, 2020 • edited

coveralls commented Apr 5, 2020

Essential cookies

Always active

Analytics cookies

huyang2229 commented Apr 5, 2020 •

edited