Enhancement（機能追加・改善） #2617: prototype.js や script.aculo.us を使用している箇所を全て jQuery で置き換える - OpenPNE 3 - OpenPNE Issue Tracking System

Actions

Copy link

Enhancement（機能追加・改善） #2617

closed

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

Added by Youichi Kimura about 13 years ago. Updated over 12 years ago.

Status:

Fixed（完了）

Priority:

High（高め）

Assignee:

Shouta Kashiwagi

Target version:

OpenPNE 3.8beta1

Start date:

2011-11-22

Due date:

% Done:

100%

Estimated time:

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 用に書き換える

Files

english_nav.png (36.4 KB) english_nav.png

Fumie Toyooka, 2012-04-04 14:35

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Youichi Kimura about 13 years ago

Status changed from New（新規） to Pending Review（レビュー待ち）
% Done changed from 0 to 50

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

Actions

Copy link

Updated by Minoru Takai about 13 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(

Actions

Copy link

Updated by Minoru Takai almost 13 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 を本来のものに変更してください。

Actions

Copy link