Enhancement(機能追加・改善) #2617
完了prototype.js や script.aculo.us を使用している箇所を全て jQuery で置き換える
Youichi Kimura さんが約13年前に追加. 12年以上前に更新.
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 用に書き換える
ファイル
Youichi Kimura さんがほぼ13年前に更新
- ステータス を New(新規) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 25c254996616c408ede16deaab2c6207277988a1 で適用されました。
Minoru Takai さんがほぼ13年前に更新
- ステータス を 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(
Minoru Takai さんがほぼ13年前に更新
- 優先度 を 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 を本来のものに変更してください。
Youichi Kimura さんがほぼ13年前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
更新履歴 7efa6bdad311898fb37dbfe9e6517c0787c632f8 で適用されました。
Youichi Kimura さんがほぼ13年前に更新
- 優先度 を Immediate(今すぐ) から Normal(通常) に変更
note-2 で指摘されている不具合について修正しました。現在、該当する管理画面ページはこちらの環境では正しく表示・操作できています。
Youichi Kimura さんがほぼ13年前に更新
- 説明 を更新 (差分)
- ステータス を Pending Review(レビュー待ち) から Accepted(着手) に変更
- 進捗率 を 50 から 0 に変更
当チケットの仕様を、jQuery などの JavaScript ライブラリを ajax.googleapis.com など外部からロードする方法から OpenPNE3 コアに同梱する方法に変更しました。
Youichi Kimura さんがほぼ13年前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 207729a0c01b2abcc10902fd08bea021e17b27a9 で適用されました。
Shouta Kashiwagi さんが12年以上前に更新
- 対象バージョン を OpenPNE 3.7.0 から 252 に変更
HOUOUでの成果物はOpenPNE3.7.1にて取り込むことになったため、対象バージョンを変更します。
Shouta Kashiwagi さんが12年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
以下のバグを発見しましたので、一旦差し戻しとさせていただきます。
- 管理画面の「プロフィール項目設定」において、プロフィール項目の並び替えが効かない
- 管理画面の「ガジェット設定」において、ガジェットを並び替えるとその要素内のガジェットがすべて削除される
Youichi Kimura さんが12年以上前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
更新履歴 88138e9fe524fa1403d9f8185b71644b96222f46 で適用されました。
Fumie Toyooka さんが12年以上前に更新
- ファイル english_nav.png english_nav.png を追加
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
バグを発見しました¶
管理画面>ナビゲーション設定
の全項目にて、「項目名 (en)」の文字列が登録されているにも拘わらず、
非表示になっています。
ですので、この項目に登録した英語の項目名が表示されるように修正お願いします。
Shouta Kashiwagi さんが12年以上前に更新
- ステータス を Rejected(差し戻し) から Accepted(着手) に変更
- 進捗率 を 50 から 0 に変更
Shouta Kashiwagi さんが12年以上前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
- 085e4e9 Revert "eager loading in fetching i18n translation records (fixes #1898)"
なお,masterでは↑のコミットが取り込まれている為,masterでは上記のようなエラーは発生しません.
Shouta Kashiwagi さんが12年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Accepted(着手) に変更
- 進捗率 を 50 から 0 に変更
Shouta Kashiwagi さんが12年以上前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
Yuya Watanabe さんが12年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
Shouta Kashiwagi さんが12年以上前に更新
- ステータス を Pending Testing(テスト待ち) から Fixed(完了) に変更
- 進捗率 を 70 から 100 に変更
テストOKとします.
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)
Yuya Watanabe さんが12年以上前に更新
- ステータス を Fixed(完了) から Pending Fixing(修正待ち) に変更
- 進捗率 を 100 から 0 に変更
Yuya Watanabe さんが12年以上前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
Yuya Watanabe さんが12年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
note-28 の内容についてはレビューOKとしましたが, release-3.8beta1 には含まれていません.
Youichi Kimura さんが12年以上前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
更新履歴 0acdde3352c507772804929158f3d3d11c39d24e で適用されました。
Yuya Watanabe さんが12年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
Kiwa Sakai さんが12年以上前に更新
- ステータス を Pending Testing(テスト待ち) から Fixed(完了) に変更
- 進捗率 を 70 から 100 に変更
【3.8beta1テスト表】https://docs.google.com/a/openpne.jp/spreadsheet/ccc?key=0AsdDZCDQSqiYdDFCSGdTZzlYQWV6bDFkTmI2N2p2emc#gid=0
このチケットは閉じます。