Project

General

Profile

Actions

Bug #21924

closed

suggest wizard does not work properly with TCA fields of type "select"

Added by Franz Koch about 14 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
Start date:
2010-01-07
Due date:
% Done:

100%

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

Description

Also the pending documentation claims that the new suggest wizard is working nicely with group and select fields, it's not the case for select fields for two reasons:
a) 'foreign_table' is not taken into account for the search, only 'allowed' which is not present in select field configurations by default

b) once a) is fixed, the JS part of the suggest box is handling the search results wrong because the ID values of the generated option fields always have the tablename prepended as it's needed for group fields, but not select fields

The attached patch has as first step a patch for a). I didn't have a closer look at the JS part yet - maybe if one who knows what to do could provide a patch for the JS? Thanks.

The attatched patch is also introducing a new configuration option for the wizard, that allows to define additional search fields in which the wizard should search in.
(issue imported from #M13172)


Files

suggest-wizard-selectFields.diff (1.85 KB) suggest-wizard-selectFields.diff Administrator Admin, 2010-01-07 19:05
suggest-wizard-selectFields_v2.diff (4.63 KB) suggest-wizard-selectFields_v2.diff Administrator Admin, 2010-01-07 19:46
patch_4-6.patch (5.92 KB) patch_4-6.patch Tolleiv Nietsch, 2012-05-02 15:04

Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Feature #17874: Add suggest-like feature to TCEformsClosedRupert Germann2007-12-02

Actions
Actions #1

Updated by Franz Koch about 14 years ago

added final patch including changes in JS.

Actions #2

Updated by Tolleiv Nietsch almost 12 years ago

  • Status changed from New to Accepted
  • Assignee set to Tolleiv Nietsch
  • Target version changed from 0 to 4.7.1

Let's see if I can help with this one.

Actions #3

Updated by Tolleiv Nietsch almost 12 years ago

Working version for 4-6 - going to take care of master in the next days.

Actions #4

Updated by Gerrit Code Review almost 12 years ago

  • Status changed from Accepted to Under Review

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

Actions #5

Updated by Gerrit Code Review about 11 years ago

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

Actions #6

Updated by Gerrit Code Review almost 11 years ago

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

Actions #7

Updated by Gerrit Code Review over 10 years ago

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

Actions #8

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

Actions #9

Updated by Gerrit Code Review over 10 years ago

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

Actions #10

Updated by Gerrit Code Review about 10 years ago

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

Actions #11

Updated by Markus Klein about 10 years ago

  • Is Regression set to No

How to test

Step 1
create the typo3conf/extTables.php file and add this content:

<?php

$GLOBALS['TCA']['be_groups']['columns']['subgroup']['config']['renderMode'] = 'checkbox';
$GLOBALS['TCA']['be_groups']['columns']['subgroup']['config']['wizards']['suggest'] = array('type' => 'suggest');

Step 2
Clear All Cache

Step 3
If you don't have a BE user group, add one.
Save and close

Step 4
Create another BE user group. See the groups listed in the subgroups field. They all have a checkbox in front now.
Search for one of the groups in the wizard and select it there.
The item in the list should now be checked.

Actions #12

Updated by Tolleiv Nietsch about 10 years ago

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

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF