Project

General

Profile

Actions

Bug #23909

closed

sys_action - not needed check for "table" in viewRecordList action

Added by Krystian Szymukowicz over 13 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Backend User Interface
Target version:
Start date:
2010-11-02
Due date:
% Done:

100%

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

Description

PROBLEM:
When you define new action type of "Record List" (sys_action records) you have field "List only table". This filed could have been empty in previous relases of "sys_action". Now if its empty and you go into the action you have message:

Action error!
Error: The action configuration is not finished.

This is a regresion.

The label says "List only table" - so if I do not select anything I like to show all records, but this is not possible right now. What if I want to define action: "Edit records of seminar extension" and if user click he should be able to see all the records connected with extension seminar (in one sysfolder)

There is also end user usability issue connected with this. (and this is most important for me) If you select table then "Select field" list will show up which is in 95% not needed by end user and only scares him with not needed options. See the attached screenshot to see what I am referring to

SOLUTION:
Remove the check for the "table" in the "List record" action.

(issue imported from #M16213)


Files

bug_0016213 _overview.gif (60.7 KB) bug_0016213 _overview.gif Administrator Admin, 2010-11-02 10:54
0016213_sys_action.patch (511 Bytes) 0016213_sys_action.patch Administrator Admin, 2010-11-02 10:54
0016213_sys_action_44.patch (598 Bytes) 0016213_sys_action_44.patch Administrator Admin, 2010-11-26 15:49
0016213_sys_action_trunk.patch (598 Bytes) 0016213_sys_action_trunk.patch Administrator Admin, 2010-11-26 15:49
Actions #1

Updated by Krystian Szymukowicz over 13 years ago

Plese test:
0016213_sys_action_44.patch
0016213_sys_action_trunk.patch

Actions #2

Updated by Alexander Opitz over 10 years ago

  • Status changed from New to Needs Feedback
  • Target version deleted (0)
  • Is Regression set to No

Hi,

as this issue is very old. Does the problem still exists within newer versions of TYPO3 CMS (4.5 or 6.1)?

Actions #3

Updated by Krystian Szymukowicz over 10 years ago

Hi,

Yes. This issue is still there in 4.7.x and 6.1.x

Just search for:

if ($this->id 0 || $this->table '')

Its only in one file.

grtz
Krystian

Actions #4

Updated by Alexander Opitz over 10 years ago

  • Category set to Backend User Interface
  • Status changed from Needs Feedback to New
Actions #5

Updated by Mathias Schreiber over 9 years ago

  • Target version set to 7.5
Actions #6

Updated by Benni Mack over 8 years ago

  • Target version changed from 7.5 to 7 LTS
Actions #7

Updated by Gerrit Code Review over 8 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/44096

Actions #8

Updated by Mathias Schreiber over 8 years ago

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

Updated by Krystian Szymukowicz over 8 years ago

Thank you :)

Actions #10

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF