Project

General

Profile

Actions

Bug #44287

closed

sys_action's "task" doesn't show pid field

Added by Anonymous almost 12 years ago. Updated about 6 years ago.

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

100%

Estimated time:
TYPO3 Version:
4.5
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

when creating a sql query via sys_actions task module it's not possible to have the pid field in the result table.

this is because t3lib_fullsearch that is being used to render the result excludes "pid" and "deleted" from the list of query fields as long there's no explicitly defined list of valid fields:

see t3lib/class.t3lib_fullsearch.php

1074         function resultRowTitles($row, $conf, $table) {
1075                 $SET = $GLOBALS['SOBE']->MOD_SETTINGS;
1076
1077                         $tableHeader = array();
1078
1079                         // Start header row
1080                 $tableHeader[] = '<thead><tr class="bgColor5">';
1081
1082                         // Iterate over given columns
1083                 foreach ($row as $fieldName => $fieldValue) {
1084                         if (t3lib_div::inList($SET['queryFields'], $fieldName) || (!$SET['queryFields'] && $fieldName != 'pid' && $fieldName != 'deleted')) {
1085                                 $THparams = (strlen($fieldValue) < 50) ? ' style="white-space:nowrap;"' : '';
1086

the attached patch adds all fields that were selected in the query editor to the list of valid queryFields so t3lib_fullsearch doesn't kick out "pid" and "deleted".


Files

sys_action_task_fields.txt (780 Bytes) sys_action_task_fields.txt Anonymous, 2013-01-03 15:20
Actions #1

Updated by Gerrit Code Review over 11 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 https://review.typo3.org/20980

Actions #2

Updated by Gerrit Code Review over 11 years ago

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

Actions #3

Updated by Gerrit Code Review over 11 years ago

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

Actions #4

Updated by Gerrit Code Review almost 10 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/20980

Actions #5

Updated by Anonymous almost 10 years ago

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

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF