Feeling Lucky
?
?
[#28053] - Fix some issues when saving menus
- Fixed in Code Base
- 25 Feb 2020
- Medium
- Build: staging
- # 28053
- Diff
- SharkyKZ:patch-4
- Pending Hound Hound is busy sniffing around... Details
User tests: Successful: Unsuccessful:
Summary of Changes
Append ID to redirect URL so it doesn't get lost.
Clear session data when applying changes to load data from database.
Testing Instructions
Create two menus (menu1 and menu2).
Edit menu2.
Change its Menu Type to menu1.
Click Save.
After saving fails, change its menu type to menu3 and save.
View menus.
Edit a menu.
Remove its menu type.
Use browser tools to circumvent JS validation (remove required class and attribute from menu type field).
Click Save.
After saving fails, edit menu type back to what it was.
Save.
Expected result
- menu2 type changed to menu3.
- Menu saved.
Actual result
- A new menu with type menu3 is created.
- Saving fails with error:
Save failed with the following error: Menu type exists:...
Documentation Changes Required
No.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_menus |
infograf768
- comment
- 25 Feb 2020
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28053.
| Status | Pending | ⇒ | Ready to Commit |
infograf768
- comment
- 25 Feb 2020
rtc
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28053.
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-02-25 22:39:15 |
| Closed_By | ⇒ | rdeutz | |
| Labels |
Added:
?
?
|
||
I have tested this item✅ successfully on 1a681df
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28053.