[com_fields] Include group state only when not on back end list

[laoneo]

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.

[infograf768]

Tested successfully (I had again problems with SHA @mbabker )

[brianteeman]

@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/

[Bakual]

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?

[laoneo]

Yes, it considers the group state only on the front. We never consider the group state on the back end for lists.

[Bakual]

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).

[brianteeman]

I have tested this item ✅ successfully on c9b6f4d

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12696.}

[laoneo]

Comment added

[brianteeman]

I have tested this item ✅ successfully on f4c413c

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12696.}

[laoneo]

We have here two successful tests, can be set to RTC.

[Bakual]

Lets just merge it instead :)

[laoneo]

Even better. Thanks.