Feeling Lucky
?
?
[#13475] - Remove invalid ordering options from language installer
- Fixed in Code Base
- 5 Jan 2017
- Medium
- Build: staging
- # 13475
- Diff
- Bakual:RemoveInvalidOrdering
User tests: Successful: Unsuccessful:
PR #13256 changed the way the language installer works.
I forgot to remove the ordering option for ID which is now invalid.
Summary of Changes
Removes the options from the dropdown
Testing Instructions
- Go to Extension -> Manage -> Install Languages
- After applying this PR, the options to order by ID are no longer present.
Documentation Changes Required
None
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_installer |
Bakual
- comment
- 5 Jan 2017
Judging from the screenshots you need to update your local installation to current staging (or nightly) before applying the PR.
You still seem to have the old install view
RonakParmar
- comment
- 5 Jan 2017
I have tested this item
Downloaded latest Joomla staging branch and tested.
PR works fine.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13475.
franz-wohlkoenig
- comment
- 5 Jan 2017
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13475.
zero-24
- | Milestone |
Added: |
||
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
?
|
||
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-01-05 19:43:42 |
| Closed_By | ⇒ | rdeutz | |
| Labels |
Added:
?
|
||
@Bakual I have attached before and after patch applied screen dumps please check them, Are they correct or not? So I can successfully test this PR. Because, You mentioned that after applying this PR, the options to order by ID are no longer present, still I can see ID option in drop-down.
Before PR:
After PR:
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13475.