Bug #34631

Show correct record title for be_groups

Added by Joh. Feustel almost 8 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2012-03-07
Due date:
% Done:

100%

TYPO3 Version:
4.7
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Currently it lists groups only using field title of be_groups table regardless of any label definitions in TCA.
Use t3lib_BEfunc::getRecordTitle() instead.
Function is called in following modules:

typo3/sysext/belog/mod/index.php
typo3/sysext/beuser/mod/index.php
typo3/sysext/perm/mod1/class.sc_mod_web_perm_ajax.php
typo3/sysext/perm/mod1/index.php
typo3/sysext/version/ws/index.php:

Test:

34631.patch View (914 Bytes) Joh. Feustel, 2012-03-07 11:17

screen.png View - Screenshot showing changed output (21.2 KB) Bernhard Kraft, 2013-08-13 10:03

extTables.php View - Test-Configuration (1.41 KB) Bernhard Kraft, 2013-08-13 10:03


Related issues

Related to TYPO3 Core - Bug #27100: use t3lib_BEfunc::getRecordTitle in module header and beuser module Closed 2011-05-27

Associated revisions

Revision 8a2b0404 (diff)
Added by Markus Klein over 6 years ago

[BUGFIX] Show correct record title for be_groups and be_users

Currently it lists groups only using field title of
be_groups table regardless of any label definitions in TCA.
Use BackendUtility::getRecordTitle() instead.

Change-Id: I051c29687cb51d1883846b3cb780409c1e1cadc4
Resolves: #34631
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/9631
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters

Revision 429e13d5 (diff)
Added by Markus Klein about 6 years ago

[BUGFIX] Show correct record title for be_groups and be_users

Currently it lists groups only using field title of
be_groups table regardless of any label definitions in TCA.
Use BackendUtility::getRecordTitle() instead.

Change-Id: I051c29687cb51d1883846b3cb780409c1e1cadc4
Resolves: #34631
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22374
Reviewed-by: Markus Klein
Tested-by: Markus Klein

Revision f76c7ea8 (diff)
Added by Markus Klein about 6 years ago

[BUGFIX] Show correct record title for be_groups and be_users

Currently it lists groups only using field title of
be_groups table regardless of any label definitions in TCA.
Use BackendUtility::getRecordTitle() instead.

Change-Id: I051c29687cb51d1883846b3cb780409c1e1cadc4
Resolves: #34631
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/23142
Reviewed-by: Bernhard Kraft
Tested-by: Bernhard Kraft
Reviewed-by: Markus Klein
Tested-by: Markus Klein

History

#1 Updated by Joh. Feustel almost 8 years ago

#2 Updated by Gerrit Code Review almost 8 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9631

#3 Updated by Gerrit Code Review almost 8 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9631

#4 Updated by Gerrit Code Review about 7 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/9631

#5 Updated by Gerrit Code Review about 7 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/9631

#6 Updated by Gerrit Code Review about 7 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/9631

#7 Updated by Gerrit Code Review about 7 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/9631

#8 Updated by Gerrit Code Review about 7 years ago

Patch set 7 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/9631

#9 Updated by Gerrit Code Review about 7 years ago

Patch set 8 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/9631

#10 Updated by Gerrit Code Review over 6 years ago

Patch set 9 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/9631

#11 Updated by Gerrit Code Review over 6 years ago

Patch set 10 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/9631

#12 Updated by Gerrit Code Review over 6 years ago

Patch set 11 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/9631

#13 Updated by Gerrit Code Review over 6 years ago

Patch set 12 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/9631

#14 Updated by Gerrit Code Review over 6 years ago

Patch set 13 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/9631

#15 Updated by Gerrit Code Review over 6 years ago

Patch set 14 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/9631

#16 Updated by Gerrit Code Review over 6 years ago

Patch set 1 for branch TYPO3_6-1 has been pushed to the review server.
It is available at https://review.typo3.org/22374

#17 Updated by Gerrit Code Review over 6 years ago

Patch set 15 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/9631

#18 Updated by Bernhard Kraft over 6 years ago

Tested current patch from review.typo3.org
https://review.typo3.org/#/c/9631/

and found it working properly. See attached files for screenshot and test-configuration "typo3conf/extTables.php"

#19 Updated by Gerrit Code Review over 6 years ago

Patch set 16 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/9631

#20 Updated by Gerrit Code Review over 6 years ago

Patch set 2 for branch TYPO3_6-1 has been pushed to the review server.
It is available at https://review.typo3.org/22374

#21 Updated by Gerrit Code Review over 6 years ago

Patch set 1 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/23142

#22 Updated by Markus Klein over 6 years ago

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

#23 Updated by Gerrit Code Review over 6 years ago

  • Status changed from Resolved to Under Review

Patch set 3 for branch TYPO3_6-1 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/22374

#24 Updated by Gerrit Code Review about 6 years ago

Patch set 4 for branch TYPO3_6-1 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/22374

#25 Updated by Gerrit Code Review about 6 years ago

Patch set 2 for branch TYPO3_6-0 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/23142

#26 Updated by Gerrit Code Review about 6 years ago

Patch set 5 for branch TYPO3_6-1 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/22374

#27 Updated by Gerrit Code Review about 6 years ago

Patch set 3 for branch TYPO3_6-0 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/23142

#28 Updated by Markus Klein about 6 years ago

  • Status changed from Under Review to Resolved

#29 Updated by Benni Mack over 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF