Bug #21924
closedsuggest wizard does not work properly with TCA fields of type "select"
100%
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
Updated by Franz Koch about 15 years ago
added final patch including changes in JS.
Updated by Tolleiv Nietsch almost 13 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.
Updated by Tolleiv Nietsch almost 13 years ago
- File patch_4-6.patch patch_4-6.patch added
Working version for 4-6 - going to take care of master in the next days.
Updated by Gerrit Code Review almost 13 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
Updated by Gerrit Code Review almost 12 years ago
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/18689
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/18689
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/18689
Updated by Gerrit Code Review about 11 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
Updated by Gerrit Code Review about 11 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
Updated by Gerrit Code Review about 11 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
Updated by Markus Klein about 11 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.
Updated by Tolleiv Nietsch about 11 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset b427ae1086a6041a69ba2466c4860c4c70a70d17.
Updated by Riccardo De Contardi over 7 years ago
- Status changed from Resolved to Closed