Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Add the module name to tl_user_group::getExcludedFields() #7868

Closed
daniel-berger-nemeth opened this issue Jun 7, 2015 · 6 comments
Closed
Assignees
Labels
Milestone

Comments

@daniel-berger-nemeth
Copy link

Hi community!

I'm often using a lot of extensions for my Contao website. When creating a user group, I sometimes have the problem, for example in tl_content, that the label of two different extensions has the same value (e.g. Title).

I know, the intention of every extension developer should be using self-explaining field labels, but thats not the standard.

My question to the community: Would it be a good idea to add the module name after the field label (e.g. Title [core] or Website title [x-seo]) in the system/modules/core/dca/tl_user_group.php file?

Thanks,
Daniel

@Toflar
Copy link
Member

Toflar commented Jun 8, 2015

Imho best would be to show the table and field name. So something like that:

  • Title (tl_page.title)
  • Website title (tl_page.x-seo_title)
  • etc.

I'm +1 on that feature. Should be fairly easy to implement but it's for Contao 4.1 so we actually need to move that issue. How shall we proceed with issues that actually belong to contao-core-bundle but have been created here @contao/developers?

@daniel-berger-nemeth
Copy link
Author

With the code from Contao 3.5, the DCA tablename is provided within the loop. So this was a nice idea Toflar.

Thx.

@leofeyer leofeyer added this to the 4.1.0 milestone Jun 8, 2015
@daniel-berger-nemeth
Copy link
Author

The same feature would be nice for the Edit multiple mode in tl_content. Should I open a seperate ticket?

@aschempp
Copy link
Member

So the real difference would be to add the database field name to the permissions and edit-multiple, right?

@daniel-berger-nemeth
Copy link
Author

Yes.

@leofeyer leofeyer self-assigned this Oct 8, 2015
leofeyer added a commit to contao/core-bundle that referenced this issue Oct 8, 2015
@leofeyer
Copy link
Member

leofeyer commented Oct 8, 2015

Implemented for user groups and the "edit multiple" mode in contao/core-bundle@d1d2d4e.

@leofeyer leofeyer closed this as completed Oct 8, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants