[#12696] - [com_fields] Include group state only when not on back end list
- Fixed in Code Base
- 14 Dec 2016
- Medium
- Build: staging
- Easy Test
- # 12696
- Diff
- Digital-Peak:group-state
- Success continuous-integration/drone the build was successful Details
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Pending JTracker/HumanTestResults Human Test Results: 1 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
Pull Request for Issue #12692.
Summary of Changes
Shows only fields in the form on the back end for published groups.
Testing Instructions
- Create a Field Group for Articles.
- Create a Field and assign it to that group
- Unpublish the Field Group.
Expected result
The field should not be proposed in the edit form as its Group is unpublished.
Actual result
The field is still proposed under the tab of its Field Group when editing an article.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Category | ⇒ | Administration Components |
| Title |
|
||||||
| Title |
|
||||||
@infograf768 - again - clear your browser cache
On 2 November 2016 at 16:37, infograf768 notifications@github.com wrote:
Tested successfully (I had again problems with SHA @mbabker
https://github.com/mbabker )—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#12696 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8ZZWatY5hgeW0LJNZYwkSP9C11YKks5q6LxjgaJpZM4KnGQj
.
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
https://brian.teeman.net/ http://brian.teeman.net/
| Category | Administration Components | ⇒ | Administration com_fields Components |
Can you maybe add a comment as to why we even have that check? Because it confused me as well.
Now I think it's just so we still see the fields in the fields manager, right?
Yes, it considers the group state only on the front. We never consider the group state on the back end for lists.
Agreed. Can you add a comment to it? Because I don't think we have that case anywhere else (usually we have different models in front- and backend).
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12696.
Comment added
| Easy | No | ⇒ | Yes |
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12696.
We have here two successful tests, can be set to RTC.
Lets just merge it instead :)
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-12-14 07:25:40 |
| Closed_By | ⇒ | Bakual |
| Milestone |
Added: |
||
Even better. Thanks.
cpfeifer
-
Tested successfully (I had again problems with SHA @mbabker )