Skip to content

[plugin]video をPHP7環境で動作させる #60#61

Merged
big2men merged 2 commits into
open-qhm:masterfrom
big2men:feature/#60-fix-video-plugin
Apr 5, 2017
Merged

[plugin]video をPHP7環境で動作させる #60#61
big2men merged 2 commits into
open-qhm:masterfrom
big2men:feature/#60-fix-video-plugin

Conversation

@big2men

@big2men big2men commented Apr 5, 2017

Copy link
Copy Markdown
Member

このプルリクエストは何なのか?

connected to #60

  • PHP7.1で video プラグインが動作しない
  • エラー発生箇所は文字列を格納した変数に対して配列操作を行っていた箇所(どうしてこうなった)
    Uncaught Error: [] operator not supported for strings  in /path/to/qhm/plugin/video.inc.php:252
    
  • 普通に配列で初期化して動作確認した

どうやってテストすればいいのか?

PHP7.1 で動作確認すること。

  1. QHMにログインする
  2. FrontPage を編集
  3. どこかに #video(hoge.mp4) とページに書く(動画ファイルは無くてもOK)
  4. 保存後、表示できればOK

タスク

  • レビュー
  • バージョンアップ(パッチバージョン ⬆️ )
  • マージ

@big2men big2men self-assigned this Apr 5, 2017
@big2men big2men requested a review from a user April 5, 2017 14:39
@big2men

big2men commented Apr 5, 2017

Copy link
Copy Markdown
Member Author

@tanakakc レビュー無しでマージできないようなのでレビューよろしくですm(_ _)m
コードレビューだけでOKです。

@ghost ghost left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMです!

一応、僕と @big2men は強制的にマージできる権限はつけてあるので、レビューなしでマージ可能なはずです!

@big2men

big2men commented Apr 5, 2017

Copy link
Copy Markdown
Member Author

@tanakakc あ、そうなの?なんかすごいダメよ感出してきたからびびっちゃったw
ありがとう!

@big2men big2men merged commit d72167d into open-qhm:master Apr 5, 2017
@big2men big2men deleted the feature/#60-fix-video-plugin branch April 5, 2017 15:23
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.

1 participant