Opened 2 months ago
Closed 8 weeks ago
#23813 closed defect (bug) (fixed)
3.6-Menus: Menu select should use get instead of post
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.6 |
| Component: | Menus | Version: | trunk |
| Severity: | normal | Keywords: | 3.6-menus commit |
| Cc: |
Description (last modified by DrewAPicture)
@helen commented on ticket:23607:20:
Thinking the select a menu form should be get instead of post.
There's a patch on the tracking ticket: ticket:23607:23607.diff
Attachments (3)
Change History (12)
comment:1
DrewAPicture — 2 months ago
- Description modified (diff)
Meant to mention that my reasoning for this is so that the URL changes (bookmarking, copying the link, just looking nice) and so that if the user refreshes for some reason after switching the menu it doesn't prompt them about re-submitting form data.
- Keywords commit has-patch added
23813.1.diff addresses Helen's second concern on ticket:23607:20.
Looks like we use an empty action for forms around the admin rather than just omitting it. Also, there are more forms on the screen. Maybe 23813.2.diff?
comment:7
markjaquith — 8 weeks ago
- Owner set to markjaquith
- Resolution set to fixed
- Status changed from new to closed
In 23807:
comment:8
johnbillion — 8 weeks ago
- Keywords has-patch removed
- Resolution fixed deleted
- Status changed from closed to reopened
This has had the side effect that after adding a new menu you remain on the new menu screen.
comment:9
DrewAPicture — 8 weeks ago
- Resolution set to fixed
- Status changed from reopened to closed
Fixed in [23810]
23813.diff should do it.