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

NOJIRA: fix src mime type #1

Open
wants to merge 1 commit into
base: 3.0.2p1
Choose a base branch
from
Open

Conversation

maneman8000
Copy link

https://kaizen.qiita.com/maneman8000/items/d8761fd893e49aeeba0f
こちらの内容の修正のためのPRです

@glidenote
Copy link
Member

これバグなら本家にPRを出してmergeしてもらうのが本来は良さそうですね。

https://github.com/thoughtbot/paperclip/pulls?utf8=%E2%9C%93&q=is%3Apr
を見るとPRだしてもmergeされるまでかなり時間かかりそうなので、
今回はこの対応で行くのが良さそうですが。

@maneman8000
Copy link
Author

@glidenote はい。そうなのですが使っているバージョンが古いのでなかなかmergeしてもらえない気がしてます・・・

@maneman8000
Copy link
Author

確認すべきテーブル&カラム一覧

BP/K2 table column エラーあり?
K2 v2_site_scripts file_content_type あり
K2 v2_projects image_content_type -
K2 v2_variation_versions image_content_type -
K2 v2_attachments file_content_type -
K2 users avatar_image_content_type -
BP bp_offer_reports jst_csv_file_content_type, utc_csv_file_content_type, pst_csv_file_content_type -
BP domains js_content_type あり
BP offers image_original_content_type -
BP offer_sources file_content_type -
BP posts image_content_type -
BP post_sources file_content_type -
BP tmp_uploads file_content_type あり (.ppt, .rtf)

@maneman8000
Copy link
Author

distinct した結果(エラーが出てないもののみ)

  • v2_projects.image_content_type
image/png
image/jpeg
image/bmp
  • v2_variation_versions.image_content_type
image/png
image/jpeg
image/gif
  • v2_attachments.file_content_type
image/jpeg
image/png
application/pdf
application/vnd.openxmlformats-officedocument.presentationml.presentation
image/gif
application/zip
application/vnd.openxmlformats-officedocument.spreadsheetml.sheet
image/vnd.adobe.photoshop
application/postscript
application/octet-stream
text/plain
audio/x-scpls
application/x-stuffit
text/richtext
application/x-photoshop
application/vnd.ms-powerpoint
application/x-zip-compressed
text/rtf
image/x-photoshop
application/msword
image/svg+xml
text/javascript
image/psd
application/vnd.openxmlformats-officedocument.wordprocessingml.document
application/vnd.ms-excel
  • users.avatar_image_content_type
image/jpeg
image/png
image/gif
  • bp_offer_reports.jst_csv_file_content_type, utc_csv_file_content_type, pst_csv_file_content_type
text/plain
  • offers.image_original_content_type
image/png
image/jpeg
image/gif
  • offer_sources.file_content_type
image/jpeg
image/png
application/zip
application/pdf
application/vnd.openxmlformats-officedocument.spreadsheetml.sheet
application/vnd.openxmlformats-officedocument.presentationml.presentation
image/vnd.adobe.photoshop
text/plain
application/postscript
  • posts.image_content_type
image/jpeg
image/png
image/gif
  • post_sources.file_content_type
image/jpeg
image/png
image/gif
image/svg+xml
image/tiff
text/plain
image/x-bmp
image/vnd.adobe.photoshop
  • tmp_uploads.file_content_type

( length < 251 のもののみ)

image/jpeg
application/zip
application/pdf
image/gif
image/png
text/plain
application/vnd.openxmlformats-officedocument.presentationml.presentation
application/octet-stream
application/x-apple-diskimage
application/postscript
application/vnd.openxmlformats-officedocument.spreadsheetml.sheet
image/vnd.adobe.photoshop

@maneman8000
Copy link
Author

ローカル環境にてざっと動作確認完了しました

  • スクリーンショット再キャプチャ

image/png => OK

  • スクリーンショット手動アップデート

image/jpeg => OK
image/png => OK
image/gif => OK
image/bmp => OK

  • アタッチメント

application/zip => OK
application/pdf => OK
application/vnd.openxmlformats-officedocument.spreadsheetml.sheet => OK
application/vnd.openxmlformats-officedocument.presentationml.presentation => OK
text/rtf => OK
text/javascript => OK
image/svg+xml => ローカル環境の問題でNG(本番はOK。ローカルで戻して実行はNG)

  • sitejs 更新

application/javascript => OK

  • distjs 更新

application/javascript => OK

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

Successfully merging this pull request may close these issues.

2 participants