プロジェクト

全般

プロフィール

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

prototype.js や script.aculo.us を使用している箇所を全て jQuery で置き換える

Youichi Kimura12年以上前に追加. 約12年前に更新.

ステータス:
Fixed(完了)
優先度:
High(高め)
対象バージョン:
開始日:
2011-11-22
期日:
進捗率:

100%


説明

Overview (概要)

prototype.js と script.aculo.us が含まれている sfProtoculousPlugin を削除し、OpenPNE3 の JavaScript コード中に含まれている prototype.js または script.aculo.us 依存なコードを jQuery 用に書き換える。

Spec (仕様)

  • sfProtoculousPlugin を削除
  • jQuery 及び jQuery UI を同梱する
  • ビューでのJavaScriptの読み込みで %SF_PROTOTYPE_WEB_DIR%/js/prototype などの代わりに jquery.min.js を使用して jQuery をロードする
  • 既存の JavaScript コードの prototype.js または script.aculo.us 依存なコードを jQuery 用に書き換える

english_nav.png 表示 (36.4 KB) Fumie Toyooka, 2012-04-04 14:35


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #3144: prototype.jsからjQueryへの移行の対応漏れにより、日記で絵文字が入力できない Won't fix(対応せず) 2012-08-01

関係しているリビジョン

リビジョン cb6fc103 (差分)
Youichi Kimura12年以上前に追加

remove sfProtoculousPlugin (refs #2617)

リビジョン 25c25499 (差分)
Youichi Kimura12年以上前に追加

rewrite to use jQuery instead of prototype.js (fixes #2617)

リビジョン 7efa6bda (差分)
Youichi Kimura約12年前に追加

rewrite to use jQuery instead of prototype.js (fixes #2617)

リビジョン 1a8eb4e2 (差分)
Youichi Kimura約12年前に追加

add sfProtoculousPlugin for compatibility (fixes #2617)

リビジョン 0a55fa43 (差分)
Youichi Kimura約12年前に追加

rename JavascriptHelper.php in sfProtoculousPlugin to avoid helper name conflict (fixes #2617)

リビジョン 3f9d1036 (差分)
Youichi Kimura約12年前に追加

don't make element that is'nt mark with sortable CSS class as movable (refs #2617)

リビジョン 207729a0 (差分)
Youichi Kimura約12年前に追加

include jquery.min.js and jquery-ui.min.js into project tree (fixes #2617)

リビジョン 88138e9f (差分)
Youichi Kimura約12年前に追加

fix sort order not applied correctly in backend config page (gadget, profile) (fixes #2617)

リビジョン 3bd1bac4 (差分)
Youichi Kimura約12年前に追加

remove sfProtoculousPlugin (refs #2617)

リビジョン 10c0ffec (差分)
Youichi Kimura約12年前に追加

rewrite to use jQuery instead of prototype.js (fixes #2617)

リビジョン 5337346d (差分)
Youichi Kimura約12年前に追加

rewrite to use jQuery instead of prototype.js (fixes #2617)

リビジョン e5ecb058 (差分)
Youichi Kimura約12年前に追加

add sfProtoculousPlugin for compatibility (fixes #2617)

リビジョン 329a54f9 (差分)
Youichi Kimura約12年前に追加

rename JavascriptHelper.php in sfProtoculousPlugin to avoid helper name conflict (fixes #2617)

リビジョン 5a1dba21 (差分)
Youichi Kimura約12年前に追加

don't make element that is'nt mark with sortable CSS class as movable (refs #2617)

リビジョン 31db1bd2 (差分)
Youichi Kimura約12年前に追加

include jquery.min.js and jquery-ui.min.js into project tree (fixes #2617)

リビジョン f2698b58 (差分)
Youichi Kimura約12年前に追加

fix sort order not applied correctly in backend config page (gadget, profile) (fixes #2617)

リビジョン 3da30b7b (差分)
Youichi Kimura約12年前に追加

fix some gadgets sortable in PC view (fixes #2617)

リビジョン 0acdde33 (差分)
Youichi Kimura約12年前に追加

fix some gadgets sortable in PC view (fixes #2617)

履歴

#1 Youichi Kimura12年以上前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

更新履歴 25c254996616c408ede16deaab2c6207277988a1 で適用されました。

#2 Minoru Takai12年以上前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

修正を追加するか、あるいは別チケットか何かで次の問題に対応しておく必要があるかと思います。

  • pc_backend.php/sns/richTextarea
  • pc_backend.php/community/categoryList
  • pc_backend.php/profile/list
  • pc_backend.php/navigation/list

などにアクセスすると

Fatal error: Call to undefined function sortable_element() in
/path/to/apps/pc_backend/modules/profile/templates/listSuccess.php on line 56

のようなエラーが生じます。

これは上記に示したようなテンプレート内で、もともと plugins/sfProtoculousPlugin/lib/helper/JavascriptHelper.php で定義されていた sortable_element() メソッドを用いているためで、このチケットでは恐らく(少なくとも) sortable_element() メソッドが使われていた部分の書き換えも行う必要があるはずです。

手元で master ブランチを用いてインストールした際のソースコードを調べた結果を示しておきます。

$ ack 'sortable_element'

lib/vendor/symfony/lib/task/project/validation/sfDeprecatedHelpersValidation.class.php
56:      'remote_function', 'observe_field', 'observe_form', 'visual_effect', 'sortable_element',

apps/pc_backend/modules/sns/templates/richTextareaSuccess.php
37:<?php echo sortable_element('button', array(

apps/pc_backend/modules/navigation/templates/listSuccess.php
54:<?php echo sortable_element('type_'.str_replace(' ', '_', $type), array(

apps/pc_backend/modules/community/templates/_categoryListForm.php
65:<?php echo sortable_element('type_'.$type, array(

apps/pc_backend/modules/profile/templates/listSuccess.php
56:<?php echo sortable_element('profiles',array(
117:<?php echo sortable_element('profile_options_'.$value->getId(),array(

plugins/opOpenSocialPlugin/apps/pc_frontend/modules/application/templates/listSuccess.php
32:<?php echo sortable_element('order', array(

#3 Minoru Takai約12年前に更新

  • 優先度Normal(通常) から Immediate(今すぐ) に変更

note-2 で指摘している不具合についてですが、現在 OpenPNE3 は 3.6.x 系統、 master (旧3.7)ブランチ、 HOUOU (新3.7)系統がありますが(※)、 OpenPNE3 リポジトリの master ブランチを触る開発者が管理画面から一部のページにアクセスできない問題(note-2 の内容)を受けています。

(※)少なくとも、という意味で、ここでは 3.4.x 系統などは敢えて示していません。

このチケットでの経緯を知らないと、 master ブランチを触る開発者は不毛な調査をする羽目になり結果として生産性を落とすことに繋がるため、チケットでの改善内容とは別に note-2 の致命的な不具合について可及的速やかに対応すべきであることを示しておきます。

この指摘の意味で、 Priority を一時的に Immediate にします。致命的な不具合を解消した後は Priority を本来のものに変更してください。

#4 Youichi Kimura約12年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

更新履歴 7efa6bdad311898fb37dbfe9e6517c0787c632f8 で適用されました。

#5 Youichi Kimura約12年前に更新

更新履歴 1a8eb4e21f194cfed860b4341a03c284762d43a3 で適用されました。

#6 Youichi Kimura約12年前に更新

更新履歴 0a55fa430b5dd82c2af911a438e51d980eba49f6 で適用されました。

#7 Youichi Kimura約12年前に更新

  • 優先度Immediate(今すぐ) から Normal(通常) に変更

note-2 で指摘されている不具合について修正しました。現在、該当する管理画面ページはこちらの環境では正しく表示・操作できています。

#8 Youichi Kimura約12年前に更新

  • 説明 を更新 (diff)
  • ステータスPending Review(レビュー待ち) から Accepted(着手) に変更
  • 進捗率50 から 0 に変更

当チケットの仕様を、jQuery などの JavaScript ライブラリを ajax.googleapis.com など外部からロードする方法から OpenPNE3 コアに同梱する方法に変更しました。

#9 Youichi Kimura約12年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

更新履歴 207729a0c01b2abcc10902fd08bea021e17b27a9 で適用されました。

#10 Shouta Kashiwagi約12年前に更新

  • 対象バージョンOpenPNE 3.7.0 から 252 に変更

HOUOUでの成果物はOpenPNE3.7.1にて取り込むことになったため、対象バージョンを変更します。

#11 Shouta Kashiwagi約12年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

以下のバグを発見しましたので、一旦差し戻しとさせていただきます。

  • 管理画面の「プロフィール項目設定」において、プロフィール項目の並び替えが効かない
  • 管理画面の「ガジェット設定」において、ガジェットを並び替えるとその要素内のガジェットがすべて削除される

#12 Youichi Kimura約12年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

更新履歴 88138e9fe524fa1403d9f8185b71644b96222f46 で適用されました。

#13 Youichi Kimura約12年前に更新

  • 優先度Normal(通常) から Urgent(急いで) に変更

#14 Shouta Kashiwagi約12年前に更新

  • 優先度Urgent(急いで) から High(高め) に変更

#15 Shouta Kashiwagi約12年前に更新

  • 対象バージョン252 から OpenPNE 3.8beta1 に変更

#16 Youichi Kimura約12年前に更新

更新履歴 10c0ffec716301658146a93562b2918d0cf90644 で適用されました。

#17 Youichi Kimura約12年前に更新

更新履歴 5337346d6b4e33b16f273ca924b766c523b0e8c1 で適用されました。

#18 Youichi Kimura約12年前に更新

更新履歴 e5ecb058f8ff17cf47a157e502022f18af5514b1 で適用されました。

#19 Youichi Kimura約12年前に更新

更新履歴 329a54f98768cb712d1b54b075e5621357776b3f で適用されました。

#20 Youichi Kimura約12年前に更新

更新履歴 31db1bd266203d2f8448bf75c288bbb0061a1cdb で適用されました。

#21 Youichi Kimura約12年前に更新

更新履歴 f2698b584443095a43060b09a4d9e67a4384749c で適用されました。

#22 Fumie Toyooka約12年前に更新

  • ファイル english_nav.png 表示 を追加
  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

バグを発見しました

管理画面>ナビゲーション設定
の全項目にて、「項目名 (en)」の文字列が登録されているにも拘わらず、
非表示になっています。

ですので、この項目に登録した英語の項目名が表示されるように修正お願いします。

#23 Shouta Kashiwagi約12年前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更
  • 進捗率50 から 0 に変更

#24 Shouta Kashiwagi約12年前に更新

  • 担当者Youichi Kimura から Shouta Kashiwagi に変更

#25 Shouta Kashiwagi約12年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更
以下のコミットをrelease-3.8beta1に取り込みました
  • 085e4e9 Revert "eager loading in fetching i18n translation records (fixes #1898)"

なお,masterでは↑のコミットが取り込まれている為,masterでは上記のようなエラーは発生しません.

#26 Shouta Kashiwagi約12年前に更新

  • ステータスPending Review(レビュー待ち) から Accepted(着手) に変更
  • 進捗率50 から 0 に変更

#27 Shouta Kashiwagi約12年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

#28 Youichi Kimura約12年前に更新

更新履歴 3da30b7b8cda8d1200e95b55ea3c0c4a2fa2e964 で適用されました。

#29 Yuya Watanabe約12年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#30 Shouta Kashiwagi約12年前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

テストOKとします.

#31 Yuya Watanabe約12年前に更新

master

cb6fc103e76b915d66158983d1971d49c6a260d2 remove sfProtoculousPlugin (refs #2617)
25c254996616c408ede16deaab2c6207277988a1 rewrite to use jQuery instead of prototype.js (fixes #2617)
7efa6bdad311898fb37dbfe9e6517c0787c632f8 rewrite to use jQuery instead of prototype.js (fixes #2617)
1a8eb4e21f194cfed860b4341a03c284762d43a3 add sfProtoculousPlugin for compatibility (fixes #2617)
0a55fa430b5dd82c2af911a438e51d980eba49f6 rename JavascriptHelper.php in sfProtoculousPlugin to avoid helper name conflict (fixes #2617)
3f9d1036cb351c88ec0eb87f2fdfa6079fafbbf9 don't make element that is'nt mark with sortable CSS class as movable (refs #2617)
207729a0c01b2abcc10902fd08bea021e17b27a9 include jquery.min.js and jquery-ui.min.js into project tree (fixes #2617)
88138e9fe524fa1403d9f8185b71644b96222f46 fix sort order not applied correctly in backend config page (gadget, profile) (fixes #2617)
3da30b7b8cda8d1200e95b55ea3c0c4a2fa2e964 fix some gadgets sortable in PC view (fixes #2617)

release-3.8beta1

3bd1bac4ed298a212678877491fe242a9908e43b remove sfProtoculousPlugin (refs #2617)
10c0ffec716301658146a93562b2918d0cf90644 rewrite to use jQuery instead of prototype.js (fixes #2617)
5337346d6b4e33b16f273ca924b766c523b0e8c1 rewrite to use jQuery instead of prototype.js (fixes #2617)
e5ecb058f8ff17cf47a157e502022f18af5514b1 add sfProtoculousPlugin for compatibility (fixes #2617)
329a54f98768cb712d1b54b075e5621357776b3f rename JavascriptHelper.php in sfProtoculousPlugin to avoid helper name conflict (fixes #2617)
5a1dba21c6f9096f1ad802e626421f8d6ffc9eac don't make element that is'nt mark with sortable CSS class as movable (refs #2617)
31db1bd266203d2f8448bf75c288bbb0061a1cdb include jquery.min.js and jquery-ui.min.js into project tree (fixes #2617)
f2698b584443095a43060b09a4d9e67a4384749c fix sort order not applied correctly in backend config page (gadget, profile) (fixes #2617)

#32 Yuya Watanabe約12年前に更新

  • ステータスFixed(完了) から Pending Fixing(修正待ち) に変更
  • 進捗率100 から 0 に変更

#33 Yuya Watanabe約12年前に更新

  • ステータスPending Fixing(修正待ち) から Accepted(着手) に変更

#34 Yuya Watanabe約12年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

#35 Yuya Watanabe約12年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

note-28 の内容についてはレビューOKとしましたが, release-3.8beta1 には含まれていません.

#36 Youichi Kimura約12年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

更新履歴 0acdde3352c507772804929158f3d3d11c39d24e で適用されました。

#37 Yuya Watanabe約12年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#38 Kiwa Sakai約12年前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

【3.8beta1テスト表】https://docs.google.com/a/openpne.jp/spreadsheet/ccc?key=0AsdDZCDQSqiYdDFCSGdTZzlYQWV6bDFkTmI2N2p2emc#gid=0
このチケットは閉じます。

他の形式にエクスポート: Atom PDF