Molunerfinn / PicGo

:rocket:A simple & beautiful tool for pictures uploading built by vue-cli-electron-builder
https://molunerfinn.com/PicGo/
MIT License
23.86k stars 2.22k forks source link

[Bug]: 未打开“时间戳重命名”,从剪贴板上传时自动出现时间戳重命名 #1131

Closed zhang-stephen closed 1 year ago

zhang-stephen commented 1 year ago

前置阅读 | Pre-reading

PicGo的版本 | PicGo Version

v2.4.0-beta.1

系统信息 | System Information

Windows

问题重现 | Bug reproduce

  1. 复制图片内容
  2. PicGo上传图床为sm.ms,配置如图 image
  3. 在主界面点击“剪贴板上传”或者使用快捷键上传
  4. PicGo行为如图(Win11 Bug, 截图/录屏的亮度会被异常提高): bug

相关日志 | Logs

2023-05-19 14:22:40 [PicGo INFO] Before transform 
2023-05-19 14:22:40 [PicGo INFO] Transforming... Current transformer is [path] 
2023-05-19 14:22:40 [PicGo INFO] Before upload 
2023-05-19 14:22:40 [PicGo INFO] beforeUploadPlugins: renameFn running 
2023-05-19 14:22:50 [PicGo INFO] Uploading... Current uploader is [smms] 
2023-05-19 14:22:53 [PicGo SUCCESS] 
https://s2.loli.net/2023/05/19/vnT4aUOGZkHLIQz.png 
2023-05-19 14:29:26 [PicGo INFO] Before transform 
2023-05-19 14:29:26 [PicGo INFO] Transforming... Current transformer is [path] 
2023-05-19 14:29:27 [PicGo INFO] Before upload 
2023-05-19 14:29:27 [PicGo INFO] beforeUploadPlugins: renameFn running 
2023-05-19 14:29:35 [PicGo INFO] Uploading... Current uploader is [smms] 
2023-05-19 14:29:38 [PicGo SUCCESS] 
https://s2.loli.net/2023/05/19/R7QWBgPhxE6aomu.png 
Molunerfinn commented 1 year ago

对 picgo 来说,剪贴板图片统一是没有名字的(比如你截了一张图,这个时候又没保存成文件,哪来的文件名?)因此会统一用时间戳命名。这个不是 bug,是设计如此,而且从 picgo 有剪贴板图片上传以来一直如此。

zhang-stephen commented 1 year ago

对 picgo 来说,剪贴板图片统一是没有名字的(比如你截了一张图,这个时候又没保存成文件,哪来的文件名?)因此会统一用时间戳命名。这个不是 bug,是设计如此,而且从 picgo 有剪贴板图片上传以来一直如此。

原来如此,但是在打开“上传前重命名图片”选项之后,这个动作的存在就显得不是那么合理了,此时不应该以用户操作为准吗? 我认为在不打开此选项的情况下,自动应用时间戳是合理的,打开之后应该等待用户输入为准,可以作为一个enhancement吗?

Molunerfinn commented 1 year ago

因为在 picgo 层面,所有的截图和普通图片上传都需要从本地一个具体的路径上传。只不过截图的话, picgo 会先将其存到 picgo 的一个临时目录下,这个时候落地的文件名已经是带有时间戳的名字。所以对于「上传前重命名」的内置插件来说,它获取的只是这个文件本身的名字而已,并不知道是截图落地的图片还是普通图片。

Molunerfinn commented 1 year ago

这里的 bug 其实是打开重命名弹窗应该立即显示文件名而不是等了一会才显示。另外后续的优化会让你鼠标点击的时候直接激活完整的文件名,这样如果不满意的文件名可以直接一键删除,但是肯定还是会显示默认的文件名

bug 放到这个 issue 处理:https://github.com/Molunerfinn/PicGo/issues/1130