Feeling Lucky
?
[#13115] - [com_fields] No need for an alias in fields groups.
- Fixed in Code Base
- 9 Dec 2016
- Medium
- Build: staging
- # 13115
- Diff
- Bakual:RemoveAliasFromFieldsGroups
- Success continuous-integration/drone the build was successful Details
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Success JTracker/HumanTestResults Human Test Results: 2 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
When I rewrote com_fields to use groups instead of categories, I just copy-pasted the alias field. But it is in fact not used anywhere and thus can be removed.
Summary of Changes
This PR removes all traces to an alias in fields groups
Testing Instructions
- Delete the alias column from your database table #__fields_groups
- Check that creating/editing/saving/deleting field groups still works
- Check that fields behave as expected
Documentation Changes Required
None
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | SQL Administration com_admin Postgresql com_fields Installation |
alikon
- comment
- 7 Dec 2016
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13115.
| Milestone |
Added: |
||
| Status | Pending | ⇒ | Ready to Commit |
| Milestone |
Added: |
||
jeckodevelopment
- comment
- 7 Dec 2016
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13115.
wilsonge
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-12-09 13:29:30 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
|
||
cpfeifer
-
I have tested this item✅ successfully on 2b9b286
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13115.