LancersのCakePHPバージョンアップ施策について

ランサーズ株式会社
エンジニア／秋山茂昌
バージョンアップ施策について
〜 1.3系から3.x系までの道のり〜

cakephp1.3から2.8で詰まったところ
● ランサーズとは

はじめに
● ランサーズとは

なぜ、バージョンアップするの？

なぜ、バージョンアップするのか？
● 過去にもリプレース行おうとしていた

● KPI改善や機能追加/改善を優先的に行ってきた

● KPI改善や機能追加/改善を優先的に行ってきた
→ ソースコードの肥大化
→ UTがほとんどない状態

● ソースコード量
ファイル数行数
コントローラー 174 約6万
バッチファイル 127 約5万
UT 505 約20万
UT抜き 1,386 約40万
全体 1,891 約60万

目的：開発リソースの最大化

● レガシーコードからの脱却

● サポートが終了している
○ cakephp1.3, php5.3等

● cakephp2系もサポート終了しそう

● cakephp2系もサポート終了しそう
● CakePHP3.x, PHP7.xだったりするサービスも

どんな感じで進めるの？

● バージョンアップの前準備として

今あるUTを動くようにする
Nginx + php-fpm化
● バージョンアップの前準備として

1.3 → 2.8
2.8 → 2.9
2.9 → 3.3
3.3 → 3.4
3.4 → 3.5
5.3 → 5.6
5.6 → 7.1
3.5 → 3.6 .. more

誰がバージョンアップやってるの？

部署：開発部
名前：秋山茂昌
関西弁のなかなか出ない大阪出身
● インフラ
● 金澤さん
● エンジニア
● 小林さん
● リーダー
誰がバージョンアップやってるの？

● Upgrade Shellの実行だけでは動かない
○ cakephp2.0から2.8の差分を取り込んでいるわけでな
かった
○ 変更点ドキュメントを2.0から読み直すことになった

● エラーが不明確
PHP Fatal error:
Allowed memory size of 536870912 bytes exhausted (tried to allocate 1664
bytes) in
/path/to/vendor/cakephp/cakephp/lib/Cake/Controller/Controller.php on line 335
PHP Fatal error:
Access to undeclared static property:App::$_classMap in
/path/to/vendor/cakephp/cakephp/lib/Cake/Core/App.php on line 535

● エラーが不明確
● こんなエラーはだいたいComponent, Helperの
読み込み失敗
PHP Fatal error:
Allowed memory size of 536870912 bytes exhausted (tried to allocate 1664
bytes) in
/path/to/vendor/cakephp/cakephp/lib/Cake/Controller/Controller.php on line 335
PHP Fatal error:
Access to undeclared static property:App::$_classMap in
/path/to/vendor/cakephp/cakephp/lib/Cake/Core/App.php on line 535

● HtmlHelper::link, FormHelper::selectの
仕様が変わっている
○ 仕様に合わせたラッパークラスの作成をして、
そっちを利用するようにテンプレートの修正
■ HtmlHelper::linkはjavascript::confirm()でアラート表示する方法が
変わってたり
■ FormHelper::selectはselectedの指定方法が変更になっていたり

● HtmlHelper::link, FormHelper::selectの
仕様が変わっている
○ 仕様に合わせたラッパークラスの作成をして、
そっちを利用するようにテンプレートの修正
■ HtmlHelper::linkはjavascript::confirm()でアラート表示する方法が
変わってたり
■ FormHelper::selectはselectedの指定方法が変更になっていたり
○ その他にも修正点が多く時間がかかる

● CakePHPに則ってLancersが組み立てられていない

● CakePHPに則ってLancersが組み立てられていない
○ 親クラスを意識して作られていなかったり
○ 1ファイル内のコード量の多さ
○ 冗長コードがたくさん
○ Noticeがたくさん

どんな感じですすめているのか

● CakePHP2.8にアップデートしたブランチを作成して、
そっちで動作を確認しつつコントローラー単位で修正を
行っていました。

● また、別軸でCakePHP2.8の動作担保として
Codeceptionを利用し手動チェックを減らしています。

● が、
毎週masterから最新を取り込むのが面倒くさく、、

● が、
毎週masterから最新を取り込むのが面倒くさく、、
● 同じような修正点が多くて辛い、、、

オレオレupgrade shellを作った

オレオレUpgrade Shellを作った
● App::import → App::uses

● new FooHelper() → new FooHelper(new View())

● new HogeComponent() →
new HogeComponent(new ComponentCollection())

● new HogeComponent() →
new HogeComponent(new ComponentCollection())
● new BarController() →
new BarController(
new CakeRequest(), new CakeResponse()
)

● $this->param, $this->data等の参照変更
→ $this->request->param, $this->request->data

● $this->param, $this->data等の参照変更
→ $this->request->param, $this->request->data
● その他多数

ピボットしました
● 事前にCakePHP1.3で出来ることをする

○ viewファイルで、Helperの参照方法
■ $lancers → $this->Lancers

○ viewファイルで、Helperの参照方法
■ $lancers → $this->Lancers
○ $this->param, $this->dataの参照変更
■ $this->request->param, $this->request->data

● CircleCIでチェックし変更をお願いする

● 纏めてCakePHP2.8に切り替える怖さより
1コントローラーずつでも移行できたら安心感あるよね

● 纏めてCakePHP2.8に切り替える怖さより
1コントローラーずつでも移行できたら安心感あるよね
● CakePHP1.3のapp/webroot/index.phpに細工

● CakePHP1.3とCakePHP2.8の同居！

● 事前にComponent, Helper, Viewあたりは
CakePHP2.8側に用意する

● 事前にComponent, Helper, Viewあたりは
CakePHP2.8側に用意する
● 1コントローラー毎に移行
○ UTも含めて移行
○ Codeceptionで動作を担保

● 纏めてどっかんより、協力を求めるような形で出来る

● 纏めてどっかんより、協力を求めるような形で出来る
● いつ変更するのか不安感を無くす事ができる

今後はどんな感じで進めるのか

● phpのバージョンアップ
○ カナリアテスト

● CakePHPのバージョンアップ

● CakePHPのバージョンアップ
○ 2バージョンを用意して置き換えていく
○ Codeceptionがあるから安心
○ UTも2バージョンを用意する

まとめ
● 事前にできることはやる
● Upgrade shellをあまり信じないこと
● 実はバージョン間の同居できる
● これからも慎重かつ大胆にバージョンアップ

絶賛、募集中
ランサーズではエンジニアを絶賛募集中です！
● 弊社HP、Wantedly等
ご応募お待ちしてます

ご視聴ありがとうございました！

LancersのCakePHPバージョンアップ施策について

Recommandé

Recommandé

Contenu connexe

Tendances

Tendances (20)

Similaire à LancersのCakePHPバージョンアップ施策について

Similaire à LancersのCakePHPバージョンアップ施策について (20)

LancersのCakePHPバージョンアップ施策について

Notes de l'éditeur