[#13021] - Joomla 4.0: Removing modern SEF option
- Fixed in Code Base
- 18 Dec 2016
- Medium
- Build: 4.0-dev
- # 13021
- Diff
- Hackwar:patch-14
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
User tests: Successful: Unsuccessful:
This removes the option for modern routing from the components configuration.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_contact |
| Labels |
Added:
?
|
||
| Category | Administration com_contact | ⇒ | Administration com_contact com_content |
| Category | Administration com_contact com_content | ⇒ | Administration com_contact com_content com_newsfeeds |
Yes, you are right. Done.
I have tested this item
anyway this is not what this PR does, so tested with success on code review
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13021.
I don't want to remove the IDs by default. We are just using the modern routing in all cases, but otherwise that is just another option that people can select. So keeping that 0 is correct for me.
ups just noticed you still also have sef_advanced in com_users
https://github.com/joomla/joomla-cms/blob/4.0-dev/administrator/components/com_users/config.xml#L244
I have tested this item
as comments above
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13021.
| Category | Administration com_contact com_content com_newsfeeds | ⇒ | Administration com_contact com_content com_newsfeeds com_users |
Removed the option in com_users, too.
wilsonge
- | Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-12-18 11:43:26 |
| Closed_By | ⇒ | wilsonge |
zero-24
- | Milestone |
Added: |
||
@Hackwar i think you also need to remove
showon="sef_advanced:1"in sef ids field