プロジェクト

全般

プロフィール

Vote #75029

未完了

Redirect back is overheated

Admin Redmine さんが約4年前に追加.

ステータス:
New
優先度:
通常
担当者:
-
カテゴリ:
-
対象バージョン:
-
開始日:
2022/05/09
期日:
進捗率:

0%

予定工数:
category_id:
0
version_id:
0
issue_org_id:
16314
author_id:
66834
assigned_to_id:
1
comments:
2
status_id:
1
tracker_id:
3
plus1:
0
affected_version:
closed_on:
affected_version_id:
ステータス-->[New]

説明

Take a look at these code lines:

source:trunk/app/controllers/application_controller.rb@13036#L368
It is not used at all.

source:trunk/app/controllers/application_controller.rb@13036#L399
It uses same functionality that method above (relies on HTTP_REFERER).

source:trunk/app/helpers/application_helper.rb@13036#L1100
It could be replaced with helper_method :back_url.

source:trunk/app/controllers/application_controller.rb@13036#L390
This condition is never satisfied. I did not find an usage.


journals

Anton Orel wrote:
> Take a look at these code lines:
>
> source:app/controllers/application_controller.rb#L368
> It is not used at all.

It's used at least in @ContextMenusController@.

> source:app/controllers/application_controller.rb#L399
> It uses same functionality that method above (relies on HTTP_REFERER).

Not the same functionnality, one relies on the :back_url param, the other one on HTTP_REFERER. But please provide a patch if you think they can be merged in a single one.

> source:app/helpers/application_helper.rb#L1100
> It could be replaced with `helper_method :back_url`.

Please provide a patch.

> source:app/controllers/application_controller.rb#L390
> This condition is never satisfied. I did not find an usage.

Used at least in AccountController.
--------------------------------------------------------------------------------

--------------------------------------------------------------------------------

表示するデータがありません

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

いいね!0