Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

背景音乐数量显示错误：未校验是否是音乐文件 #5762

Open
1 task done
duoduo13123678 opened this issue Feb 28, 2025 · 11 comments · May be fixed by #5798
Open
1 task done

背景音乐数量显示错误：未校验是否是音乐文件 #5762

duoduo13123678 opened this issue Feb 28, 2025 · 11 comments · May be fixed by #5798
Labels
· Bug 可合并 已有 Pull Request 处理了该 Issue

Comments

@duoduo13123678
Copy link

检查项

描述

资源管理器显示PCL2的音乐文件夹只有13首音乐，但是PCL2显示有19首，无论怎么刷新都没用，甚至更新了启动器都一样，以下是截图和Log

Image

重现步骤

  1. 添加/删除一些音乐（我是多次删除了很多音乐出现的）
  2. 刷新背景音乐，可能会出现该情况

日志与附件

Log2.txt
Log3.txt
Log4.txt
Log5.txt
Log1.txt
@duoduo13123678 duoduo13123678 added · Bug 新提交 需要社区初步确认其有效的新提交 labels Feb 28, 2025
@wyc-26
Copy link

wyc-26 commented Feb 28, 2025

需要 Author 提供音乐文件列表

[!TIP]
ogg 格式会被自动移出播放列表

人家是显示的比实际的多……

@shimoranla
Copy link
Collaborator

PCL2/Plain Craft Launcher 2/Pages/PageSetup/PageSetupUI.xaml.vb

Lines 354 to 358 in 503244a

If MusicAllList.Any Then
PanMusicVolume.Visibility = Visibility.Visible
PanMusicDetail.Visibility = Visibility.Visible
BtnMusicClear.Visibility = Visibility.Visible
CardMusic.Title = "背景音乐（" & EnumerateFiles(Path & "PCL\Musics\").Count & " 首）"

可能是把非音乐文件算进去了.....

@shimoranla
Copy link
Collaborator

找了最新快照版，验证了我的猜想

Image

Image

Author 的情况可能是因为音乐图和缩略图+文件夹配置文件导致计算不准确

@MoYuan-CN
Copy link
Collaborator

背景音乐数量显示错误：未校验是否是音乐文件

@MoYuan-CN MoYuan-CN added 等待确认 已经过社区确认，等待开发者确认 and removed 新提交 需要社区初步确认其有效的新提交 labels Feb 28, 2025
@LTCatt LTCatt changed the title 背景音乐数量非真实数量 背景音乐数量显示错误：未校验是否是音乐文件 Feb 28, 2025
@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟩 低 优先度：低 and removed 等待确认 已经过社区确认，等待开发者确认 labels Feb 28, 2025
@lactobionicAcid
Copy link
Contributor

背景图片亦是如此，并且 PCL 还会尝试读取不是图片的文件，然后报错。

Image

@duoduo13123678
Copy link
Author

兄弟们我发现了，有几个系统级别隐藏文件（音乐的缩略图），不是PCL2的锅，但也希望PCL能判断一下文件格式，不是音乐就跳过

@HexDragon-Bot HexDragon-Bot closed this as not planned Won't fix, can't repro, duplicate, stale Mar 2, 2025
@HexDragon-Bot HexDragon-Bot added 忽略 因为各种杂项原因而关闭 and removed 处理中 开发者正在调查或处理该项 🟩 低 优先度：低 labels Mar 2, 2025
@wyc-26
Copy link

wyc-26 commented Mar 2, 2025

那你也别关啊……

@Deep-Dark-Forest
Copy link
Collaborator

兄弟们我发现了，有几个系统级别隐藏文件（音乐的缩略图），不是PCL2的锅，但也希望PCL能判断一下文件格式，不是音乐就跳过

让龙猫看看吧

@wyc-26
Copy link

wyc-26 commented Mar 2, 2025

把非音乐文件算进去确实是 PCL 的 Bug，也不止你一个人遇到了这个问题

@wyc-26
Copy link

wyc-26 commented Mar 2, 2025

兄弟们我发现了，有几个系统级别隐藏文件（音乐的缩略图），不是PCL2的锅，但也希望PCL能判断一下文件格式，不是音乐就跳过

让龙猫看看吧

龙猫都已经加了处理中了……

@Deep-Dark-Forest Deep-Dark-Forest added 处理中 开发者正在调查或处理该项 🟩 低 优先度：低 and removed 忽略 因为各种杂项原因而关闭 labels Mar 2, 2025
@duoduo13123678
Copy link
Author

抱歉我不应该close的（哭）

@Ciilu Ciilu linked a pull request Mar 7, 2025 that will close this issue
Open
@LTCatt LTCatt removed the 处理中 开发者正在调查或处理该项 label Mar 8, 2025
@LTCatt LTCatt added 可合并 已有 Pull Request 处理了该 Issue and removed 🟩 低 优先度：低 labels Mar 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
· Bug 可合并 已有 Pull Request 处理了该 Issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

解决 #5762 ：仅合法音乐纳入背景音乐数量统计 Ciilu/PCL2
8 participants
@LTCatt @MoYuan-CN @HexDragon-Bot @lactobionicAcid @Deep-Dark-Forest @duoduo13123678 @wyc-26 @shimoranla