Wing icon indicating copy to clipboard operation
Wing copied to clipboard
verified publisher icon Tokinx

feat: 添加了首页文章列表置顶功能

Open moxiaonai opened this issue 2 years ago • 2 comments

如题：添加了首页文章列表置顶功能，效果如下图

image

moxiaonai avatar Feb 26 '23 14:02 moxiaonai

非常感谢这个PR，但是代码执行效率似乎有待提升与优化。 我认为应该从order入手将sticky的文章排到最上面，而不是分两次查询将结果合并。

再次感谢你的支持与贡献，此PR可能暂时不会合并到主分支。

Tokinx avatar Feb 27 '23 06:02 Tokinx

非常感谢这个PR，但是代码执行效率似乎有待提升与优化。 我认为应该从order入手将sticky的文章排到最上面，而不是分两次查询将结果合并。

再次感谢你的支持与贡献，此PR可能暂时不会合并到主分支。

@Tokinx 是的，按排序处理一次查询应该是最优解，wp默认排序置顶应该会返回在最上面，但是因为现在的代码已经有了orderby=>date，我试着调整了现在的自定义查询条件，试了一下没有实现，最后没敢大改怕影响到其他统计， 上面的方案是参考了dux的做法，他的方案是为了满足自定义置顶id，放在这里确实不是最合适的，期待大佬有更优雅的方式

moxiaonai avatar Feb 28 '23 02:02 moxiaonai