submit on admin >> filters redirects to home page [#50982]

When changing the default input format on the admin >> filters page, hitting the "Set default format" button, correctly changes the default format, but returns the user to the site home page. I believe it should return back to the admin >> filters page.

Comment	File	Size	Author
#7	filters.module_0.patch	1.23 KB	kuba.zygmunt
#7
#4	filters.module.patch	1.55 KB	kuba.zygmunt
#4
	all_roles_default_format_2.patch	1.94 KB	RayZ

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

RayZ CreditAttribution: RayZ commented 23 February 2006 at 18:27

Ignore the patch ... it was stolen from this issue. I'd uploaded the patch to a follow-up to that issue, previewed my followup, then submitted this issue in another browser window and it stole the patch from my other upload. Guess you can't work on more than one issue at a time.

Comment #2

RayZ CreditAttribution: RayZ commented 24 February 2006 at 21:27

Priority:

Normal

» Critical

IMHO this should be fixed before 4.7 release, so I bumped it to critical. I suspect this issue is a result of this patch: http://drupal.org/node/49799

Comment #3

RayZ CreditAttribution: RayZ commented 24 February 2006 at 21:34

#2 at http://drupal.org/node/51189 may also be helpful, as I suspect that was caused by the same patch.

Comment #4

kuba.zygmunt CreditAttribution: kuba.zygmunt commented 25 February 2006 at 10:03

Status:

Active

» Needs review

File	Size
filters.module.patch	1.55 KB

I've changed a little by adding function filter_admin_overview_submit which sets default filter (part of code moved from filter_admin_overview) and proper redirect url.

Comment #5

chx CreditAttribution: chx commented 25 February 2006 at 10:30

Status:

Needs review

» Reviewed & tested by the community

Nice application of submit.

Comment #6

chx CreditAttribution: chx commented 25 February 2006 at 12:28

Status:

Reviewed & tested by the community

» Needs work

If http://drupal.org/node/51189 goes in this will need a little work, because the initial problem is cured by that one. This patch however makes use of submit which is always good.