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.

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

Feature/saver/download/1 #32

Open
wants to merge 9 commits into
base: devel
Choose a base branch
from

Conversation

bluelovers
Copy link

修正檔案名稱 避免出現圖檔的副檔名為 .dat 或者 .php 這種狀況

@danny0838
Copy link
Owner

這算是積存已久的問題了,理想上要充分確認目標檔案格式,應該要抓連結的 header 分析,但如此一來,原來的副檔名偵測機制可能也要改成辨識 MIME,總之是個大工程。

感謝您的熱情。但目前暫不傾向用這方式處理問題,尤其圖檔格式很多種,一律預設為 .jpg 也不太妥當。

另外,有注意到您習慣調整排版,其實個人也不太喜歡原來 ScrapBook 程式碼的排版,但考量比對程式碼時減少不必要的更動,暫時傾向不改動,除了像改寫整個函式時順便把排版整個改過來。如果要改,也建議將單純改排版/優化程式碼的部分獨立為一個 commit。

@bluelovers
Copy link
Author

個人感覺加上副檔名 總比看見一堆 .dat 好
從 檔案本身內容來辨識附檔名的話 太過複雜了
這個修正也只更動到附檔名為 .dat 的
另外為了辨識是否是自動補上副檔名,所以是以附加在 .dat 之後 來做處理
這樣一來如果有需要對檔案的真正副檔名作處理時 只需要搜尋檔名內有 .dat. 的檔案

就算是日後增加功能 或者 有另外的附加功能實現判斷檔名時 仍然可以輕鬆過濾來做處理

所以還是希望能夠加上這個功能xd

我盡量已經沒有更動到原有的排版了
如果有大概是編輯器的自動修剪行尾空白導致的

@danny0838
Copy link
Owner

我瞭解您的考量,不過若要自動化處理,一般會從 index.html 出發去掃描各節點及連結的檔案,副檔名不是太大的問題;且若本來連結的檔名就是如此,也未用 header 改寫,強制修改副檔名反而造成失真。

其他問題包括:

  • 圖檔格式很多,不宜一律視作 jpg
  • 副檔名只偵測 dat, php, htm 也不夠,還有 html 或動態網頁常見的 cgi, asp, jsp, ...... 族繁不及備載

您的做法稍嫌複雜,且仍不夠充分解決整個相關問題,加上會永久影響資料且有失真疑慮,因此在有完整解決方案以前會先維持現狀,還望理解。

@bluelovers
Copy link
Author

做成選項如何? 可選擇是否開啟這個功能 也可自行設定要偵測那些副檔名

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants