search

Izumiko / jellyfin-danmaku

Jellyfin danmaku extension
MIT License
130 stars 12 forks source link

[BUG] #36

Closed Xarth-Mai closed 6 months ago

Izumiko commented 6 months ago

你再看看，发现之前手滑打错单词了

Izumiko commented 6 months ago

改这段的时候是为了改善一下OVA相关的搜索和显示，正常的TVSeries忘记测了

Izumiko commented 6 months ago

我先把发送界面那个回滚了吧，没设计好

Izumiko commented 6 months ago

开了呀

Izumiko commented 3 months ago

那估计是我这个 b6f058a 没修完全。看样子不应该直接destory弹幕对象，我把wrapper隐藏了一下，然后让后续的函数重新创建弹幕对象吧。 4c839a3 这个看着貌似可以了

Izumiko commented 2 months ago

image

这个DanDanPlay的弹幕和B站野生视频的弹幕居然也有重复 现在这个版本看轻音少女经常遇到这种三条重复的弹幕/(ㄒoㄒ)/~~

也测了下JOJO的第一季,也有这样额外的两个野生资源,但是没有重复的弹幕

这是看轻音时获取到的源,重复的都来自后两个野生资源:

[
    {
        "url": "https://www.bilibili.com/bangumi/play/ep21286",
        "shift": 0
    },
    {
        "url": "http://www.bilibili.com/video/av1700631/index_8.html",
        "shift": 0
    },
    {
        "url": "http://www.bilibili.com/video/av29176/index_8.html",
        "shift": 0
    }
]

捣鼓了半天好像只有给这种野生资源单独加个开关能比较简单解决

没想到不同来源不仅cid不一样，连用户名都不一样。修改了一下重复弹幕的判定，去掉了用户名 d9bba73162d39f85a191fcfc808b0239f5e03bea ，现在只用时间,模式,颜色,内容作为key

Xarth-Mai commented 2 months ago

没想到不同来源不仅cid不一样，连用户名都不一样。修改了一下重复弹幕的判定，去掉了用户名 d9bba73162d39f85a191fcfc808b0239f5e03bea ，现在只用时间,模式,颜色,内容

源数据确实很复杂，如我上面发的图，有个源的时间和另两个源差了一秒 甚至一条来自B站一条来自弹弹😨

Xarth-Mai commented 3 days ago

弹幕来源过滤出现了BUG 好像是这个提交https://github.com/Izumiko/jellyfin-danmaku/commit/0e5a3b008783c9cecfacd75a002d3d062fc8c29a

const user = comment.p.split(',')[3];
const modeId = parseInt(comment.p.split(',')[1], 10);
return !danmakuFilteRule.test(comment.m) && enabledMode.includes(modeId);

最后一行应该是

return !danmakuFilteRule.test(user) && enabledMode.includes(modeId);

原先这里定义了一个user却没有引用🤔