Skip to content

スマホ対応版の動画配信モジュールをコミット#2

Merged
kteraguchi merged 2 commits intonetcommons:developfrom
go-to:develop
Dec 5, 2013
Merged

スマホ対応版の動画配信モジュールをコミット#2
kteraguchi merged 2 commits intonetcommons:developfrom
go-to:develop

Conversation

@go-to
Copy link
Collaborator

@go-to go-to commented Aug 28, 2013

・動画配信モジュールのスマホ対応
・IE以外でチャンネル編集リンクをクリックした際にブロックが表示されなくなるバグを修正

goto added 2 commits August 28, 2013 16:16
IE以外でチャンネル編集リンクをクリックした際に、ブロックが表示されなくなるバグを修正
@kteraguchi
Copy link
Collaborator

動画ファイルアップロードができなくなってます。

multimedia/validator/Validator_ItemUpload.class.php
48行目あたりの、ffmpegライブラリの存在チェックが変更されています。

multimediaAction->hasFfmpegLibでffmpeg-phpのロードチェックを行っていたのですが、
multimediaAction->hasFfmpegでffmpeg実行ファイルの存在チェックに変わっています。

これは、ffmpeg-phpを利用せず、ffmpeg実行ファイルを直接利用することに変更したという意図でしょうか?

既に動作している環境でも、動作するようにしたいと思っています。

@nagasheep
Copy link
Collaborator

ffmpeg-php を使用しないようにしたことは確かです。
既に動作している環境で問題になるケースとしてはどういうものでしょう。

定数で定義している MULTIMEDIA_FFMPEG_PATH の値が異なる場合とか?

@kteraguchi
Copy link
Collaborator

そうです。
MULTIMEDIA_FFMPEG_PATHが異なっています。

こっちの環境では
/usr/lib64/php/modules
にffmpeg.soがあります。

@kteraguchi
Copy link
Collaborator

ffmpeg.soは関係ないですね。
間違えました。

ffmpegは
/usr/local/bin/
にあります。

MULTIMEDIA_FFMPEG_PATHを/usr/local/bin/ffmpegに変更して試したところ、
MP4Boxが見つからないと表示されました。

MP4Boxも必須ということでしょうか?

@nagasheep
Copy link
Collaborator

永原@携帯です。
MP4Boxは必須です。
今回、iOS系でも動くように、mp4に変換します。
で、擬似ストリーミングするために、動画のインデックスをファイルの先頭に移動する必要があります。
MP4Boxはそのインデックス移動のために必要なんです。

@go-to
Copy link
Collaborator Author

go-to commented Aug 30, 2013

出遅れました。

動画モジュールを動かすのに必要な環境が、
ffmpeg+ffmpeg-phpの組み合わせから、
ffmpeg+MP4Boxの組み合わせに変更になっています。

なので、その旨をどこかに記載しないと、現状の動画モジュールを使っている方がアップデートした際に、
動かない!という状況に陥ってしまいそうですね。

注意書きは、どこに入れましょう?
動画モジュールのディレクトリの中にREADME的なファイルでも追加しますか。

@kteraguchi
Copy link
Collaborator

了解しました。

MP4Boxの環境を作って動作確認してからリリースしようと考えております。
注意書きに関しては、リリースする際にアナウンスします。

また、既存利用者のために、
チャンネル編集リンクのバグ修正のみマージしたバージョンと、
スマホ対応したバージョンを分けたいと思っています。

チャンネル編集リンクのバグ修正のみを抽出できますでしょうか?

@nagasheep
Copy link
Collaborator

永原です。
今気づいたのですが、動画モジュール、モバイルのメニューに出てこないですね。
モバイル対応ということでは、まだ足りないかな。

kteraguchi pushed a commit that referenced this pull request Dec 5, 2013
Conflicts:
	multimedia/CHANGES.txt

#2 一旦プルリクエスト分をマージします。
既知の問題については、改めてissueに登録して対応していきましょう。
@kteraguchi kteraguchi merged commit a590959 into netcommons:develop Dec 5, 2013
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.

3 participants