Bug #40342

Story #69617: FormEngine bugs

"Allowed excludefields" setting from XML flexforms uncomplete / wrong section

Added by Frank Frewer almost 7 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Backend User Interface
Target version:
Start date:
2012-08-28
Due date:
% Done:

100%

TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
nightmare
Is Regression:
No
Sprint Focus:

Description

In the usergroup settings, under "allowed excludefields", exclude fields from flexforms XML doesn't have a label and are in my opinion in the wrong section "Page".

You can replicate the actual bug by installing for example extension tt_address v2.2.1. Under "allowed excludefields" search for singleRecord. You will find only "(singleRecord)" without any label under section "Page" and not in section "Page Content".
Field "singleRecord" is defined in "tt_address/pi1/flexform.xml".

cattura1.png View (39.6 KB) Riccardo De Contardi, 2015-09-14 14:39

cattura2.png View (23.3 KB) Riccardo De Contardi, 2015-09-14 14:39


Related issues

Duplicated by Grid Elements (former official tracker) - now moved to Gitlab! - Bug #63539: <exclude>1</exclude> not working as expected Rejected 2014-12-03

Associated revisions

Revision 5e205b5b (diff)
Added by Mathias Schreiber almost 4 years ago

[BUGFIX] FormEngine: Fix "Allowed excludefields" settings from FlexForm

Creates an own section in usergroup settings for each plugin with
exclude-Tag in at least one field in XML flexform.
Shows the correct label for each field in this section.

Resolves: #40342
Releases: master
Change-Id: I95c969281c689e7502506f5d358cfc06b4324cfb
Reviewed-on: https://review.typo3.org/43308
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>
Reviewed-by: Frank Nägler <>
Tested-by: Frank Nägler <>

History

#1 Updated by Christian Buelter about 6 years ago

For me it seems that in TYPO3 6.1.1 the plugin flexform exclude fields are completely missing in the "Allow excludefields" form element in the backend user group record.

That means it is impossible to give editors rights to fields marked as "exclude field" by the extension. Tested with tt_address 2.3.3.

#2 Updated by Mathias Schreiber over 4 years ago

  • Target version set to 7.4 (Backend)
  • Complexity set to nightmare
  • Is Regression set to No

I'm not sure how to tackle this.
Technically it's a nested condition.
Imagine you want to disallow access to the field flexform itself.
How would one do this?

#3 Updated by Riccardo De Contardi over 4 years ago

  • TYPO3 Version changed from 4.7 to 6.2

This is still present in TYPO3 6.2.12

#4 Updated by Susanne Moog about 4 years ago

  • Target version changed from 7.4 (Backend) to 7.5

#5 Updated by Riccardo De Contardi almost 4 years ago

I made a test in both 6.2.15 and 7.5-dev (latest master) and the latest version of tt_address (2.3.5)

On version 6.2.15 the allow-excludefield "singleRecord" is shown in cattura1.png: as you can see it is shown in the "Page content" list (the "Page content" list is just before "website user")

On version 7.5-dev the allow-excludefield "singleRecord" is shown in cattura2.png: as you can see it is shown in the "Page content" list (the "Page content" list is just before "website user")

The label is still missing

#6 Updated by Gerrit Code Review almost 4 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43308

#7 Updated by Gerrit Code Review almost 4 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43308

#8 Updated by Gerrit Code Review almost 4 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43308

#9 Updated by Gerrit Code Review almost 4 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43308

#10 Updated by Benni Mack almost 4 years ago

  • Sprint Focus set to Stabilization Sprint

#11 Updated by Gerrit Code Review almost 4 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43308

#12 Updated by Wouter Wolters almost 4 years ago

  • Target version changed from 7.5 to 7 LTS
  • Sprint Focus deleted (Stabilization Sprint)

#13 Updated by Gerrit Code Review almost 4 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43308

#14 Updated by Gerrit Code Review almost 4 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/43308

#15 Updated by Gerrit Code Review almost 4 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/43308

#16 Updated by Gerrit Code Review almost 4 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/43308

#17 Updated by Christian Kuhn almost 4 years ago

  • Parent task set to #69617

#18 Updated by Gerrit Code Review almost 4 years ago

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/43308

#19 Updated by Gerrit Code Review almost 4 years ago

Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/43308

#20 Updated by Mathias Schreiber almost 4 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#21 Updated by Frank Frewer almost 4 years ago

As this is a bug - should there not be a fix for v6.2 too?

#22 Updated by Riccardo De Contardi almost 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF