Project

General

Profile

Enhancement(機能追加・改善) #3887

OpenPNE3に同梱されているsymfonyを1.4.20に更新する

Added by Youichi Kimura over 3 years ago. Updated over 2 years ago.

Status:
Rejected(差し戻し)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2017-02-07
Due date:
% Done:

50%


Description

Overview (概要)

現在 OpenPNE3 には symfony 1.4.13 (2011/08/05) が同梱されているが、symfony 1.x の最新版は 1.4.20 (2012/11/25) がリリースされているためこのバージョンに更新する。

なお、symfony 1.4.20 には security fix も含まれていたがこれは既に #3265 にて先行して該当する箇所のみ取り込まれている。

symfony 1.x の変更履歴は下記の URL に記載されている。

https://github.com/symfony/symfony1/blob/v1.4.20/CHANGELOG

目立った変化としては ./symfony openpne:version などの実行時や pc_frontend_dev.php で表示されていた ob_start 関連の警告が修正されている。

$ ./symfony openpne:version
PHP Warning:  ob_start(): function '' not found or invalid function name in /home/upsilon/git/openpne3/pnejp/lib/vendor/symfony/lib/config/sfApplicationConfiguration.class.php on line 155

Warning: ob_start(): function '' not found or invalid function name in /home/upsilon/git/openpne3/pnejp/lib/vendor/symfony/lib/config/sfApplicationConfiguration.class.php on line 155
PHP Notice:  ob_start(): failed to create buffer in /home/upsilon/git/openpne3/pnejp/lib/vendor/symfony/lib/config/sfApplicationConfiguration.class.php on line 155

Notice: ob_start(): failed to create buffer in /home/upsilon/git/openpne3/pnejp/lib/vendor/symfony/lib/config/sfApplicationConfiguration.class.php on line 155
Core versions:
 OpenPNE                       3.8.17
 symfony                       1.4.13
OpenPNE plugin versions:
 opAuthMailAddressPlugin       1.3.5
 opSkinThemePlugin             1.0.11
 opTimelinePlugin              1.2.6
 opCommunityTopicPlugin        1.1.2
 opDiaryPlugin                 1.5.1
 opMessagePlugin               2.0.0

Spec (仕様)

symfony 1.4.20 は http://symfony.com/legacy からダウンロードできる。

また、過去に symfony のバージョンを更新した時には、symfony に独自に適用していたパッチが巻き戻ってしまう問題 (#2780) が起きている。
更新時に再度適用する必要のあるパッチは ./data/patches 以下に置かれているため、これらを適用し #2408 のバグが再び発生していないことを確認する必要がある。

さらに #2822./lib/vendor/symfony/plugins/sfPropelPlugin を削除しているため、更新時に sfPropelPlugin が追加された場合には削除する必要がある。


Subtasks

Backport(バックポート) #4141: OpenPNE3に同梱されているsymfonyを1.4.20に更新するPending Review(レビュー待ち)kaoru n

Backport(バックポート) #4142: OpenPNE3に同梱されているsymfonyを1.4.20に更新するPending Review(レビュー待ち)kaoru n

History

#1 Updated by kaoru n almost 3 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • Assignee set to kaoru n
  • % Done changed from 0 to 50

https://github.com/openpne/OpenPNE3/pull/435
にてプルリクエストを行いました。
レビューをお願いします。

下記手順で更新しました。

1. symfony 1.4.20 に置き換え

$ cd lib/vendor/
$ rm -rf symfony
$ cp ~/Downloads/symfony-1.4.20.tgz ./
$ tar zxvf symfony-1.4.20.tgz
$ mv symfony-1.4.20 symfony
$ rm symfony-1.4.20.tgz
$ rm package.xml

2. sfPropelPlugin 削除 ( #2822 )

$ rm -rf lib/vendor/symfony/lib/plugins/sfPropelPlugin

3. ./data/patches 以下のパッチ適用

$ patch -p1 --dry-run < data/patches/t1565.patch
patching file lib/vendor/symfony/lib/request/sfWebRequest.class.php
Hunk #1 succeeded at 256 (offset 17 lines).
Hunk #2 succeeded at 278 (offset 17 lines).

$ patch -p1 < data/patches/t1565.patch
patching file lib/vendor/symfony/lib/request/sfWebRequest.class.php
Hunk #1 succeeded at 256 (offset 17 lines).
Hunk #2 succeeded at 278 (offset 17 lines).

$ patch -p1 --dry-run < data/patches/t2408.patch
patching file lib/vendor/symfony/lib/helper/PartialHelper.php
Hunk #1 succeeded at 215 (offset 2 lines).

$ patch -p1 < data/patches/t2408.patch
patching file lib/vendor/symfony/lib/helper/PartialHelper.php
Hunk #1 succeeded at 215 (offset 2 lines).

$ patch -p1 --dry-run < data/patches/t3956.patch
patching file lib/vendor/symfony/lib/config/sfFactoryConfigHandler.class.php
patching file lib/vendor/symfony/lib/response/sfWebResponse.class.php

$ patch -p1 < data/patches/t3956.patch
patching file lib/vendor/symfony/lib/config/sfFactoryConfigHandler.class.php
patching file lib/vendor/symfony/lib/response/sfWebResponse.class.php

$ rm lib/vendor/symfony/lib/request/sfWebRequest.class.php.orig

#2 Updated by kaoru n almost 3 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

symfony 1.4.20 は http://symfony.com/legacy からダウンロードできる。

上記のリンクからアーカイブをダウンロードした場合、lib/vendor/symfony/test が含まれていない。
svn から取得した場合は含まれるので、いったん差し戻してsvnから取得します。

#3 Updated by kaoru n almost 3 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

#4 Updated by kaoru n almost 3 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

https://github.com/openpne/OpenPNE3/pull/435
更新しました。
レビューをお願いします。

下記手順で更新しました。

1. symfony 1.4.20 に置き換え

$ cd lib/vendor/
$ rm -rf symfony
$ svn checkout http://svn.symfony-project.com/tags/RELEASE_1_4_20/ symfony
$ cd ../../
$ cp /var/www/work/jobeet/symfony ./
$ rm -rf lib/vendor/symfony/.svn/ lib/vendor/symfony/lib/event_dispatcher/.svn/ lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/i18n/.svn/ lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/lib/vendor/doctrine/.svn/ lib/vendor/symfony/lib/plugins/sfDoctrinePlugin/web/.svn/ lib/vendor/symfony/lib/vendor/lime/.svn/ lib/vendor/symfony/lib/yaml/.svn/

2. sfPropelPlugin 削除 ( #2822 )

$ rm -rf lib/vendor/symfony/lib/plugins/sfPropelPlugin

3. ./data/patches 以下のパッチ適用

$ patch -p1 --dry-run < data/patches/t1565.patch
patching file lib/vendor/symfony/lib/request/sfWebRequest.class.php
Hunk #1 succeeded at 256 (offset 17 lines).
Hunk #2 succeeded at 278 (offset 17 lines).

$ patch -p1 < data/patches/t1565.patch
patching file lib/vendor/symfony/lib/request/sfWebRequest.class.php
Hunk #1 succeeded at 256 (offset 17 lines).
Hunk #2 succeeded at 278 (offset 17 lines).

$ patch -p1 --dry-run < data/patches/t2408.patch
patching file lib/vendor/symfony/lib/helper/PartialHelper.php
Hunk #1 succeeded at 215 (offset 2 lines).

$ patch -p1 < data/patches/t2408.patch
patching file lib/vendor/symfony/lib/helper/PartialHelper.php
Hunk #1 succeeded at 215 (offset 2 lines).

$ patch -p1 --dry-run < data/patches/t3956.patch
patching file lib/vendor/symfony/lib/config/sfFactoryConfigHandler.class.php
patching file lib/vendor/symfony/lib/response/sfWebResponse.class.php

$ patch -p1 < data/patches/t3956.patch
patching file lib/vendor/symfony/lib/config/sfFactoryConfigHandler.class.php
patching file lib/vendor/symfony/lib/response/sfWebResponse.class.php

$ rm lib/vendor/symfony/lib/helper/PartialHelper.php.orig lib/vendor/symfony/lib/request/sfWebRequest.class.php.orig

#5 Updated by kaoru n over 2 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0

対象バージョン変更により修正内容の確認が必要であるため差し戻します。

Also available in: Atom PDF