Project

General

Profile

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

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

Added by Youichi Kimura over 7 years ago. Updated about 7 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Target version:
Start date:
2011-11-22
Due date:
% Done:

100%


Description

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 View (36.4 KB) Fumie Toyooka, 2012-04-04 14:35


Related issues

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

Associated revisions

Revision cb6fc103 (diff)
Added by Youichi Kimura over 7 years ago

remove sfProtoculousPlugin (refs #2617)

Revision 25c25499 (diff)
Added by Youichi Kimura over 7 years ago

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

Revision 7efa6bda (diff)
Added by Youichi Kimura over 7 years ago

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

Revision 1a8eb4e2 (diff)
Added by Youichi Kimura over 7 years ago

add sfProtoculousPlugin for compatibility (fixes #2617)

Revision 0a55fa43 (diff)
Added by Youichi Kimura over 7 years ago

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

Revision 3f9d1036 (diff)
Added by Youichi Kimura over 7 years ago

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

Revision 207729a0 (diff)
Added by Youichi Kimura over 7 years ago

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

Revision 88138e9f (diff)
Added by Youichi Kimura over 7 years ago

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

Revision 3bd1bac4 (diff)
Added by Youichi Kimura about 7 years ago

remove sfProtoculousPlugin (refs #2617)

Revision 10c0ffec (diff)
Added by Youichi Kimura about 7 years ago

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

Revision 5337346d (diff)
Added by Youichi Kimura about 7 years ago

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

Revision e5ecb058 (diff)
Added by Youichi Kimura about 7 years ago

add sfProtoculousPlugin for compatibility (fixes #2617)

Revision 329a54f9 (diff)
Added by Youichi Kimura about 7 years ago

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

Revision 5a1dba21 (diff)
Added by Youichi Kimura about 7 years ago

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

Revision 31db1bd2 (diff)
Added by Youichi Kimura about 7 years ago

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

Revision f2698b58 (diff)
Added by Youichi Kimura about 7 years ago

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

Revision 3da30b7b (diff)
Added by Youichi Kimura about 7 years ago

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

Revision 0acdde33 (diff)
Added by Youichi Kimura about 7 years ago

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

History

#1 Updated by Youichi Kimura over 7 years ago

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

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

#2 Updated by Minoru Takai over 7 years ago

  • Status changed from Pending Review(レビュー待ち) to 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 Updated by Minoru Takai over 7 years ago

  • Priority changed from Normal(通常) to 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 Updated by Youichi Kimura over 7 years ago

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

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

#5 Updated by Youichi Kimura over 7 years ago

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

#6 Updated by Youichi Kimura over 7 years ago

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

#7 Updated by Youichi Kimura over 7 years ago

  • Priority changed from Immediate(今すぐ) to Normal(通常)

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

#8 Updated by Youichi Kimura over 7 years ago

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

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

#9 Updated by Youichi Kimura over 7 years ago

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

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

#10 Updated by Shouta Kashiwagi over 7 years ago

  • Target version changed from OpenPNE 3.7.0 to 252

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

#11 Updated by Shouta Kashiwagi over 7 years ago

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

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

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

#12 Updated by Youichi Kimura over 7 years ago

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

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

#13 Updated by Youichi Kimura over 7 years ago

  • Priority changed from Normal(通常) to Urgent(急いで)

#14 Updated by Shouta Kashiwagi over 7 years ago

  • Priority changed from Urgent(急いで) to High(高め)

#15 Updated by Shouta Kashiwagi about 7 years ago

  • Target version changed from 252 to OpenPNE 3.8beta1

#16 Updated by Youichi Kimura about 7 years ago

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

#17 Updated by Youichi Kimura about 7 years ago

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

#18 Updated by Youichi Kimura about 7 years ago

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

#19 Updated by Youichi Kimura about 7 years ago

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

#20 Updated by Youichi Kimura about 7 years ago

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

#21 Updated by Youichi Kimura about 7 years ago

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

#22 Updated by Fumie Toyooka about 7 years ago

  • File english_nav.png View added
  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

バグを発見しました

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

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

#23 Updated by Shouta Kashiwagi about 7 years ago

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

#24 Updated by Shouta Kashiwagi about 7 years ago

  • Assignee changed from Youichi Kimura to Shouta Kashiwagi

#25 Updated by Shouta Kashiwagi about 7 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50
以下のコミットをrelease-3.8beta1に取り込みました
  • 085e4e9 Revert "eager loading in fetching i18n translation records (fixes #1898)"

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

#26 Updated by Shouta Kashiwagi about 7 years ago

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

#27 Updated by Shouta Kashiwagi about 7 years ago

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

#28 Updated by Youichi Kimura about 7 years ago

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

#29 Updated by Yuya Watanabe about 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#30 Updated by Shouta Kashiwagi about 7 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テストOKとします.

#31 Updated by Yuya Watanabe about 7 years ago

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 Updated by Yuya Watanabe about 7 years ago

  • Status changed from Fixed(完了) to Pending Fixing(修正待ち)
  • % Done changed from 100 to 0

#33 Updated by Yuya Watanabe about 7 years ago

  • Status changed from Pending Fixing(修正待ち) to Accepted(着手)

#34 Updated by Yuya Watanabe about 7 years ago

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

#35 Updated by Yuya Watanabe about 7 years ago

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

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

#36 Updated by Youichi Kimura about 7 years ago

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

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

#37 Updated by Yuya Watanabe about 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#38 Updated by Kiwa Sakai about 7 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

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

Also available in: Atom PDF